Hi David, On Tue, Oct 06, 2015 at 04:50:35PM -0700, David Daney wrote: > From: "Sean O. Stalley" <sean.stalley@xxxxxxxxx> > > Add registers defined in PCI-SIG's Enhanced allocation ECN. > > Signed-off-by: Sean O. Stalley <sean.stalley@xxxxxxxxx> > [david.daney@xxxxxxxxxx: Added more definitions for PCI_EA_BEI_*] > Signed-off-by: Signed-off-by: David Daney <david.daney@xxxxxxxxxx> > --- > include/uapi/linux/pci_regs.h | 44 ++++++++++++++++++++++++++++++++++++++++++- > 1 file changed, 43 insertions(+), 1 deletion(-) > > diff --git a/include/uapi/linux/pci_regs.h b/include/uapi/linux/pci_regs.h > index 413417f..352e418 100644 > --- a/include/uapi/linux/pci_regs.h > +++ b/include/uapi/linux/pci_regs.h > @@ -216,7 +216,8 @@ > #define PCI_CAP_ID_MSIX 0x11 /* MSI-X */ > #define PCI_CAP_ID_SATA 0x12 /* SATA Data/Index Conf. */ > #define PCI_CAP_ID_AF 0x13 /* PCI Advanced Features */ > -#define PCI_CAP_ID_MAX PCI_CAP_ID_AF > +#define PCI_CAP_ID_EA 0x14 /* PCI Enhanced Allocation */ > +#define PCI_CAP_ID_MAX PCI_CAP_ID_EA > #define PCI_CAP_LIST_NEXT 1 /* Next capability in the list */ > #define PCI_CAP_FLAGS 2 /* Capability defined flags (16 bits) */ > #define PCI_CAP_SIZEOF 4 > @@ -353,6 +354,47 @@ > #define PCI_AF_STATUS_TP 0x01 > #define PCI_CAP_AF_SIZEOF 6 /* size of AF registers */ > > +/* PCI Enhanced Allocation registers */ > + > +#define PCI_EA_NUM_ENT 2 /* Number of Capability Entries */ > +#define PCI_EA_NUM_ENT_MASK 0x3f /* Num Entries Mask */ Can you tweak this a bit to match the style of the rest of the file a bit closer? - no indent (one space) for register offsets - two spaces before masks for register fields - write out masks showing register width - use explicit mask, not BIT() > +#define PCI_EA_FIRST_ENT 4 /* First EA Entry in List */ > +#define PCI_EA_FIRST_ENT_BRIDGE 8 /* First EA Entry for Bridges */ > +#define PCI_EA_ES 0x7 /* Entry Size */ #define PCI_EA_ES 0x00000007 /* Entry Size */ > +#define PCI_EA_BEI(x) (((x) >> 4) & 0xf) /* BAR Equivalent Indicator */ > +/* 0-5 map to BARs 0-5 respectively */ > +#define PCI_EA_BEI_BAR0 0 > +#define PCI_EA_BEI_BAR5 5 > +#define PCI_EA_BEI_BRIDGE 6 /* Resource behind bridge */ > +#define PCI_EA_BEI_ENI 7 /* Equivalent Not Indicated */ > +#define PCI_EA_BEI_ROM 8 /* Expansion ROM */ > +/* 9-14 map to VF BARs 0-5 respectively */ > +#define PCI_EA_BEI_VF_BAR0 9 > +#define PCI_EA_BEI_VF_BAR5 14 > +#define PCI_EA_BEI_RESERVED 15 /* Reserved - Treat like ENI */ > + > +#define PCI_EA_PP(x) (((x) >> 8) & 0xff) /* Primary Properties */ > +#define PCI_EA_SP(x) (((x) >> 16) & 0xff) /* Secondary Properties */ > +#define PCI_EA_P_MEM 0x00 /* Non-Prefetch Memory */ What does the "_P_" stand for? Maybe it could be dropped? Oh, I suppose it stands for "property." > +#define PCI_EA_P_MEM_PREFETCH 0x01 /* Prefetchable Memory */ > +#define PCI_EA_P_IO 0x02 /* I/O Space */ > +#define PCI_EA_P_VIRT_MEM_PREFETCH 0x03 /* VF Prefetchable Memory */ > +#define PCI_EA_P_VIRT_MEM 0x04 /* VF Non-Prefetch Memory */ _VIRT_MEM suggests "virtual memory"; maybe you could use "VF_MEM" instead? > +#define PCI_EA_P_BRIDGE_MEM 0x05 /* Bridge Non-Prefetch Memory */ > +#define PCI_EA_P_BRIDGE_MEM_PREFETCH 0x06 /* Bridge Prefetchable Memory */ > +#define PCI_EA_P_BRIDGE_IO 0x07 /* Bridge I/O Space */ > +/* 0x08-0xfc reserved */ > +#define PCI_EA_P_MEM_RESERVED 0xfd /* Reserved Memory */ > +#define PCI_EA_P_IO_RESERVED 0xfe /* Reserved I/O Space */ > +#define PCI_EA_P_UNAVAILABLE 0xff /* Entry Unavailable */ > +#define PCI_EA_WRITEABLE BIT(30) /* Writable, 1 = RW, 0 = HwInit */ > +#define PCI_EA_ENABLE BIT(31) /* Enable for this entry */ #define PCI_EA_WRITABLE 0x40000000 #define PCI_EA_ENABLE 0x80000000 (note s/WRITEABLE/WRITABLE/) > +#define PCI_EA_BASE 4 /* Base Address Offset */ > +#define PCI_EA_MAX_OFFSET 8 /* MaxOffset (resource length) */ > +/* bit 0 is reserved */ > +#define PCI_EA_IS_64 BIT(1) /* 64-bit field flag */ > +#define PCI_EA_FIELD_MASK 0xfffffffc /* For Base & Max Offset */ > + > /* PCI-X registers (Type 0 (non-bridge) devices) */ > > #define PCI_X_CMD 2 /* Modes & Features */ > -- > 1.9.1 > > -- > To unsubscribe from this list: send the line "unsubscribe linux-kernel" in > the body of a message to majordomo@xxxxxxxxxxxxxxx > More majordomo info at http://vger.kernel.org/majordomo-info.html > Please read the FAQ at http://www.tux.org/lkml/ -- To unsubscribe from this list: send the line "unsubscribe linux-pci" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html