From 9007ffd3d1b7c079c791e90ec52243eac385979f Mon Sep 17 00:00:00 2001 From: Werner Zeh Date: Wed, 6 Jul 2016 08:47:56 +0200 Subject: PCI: Use PCI_DEVFN macro instead of DEV_FUNC There are several different macros available to convert a PCI device and function to a single 8 bit value. One is PCI_DEVFN and is defined in device/pci_def.h. The other is DEV_FUNC and is defined in several intel fsp based chipset implementations. In fsp_broadwell_de DEV_FUNC is even used without being defined at all. This patch unifies the situation so that only PCI_DEVFN is used. Change-Id: Ia1c6d7f3683badc66d15053846936d88aa836632 Signed-off-by: Werner Zeh Reviewed-on: https://review.coreboot.org/15546 Tested-by: build bot (Jenkins) Reviewed-by: Paul Menzel Reviewed-by: Stefan Reinauer --- src/soc/intel/fsp_baytrail/include/soc/pci_devs.h | 72 +++++++++++----------- .../intel/fsp_broadwell_de/include/soc/pci_devs.h | 38 ++++++------ src/soc/intel/quark/include/soc/pci_devs.h | 5 +- 3 files changed, 59 insertions(+), 56 deletions(-) (limited to 'src/soc') diff --git a/src/soc/intel/fsp_baytrail/include/soc/pci_devs.h b/src/soc/intel/fsp_baytrail/include/soc/pci_devs.h index 2d0dac11b8..f9c978da32 100644 --- a/src/soc/intel/fsp_baytrail/include/soc/pci_devs.h +++ b/src/soc/intel/fsp_baytrail/include/soc/pci_devs.h @@ -17,49 +17,49 @@ #ifndef _BAYTRAIL_PCI_DEVS_H_ #define _BAYTRAIL_PCI_DEVS_H_ +#include + #define BUS0 0 /* All these devices live on bus 0 with the associated device and function */ -#define DEV_FUNC(x,y) ((x<<3) | y) - /* SoC transaction router */ #define SOC_DEV 0x0 #define SOC_FUNC 0 # define SOC_DEVID 0x0f00 -# define SOC_DEV_FUNC DEV_FUNC(SOC_DEV,SOC_FUNC) +# define SOC_DEV_FUNC PCI_DEVFN(SOC_DEV,SOC_FUNC) /* Graphics and Display */ #define GFX_DEV 0x2 #define GFX_FUNC 0 # define GFX_DEVID 0x0f31 -# define GFX_DEV_FUNC DEV_FUNC(GFX_DEV,GFX_FUNC) +# define GFX_DEV_FUNC PCI_DEVFN(GFX_DEV,GFX_FUNC) /* MIPI */ #define MIPI_DEV 0x3 #define MIPI_FUNC 0 # define MIPI_DEVID 0x0f38 -# define MIPI_DEV_FUNC DEV_FUNC(MIPI_DEV,MIPI_FUNC) +# define MIPI_DEV_FUNC PCI_DEVFN(MIPI_DEV,MIPI_FUNC) /* SDIO Port */ #define EMMC_DEV 0x10 #define EMMC_FUNC 0 # define EMMC_DEVID 0x0f14 -# define EMMC_DEV_FUNC DEV_FUNC(EMMC_DEV,EMMC_FUNC) +# define EMMC_DEV_FUNC PCI_DEVFN(EMMC_DEV,EMMC_FUNC) /* SDIO Port */ #define SDIO_DEV 0x11 #define SDIO_FUNC 0 # define SDIO_DEVID 0x0f15 -# define SDIO_DEV_FUNC DEV_FUNC(SDIO_DEV,SDIO_FUNC) +# define SDIO_DEV_FUNC PCI_DEVFN(SDIO_DEV,SDIO_FUNC) /* SD Port */ #define SD_DEV 0x12 #define SD_FUNC 0 # define SD_DEVID 0x0f16 -# define SD_DEV_FUNC DEV_FUNC(SD_DEV,SD_FUNC) +# define SD_DEV_FUNC PCI_DEVFN(SD_DEV,SD_FUNC) /* SATA */ #define SATA_DEV 0x13 @@ -74,7 +74,7 @@ # define SATA_SP 0xD0 # define SATA_BIST1 0xE4 # define SATA_BIST2 0xE8 -# define SATA_DEV_FUNC DEV_FUNC(SATA_DEV,SATA_FUNC) +# define SATA_DEV_FUNC PCI_DEVFN(SATA_DEV,SATA_FUNC) #define SATA_MA_BDFO PCI_DEV(BUS0, SATA_DEV, SATA_FUNC), SATA_MA #define SATA_SP_BDFO PCI_DEV(BUS0, SATA_DEV, SATA_FUNC), SATA_SP @@ -88,25 +88,25 @@ # define XHCI_FUS_REG 0xE0 # define XHCI_FUNC_DISABLE (1 << 0) # define XHCI_USB2PR_REG 0xD0 -# define XHCI_DEV_FUNC DEV_FUNC(XHCI_DEV,XHCI_FUNC) +# define XHCI_DEV_FUNC PCI_DEVFN(XHCI_DEV,XHCI_FUNC) /* LPE Audio */ #define LPE_DEV 0x15 #define LPE_FUNC 0 # define LPE_DEVID 0x0f28 -# define LPE_DEV_FUNC DEV_FUNC(LPE_DEV,LPE_FUNC) +# define LPE_DEV_FUNC PCI_DEVFN(LPE_DEV,LPE_FUNC) /* OTG */ #define OTG_DEV 0x16 #define OTG_FUNC 0 # define OTG_DEVID 0x0f37 -# define OTG_DEV_FUNC DEV_FUNC(LPE_DEV,LPE_FUNC) +# define OTG_DEV_FUNC PCI_DEVFN(LPE_DEV,LPE_FUNC) /* MMC Port */ #define MMC45_DEV 0x17 #define MMC45_FUNC 0 # define MMC45_DEVID 0x0f50 -# define MMC45_DEV_FUNC DEV_FUNC(MMC45_DEV,MMC45_FUNC) +# define MMC45_DEV_FUNC PCI_DEVFN(MMC45_DEV,MMC45_FUNC) /* Serial IO 1 */ #define SIO1_DEV 0x18 @@ -134,26 +134,26 @@ # define I2C7_DEV SIO1_DEV # define I2C7_FUNC 7 # define I2C7_DEVID 0x0f47 -# define SIO_DMA1_DEV_FUNC DEV_FUNC(SIO_DMA1_DEV,SIO_DMA1_FUNC) -# define I2C1_DEV_FUNC DEV_FUNC(I2C1_DEV,I2C1_FUNC) -# define I2C2_DEV_FUNC DEV_FUNC(I2C2_DEV,I2C2_FUNC) -# define I2C3_DEV_FUNC DEV_FUNC(I2C3_DEV,I2C3_FUNC) -# define I2C4_DEV_FUNC DEV_FUNC(I2C4_DEV,I2C4_FUNC) -# define I2C5_DEV_FUNC DEV_FUNC(I2C5_DEV,I2C5_FUNC) -# define I2C6_DEV_FUNC DEV_FUNC(I2C6_DEV,I2C6_FUNC) -# define I2C7_DEV_FUNC DEV_FUNC(I2C7_DEV,I2C7_FUNC) +# define SIO_DMA1_DEV_FUNC PCI_DEVFN(SIO_DMA1_DEV,SIO_DMA1_FUNC) +# define I2C1_DEV_FUNC PCI_DEVFN(I2C1_DEV,I2C1_FUNC) +# define I2C2_DEV_FUNC PCI_DEVFN(I2C2_DEV,I2C2_FUNC) +# define I2C3_DEV_FUNC PCI_DEVFN(I2C3_DEV,I2C3_FUNC) +# define I2C4_DEV_FUNC PCI_DEVFN(I2C4_DEV,I2C4_FUNC) +# define I2C5_DEV_FUNC PCI_DEVFN(I2C5_DEV,I2C5_FUNC) +# define I2C6_DEV_FUNC PCI_DEVFN(I2C6_DEV,I2C6_FUNC) +# define I2C7_DEV_FUNC PCI_DEVFN(I2C7_DEV,I2C7_FUNC) /* Trusted Execution Engine */ #define TXE_DEV 0x1a #define TXE_FUNC 0 # define TXE_DEVID 0x0f18 -# define TXE_DEV_FUNC DEV_FUNC(TXE_DEV,TXE_FUNC) +# define TXE_DEV_FUNC PCI_DEVFN(TXE_DEV,TXE_FUNC) /* HD Audio */ #define HDA_DEV 0x1b #define HDA_FUNC 0 # define HDA_DEVID 0x0f04 -# define HDA_DEV_FUNC DEV_FUNC(HDA_DEV,HDA_FUNC) +# define HDA_DEV_FUNC PCI_DEVFN(HDA_DEV,HDA_FUNC) # define HDA_AZUBAR 0x14 # define HDA_MMLA 0x64 # define HDA_MMUA 0x68 @@ -175,16 +175,16 @@ # define PCIE_PORT4_DEV PCIE_DEV # define PCIE_PORT4_FUNC 3 # define PCIE_PORT4_DEVID 0x0f4e -# define PCIE_PORT1_DEV_FUNC DEV_FUNC(PCIE_DEV,PCIE_PORT1_FUNC) -# define PCIE_PORT2_DEV_FUNC DEV_FUNC(PCIE_DEV,PCIE_PORT2_FUNC) -# define PCIE_PORT3_DEV_FUNC DEV_FUNC(PCIE_DEV,PCIE_PORT3_FUNC) -# define PCIE_PORT4_DEV_FUNC DEV_FUNC(PCIE_DEV,PCIE_PORT4_FUNC) +# define PCIE_PORT1_DEV_FUNC PCI_DEVFN(PCIE_DEV,PCIE_PORT1_FUNC) +# define PCIE_PORT2_DEV_FUNC PCI_DEVFN(PCIE_DEV,PCIE_PORT2_FUNC) +# define PCIE_PORT3_DEV_FUNC PCI_DEVFN(PCIE_DEV,PCIE_PORT3_FUNC) +# define PCIE_PORT4_DEV_FUNC PCI_DEVFN(PCIE_DEV,PCIE_PORT4_FUNC) /* EHCI */ #define EHCI_DEV 0x1d #define EHCI_FUNC 0 # define EHCI_DEVID 0x0f34 -# define EHCI_DEV_FUNC DEV_FUNC(EHCI_DEV,EHCI_FUNC) +# define EHCI_DEV_FUNC PCI_DEVFN(EHCI_DEV,EHCI_FUNC) /* Serial IO 2 */ #define SIO2_DEV 0x1e @@ -206,12 +206,12 @@ # define SPI_DEV SIO2_DEV # define SPI_FUNC 5 # define SPI_DEVID 0xf0e -# define SIO_DMA2_DEV_FUNC DEV_FUNC(SIO_DMA2_DEV,SIO_DMA2_FUNC) -# define PWM1_DEV_FUNC DEV_FUNC(PWM1_DEV,PWM1_FUNC) -# define PWM2_DEV_FUNC DEV_FUNC(PWM2_DEV,PWM2_FUNC) -# define HSUART1_DEV_FUNC DEV_FUNC(HSUART1_DEV,HSUART1_FUNC) -# define HSUART2_DEV_FUNC DEV_FUNC(HSUART2_DEV,HSUART2_FUNC) -# define SPI_DEV_FUNC DEV_FUNC(SPI_DEV,SPI_FUNC) +# define SIO_DMA2_DEV_FUNC PCI_DEVFN(SIO_DMA2_DEV,SIO_DMA2_FUNC) +# define PWM1_DEV_FUNC PCI_DEVFN(PWM1_DEV,PWM1_FUNC) +# define PWM2_DEV_FUNC PCI_DEVFN(PWM2_DEV,PWM2_FUNC) +# define HSUART1_DEV_FUNC PCI_DEVFN(HSUART1_DEV,HSUART1_FUNC) +# define HSUART2_DEV_FUNC PCI_DEVFN(HSUART2_DEV,HSUART2_FUNC) +# define SPI_DEV_FUNC PCI_DEVFN(SPI_DEV,SPI_FUNC) /* Platform Controller Unit */ @@ -222,9 +222,9 @@ # define SMBUS_DEV PCU_DEV # define SMBUS_FUNC 3 # define SMBUS_DEVID 0x0f12 -# define LPC_DEV_FUNC DEV_FUNC(LPC_DEV,LPC_FUNC) +# define LPC_DEV_FUNC PCI_DEVFN(LPC_DEV,LPC_FUNC) # define LPC_BDF PCI_DEV(0, LPC_DEV, LPC_FUNC) -# define SMBUS_DEV_FUNC DEV_FUNC(SMBUS_DEV,SMBUS_FUNC) +# define SMBUS_DEV_FUNC PCI_DEVFN(SMBUS_DEV,SMBUS_FUNC) #define INITIAL_TIMESTAMP_LOCATION HDA_MMUA_BDFO #define BEFORE_CAR_TIMESTAMP_LOCATION SATA_BIST1_BDFO diff --git a/src/soc/intel/fsp_broadwell_de/include/soc/pci_devs.h b/src/soc/intel/fsp_broadwell_de/include/soc/pci_devs.h index 646ed3fecb..99236018c5 100644 --- a/src/soc/intel/fsp_broadwell_de/include/soc/pci_devs.h +++ b/src/soc/intel/fsp_broadwell_de/include/soc/pci_devs.h @@ -17,38 +17,40 @@ #ifndef _SOC_PCI_DEVS_H_ #define _SOC_PCI_DEVS_H_ +#include + #define BUS0 0 #define SOC_DEV 0 #define SOC_FUNC 0 #define SOC_DEVID 0x2F00 #define SOC_DEVID_ES2 0x6F00 -#define SOC_DEV_FUNC DEV_FUNC(SOC_DEV, SOC_FUNC) +#define SOC_DEV_FUNC PCI_DEVFN(SOC_DEV, SOC_FUNC) #define LPC_DEV 31 #define LPC_FUNC 0 #define LPC_DEVID 0x8C42 #define LPC_DEVID_ES2 0x8C54 -#define LPC_DEV_FUNC DEV_FUNC(LPC_DEV, LPC_FUNC) +#define LPC_DEV_FUNC PCI_DEVFN(LPC_DEV, LPC_FUNC) #define SATA_DEV 31 #define SATA_FUNC 2 #define AHCI_DEVID 0x8C02 -#define SATA_DEV_FUNC DEV_FUNC(SATA_DEV, SATA_FUNC) +#define SATA_DEV_FUNC PCI_DEVFN(SATA_DEV, SATA_FUNC) #define SATA2_DEV 31 #define SATA2_FUNC 5 -#define SATA2_DEV_FUNC DEV_FUNC(SATA2_DEV, SATA2_FUNC) +#define SATA2_DEV_FUNC PCI_DEVFN(SATA2_DEV, SATA2_FUNC) #define EHCI1_DEV 29 #define EHCI1_FUNC 0 #define EHCI1_DEVID 0x8C26 -#define EHCI1_DEV_FUNC DEV_FUNC(EHCI_DEV1, EHCI_FUNC1) +#define EHCI1_DEV_FUNC PCI_DEVFN(EHCI_DEV1, EHCI_FUNC1) #define EHCI2_DEV 26 #define EHCI2_FUNC 0 #define EHCI2_DEVID 0x8C2D -#define EHCI2_DEV_FUNC DEV_FUNC(EHCI_DEV2, EHCI_FUNC2) +#define EHCI2_DEV_FUNC PCI_DEVFN(EHCI_DEV2, EHCI_FUNC2) #define XHCI_DEV 20 #define XHCI_FUNC 0 @@ -56,22 +58,22 @@ #define XHCI_FUS_REG 0xE0 #define XHCI_FUNC_DISABLE (1 << 0) #define XHCI_USB2PR_REG 0xD0 -#define XHCI_DEV_FUNC DEV_FUNC(XHCI_DEV, XHCI_FUNC) +#define XHCI_DEV_FUNC PCI_DEVFN(XHCI_DEV, XHCI_FUNC) #define GBE_DEV 25 #define GBE_FUNC 0 #define GBE_DEVID 0x8C33 -#define GBE_DEV_FUNC DEV_FUNC(GBE_DEV, GBE_FUNC) +#define GBE_DEV_FUNC PCI_DEVFN(GBE_DEV, GBE_FUNC) #define ME_DEV 22 #define ME_FUNC 0 #define ME_DEVID 0x8C3A -#define ME_DEV_FUNC DEV_FUNC(ME_DEV, ME_FUNC) +#define ME_DEV_FUNC PCI_DEVFN(ME_DEV, ME_FUNC) #define HDA_DEV 27 #define HDA_FUNC 0 #define HDA_DEVID 0x8C20 -#define HDA_DEV_FUNC DEV_FUNC(HDA_DEV, HDA_FUNC) +#define HDA_DEV_FUNC PCI_DEVFN(HDA_DEV, HDA_FUNC) #define PCIE_DEV 28 #define PCIE_PORT1_DEV PCIE_DEV @@ -98,13 +100,13 @@ #define PCIE_PORT8_DEV PCIE_DEV #define PCIE_PORT8_FUNC 7 #define PCIE_PORT8_DEVID 0x8C1E -#define PCIE_PORT1_DEV_FUNC DEV_FUNC(PCIE_DEV, PCIE_PORT1_FUNC) -#define PCIE_PORT2_DEV_FUNC DEV_FUNC(PCIE_DEV, PCIE_PORT2_FUNC) -#define PCIE_PORT3_DEV_FUNC DEV_FUNC(PCIE_DEV, PCIE_PORT3_FUNC) -#define PCIE_PORT4_DEV_FUNC DEV_FUNC(PCIE_DEV, PCIE_PORT4_FUNC) -#define PCIE_PORT5_DEV_FUNC DEV_FUNC(PCIE_DEV, PCIE_PORT5_FUNC) -#define PCIE_PORT6_DEV_FUNC DEV_FUNC(PCIE_DEV, PCIE_PORT6_FUNC) -#define PCIE_PORT7_DEV_FUNC DEV_FUNC(PCIE_DEV, PCIE_PORT7_FUNC) -#define PCIE_PORT8_DEV_FUNC DEV_FUNC(PCIE_DEV, PCIE_PORT8_FUNC) +#define PCIE_PORT1_DEV_FUNC PCI_DEVFN(PCIE_DEV, PCIE_PORT1_FUNC) +#define PCIE_PORT2_DEV_FUNC PCI_DEVFN(PCIE_DEV, PCIE_PORT2_FUNC) +#define PCIE_PORT3_DEV_FUNC PCI_DEVFN(PCIE_DEV, PCIE_PORT3_FUNC) +#define PCIE_PORT4_DEV_FUNC PCI_DEVFN(PCIE_DEV, PCIE_PORT4_FUNC) +#define PCIE_PORT5_DEV_FUNC PCI_DEVFN(PCIE_DEV, PCIE_PORT5_FUNC) +#define PCIE_PORT6_DEV_FUNC PCI_DEVFN(PCIE_DEV, PCIE_PORT6_FUNC) +#define PCIE_PORT7_DEV_FUNC PCI_DEVFN(PCIE_DEV, PCIE_PORT7_FUNC) +#define PCIE_PORT8_DEV_FUNC PCI_DEVFN(PCIE_DEV, PCIE_PORT8_FUNC) #endif /* _SOC_PCI_DEVS_H_ */ diff --git a/src/soc/intel/quark/include/soc/pci_devs.h b/src/soc/intel/quark/include/soc/pci_devs.h index 0c99eb9de4..1aecf6f426 100644 --- a/src/soc/intel/quark/include/soc/pci_devs.h +++ b/src/soc/intel/quark/include/soc/pci_devs.h @@ -19,6 +19,7 @@ #define _QUARK_PCI_DEVS_H_ #include +#include #include /* DEVICE 0 (Memory Controller Hub) */ @@ -54,12 +55,12 @@ #define PCIE_DEV 0x17 #define PCIE_PORT0_DEV PCIE_DEV #define PCIE_PORT0_FUNC 0 -#define PCIE_PORT0_DEV_FUNC DEV_FUNC(PCIE_DEV, PCIE_PORT0_FUNC) +#define PCIE_PORT0_DEV_FUNC PCI_DEVFN(PCIE_DEV, PCIE_PORT0_FUNC) #define PCIE_PORT0_BDF PCI_DEV(PCI_BUS_NUMBER_QNC, PCIE_DEV, PCIE_PORT0_FUNC) #define PCIE_PORT1_DEV PCIE_DEV #define PCIE_PORT1_FUNC 1 -#define PCIE_PORT1_DEV_FUNC DEV_FUNC(PCIE_DEV,PCIE_PORT1_FUNC) +#define PCIE_PORT1_DEV_FUNC PCI_DEVFN(PCIE_DEV,PCIE_PORT1_FUNC) #define PCIE_PORT1_BDF PCI_DEV(PCI_BUS_NUMBER_QNC, PCIE_DEV, PCIE_PORT1_FUNC) /* Platform Controller Unit */ -- cgit v1.2.3