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 +++++++++++------------ 1 file changed, 36 insertions(+), 36 deletions(-) (limited to 'src/soc/intel/fsp_baytrail/include') 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 -- cgit v1.2.3