* [U-Boot] [PATCHv2 0/8] dm: pci: add Freescale PowerPC PCIe driver
@ 2019-05-23 12:21 Z.q. Hou
2019-05-23 12:22 ` [U-Boot] [PATCHv2 1/8] powerpc: mpc85xx: Move CONFIG_FSL_PCIE_DISABLE_ASPM to Kconfig Z.q. Hou
` (7 more replies)
0 siblings, 8 replies; 29+ messages in thread
From: Z.q. Hou @ 2019-05-23 12:21 UTC (permalink / raw)
To: u-boot
From: Hou Zhiqiang <Zhiqiang.Hou@nxp.com>
Add PCIe DM driver for Freescale PowerPC PCIe controllers.
Hou Zhiqiang (8):
powerpc: mpc85xx: Move CONFIG_FSL_PCIE_DISABLE_ASPM to Kconfig
powerpc: mpc85xx: Move CONFIG_FSL_PCIE_RESET to Kconfig
powerpc: mpc85xx: Update the condition to compile PCI routines
powerpc: T208xQDS: Compile the legacy PCIe routines conditionally
t2080: dts: Added PCIe DT nodes
dm: pci: add Freescale PowerPC PCIe driver
powerpc: T208xQDS: Disable legacy PCIe driver
configs: T2080QDS: Enable PCIe driver
arch/powerpc/cpu/mpc85xx/Kconfig | 29 +
arch/powerpc/cpu/mpc85xx/pci.c | 2 +-
arch/powerpc/dts/t2080.dtsi | 48 ++
arch/powerpc/include/asm/config_mpc85xx.h | 5 -
board/freescale/t208xqds/pci.c | 2 +
configs/T2080QDS_NAND_defconfig | 4 +
configs/T2080QDS_SDCARD_defconfig | 4 +
configs/T2080QDS_SPIFLASH_defconfig | 4 +
configs/T2080QDS_defconfig | 4 +
drivers/pci/Kconfig | 8 +
drivers/pci/Makefile | 1 +
drivers/pci/pcie_fsl.c | 611 ++++++++++++++++++++++
drivers/pci/pcie_fsl.h | 57 ++
drivers/pci/pcie_fsl_fixup.c | 51 ++
include/configs/BSC9132QDS.h | 1 -
include/configs/C29XPCIE.h | 1 -
include/configs/MPC8536DS.h | 1 -
include/configs/MPC8544DS.h | 1 -
include/configs/MPC8548CDS.h | 1 -
include/configs/MPC8568MDS.h | 1 -
include/configs/MPC8569MDS.h | 1 -
include/configs/MPC8572DS.h | 1 -
include/configs/P1010RDB.h | 1 -
include/configs/P1022DS.h | 1 -
include/configs/P1023RDB.h | 1 -
include/configs/T208xQDS.h | 19 -
include/configs/T208xRDB.h | 1 -
include/configs/UCP1020.h | 1 -
include/configs/controlcenterd.h | 1 -
include/configs/p1_p2_rdb_pc.h | 1 -
include/configs/p1_twr.h | 1 -
include/configs/sbc8548.h | 1 -
include/configs/xpedite537x.h | 1 -
include/configs/xpedite550x.h | 1 -
34 files changed, 824 insertions(+), 44 deletions(-)
create mode 100644 drivers/pci/pcie_fsl.c
create mode 100644 drivers/pci/pcie_fsl.h
create mode 100644 drivers/pci/pcie_fsl_fixup.c
--
2.17.1
^ permalink raw reply [flat|nested] 29+ messages in thread
* [U-Boot] [PATCHv2 1/8] powerpc: mpc85xx: Move CONFIG_FSL_PCIE_DISABLE_ASPM to Kconfig
2019-05-23 12:21 [U-Boot] [PATCHv2 0/8] dm: pci: add Freescale PowerPC PCIe driver Z.q. Hou
@ 2019-05-23 12:22 ` Z.q. Hou
2019-06-20 9:44 ` Prabhakar Kushwaha
2019-05-23 12:22 ` [U-Boot] [PATCHv2 2/8] powerpc: mpc85xx: Move CONFIG_FSL_PCIE_RESET " Z.q. Hou
` (6 subsequent siblings)
7 siblings, 1 reply; 29+ messages in thread
From: Z.q. Hou @ 2019-05-23 12:22 UTC (permalink / raw)
To: u-boot
From: Hou Zhiqiang <Zhiqiang.Hou@nxp.com>
Use the Kconfig option to select the PCIe ASPM errata.
Signed-off-by: Hou Zhiqiang <Zhiqiang.Hou@nxp.com>
---
V2:
- New patch.
arch/powerpc/cpu/mpc85xx/Kconfig | 8 ++++++++
arch/powerpc/include/asm/config_mpc85xx.h | 5 -----
2 files changed, 8 insertions(+), 5 deletions(-)
diff --git a/arch/powerpc/cpu/mpc85xx/Kconfig b/arch/powerpc/cpu/mpc85xx/Kconfig
index 0057f195b3..02bc1caa3f 100644
--- a/arch/powerpc/cpu/mpc85xx/Kconfig
+++ b/arch/powerpc/cpu/mpc85xx/Kconfig
@@ -702,6 +702,7 @@ config ARCH_P1011
select SYS_FSL_ERRATUM_A005125
select SYS_FSL_ERRATUM_ELBC_A001
select SYS_FSL_ERRATUM_ESDHC111
+ select FSL_PCIE_DISABLE_ASPM
select SYS_FSL_HAS_DDR3
select SYS_FSL_HAS_SEC
select SYS_FSL_SEC_BE
@@ -716,6 +717,7 @@ config ARCH_P1020
select SYS_FSL_ERRATUM_A005125
select SYS_FSL_ERRATUM_ELBC_A001
select SYS_FSL_ERRATUM_ESDHC111
+ select FSL_PCIE_DISABLE_ASPM
select SYS_FSL_HAS_DDR3
select SYS_FSL_HAS_SEC
select SYS_FSL_SEC_BE
@@ -735,6 +737,7 @@ config ARCH_P1021
select SYS_FSL_ERRATUM_A005125
select SYS_FSL_ERRATUM_ELBC_A001
select SYS_FSL_ERRATUM_ESDHC111
+ select FSL_PCIE_DISABLE_ASPM
select SYS_FSL_HAS_DDR3
select SYS_FSL_HAS_SEC
select SYS_FSL_SEC_BE
@@ -782,6 +785,7 @@ config ARCH_P1024
select SYS_FSL_ERRATUM_A005125
select SYS_FSL_ERRATUM_ELBC_A001
select SYS_FSL_ERRATUM_ESDHC111
+ select FSL_PCIE_DISABLE_ASPM
select SYS_FSL_HAS_DDR3
select SYS_FSL_HAS_SEC
select SYS_FSL_SEC_BE
@@ -802,6 +806,7 @@ config ARCH_P1025
select SYS_FSL_ERRATUM_A005125
select SYS_FSL_ERRATUM_ELBC_A001
select SYS_FSL_ERRATUM_ESDHC111
+ select FSL_PCIE_DISABLE_ASPM
select SYS_FSL_HAS_DDR3
select SYS_FSL_HAS_SEC
select SYS_FSL_SEC_BE
@@ -1431,6 +1436,9 @@ config SYS_P4080_ERRATUM_SERDES_A001
config SYS_P4080_ERRATUM_SERDES_A005
bool
+config FSL_PCIE_DISABLE_ASPM
+ bool
+
config SYS_FSL_QORIQ_CHASSIS1
bool
diff --git a/arch/powerpc/include/asm/config_mpc85xx.h b/arch/powerpc/include/asm/config_mpc85xx.h
index 7c963cdc35..946e74a93b 100644
--- a/arch/powerpc/include/asm/config_mpc85xx.h
+++ b/arch/powerpc/include/asm/config_mpc85xx.h
@@ -61,19 +61,16 @@
/* P1011 is single core version of P1020 */
#elif defined(CONFIG_ARCH_P1011)
#define CONFIG_TSECV2
-#define CONFIG_FSL_PCIE_DISABLE_ASPM
#define CONFIG_USB_MAX_CONTROLLER_COUNT 2
#elif defined(CONFIG_ARCH_P1020)
#define CONFIG_TSECV2
-#define CONFIG_FSL_PCIE_DISABLE_ASPM
#ifndef CONFIG_USB_MAX_CONTROLLER_COUNT
#define CONFIG_USB_MAX_CONTROLLER_COUNT 2
#endif
#elif defined(CONFIG_ARCH_P1021)
#define CONFIG_TSECV2
-#define CONFIG_FSL_PCIE_DISABLE_ASPM
#define QE_MURAM_SIZE 0x6000UL
#define MAX_QE_RISC 1
#define QE_NUM_OF_SNUM 28
@@ -95,14 +92,12 @@
/* P1024 is lower end variant of P1020 */
#elif defined(CONFIG_ARCH_P1024)
#define CONFIG_TSECV2
-#define CONFIG_FSL_PCIE_DISABLE_ASPM
#define CONFIG_USB_MAX_CONTROLLER_COUNT 2
/* P1025 is lower end variant of P1021 */
#elif defined(CONFIG_ARCH_P1025)
#define CONFIG_USB_MAX_CONTROLLER_COUNT 1
#define CONFIG_TSECV2
-#define CONFIG_FSL_PCIE_DISABLE_ASPM
#define QE_MURAM_SIZE 0x6000UL
#define MAX_QE_RISC 1
#define QE_NUM_OF_SNUM 28
--
2.17.1
^ permalink raw reply related [flat|nested] 29+ messages in thread
* [U-Boot] [PATCHv2 2/8] powerpc: mpc85xx: Move CONFIG_FSL_PCIE_RESET to Kconfig
2019-05-23 12:21 [U-Boot] [PATCHv2 0/8] dm: pci: add Freescale PowerPC PCIe driver Z.q. Hou
2019-05-23 12:22 ` [U-Boot] [PATCHv2 1/8] powerpc: mpc85xx: Move CONFIG_FSL_PCIE_DISABLE_ASPM to Kconfig Z.q. Hou
@ 2019-05-23 12:22 ` Z.q. Hou
2019-06-20 9:44 ` Prabhakar Kushwaha
2019-05-23 12:22 ` [U-Boot] [PATCHv2 3/8] powerpc: mpc85xx: Update the condition to compile PCI routines Z.q. Hou
` (5 subsequent siblings)
7 siblings, 1 reply; 29+ messages in thread
From: Z.q. Hou @ 2019-05-23 12:22 UTC (permalink / raw)
To: u-boot
From: Hou Zhiqiang <Zhiqiang.Hou@nxp.com>
Use the Kconfig option to select the PCIe reset errata.
Signed-off-by: Hou Zhiqiang <Zhiqiang.Hou@nxp.com>
---
V2:
- New patch.
arch/powerpc/cpu/mpc85xx/Kconfig | 21 +++++++++++++++++++++
include/configs/BSC9132QDS.h | 1 -
include/configs/C29XPCIE.h | 1 -
include/configs/MPC8536DS.h | 1 -
include/configs/MPC8544DS.h | 1 -
include/configs/MPC8548CDS.h | 1 -
include/configs/MPC8568MDS.h | 1 -
include/configs/MPC8569MDS.h | 1 -
include/configs/MPC8572DS.h | 1 -
include/configs/P1010RDB.h | 1 -
include/configs/P1022DS.h | 1 -
include/configs/P1023RDB.h | 1 -
include/configs/T208xQDS.h | 1 -
include/configs/T208xRDB.h | 1 -
include/configs/UCP1020.h | 1 -
include/configs/controlcenterd.h | 1 -
include/configs/p1_p2_rdb_pc.h | 1 -
include/configs/p1_twr.h | 1 -
include/configs/sbc8548.h | 1 -
include/configs/xpedite537x.h | 1 -
include/configs/xpedite550x.h | 1 -
21 files changed, 21 insertions(+), 20 deletions(-)
diff --git a/arch/powerpc/cpu/mpc85xx/Kconfig b/arch/powerpc/cpu/mpc85xx/Kconfig
index 02bc1caa3f..5c23efcf66 100644
--- a/arch/powerpc/cpu/mpc85xx/Kconfig
+++ b/arch/powerpc/cpu/mpc85xx/Kconfig
@@ -533,6 +533,7 @@ config ARCH_BSC9132
select SYS_FSL_ERRATUM_ESDHC111
select SYS_FSL_ERRATUM_I2C_A004447
select SYS_FSL_ERRATUM_IFC_A002769
+ select FSL_PCIE_RESET
select SYS_FSL_HAS_DDR3
select SYS_FSL_HAS_SEC
select SYS_FSL_SEC_BE
@@ -551,6 +552,7 @@ config ARCH_C29X
select SYS_FSL_DDR_VER_46
select SYS_FSL_ERRATUM_A005125
select SYS_FSL_ERRATUM_ESDHC111
+ select FSL_PCIE_RESET
select SYS_FSL_HAS_DDR3
select SYS_FSL_HAS_SEC
select SYS_FSL_SEC_BE
@@ -566,6 +568,7 @@ config ARCH_MPC8536
select FSL_LAW
select SYS_FSL_ERRATUM_A004508
select SYS_FSL_ERRATUM_A005125
+ select FSL_PCIE_RESET
select SYS_FSL_HAS_DDR2
select SYS_FSL_HAS_DDR3
select SYS_FSL_HAS_SEC
@@ -594,6 +597,7 @@ config ARCH_MPC8544
bool
select FSL_LAW
select SYS_FSL_ERRATUM_A005125
+ select FSL_PCIE_RESET
select SYS_FSL_HAS_DDR2
select SYS_FSL_HAS_SEC
select SYS_FSL_SEC_BE
@@ -609,6 +613,7 @@ config ARCH_MPC8548
select SYS_FSL_ERRATUM_NMG_LBC103
select SYS_FSL_ERRATUM_NMG_ETSEC129
select SYS_FSL_ERRATUM_I2C_A004447
+ select FSL_PCIE_RESET
select SYS_FSL_HAS_DDR2
select SYS_FSL_HAS_DDR1
select SYS_FSL_HAS_SEC
@@ -633,6 +638,7 @@ config ARCH_MPC8560
config ARCH_MPC8568
bool
select FSL_LAW
+ select FSL_PCIE_RESET
select SYS_FSL_HAS_DDR2
select SYS_FSL_HAS_SEC
select SYS_FSL_SEC_BE
@@ -643,6 +649,7 @@ config ARCH_MPC8569
select FSL_LAW
select SYS_FSL_ERRATUM_A004508
select SYS_FSL_ERRATUM_A005125
+ select FSL_PCIE_RESET
select SYS_FSL_HAS_DDR3
select SYS_FSL_HAS_SEC
select SYS_FSL_SEC_BE
@@ -657,6 +664,7 @@ config ARCH_MPC8572
select SYS_FSL_ERRATUM_A005125
select SYS_FSL_ERRATUM_DDR_115
select SYS_FSL_ERRATUM_DDR111_DDR134
+ select FSL_PCIE_RESET
select SYS_FSL_HAS_DDR2
select SYS_FSL_HAS_DDR3
select SYS_FSL_HAS_SEC
@@ -681,6 +689,7 @@ config ARCH_P1010
select SYS_FSL_ERRATUM_P1010_A003549
select SYS_FSL_ERRATUM_SEC_A003571
select SYS_FSL_ERRATUM_IFC_A003399
+ select FSL_PCIE_RESET
select SYS_FSL_HAS_DDR3
select SYS_FSL_HAS_SEC
select SYS_FSL_SEC_BE
@@ -718,6 +727,7 @@ config ARCH_P1020
select SYS_FSL_ERRATUM_ELBC_A001
select SYS_FSL_ERRATUM_ESDHC111
select FSL_PCIE_DISABLE_ASPM
+ select FSL_PCIE_RESET
select SYS_FSL_HAS_DDR3
select SYS_FSL_HAS_SEC
select SYS_FSL_SEC_BE
@@ -738,6 +748,7 @@ config ARCH_P1021
select SYS_FSL_ERRATUM_ELBC_A001
select SYS_FSL_ERRATUM_ESDHC111
select FSL_PCIE_DISABLE_ASPM
+ select FSL_PCIE_RESET
select SYS_FSL_HAS_DDR3
select SYS_FSL_HAS_SEC
select SYS_FSL_SEC_BE
@@ -759,6 +770,7 @@ config ARCH_P1022
select SYS_FSL_ERRATUM_ELBC_A001
select SYS_FSL_ERRATUM_ESDHC111
select SYS_FSL_ERRATUM_SATA_A001
+ select FSL_PCIE_RESET
select SYS_FSL_HAS_DDR3
select SYS_FSL_HAS_SEC
select SYS_FSL_SEC_BE
@@ -772,6 +784,7 @@ config ARCH_P1023
select SYS_FSL_ERRATUM_A004508
select SYS_FSL_ERRATUM_A005125
select SYS_FSL_ERRATUM_I2C_A004447
+ select FSL_PCIE_RESET
select SYS_FSL_HAS_DDR3
select SYS_FSL_HAS_SEC
select SYS_FSL_SEC_BE
@@ -786,6 +799,7 @@ config ARCH_P1024
select SYS_FSL_ERRATUM_ELBC_A001
select SYS_FSL_ERRATUM_ESDHC111
select FSL_PCIE_DISABLE_ASPM
+ select FSL_PCIE_RESET
select SYS_FSL_HAS_DDR3
select SYS_FSL_HAS_SEC
select SYS_FSL_SEC_BE
@@ -807,6 +821,7 @@ config ARCH_P1025
select SYS_FSL_ERRATUM_ELBC_A001
select SYS_FSL_ERRATUM_ESDHC111
select FSL_PCIE_DISABLE_ASPM
+ select FSL_PCIE_RESET
select SYS_FSL_HAS_DDR3
select SYS_FSL_HAS_SEC
select SYS_FSL_SEC_BE
@@ -824,6 +839,7 @@ config ARCH_P2020
select SYS_FSL_ERRATUM_A005125
select SYS_FSL_ERRATUM_ESDHC111
select SYS_FSL_ERRATUM_ESDHC_A001
+ select FSL_PCIE_RESET
select SYS_FSL_HAS_DDR3
select SYS_FSL_HAS_SEC
select SYS_FSL_SEC_BE
@@ -1079,6 +1095,7 @@ config ARCH_T2080
select SYS_FSL_ERRATUM_A007907
select SYS_FSL_ERRATUM_A009942
select SYS_FSL_ERRATUM_ESDHC111
+ select FSL_PCIE_RESET
select SYS_FSL_HAS_DDR3
select SYS_FSL_HAS_SEC
select SYS_FSL_QORIQ_CHASSIS2
@@ -1103,6 +1120,7 @@ config ARCH_T2081
select SYS_FSL_ERRATUM_A007212
select SYS_FSL_ERRATUM_A009942
select SYS_FSL_ERRATUM_ESDHC111
+ select FSL_PCIE_RESET
select SYS_FSL_HAS_DDR3
select SYS_FSL_HAS_SEC
select SYS_FSL_QORIQ_CHASSIS2
@@ -1439,6 +1457,9 @@ config SYS_P4080_ERRATUM_SERDES_A005
config FSL_PCIE_DISABLE_ASPM
bool
+config FSL_PCIE_RESET
+ bool
+
config SYS_FSL_QORIQ_CHASSIS1
bool
diff --git a/include/configs/BSC9132QDS.h b/include/configs/BSC9132QDS.h
index f385509daf..3522f03009 100644
--- a/include/configs/BSC9132QDS.h
+++ b/include/configs/BSC9132QDS.h
@@ -60,7 +60,6 @@
#define CONFIG_PCIE1 /* PCIE controller 1 (slot 1) */
#define CONFIG_FSL_PCI_INIT /* Use common FSL init code */
#define CONFIG_PCI_INDIRECT_BRIDGE /* indirect PCI bridge support */
-#define CONFIG_FSL_PCIE_RESET /* need PCIe reset errata */
#define CONFIG_SYS_PCI_64BIT /* enable 64-bit PCI resources */
/*
diff --git a/include/configs/C29XPCIE.h b/include/configs/C29XPCIE.h
index ebf44b5529..f9b8e446c1 100644
--- a/include/configs/C29XPCIE.h
+++ b/include/configs/C29XPCIE.h
@@ -69,7 +69,6 @@
#define CONFIG_PCIE1 /* PCIE controller 1 (slot 1) */
#define CONFIG_FSL_PCI_INIT /* Use common FSL init code */
#define CONFIG_PCI_INDIRECT_BRIDGE
-#define CONFIG_FSL_PCIE_RESET /* need PCIe reset errata */
#define CONFIG_SYS_PCI_64BIT /* enable 64-bit PCI resources */
/*
diff --git a/include/configs/MPC8536DS.h b/include/configs/MPC8536DS.h
index 1413b3dcfe..bea4021793 100644
--- a/include/configs/MPC8536DS.h
+++ b/include/configs/MPC8536DS.h
@@ -36,7 +36,6 @@
#define CONFIG_PCIE3 1 /* PCIE controller 3 (ULI bridge) */
#define CONFIG_FSL_PCI_INIT 1 /* Use common FSL init code */
#define CONFIG_PCI_INDIRECT_BRIDGE 1 /* indirect PCI bridge support */
-#define CONFIG_FSL_PCIE_RESET 1 /* need PCIe reset errata */
#define CONFIG_SYS_PCI_64BIT 1 /* enable 64-bit PCI resources */
diff --git a/include/configs/MPC8544DS.h b/include/configs/MPC8544DS.h
index 280b873aee..2cbe855235 100644
--- a/include/configs/MPC8544DS.h
+++ b/include/configs/MPC8544DS.h
@@ -16,7 +16,6 @@
#define CONFIG_PCIE3 1 /* PCIE controller 3 (ULI bridge) */
#define CONFIG_FSL_PCI_INIT 1 /* Use common FSL init code */
#define CONFIG_PCI_INDIRECT_BRIDGE 1 /* indirect PCI bridge support */
-#define CONFIG_FSL_PCIE_RESET 1 /* need PCIe reset errata */
#define CONFIG_SYS_PCI_64BIT 1 /* enable 64-bit PCI resources */
#define CONFIG_ENV_OVERWRITE
diff --git a/include/configs/MPC8548CDS.h b/include/configs/MPC8548CDS.h
index be600becfe..b37601c794 100644
--- a/include/configs/MPC8548CDS.h
+++ b/include/configs/MPC8548CDS.h
@@ -20,7 +20,6 @@
#undef CONFIG_PCI2
#define CONFIG_FSL_PCI_INIT 1 /* Use common FSL init code */
#define CONFIG_PCI_INDIRECT_BRIDGE 1 /* indirect PCI bridge support */
-#define CONFIG_FSL_PCIE_RESET 1 /* need PCIe reset errata */
#define CONFIG_SYS_PCI_64BIT 1 /* enable 64-bit PCI resources */
#define CONFIG_ENV_OVERWRITE
diff --git a/include/configs/MPC8568MDS.h b/include/configs/MPC8568MDS.h
index 9b3485ed4b..bf27427869 100644
--- a/include/configs/MPC8568MDS.h
+++ b/include/configs/MPC8568MDS.h
@@ -16,7 +16,6 @@
#define CONFIG_PCIE1 1 /* PCIE controller */
#define CONFIG_FSL_PCI_INIT 1 /* use common fsl pci init code */
#define CONFIG_PCI_INDIRECT_BRIDGE 1 /* indirect PCI bridge support */
-#define CONFIG_FSL_PCIE_RESET 1 /* need PCIe reset errata */
#define CONFIG_SYS_PCI_64BIT 1 /* enable 64-bit PCI resources */
#define CONFIG_QE /* Enable QE */
#define CONFIG_ENV_OVERWRITE
diff --git a/include/configs/MPC8569MDS.h b/include/configs/MPC8569MDS.h
index de5a7ca959..e1f29bf107 100644
--- a/include/configs/MPC8569MDS.h
+++ b/include/configs/MPC8569MDS.h
@@ -15,7 +15,6 @@
#define CONFIG_PCIE1 1 /* PCIE controller */
#define CONFIG_FSL_PCI_INIT 1 /* use common fsl pci init code */
#define CONFIG_PCI_INDIRECT_BRIDGE 1 /* indirect PCI bridge support */
-#define CONFIG_FSL_PCIE_RESET 1 /* need PCIe reset errata */
#define CONFIG_SYS_PCI_64BIT 1 /* enable 64-bit PCI resources */
#define CONFIG_QE /* Enable QE */
#define CONFIG_ENV_OVERWRITE
diff --git a/include/configs/MPC8572DS.h b/include/configs/MPC8572DS.h
index 13fbbb3044..e3952f423b 100644
--- a/include/configs/MPC8572DS.h
+++ b/include/configs/MPC8572DS.h
@@ -27,7 +27,6 @@
#define CONFIG_PCIE3 1 /* PCIE controller 3 (ULI bridge) */
#define CONFIG_FSL_PCI_INIT 1 /* Use common FSL init code */
#define CONFIG_PCI_INDIRECT_BRIDGE 1 /* indirect PCI bridge support */
-#define CONFIG_FSL_PCIE_RESET 1 /* need PCIe reset errata */
#define CONFIG_SYS_PCI_64BIT 1 /* enable 64-bit PCI resources */
#define CONFIG_ENV_OVERWRITE
diff --git a/include/configs/P1010RDB.h b/include/configs/P1010RDB.h
index 134ffe5271..88470f69b8 100644
--- a/include/configs/P1010RDB.h
+++ b/include/configs/P1010RDB.h
@@ -127,7 +127,6 @@
#define CONFIG_PCIE2 /* PCIE controller 2 (slot 2) */
#define CONFIG_FSL_PCI_INIT /* Use common FSL init code */
#define CONFIG_PCI_INDIRECT_BRIDGE /* indirect PCI bridge support */
-#define CONFIG_FSL_PCIE_RESET /* need PCIe reset errata */
#define CONFIG_SYS_PCI_64BIT /* enable 64-bit PCI resources */
/*
diff --git a/include/configs/P1022DS.h b/include/configs/P1022DS.h
index 3ccfeca890..e785826754 100644
--- a/include/configs/P1022DS.h
+++ b/include/configs/P1022DS.h
@@ -90,7 +90,6 @@
#define CONFIG_PCIE2 /* PCIE controller 2 (slot 2) */
#define CONFIG_PCIE3 /* PCIE controller 3 (ULI bridge) */
#define CONFIG_FSL_PCI_INIT /* Use common FSL init code */
-#define CONFIG_FSL_PCIE_RESET /* need PCIe reset errata */
#define CONFIG_SYS_PCI_64BIT /* enable 64-bit PCI resources */
#define CONFIG_ENABLE_36BIT_PHYS
diff --git a/include/configs/P1023RDB.h b/include/configs/P1023RDB.h
index 4f6ee22385..1a8b450adc 100644
--- a/include/configs/P1023RDB.h
+++ b/include/configs/P1023RDB.h
@@ -24,7 +24,6 @@
#define CONFIG_PCIE2 /* PCIE controller 2 (slot 2) */
#define CONFIG_PCIE3 /* PCIE controller 3 (slot 3) */
#define CONFIG_FSL_PCI_INIT /* Use common FSL init code */
-#define CONFIG_FSL_PCIE_RESET /* need PCIe reset errata */
#define CONFIG_SYS_PCI_64BIT /* enable 64-bit PCI resources */
#ifndef __ASSEMBLY__
diff --git a/include/configs/T208xQDS.h b/include/configs/T208xQDS.h
index 9ca384cc0c..7a07679069 100644
--- a/include/configs/T208xQDS.h
+++ b/include/configs/T208xQDS.h
@@ -507,7 +507,6 @@ unsigned long get_board_ddr_clk(void);
#define CONFIG_PCIE2 /* PCIE controller 2 */
#define CONFIG_PCIE3 /* PCIE controller 3 */
#define CONFIG_PCIE4 /* PCIE controller 4 */
-#define CONFIG_FSL_PCIE_RESET /* pcie reset fix link width 2x-4x*/
#define CONFIG_FSL_PCI_INIT /* Use common FSL init code */
#define CONFIG_SYS_PCI_64BIT /* enable 64-bit PCI resources */
/* controller 1, direct to uli, tgtid 3, Base address 20000 */
diff --git a/include/configs/T208xRDB.h b/include/configs/T208xRDB.h
index 446e4268ef..77197a1750 100644
--- a/include/configs/T208xRDB.h
+++ b/include/configs/T208xRDB.h
@@ -491,7 +491,6 @@ unsigned long get_board_ddr_clk(void);
#ifdef CONFIG_PCI
#define CONFIG_PCI_INDIRECT_BRIDGE
-#define CONFIG_FSL_PCIE_RESET /* need PCIe reset errata LSZ ADD */
#define CONFIG_PCI_SCAN_SHOW /* show pci devices on startup */
#endif
diff --git a/include/configs/UCP1020.h b/include/configs/UCP1020.h
index 6a0254a55b..88237ee316 100644
--- a/include/configs/UCP1020.h
+++ b/include/configs/UCP1020.h
@@ -17,7 +17,6 @@
#define CONFIG_PCIE2 /* PCIE controller 2 (slot 2) */
#define CONFIG_FSL_PCI_INIT /* Use common FSL init code */
#define CONFIG_PCI_INDIRECT_BRIDGE /* indirect PCI bridge support */
-#define CONFIG_FSL_PCIE_RESET /* need PCIe reset errata */
#define CONFIG_SYS_PCI_64BIT /* enable 64-bit PCI resources */
#if defined(CONFIG_TARTGET_UCP1020T1)
diff --git a/include/configs/controlcenterd.h b/include/configs/controlcenterd.h
index 1908d35bcc..6d616c3773 100644
--- a/include/configs/controlcenterd.h
+++ b/include/configs/controlcenterd.h
@@ -210,7 +210,6 @@
#define CONFIG_SYS_PCI_64BIT /* enable 64-bit PCI resources */
#define CONFIG_FSL_PCI_INIT /* Use common FSL init code */
-#define CONFIG_FSL_PCIE_RESET /* need PCIe reset errata */
#define CONFIG_SYS_PCIE1_MEM_VIRT 0xc0000000
#ifdef CONFIG_PHYS_64BIT
diff --git a/include/configs/p1_p2_rdb_pc.h b/include/configs/p1_p2_rdb_pc.h
index 8fda0c1e22..513f6e4767 100644
--- a/include/configs/p1_p2_rdb_pc.h
+++ b/include/configs/p1_p2_rdb_pc.h
@@ -220,7 +220,6 @@
#define CONFIG_PCIE2 /* PCIE controller 2 (slot 2) */
#define CONFIG_FSL_PCI_INIT /* Use common FSL init code */
#define CONFIG_PCI_INDIRECT_BRIDGE /* indirect PCI bridge support */
-#define CONFIG_FSL_PCIE_RESET /* need PCIe reset errata */
#define CONFIG_SYS_PCI_64BIT /* enable 64-bit PCI resources */
#define CONFIG_ENV_OVERWRITE
diff --git a/include/configs/p1_twr.h b/include/configs/p1_twr.h
index 4f48370648..ad1637b5fc 100644
--- a/include/configs/p1_twr.h
+++ b/include/configs/p1_twr.h
@@ -35,7 +35,6 @@
#define CONFIG_PCIE2 /* PCIE controller 2 (slot 2) */
#define CONFIG_FSL_PCI_INIT /* Use common FSL init code */
#define CONFIG_PCI_INDIRECT_BRIDGE /* indirect PCI bridge support */
-#define CONFIG_FSL_PCIE_RESET /* need PCIe reset errata */
#define CONFIG_SYS_PCI_64BIT /* enable 64-bit PCI resources */
#define CONFIG_ENV_OVERWRITE
diff --git a/include/configs/sbc8548.h b/include/configs/sbc8548.h
index 9df8604af7..ba613672eb 100644
--- a/include/configs/sbc8548.h
+++ b/include/configs/sbc8548.h
@@ -49,7 +49,6 @@
#define CONFIG_SYS_PCI_64BIT 1 /* enable 64-bit PCI resources */
#endif
#ifdef CONFIG_PCIE1
-#define CONFIG_FSL_PCIE_RESET 1 /* need PCIe reset errata */
#endif
#define CONFIG_ENV_OVERWRITE
diff --git a/include/configs/xpedite537x.h b/include/configs/xpedite537x.h
index 22dd3c036e..0a87f226f8 100644
--- a/include/configs/xpedite537x.h
+++ b/include/configs/xpedite537x.h
@@ -22,7 +22,6 @@
#define CONFIG_FSL_PCI_INIT 1 /* Use common FSL init code */
#define CONFIG_PCI_INDIRECT_BRIDGE 1 /* indirect PCI bridge support */
#define CONFIG_SYS_PCI_64BIT 1 /* enable 64-bit PCI resources */
-#define CONFIG_FSL_PCIE_RESET 1 /* need PCIe reset errata */
/*
* Multicore config
diff --git a/include/configs/xpedite550x.h b/include/configs/xpedite550x.h
index a7c8dc4e33..0389874609 100644
--- a/include/configs/xpedite550x.h
+++ b/include/configs/xpedite550x.h
@@ -22,7 +22,6 @@
#define CONFIG_FSL_PCI_INIT 1 /* Use common FSL init code */
#define CONFIG_PCI_INDIRECT_BRIDGE 1 /* indirect PCI bridge support */
#define CONFIG_SYS_PCI_64BIT 1 /* enable 64-bit PCI resources */
-#define CONFIG_FSL_PCIE_RESET 1 /* need PCIe reset errata */
/*
* Multicore config
--
2.17.1
^ permalink raw reply related [flat|nested] 29+ messages in thread
* [U-Boot] [PATCHv2 3/8] powerpc: mpc85xx: Update the condition to compile PCI routines
2019-05-23 12:21 [U-Boot] [PATCHv2 0/8] dm: pci: add Freescale PowerPC PCIe driver Z.q. Hou
2019-05-23 12:22 ` [U-Boot] [PATCHv2 1/8] powerpc: mpc85xx: Move CONFIG_FSL_PCIE_DISABLE_ASPM to Kconfig Z.q. Hou
2019-05-23 12:22 ` [U-Boot] [PATCHv2 2/8] powerpc: mpc85xx: Move CONFIG_FSL_PCIE_RESET " Z.q. Hou
@ 2019-05-23 12:22 ` Z.q. Hou
2019-06-20 9:44 ` Prabhakar Kushwaha
2019-05-23 12:22 ` [U-Boot] [PATCHv2 4/8] powerpc: T208xQDS: Compile the legacy PCIe routines conditionally Z.q. Hou
` (4 subsequent siblings)
7 siblings, 1 reply; 29+ messages in thread
From: Z.q. Hou @ 2019-05-23 12:22 UTC (permalink / raw)
To: u-boot
From: Hou Zhiqiang <Zhiqiang.Hou@nxp.com>
Compile the routines of mpc85xx/pci.c when both FSL_PCI_INIT
and DM_PCI are not enabled.
Signed-off-by: Hou Zhiqiang <Zhiqiang.Hou@nxp.com>
Reviewed-by: Simon Glass <sjg@chromium.org>
---
V2:
- No change.
arch/powerpc/cpu/mpc85xx/pci.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/powerpc/cpu/mpc85xx/pci.c b/arch/powerpc/cpu/mpc85xx/pci.c
index 90ccc3427c..1d0213a513 100644
--- a/arch/powerpc/cpu/mpc85xx/pci.c
+++ b/arch/powerpc/cpu/mpc85xx/pci.c
@@ -12,7 +12,7 @@
#include <asm/cpm_85xx.h>
#include <pci.h>
-#if !defined(CONFIG_FSL_PCI_INIT)
+#if !defined(CONFIG_FSL_PCI_INIT) && !defined(CONFIG_DM_PCI)
#ifndef CONFIG_SYS_PCI1_MEM_BUS
#define CONFIG_SYS_PCI1_MEM_BUS CONFIG_SYS_PCI1_MEM_BASE
--
2.17.1
^ permalink raw reply related [flat|nested] 29+ messages in thread
* [U-Boot] [PATCHv2 4/8] powerpc: T208xQDS: Compile the legacy PCIe routines conditionally
2019-05-23 12:21 [U-Boot] [PATCHv2 0/8] dm: pci: add Freescale PowerPC PCIe driver Z.q. Hou
` (2 preceding siblings ...)
2019-05-23 12:22 ` [U-Boot] [PATCHv2 3/8] powerpc: mpc85xx: Update the condition to compile PCI routines Z.q. Hou
@ 2019-05-23 12:22 ` Z.q. Hou
2019-06-20 9:44 ` Prabhakar Kushwaha
2019-05-23 12:22 ` [U-Boot] [PATCHv2 5/8] t2080: dts: Added PCIe DT nodes Z.q. Hou
` (3 subsequent siblings)
7 siblings, 1 reply; 29+ messages in thread
From: Z.q. Hou @ 2019-05-23 12:22 UTC (permalink / raw)
To: u-boot
From: Hou Zhiqiang <Zhiqiang.Hou@nxp.com>
Compile the legacy PCIe initialization reoutines only when DM_PCI
is not enabled.
Signed-off-by: Hou Zhiqiang <Zhiqiang.Hou@nxp.com>
Reviewed-by: Simon Glass <sjg@chromium.org>
---
V2:
- No change.
board/freescale/t208xqds/pci.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/board/freescale/t208xqds/pci.c b/board/freescale/t208xqds/pci.c
index c761aea0ac..ef26f14c46 100644
--- a/board/freescale/t208xqds/pci.c
+++ b/board/freescale/t208xqds/pci.c
@@ -11,6 +11,7 @@
#include <fdt_support.h>
#include <asm/fsl_serdes.h>
+#if !defined(CONFIG_DM_PCI)
void pci_init_board(void)
{
fsl_pcie_init_board(0);
@@ -20,3 +21,4 @@ void pci_of_setup(void *blob, bd_t *bd)
{
FT_FSL_PCI_SETUP;
}
+#endif
--
2.17.1
^ permalink raw reply related [flat|nested] 29+ messages in thread
* [U-Boot] [PATCHv2 5/8] t2080: dts: Added PCIe DT nodes
2019-05-23 12:21 [U-Boot] [PATCHv2 0/8] dm: pci: add Freescale PowerPC PCIe driver Z.q. Hou
` (3 preceding siblings ...)
2019-05-23 12:22 ` [U-Boot] [PATCHv2 4/8] powerpc: T208xQDS: Compile the legacy PCIe routines conditionally Z.q. Hou
@ 2019-05-23 12:22 ` Z.q. Hou
2019-06-20 9:44 ` Prabhakar Kushwaha
2019-05-23 12:22 ` [U-Boot] [PATCHv2 6/8] dm: pci: add Freescale PowerPC PCIe driver Z.q. Hou
` (2 subsequent siblings)
7 siblings, 1 reply; 29+ messages in thread
From: Z.q. Hou @ 2019-05-23 12:22 UTC (permalink / raw)
To: u-boot
From: Hou Zhiqiang <Zhiqiang.Hou@nxp.com>
T2080 integrated 4 PCIe controllers, which is compatible with
the PCI Expressâ„¢ Base Specification, Revision 3.0, and this
patch is to add DT node for each PCIe controller.
Signed-off-by: Hou Zhiqiang <Zhiqiang.Hou@nxp.com>
Reviewed-by: Simon Glass <sjg@chromium.org>
---
V2:
- No change.
arch/powerpc/dts/t2080.dtsi | 48 +++++++++++++++++++++++++++++++++++++
1 file changed, 48 insertions(+)
diff --git a/arch/powerpc/dts/t2080.dtsi b/arch/powerpc/dts/t2080.dtsi
index db65ea5725..89b2c618ae 100644
--- a/arch/powerpc/dts/t2080.dtsi
+++ b/arch/powerpc/dts/t2080.dtsi
@@ -59,4 +59,52 @@
clock-frequency = <0x0>;
};
};
+
+ pcie at ffe240000 {
+ compatible = "fsl,pcie-t2080", "fsl,pcie-fsl-qoriq";
+ reg = <0xf 0xfe240000 0x0 0x4000>; /* registers */
+ law_trgt_if = <0>;
+ #address-cells = <3>;
+ #size-cells = <2>;
+ device_type = "pci";
+ bus-range = <0x0 0xff>;
+ ranges = <0x01000000 0x0 0x00000000 0xf 0xf8000000 0x0 0x00010000 /* downstream I/O */
+ 0x02000000 0x0 0xe0000000 0xc 0x00000000 0x0 0x20000000>; /* non-prefetchable memory */
+ };
+
+ pcie at ffe250000 {
+ compatible = "fsl,pcie-t2080", "fsl,pcie-fsl-qoriq";
+ reg = <0xf 0xfe250000 0x0 0x1000>; /* registers */
+ law_trgt_if = <1>;
+ #address-cells = <3>;
+ #size-cells = <2>;
+ device_type = "pci";
+ bus-range = <0x0 0xff>;
+ ranges = <0x01000000 0x0 0x00000000 0xf 0xf8010000 0x0 0x00010000 /* downstream I/O */
+ 0x02000000 0x0 0xe0000000 0xc 0x20000000 0x0 0x10000000>; /* non-prefetchable memory */
+ };
+
+ pcie at ffe260000 {
+ compatible = "fsl,pcie-t2080", "fsl,pcie-fsl-qoriq";
+ reg = <0xf 0xfe260000 0x0 0x1000>; /* registers */
+ law_trgt_if = <2>;
+ #address-cells = <3>;
+ #size-cells = <2>;
+ device_type = "pci";
+ bus-range = <0x0 0xff>;
+ ranges = <0x01000000 0x0 0x00000000 0xf 0xf8020000 0x0 0x00010000 /* downstream I/O */
+ 0x02000000 0x0 0xe0000000 0xc 0x30000000 0x0 0x10000000>; /* non-prefetchable memory */
+ };
+
+ pcie at ffe270000 {
+ compatible = "fsl,pcie-t2080", "fsl,pcie-fsl-qoriq";
+ reg = <0xf 0xfe270000 0x0 0x1000>; /* registers */
+ law_trgt_if = <3>;
+ #address-cells = <3>;
+ #size-cells = <2>;
+ device_type = "pci";
+ bus-range = <0x0 0xff>;
+ ranges = <0x01000000 0x0 0x00000000 0xf 0xf8030000 0x0 0x00010000 /* downstream I/O */
+ 0x02000000 0x0 0xe0000000 0xc 0x40000000 0x0 0x10000000>; /* non-prefetchable memory */
+ };
};
--
2.17.1
^ permalink raw reply related [flat|nested] 29+ messages in thread
* [U-Boot] [PATCHv2 6/8] dm: pci: add Freescale PowerPC PCIe driver
2019-05-23 12:21 [U-Boot] [PATCHv2 0/8] dm: pci: add Freescale PowerPC PCIe driver Z.q. Hou
` (4 preceding siblings ...)
2019-05-23 12:22 ` [U-Boot] [PATCHv2 5/8] t2080: dts: Added PCIe DT nodes Z.q. Hou
@ 2019-05-23 12:22 ` Z.q. Hou
2019-06-20 9:44 ` Prabhakar Kushwaha
2019-05-23 12:22 ` [U-Boot] [PATCHv2 7/8] powerpc: T208xQDS: Disable legacy " Z.q. Hou
2019-05-23 12:22 ` [U-Boot] [PATCHv2 8/8] configs: T2080QDS: Enable " Z.q. Hou
7 siblings, 1 reply; 29+ messages in thread
From: Z.q. Hou @ 2019-05-23 12:22 UTC (permalink / raw)
To: u-boot
From: Hou Zhiqiang <Zhiqiang.Hou@nxp.com>
Add PCIe DM driver for Freescale PowerPC PCIe controllers.
Signed-off-by: Hou Zhiqiang <Zhiqiang.Hou@nxp.com>
---
V2:
- Add more detail in the PCIE_FSL help info.
- Sort the included header files.
- Romve key word 'inline' from fsl_pcie_link_up().
- Add comments for some members of structure fsl_pcie.
- Change to determine erratums by IS_ENABLED().
- Romve unnecessary header file including from pcie_fsl.h.
- Add info when can't find the DT node to fixup.
- To avoid compile error, add a null function pci_of_setup()
when CONFIG_OF_BOARD_SETUP is not enabled.
drivers/pci/Kconfig | 8 +
drivers/pci/Makefile | 1 +
drivers/pci/pcie_fsl.c | 611 +++++++++++++++++++++++++++++++++++
drivers/pci/pcie_fsl.h | 57 ++++
drivers/pci/pcie_fsl_fixup.c | 51 +++
5 files changed, 728 insertions(+)
create mode 100644 drivers/pci/pcie_fsl.c
create mode 100644 drivers/pci/pcie_fsl.h
create mode 100644 drivers/pci/pcie_fsl_fixup.c
diff --git a/drivers/pci/Kconfig b/drivers/pci/Kconfig
index 1521885bde..e679aa1a93 100644
--- a/drivers/pci/Kconfig
+++ b/drivers/pci/Kconfig
@@ -60,6 +60,14 @@ config PCIE_DW_MVEBU
Armada-8K SoCs. The PCIe controller on Armada-8K is based on
DesignWare hardware.
+config PCIE_FSL
+ bool "FSL PowerPC PCIe support"
+ depends on DM_PCI
+ help
+ Say Y here if you want to enable PCIe controller support on FSL
+ PowerPC MPC85xx, MPC86xx, B series, P series and T series SoCs.
+ This driver does not support SRIO_PCIE_BOOT feature.
+
config PCI_RCAR_GEN2
bool "Renesas RCar Gen2 PCIe driver"
depends on DM_PCI
diff --git a/drivers/pci/Makefile b/drivers/pci/Makefile
index 4923641895..d984848266 100644
--- a/drivers/pci/Makefile
+++ b/drivers/pci/Makefile
@@ -30,6 +30,7 @@ obj-$(CONFIG_SH7780_PCI) +=pci_sh7780.o
obj-$(CONFIG_PCI_TEGRA) += pci_tegra.o
obj-$(CONFIG_PCI_AARDVARK) += pci-aardvark.o
obj-$(CONFIG_PCIE_DW_MVEBU) += pcie_dw_mvebu.o
+obj-$(CONFIG_PCIE_FSL) += pcie_fsl.o pcie_fsl_fixup.o
obj-$(CONFIG_PCIE_LAYERSCAPE) += pcie_layerscape.o
obj-$(CONFIG_PCIE_LAYERSCAPE) += pcie_layerscape_fixup.o
obj-$(CONFIG_PCI_XILINX) += pcie_xilinx.o
diff --git a/drivers/pci/pcie_fsl.c b/drivers/pci/pcie_fsl.c
new file mode 100644
index 0000000000..bfb207e788
--- /dev/null
+++ b/drivers/pci/pcie_fsl.c
@@ -0,0 +1,611 @@
+// SPDX-License-Identifier: GPL-2.0+ OR X11
+/*
+ * Copyright 2019 NXP
+ * Copyright 2007-2012 Freescale Semiconductor, Inc.
+ *
+ * PCIe DM U-Boot driver for Freescale PowerPC SoCs
+ * Author: Hou Zhiqiang <Zhiqiang.Hou@nxp.com>
+ */
+
+#include <common.h>
+#include <dm.h>
+#include <malloc.h>
+#include <mapmem.h>
+#include <pci.h>
+#include <asm/fsl_pci.h>
+#include <asm/fsl_serdes.h>
+#include <asm/io.h>
+#include "pcie_fsl.h"
+
+LIST_HEAD(fsl_pcie_list);
+
+static int fsl_pcie_link_up(struct fsl_pcie *pcie);
+
+static int fsl_pcie_addr_valid(struct fsl_pcie *pcie, pci_dev_t bdf)
+{
+ struct udevice *bus = pcie->bus;
+
+ if (!pcie->enabled)
+ return -ENXIO;
+
+ if (PCI_BUS(bdf) < bus->seq)
+ return -EINVAL;
+
+ if (PCI_BUS(bdf) > bus->seq && (!fsl_pcie_link_up(pcie) || pcie->mode))
+ return -EINVAL;
+
+ if (PCI_BUS(bdf) == bus->seq && (PCI_DEV(bdf) > 0 || PCI_FUNC(bdf) > 0))
+ return -EINVAL;
+
+ if (PCI_BUS(bdf) == (bus->seq + 1) && (PCI_DEV(bdf) > 0))
+ return -EINVAL;
+
+ return 0;
+}
+
+static int fsl_pcie_read_config(struct udevice *bus, pci_dev_t bdf,
+ uint offset, ulong *valuep,
+ enum pci_size_t size)
+{
+ struct fsl_pcie *pcie = dev_get_priv(bus);
+ ccsr_fsl_pci_t *regs = pcie->regs;
+ u32 val;
+
+ if (fsl_pcie_addr_valid(pcie, bdf)) {
+ *valuep = pci_get_ff(size);
+ return 0;
+ }
+
+ bdf = bdf - PCI_BDF(bus->seq, 0, 0);
+ val = bdf | (offset & 0xfc) | ((offset & 0xf00) << 16) | 0x80000000;
+ out_be32(®s->cfg_addr, val);
+
+ sync();
+
+ switch (size) {
+ case PCI_SIZE_8:
+ *valuep = in_8((u8 *)®s->cfg_data + (offset & 3));
+ break;
+ case PCI_SIZE_16:
+ *valuep = in_le16((u16 *)((u8 *)®s->cfg_data +
+ (offset & 2)));
+ break;
+ case PCI_SIZE_32:
+ *valuep = in_le32(®s->cfg_data);
+ break;
+ }
+
+ return 0;
+}
+
+static int fsl_pcie_write_config(struct udevice *bus, pci_dev_t bdf,
+ uint offset, ulong value,
+ enum pci_size_t size)
+{
+ struct fsl_pcie *pcie = dev_get_priv(bus);
+ ccsr_fsl_pci_t *regs = pcie->regs;
+ u32 val;
+ u8 val_8;
+ u16 val_16;
+ u32 val_32;
+
+ if (fsl_pcie_addr_valid(pcie, bdf))
+ return 0;
+
+ bdf = bdf - PCI_BDF(bus->seq, 0, 0);
+ val = bdf | (offset & 0xfc) | ((offset & 0xf00) << 16) | 0x80000000;
+ out_be32(®s->cfg_addr, val);
+
+ sync();
+
+ switch (size) {
+ case PCI_SIZE_8:
+ val_8 = value;
+ out_8((u8 *)®s->cfg_data + (offset & 3), val_8);
+ break;
+ case PCI_SIZE_16:
+ val_16 = value;
+ out_le16((u16 *)((u8 *)®s->cfg_data + (offset & 2)), val_16);
+ break;
+ case PCI_SIZE_32:
+ val_32 = value;
+ out_le32(®s->cfg_data, val_32);
+ break;
+ }
+
+ return 0;
+}
+
+static int fsl_pcie_hose_read_config(struct fsl_pcie *pcie, uint offset,
+ ulong *valuep, enum pci_size_t size)
+{
+ int ret;
+ struct udevice *bus = pcie->bus;
+
+ ret = fsl_pcie_read_config(bus, PCI_BDF(bus->seq, 0, 0),
+ offset, valuep, size);
+
+ return ret;
+}
+
+static int fsl_pcie_hose_write_config(struct fsl_pcie *pcie, uint offset,
+ ulong value, enum pci_size_t size)
+{
+ struct udevice *bus = pcie->bus;
+
+ return fsl_pcie_write_config(bus, PCI_BDF(bus->seq, 0, 0),
+ offset, value, size);
+}
+
+static int fsl_pcie_hose_read_config_byte(struct fsl_pcie *pcie, uint offset,
+ u8 *valuep)
+{
+ ulong val;
+ int ret;
+
+ ret = fsl_pcie_hose_read_config(pcie, offset, &val, PCI_SIZE_8);
+ *valuep = val;
+
+ return ret;
+}
+
+static int fsl_pcie_hose_read_config_word(struct fsl_pcie *pcie, uint offset,
+ u16 *valuep)
+{
+ ulong val;
+ int ret;
+
+ ret = fsl_pcie_hose_read_config(pcie, offset, &val, PCI_SIZE_16);
+ *valuep = val;
+
+ return ret;
+}
+
+static int fsl_pcie_hose_read_config_dword(struct fsl_pcie *pcie, uint offset,
+ u32 *valuep)
+{
+ ulong val;
+ int ret;
+
+ ret = fsl_pcie_hose_read_config(pcie, offset, &val, PCI_SIZE_32);
+ *valuep = val;
+
+ return ret;
+}
+
+static int fsl_pcie_hose_write_config_byte(struct fsl_pcie *pcie, uint offset,
+ u8 value)
+{
+ return fsl_pcie_hose_write_config(pcie, offset, value, PCI_SIZE_8);
+}
+
+static int fsl_pcie_hose_write_config_word(struct fsl_pcie *pcie, uint offset,
+ u16 value)
+{
+ return fsl_pcie_hose_write_config(pcie, offset, value, PCI_SIZE_16);
+}
+
+static int fsl_pcie_hose_write_config_dword(struct fsl_pcie *pcie, uint offset,
+ u32 value)
+{
+ return fsl_pcie_hose_write_config(pcie, offset, value, PCI_SIZE_32);
+}
+
+static int fsl_pcie_link_up(struct fsl_pcie *pcie)
+{
+ ccsr_fsl_pci_t *regs = pcie->regs;
+ u16 ltssm;
+
+ if (pcie->block_rev >= PEX_IP_BLK_REV_3_0) {
+ ltssm = (in_be32(®s->pex_csr0)
+ & PEX_CSR0_LTSSM_MASK) >> PEX_CSR0_LTSSM_SHIFT;
+ return ltssm == LTSSM_L0_REV3;
+ }
+
+ fsl_pcie_hose_read_config_word(pcie, PCI_LTSSM, <ssm);
+
+ return ltssm == LTSSM_L0;
+}
+
+static bool fsl_pcie_is_agent(struct fsl_pcie *pcie)
+{
+ u8 header_type;
+
+ fsl_pcie_hose_read_config_byte(pcie, PCI_HEADER_TYPE, &header_type);
+
+ return (header_type & 0x7f) == PCI_HEADER_TYPE_NORMAL;
+}
+
+static int fsl_pcie_setup_law(struct fsl_pcie *pcie)
+{
+ struct pci_region *io, *mem, *pref;
+
+ pci_get_regions(pcie->bus, &io, &mem, &pref);
+
+ if (mem)
+ set_next_law(mem->phys_start,
+ law_size_bits(mem->size),
+ pcie->law_trgt_if);
+
+ if (io)
+ set_next_law(io->phys_start,
+ law_size_bits(io->size),
+ pcie->law_trgt_if);
+
+ return 0;
+}
+
+static void fsl_pcie_config_ready(struct fsl_pcie *pcie)
+{
+ ccsr_fsl_pci_t *regs = pcie->regs;
+
+ if (pcie->block_rev >= PEX_IP_BLK_REV_3_0) {
+ setbits_be32(®s->config, FSL_PCIE_V3_CFG_RDY);
+ return;
+ }
+
+ fsl_pcie_hose_write_config_byte(pcie, FSL_PCIE_CFG_RDY, 0x1);
+}
+
+static int fsl_pcie_setup_outbound_win(struct fsl_pcie *pcie, int idx,
+ int type, u64 phys, u64 bus_addr,
+ pci_size_t size)
+{
+ ccsr_fsl_pci_t *regs = pcie->regs;
+ pot_t *po = ®s->pot[idx];
+ u32 war, sz;
+
+ if (idx < 0)
+ return -EINVAL;
+
+ out_be32(&po->powbar, phys >> 12);
+ out_be32(&po->potar, bus_addr >> 12);
+#ifdef CONFIG_SYS_PCI_64BIT
+ out_be32(&po->potear, bus_addr >> 44);
+#else
+ out_be32(&po->potear, 0);
+#endif
+
+ sz = (__ilog2_u64((u64)size) - 1);
+ war = POWAR_EN | sz;
+
+ if (type == PCI_REGION_IO)
+ war |= POWAR_IO_READ | POWAR_IO_WRITE;
+ else
+ war |= POWAR_MEM_READ | POWAR_MEM_WRITE;
+
+ out_be32(&po->powar, war);
+
+ return 0;
+}
+
+static int fsl_pcie_setup_inbound_win(struct fsl_pcie *pcie, int idx,
+ bool pf, u64 phys, u64 bus_addr,
+ pci_size_t size)
+{
+ ccsr_fsl_pci_t *regs = pcie->regs;
+ pit_t *pi = ®s->pit[idx];
+ u32 sz = (__ilog2_u64(size) - 1);
+ u32 flag = PIWAR_LOCAL;
+
+ if (idx < 0)
+ return -EINVAL;
+
+ out_be32(&pi->pitar, phys >> 12);
+ out_be32(&pi->piwbar, bus_addr >> 12);
+
+#ifdef CONFIG_SYS_PCI_64BIT
+ out_be32(&pi->piwbear, bus_addr >> 44);
+#else
+ out_be32(&pi->piwbear, 0);
+#endif
+
+ if (IS_ENABLED(CONFIG_SYS_FSL_ERRATUM_A005434))
+ flag = 0;
+
+ flag |= PIWAR_EN | PIWAR_READ_SNOOP | PIWAR_WRITE_SNOOP;
+ if (pf)
+ flag |= PIWAR_PF;
+ out_be32(&pi->piwar, flag | sz);
+
+ return 0;
+}
+
+static int fsl_pcie_setup_outbound_wins(struct fsl_pcie *pcie)
+{
+ struct pci_region *io, *mem, *pref;
+ int idx = 1; /* skip 0 */
+
+ pci_get_regions(pcie->bus, &io, &mem, &pref);
+
+ if (io)
+ /* ATU : OUTBOUND : IO */
+ fsl_pcie_setup_outbound_win(pcie, idx++,
+ PCI_REGION_IO,
+ io->phys_start,
+ io->bus_start,
+ io->size);
+
+ if (mem)
+ /* ATU : OUTBOUND : MEM */
+ fsl_pcie_setup_outbound_win(pcie, idx++,
+ PCI_REGION_MEM,
+ mem->phys_start,
+ mem->bus_start,
+ mem->size);
+ return 0;
+}
+
+static int fsl_pcie_setup_inbound_wins(struct fsl_pcie *pcie)
+{
+ phys_addr_t phys_start = CONFIG_SYS_PCI_MEMORY_PHYS;
+ pci_addr_t bus_start = CONFIG_SYS_PCI_MEMORY_BUS;
+ u64 sz = min((u64)gd->ram_size, (1ull << 32));
+ pci_size_t pci_sz;
+ int idx;
+
+ if (pcie->block_rev >= PEX_IP_BLK_REV_2_2)
+ idx = 2;
+ else
+ idx = 3;
+
+ pci_sz = 1ull << __ilog2_u64(sz);
+
+ dev_dbg(pcie->bus, "R0 bus_start: %llx phys_start: %llx size: %llx\n",
+ (u64)bus_start, (u64)phys_start, (u64)sz);
+
+ /* if we aren't an exact power of two match, pci_sz is smaller
+ * round it up to the next power of two. We report the actual
+ * size to pci region tracking.
+ */
+ if (pci_sz != sz)
+ sz = 2ull << __ilog2_u64(sz);
+
+ fsl_pcie_setup_inbound_win(pcie, idx--, true,
+ CONFIG_SYS_PCI_MEMORY_PHYS,
+ CONFIG_SYS_PCI_MEMORY_BUS, sz);
+#if defined(CONFIG_PHYS_64BIT) && defined(CONFIG_SYS_PCI_64BIT)
+ /*
+ * On 64-bit capable systems, set up a mapping for all of DRAM
+ * in high pci address space.
+ */
+ pci_sz = 1ull << __ilog2_u64(gd->ram_size);
+ /* round up to the next largest power of two */
+ if (gd->ram_size > pci_sz)
+ pci_sz = 1ull << (__ilog2_u64(gd->ram_size) + 1);
+
+ dev_dbg(pcie->bus, "R64 bus_start: %llx phys_start: %llx size: %llx\n",
+ (u64)CONFIG_SYS_PCI64_MEMORY_BUS,
+ (u64)CONFIG_SYS_PCI_MEMORY_PHYS, (u64)pci_sz);
+
+ fsl_pcie_setup_inbound_win(pcie, idx--, true,
+ CONFIG_SYS_PCI_MEMORY_PHYS,
+ CONFIG_SYS_PCI64_MEMORY_BUS, pci_sz);
+#endif
+
+ return 0;
+}
+
+static int fsl_pcie_init_atmu(struct fsl_pcie *pcie)
+{
+ fsl_pcie_setup_outbound_wins(pcie);
+ fsl_pcie_setup_inbound_wins(pcie);
+
+ return 0;
+}
+
+static int fsl_pcie_init_port(struct fsl_pcie *pcie)
+{
+ ccsr_fsl_pci_t *regs = pcie->regs;
+ u32 val_32;
+ u16 val_16;
+
+ fsl_pcie_init_atmu(pcie);
+
+ if (IS_ENABLED(CONFIG_FSL_PCIE_DISABLE_ASPM)) {
+ val_32 = 0;
+ fsl_pcie_hose_read_config_dword(pcie, PCI_LCR, &val_32);
+ val_32 &= ~0x03;
+ fsl_pcie_hose_write_config_dword(pcie, PCI_LCR, val_32);
+ udelay(1);
+ }
+
+ if (IS_ENABLED(CONFIG_FSL_PCIE_RESET)) {
+ u16 ltssm;
+ int i;
+
+ if (pcie->block_rev >= PEX_IP_BLK_REV_3_0) {
+ /* assert PCIe reset */
+ setbits_be32(®s->pdb_stat, 0x08000000);
+ (void)in_be32(®s->pdb_stat);
+ udelay(1000);
+ /* clear PCIe reset */
+ clrbits_be32(®s->pdb_stat, 0x08000000);
+ asm("sync;isync");
+ for (i = 0; i < 100 && !fsl_pcie_link_up(pcie); i++)
+ udelay(1000);
+ } else {
+ fsl_pcie_hose_read_config_word(pcie, PCI_LTSSM, <ssm);
+ if (ltssm == 1) {
+ /* assert PCIe reset */
+ setbits_be32(®s->pdb_stat, 0x08000000);
+ (void)in_be32(®s->pdb_stat);
+ udelay(100);
+ /* clear PCIe reset */
+ clrbits_be32(®s->pdb_stat, 0x08000000);
+ asm("sync;isync");
+ for (i = 0; i < 100 &&
+ !fsl_pcie_link_up(pcie); i++)
+ udelay(1000);
+ }
+ }
+ }
+
+ if (IS_ENABLED(CONFIG_SYS_P4080_ERRATUM_PCIE_A003) &&
+ !fsl_pcie_link_up(pcie)) {
+ serdes_corenet_t *srds_regs;
+
+ srds_regs = (void *)CONFIG_SYS_FSL_CORENET_SERDES_ADDR;
+ val_32 = in_be32(&srds_regs->srdspccr0);
+
+ if ((val_32 >> 28) == 3) {
+ int i;
+
+ out_be32(&srds_regs->srdspccr0, 2 << 28);
+ setbits_be32(®s->pdb_stat, 0x08000000);
+ in_be32(®s->pdb_stat);
+ udelay(100);
+ clrbits_be32(®s->pdb_stat, 0x08000000);
+ asm("sync;isync");
+ for (i = 0; i < 100 && !fsl_pcie_link_up(pcie); i++)
+ udelay(1000);
+ }
+ }
+
+ /*
+ * The Read-Only Write Enable bit defaults to 1 instead of 0.
+ * Set to 0 to protect the read-only registers.
+ */
+ if (IS_ENABLED(CONFIG_SYS_FSL_ERRATUM_A007815))
+ clrbits_be32(®s->dbi_ro_wr_en, 0x01);
+
+ /*
+ * Enable All Error Interrupts except
+ * - Master abort (pci)
+ * - Master PERR (pci)
+ * - ICCA (PCIe)
+ */
+ out_be32(®s->peer, ~0x20140);
+
+ /* set URR, FER, NFER (but not CER) */
+ fsl_pcie_hose_read_config_dword(pcie, PCI_DCR, &val_32);
+ val_32 |= 0xf000e;
+ fsl_pcie_hose_write_config_dword(pcie, PCI_DCR, val_32);
+
+ /* Clear all error indications */
+ out_be32(®s->pme_msg_det, 0xffffffff);
+ out_be32(®s->pme_msg_int_en, 0xffffffff);
+ out_be32(®s->pedr, 0xffffffff);
+
+ fsl_pcie_hose_read_config_word(pcie, PCI_DSR, &val_16);
+ if (val_16)
+ fsl_pcie_hose_write_config_word(pcie, PCI_DSR, 0xffff);
+
+ fsl_pcie_hose_read_config_word(pcie, PCI_SEC_STATUS, &val_16);
+ if (val_16)
+ fsl_pcie_hose_write_config_word(pcie, PCI_SEC_STATUS, 0xffff);
+
+ return 0;
+}
+
+static int fsl_pcie_fixup_classcode(struct fsl_pcie *pcie)
+{
+ ccsr_fsl_pci_t *regs = pcie->regs;
+ u32 val;
+
+ setbits_be32(®s->dbi_ro_wr_en, 0x01);
+ fsl_pcie_hose_read_config_dword(pcie, PCI_CLASS_REVISION, &val);
+ val &= 0xff;
+ val |= PCI_CLASS_BRIDGE_PCI << 16;
+ fsl_pcie_hose_write_config_dword(pcie, PCI_CLASS_REVISION, val);
+ clrbits_be32(®s->dbi_ro_wr_en, 0x01);
+
+ return 0;
+}
+
+static int fsl_pcie_init_rc(struct fsl_pcie *pcie)
+{
+ return fsl_pcie_fixup_classcode(pcie);
+}
+
+static int fsl_pcie_init_ep(struct fsl_pcie *pcie)
+{
+ fsl_pcie_config_ready(pcie);
+
+ return 0;
+}
+
+static int fsl_pcie_probe(struct udevice *dev)
+{
+ struct fsl_pcie *pcie = dev_get_priv(dev);
+ ccsr_fsl_pci_t *regs = pcie->regs;
+ u16 val_16;
+
+ pcie->bus = dev;
+ pcie->block_rev = in_be32(®s->block_rev1);
+
+ list_add(&pcie->list, &fsl_pcie_list);
+ pcie->enabled = is_serdes_configured(PCIE1 + pcie->idx);
+ if (!pcie->enabled) {
+ printf("PCIe%d: %s disabled\n", pcie->idx, dev->name);
+ return 0;
+ }
+
+ fsl_pcie_setup_law(pcie);
+
+ pcie->mode = fsl_pcie_is_agent(pcie);
+
+ fsl_pcie_init_port(pcie);
+
+ printf("PCIe%d: %s ", pcie->idx, dev->name);
+
+ if (pcie->mode) {
+ printf("Endpoint");
+ fsl_pcie_init_ep(pcie);
+ } else {
+ printf("Root Complex");
+ fsl_pcie_init_rc(pcie);
+ }
+
+ if (!fsl_pcie_link_up(pcie)) {
+ printf(": %s\n", pcie->mode ? "undetermined link" : "no link");
+ return 0;
+ }
+
+ fsl_pcie_hose_read_config_word(pcie, PCI_LSR, &val_16);
+ printf(": x%d gen%d\n", (val_16 & 0x3f0) >> 4, (val_16 & 0xf));
+
+ return 0;
+}
+
+static int fsl_pcie_ofdata_to_platdata(struct udevice *dev)
+{
+ struct fsl_pcie *pcie = dev_get_priv(dev);
+ int ret;
+
+ pcie->regs = dev_remap_addr(dev);
+ if (!pcie->regs) {
+ pr_err("\"reg\" resource not found\n");
+ return -EINVAL;
+ }
+
+ ret = dev_read_u32(dev, "law_trgt_if", &pcie->law_trgt_if);
+ if (ret < 0) {
+ pr_err("\"law_trgt_if\" not found\n");
+ return ret;
+ }
+
+ pcie->idx = (dev_read_addr(dev) - 0xffe240000) / 0x10000;
+
+ return 0;
+}
+
+static const struct dm_pci_ops fsl_pcie_ops = {
+ .read_config = fsl_pcie_read_config,
+ .write_config = fsl_pcie_write_config,
+};
+
+static const struct udevice_id fsl_pcie_ids[] = {
+ { .compatible = "fsl,pcie-t2080" },
+ { }
+};
+
+U_BOOT_DRIVER(fsl_pcie) = {
+ .name = "fsl_pcie",
+ .id = UCLASS_PCI,
+ .of_match = fsl_pcie_ids,
+ .ops = &fsl_pcie_ops,
+ .ofdata_to_platdata = fsl_pcie_ofdata_to_platdata,
+ .probe = fsl_pcie_probe,
+ .priv_auto_alloc_size = sizeof(struct fsl_pcie),
+};
diff --git a/drivers/pci/pcie_fsl.h b/drivers/pci/pcie_fsl.h
new file mode 100644
index 0000000000..ce2b1afb4d
--- /dev/null
+++ b/drivers/pci/pcie_fsl.h
@@ -0,0 +1,57 @@
+/* SPDX-License-Identifier: GPL-2.0+ */
+/*
+ * Copyright 2019 NXP
+ * Copyright 2007-2012 Freescale Semiconductor, Inc.
+ *
+ * PCIe DM U-Boot driver for Freescale PowerPC SoCs
+ * Author: Hou Zhiqiang <Zhiqiang.Hou@nxp.com>
+ */
+
+#ifndef _PCIE_FSL_H_
+#define _PCIE_FSL_H_
+
+#ifdef CONFIG_SYS_FSL_PCI_VER_3_X
+#define FSL_PCIE_CAP_ID 0x70
+#else
+#define FSL_PCIE_CAP_ID 0x4c
+#endif
+/* PCIe Device Control Register */
+#define PCI_DCR (FSL_PCIE_CAP_ID + 0x08)
+/* PCIe Device Status Register */
+#define PCI_DSR (FSL_PCIE_CAP_ID + 0x0a)
+/* PCIe Link Control Register */
+#define PCI_LCR (FSL_PCIE_CAP_ID + 0x10)
+/* PCIe Link Status Register */
+#define PCI_LSR (FSL_PCIE_CAP_ID + 0x12)
+
+#ifndef CONFIG_SYS_PCI_MEMORY_BUS
+#define CONFIG_SYS_PCI_MEMORY_BUS 0
+#endif
+
+#ifndef CONFIG_SYS_PCI_MEMORY_PHYS
+#define CONFIG_SYS_PCI_MEMORY_PHYS 0
+#endif
+
+#if defined(CONFIG_SYS_PCI_64BIT) && !defined(CONFIG_SYS_PCI64_MEMORY_BUS)
+#define CONFIG_SYS_PCI64_MEMORY_BUS (64ull * 1024 * 1024 * 1024)
+#endif
+
+#define PEX_CSR0_LTSSM_MASK 0xFC
+#define PEX_CSR0_LTSSM_SHIFT 2
+#define LTSSM_L0_REV3 0x11
+#define LTSSM_L0 0x16
+
+struct fsl_pcie {
+ int idx;
+ struct udevice *bus;
+ void __iomem *regs;
+ u32 law_trgt_if; /* LAW target ID */
+ u32 block_rev; /* IP block revision */
+ bool mode; /* RC&EP mode flag */
+ bool enabled; /* Enable status */
+ struct list_head list;
+};
+
+extern struct list_head fsl_pcie_list;
+
+#endif /* _PCIE_FSL_H_ */
diff --git a/drivers/pci/pcie_fsl_fixup.c b/drivers/pci/pcie_fsl_fixup.c
new file mode 100644
index 0000000000..1a17436ac0
--- /dev/null
+++ b/drivers/pci/pcie_fsl_fixup.c
@@ -0,0 +1,51 @@
+// SPDX-License-Identifier: GPL-2.0+ OR X11
+/*
+ * Copyright 2019 NXP
+ * Copyright 2007-2012 Freescale Semiconductor, Inc.
+ *
+ * PCIe Kernel DT fixup of DM U-Boot driver for Freescale PowerPC SoCs
+ * Author: Hou Zhiqiang <Zhiqiang.Hou@nxp.com>
+ */
+
+#include <common.h>
+#ifdef CONFIG_OF_BOARD_SETUP
+#include <dm.h>
+#include <fdt_support.h>
+#include <asm/fsl_pci.h>
+#include <linux/libfdt.h>
+#include "pcie_fsl.h"
+
+static void ft_fsl_pcie_setup(void *blob, struct fsl_pcie *pcie)
+{
+ struct pci_controller *hose = dev_get_uclass_priv(pcie->bus);
+ fdt_addr_t regs_addr;
+ int off;
+
+ regs_addr = dev_read_addr(pcie->bus);
+ off = fdt_node_offset_by_compat_reg(blob, FSL_PCIE_COMPAT, regs_addr);
+ if (off < 0) {
+ printf("%s: Fail to find PCIe node@0x%pa\n",
+ FSL_PCIE_COMPAT, ®s_addr);
+ return;
+ }
+
+ if (!hose || !pcie->enabled)
+ fdt_del_node(blob, off);
+ else
+ fdt_pci_dma_ranges(blob, off, hose);
+}
+
+/* Fixup Kernel DT for PCIe */
+void pci_of_setup(void *blob, bd_t *bd)
+{
+ struct fsl_pcie *pcie;
+
+ list_for_each_entry(pcie, &fsl_pcie_list, list)
+ ft_fsl_pcie_setup(blob, pcie);
+}
+
+#else
+void pci_of_setup(void *blob, bd_t *bd)
+{
+}
+#endif
--
2.17.1
^ permalink raw reply related [flat|nested] 29+ messages in thread
* [U-Boot] [PATCHv2 7/8] powerpc: T208xQDS: Disable legacy PCIe driver
2019-05-23 12:21 [U-Boot] [PATCHv2 0/8] dm: pci: add Freescale PowerPC PCIe driver Z.q. Hou
` (5 preceding siblings ...)
2019-05-23 12:22 ` [U-Boot] [PATCHv2 6/8] dm: pci: add Freescale PowerPC PCIe driver Z.q. Hou
@ 2019-05-23 12:22 ` Z.q. Hou
2019-06-20 9:44 ` Prabhakar Kushwaha
2019-05-23 12:22 ` [U-Boot] [PATCHv2 8/8] configs: T2080QDS: Enable " Z.q. Hou
7 siblings, 1 reply; 29+ messages in thread
From: Z.q. Hou @ 2019-05-23 12:22 UTC (permalink / raw)
To: u-boot
From: Hou Zhiqiang <Zhiqiang.Hou@nxp.com>
Disable legacy PCIe driver and remove unused PCIe macros.
Signed-off-by: Hou Zhiqiang <Zhiqiang.Hou@nxp.com>
Reviewed-by: Simon Glass <sjg@chromium.org>
---
V2:
- No change.
include/configs/T208xQDS.h | 18 ------------------
1 file changed, 18 deletions(-)
diff --git a/include/configs/T208xQDS.h b/include/configs/T208xQDS.h
index 7a07679069..f58bea7e39 100644
--- a/include/configs/T208xQDS.h
+++ b/include/configs/T208xQDS.h
@@ -507,49 +507,31 @@ unsigned long get_board_ddr_clk(void);
#define CONFIG_PCIE2 /* PCIE controller 2 */
#define CONFIG_PCIE3 /* PCIE controller 3 */
#define CONFIG_PCIE4 /* PCIE controller 4 */
-#define CONFIG_FSL_PCI_INIT /* Use common FSL init code */
#define CONFIG_SYS_PCI_64BIT /* enable 64-bit PCI resources */
/* controller 1, direct to uli, tgtid 3, Base address 20000 */
#define CONFIG_SYS_PCIE1_MEM_VIRT 0x80000000
-#define CONFIG_SYS_PCIE1_MEM_BUS 0xe0000000
#define CONFIG_SYS_PCIE1_MEM_PHYS 0xc00000000ull
-#define CONFIG_SYS_PCIE1_MEM_SIZE 0x20000000 /* 512M */
#define CONFIG_SYS_PCIE1_IO_VIRT 0xf8000000
-#define CONFIG_SYS_PCIE1_IO_BUS 0x00000000
#define CONFIG_SYS_PCIE1_IO_PHYS 0xff8000000ull
-#define CONFIG_SYS_PCIE1_IO_SIZE 0x00010000 /* 64k */
/* controller 2, Slot 2, tgtid 2, Base address 201000 */
#define CONFIG_SYS_PCIE2_MEM_VIRT 0xa0000000
-#define CONFIG_SYS_PCIE2_MEM_BUS 0xe0000000
#define CONFIG_SYS_PCIE2_MEM_PHYS 0xc20000000ull
-#define CONFIG_SYS_PCIE2_MEM_SIZE 0x10000000 /* 256M */
#define CONFIG_SYS_PCIE2_IO_VIRT 0xf8010000
-#define CONFIG_SYS_PCIE2_IO_BUS 0x00000000
#define CONFIG_SYS_PCIE2_IO_PHYS 0xff8010000ull
-#define CONFIG_SYS_PCIE2_IO_SIZE 0x00010000 /* 64k */
/* controller 3, Slot 1, tgtid 1, Base address 202000 */
#define CONFIG_SYS_PCIE3_MEM_VIRT 0xb0000000
-#define CONFIG_SYS_PCIE3_MEM_BUS 0xe0000000
#define CONFIG_SYS_PCIE3_MEM_PHYS 0xc30000000ull
-#define CONFIG_SYS_PCIE3_MEM_SIZE 0x10000000 /* 256M */
#define CONFIG_SYS_PCIE3_IO_VIRT 0xf8020000
-#define CONFIG_SYS_PCIE3_IO_BUS 0x00000000
#define CONFIG_SYS_PCIE3_IO_PHYS 0xff8020000ull
-#define CONFIG_SYS_PCIE3_IO_SIZE 0x00010000 /* 64k */
/* controller 4, Base address 203000 */
#define CONFIG_SYS_PCIE4_MEM_VIRT 0xc0000000
-#define CONFIG_SYS_PCIE4_MEM_BUS 0xe0000000
#define CONFIG_SYS_PCIE4_MEM_PHYS 0xc40000000ull
-#define CONFIG_SYS_PCIE4_MEM_SIZE 0x10000000 /* 256M */
-#define CONFIG_SYS_PCIE4_IO_BUS 0x00000000
#define CONFIG_SYS_PCIE4_IO_PHYS 0xff8030000ull
-#define CONFIG_SYS_PCIE4_IO_SIZE 0x00010000 /* 64k */
#ifdef CONFIG_PCI
-#define CONFIG_PCI_INDIRECT_BRIDGE
#define CONFIG_PCI_SCAN_SHOW /* show pci devices on startup */
#endif
--
2.17.1
^ permalink raw reply related [flat|nested] 29+ messages in thread
* [U-Boot] [PATCHv2 8/8] configs: T2080QDS: Enable PCIe driver
2019-05-23 12:21 [U-Boot] [PATCHv2 0/8] dm: pci: add Freescale PowerPC PCIe driver Z.q. Hou
` (6 preceding siblings ...)
2019-05-23 12:22 ` [U-Boot] [PATCHv2 7/8] powerpc: T208xQDS: Disable legacy " Z.q. Hou
@ 2019-05-23 12:22 ` Z.q. Hou
2019-05-23 14:23 ` Bin Meng
2019-06-20 9:44 ` [U-Boot] " Prabhakar Kushwaha
7 siblings, 2 replies; 29+ messages in thread
From: Z.q. Hou @ 2019-05-23 12:22 UTC (permalink / raw)
To: u-boot
From: Hou Zhiqiang <Zhiqiang.Hou@nxp.com>
Enable the DM PCIe driver in T2080QDS defconfig.
Signed-off-by: Hou Zhiqiang <Zhiqiang.Hou@nxp.com>
---
V2:
- Enable PCIe in NAND flash, SD card and SPI flash boot defconfig.
configs/T2080QDS_NAND_defconfig | 4 ++++
configs/T2080QDS_SDCARD_defconfig | 4 ++++
configs/T2080QDS_SPIFLASH_defconfig | 4 ++++
configs/T2080QDS_defconfig | 4 ++++
4 files changed, 16 insertions(+)
diff --git a/configs/T2080QDS_NAND_defconfig b/configs/T2080QDS_NAND_defconfig
index 948a05af89..d04e223c23 100644
--- a/configs/T2080QDS_NAND_defconfig
+++ b/configs/T2080QDS_NAND_defconfig
@@ -38,6 +38,7 @@ CONFIG_MTDPARTS_DEFAULT="mtdparts=fe8000000.nor:1m(uboot),5m(kernel),128k(dtb),9
CONFIG_OF_CONTROL=y
CONFIG_DEFAULT_DEVICE_TREE="t2080qds"
CONFIG_ENV_IS_IN_NAND=y
+CONFIG_DM=y
CONFIG_FSL_CAAM=y
CONFIG_FSL_ESDHC=y
CONFIG_MTD_NOR_FLASH=y
@@ -54,6 +55,9 @@ CONFIG_PHYLIB=y
CONFIG_PHY_AQUANTIA=y
CONFIG_E1000=y
CONFIG_MII=y
+CONFIG_DM_PCI=y
+CONFIG_DM_PCI_COMPAT=y
+CONFIG_PCIE_FSL=y
CONFIG_SYS_NS16550=y
CONFIG_SPI=y
CONFIG_FSL_ESPI=y
diff --git a/configs/T2080QDS_SDCARD_defconfig b/configs/T2080QDS_SDCARD_defconfig
index 30e87df3bb..7292de750b 100644
--- a/configs/T2080QDS_SDCARD_defconfig
+++ b/configs/T2080QDS_SDCARD_defconfig
@@ -38,6 +38,7 @@ CONFIG_MTDPARTS_DEFAULT="mtdparts=fe8000000.nor:1m(uboot),5m(kernel),128k(dtb),9
CONFIG_OF_CONTROL=y
CONFIG_DEFAULT_DEVICE_TREE="t2080qds"
CONFIG_ENV_IS_IN_MMC=y
+CONFIG_DM=y
CONFIG_FSL_CAAM=y
CONFIG_FSL_ESDHC=y
CONFIG_MTD_NOR_FLASH=y
@@ -53,6 +54,9 @@ CONFIG_PHYLIB=y
CONFIG_PHY_AQUANTIA=y
CONFIG_E1000=y
CONFIG_MII=y
+CONFIG_DM_PCI=y
+CONFIG_DM_PCI_COMPAT=y
+CONFIG_PCIE_FSL=y
CONFIG_SYS_NS16550=y
CONFIG_SPI=y
CONFIG_FSL_ESPI=y
diff --git a/configs/T2080QDS_SPIFLASH_defconfig b/configs/T2080QDS_SPIFLASH_defconfig
index 0cccb3a85d..d85cc8d9d1 100644
--- a/configs/T2080QDS_SPIFLASH_defconfig
+++ b/configs/T2080QDS_SPIFLASH_defconfig
@@ -39,6 +39,7 @@ CONFIG_MTDPARTS_DEFAULT="mtdparts=fe8000000.nor:1m(uboot),5m(kernel),128k(dtb),9
CONFIG_OF_CONTROL=y
CONFIG_DEFAULT_DEVICE_TREE="t2080qds"
CONFIG_ENV_IS_IN_SPI_FLASH=y
+CONFIG_DM=y
CONFIG_FSL_CAAM=y
CONFIG_FSL_ESDHC=y
CONFIG_MTD_NOR_FLASH=y
@@ -54,6 +55,9 @@ CONFIG_PHYLIB=y
CONFIG_PHY_AQUANTIA=y
CONFIG_E1000=y
CONFIG_MII=y
+CONFIG_DM_PCI=y
+CONFIG_DM_PCI_COMPAT=y
+CONFIG_PCIE_FSL=y
CONFIG_SYS_NS16550=y
CONFIG_SPI=y
CONFIG_FSL_ESPI=y
diff --git a/configs/T2080QDS_defconfig b/configs/T2080QDS_defconfig
index 79cf4424b4..df1c5183e5 100644
--- a/configs/T2080QDS_defconfig
+++ b/configs/T2080QDS_defconfig
@@ -28,6 +28,7 @@ CONFIG_MTDPARTS_DEFAULT="mtdparts=fe8000000.nor:1m(uboot),5m(kernel),128k(dtb),9
CONFIG_OF_CONTROL=y
CONFIG_DEFAULT_DEVICE_TREE="t2080qds"
CONFIG_ENV_IS_IN_FLASH=y
+CONFIG_DM=y
CONFIG_FSL_CAAM=y
CONFIG_FSL_ESDHC=y
CONFIG_MTD_NOR_FLASH=y
@@ -43,6 +44,9 @@ CONFIG_PHYLIB=y
CONFIG_PHY_AQUANTIA=y
CONFIG_E1000=y
CONFIG_MII=y
+CONFIG_DM_PCI=y
+CONFIG_DM_PCI_COMPAT=y
+CONFIG_PCIE_FSL=y
CONFIG_SYS_NS16550=y
CONFIG_SPI=y
CONFIG_FSL_ESPI=y
--
2.17.1
^ permalink raw reply related [flat|nested] 29+ messages in thread
* [U-Boot] [PATCHv2 8/8] configs: T2080QDS: Enable PCIe driver
2019-05-23 12:22 ` [U-Boot] [PATCHv2 8/8] configs: T2080QDS: Enable " Z.q. Hou
@ 2019-05-23 14:23 ` Bin Meng
2019-05-23 16:04 ` Z.q. Hou
2019-06-20 9:44 ` [U-Boot] " Prabhakar Kushwaha
1 sibling, 1 reply; 29+ messages in thread
From: Bin Meng @ 2019-05-23 14:23 UTC (permalink / raw)
To: u-boot
On Thu, May 23, 2019 at 8:22 PM Z.q. Hou <zhiqiang.hou@nxp.com> wrote:
>
> From: Hou Zhiqiang <Zhiqiang.Hou@nxp.com>
>
> Enable the DM PCIe driver in T2080QDS defconfig.
>
> Signed-off-by: Hou Zhiqiang <Zhiqiang.Hou@nxp.com>
> ---
> V2:
> - Enable PCIe in NAND flash, SD card and SPI flash boot defconfig.
>
> configs/T2080QDS_NAND_defconfig | 4 ++++
> configs/T2080QDS_SDCARD_defconfig | 4 ++++
> configs/T2080QDS_SPIFLASH_defconfig | 4 ++++
> configs/T2080QDS_defconfig | 4 ++++
> 4 files changed, 16 insertions(+)
>
> diff --git a/configs/T2080QDS_NAND_defconfig b/configs/T2080QDS_NAND_defconfig
> index 948a05af89..d04e223c23 100644
> --- a/configs/T2080QDS_NAND_defconfig
> +++ b/configs/T2080QDS_NAND_defconfig
> @@ -38,6 +38,7 @@ CONFIG_MTDPARTS_DEFAULT="mtdparts=fe8000000.nor:1m(uboot),5m(kernel),128k(dtb),9
> CONFIG_OF_CONTROL=y
> CONFIG_DEFAULT_DEVICE_TREE="t2080qds"
> CONFIG_ENV_IS_IN_NAND=y
> +CONFIG_DM=y
> CONFIG_FSL_CAAM=y
> CONFIG_FSL_ESDHC=y
> CONFIG_MTD_NOR_FLASH=y
> @@ -54,6 +55,9 @@ CONFIG_PHYLIB=y
> CONFIG_PHY_AQUANTIA=y
> CONFIG_E1000=y
> CONFIG_MII=y
> +CONFIG_DM_PCI=y
> +CONFIG_DM_PCI_COMPAT=y
I think we should avoid adding DM_PCI_COMPAT.
> +CONFIG_PCIE_FSL=y
> CONFIG_SYS_NS16550=y
> CONFIG_SPI=y
> CONFIG_FSL_ESPI=y
> diff --git a/configs/T2080QDS_SDCARD_defconfig b/configs/T2080QDS_SDCARD_defconfig
> index 30e87df3bb..7292de750b 100644
> --- a/configs/T2080QDS_SDCARD_defconfig
> +++ b/configs/T2080QDS_SDCARD_defconfig
> @@ -38,6 +38,7 @@ CONFIG_MTDPARTS_DEFAULT="mtdparts=fe8000000.nor:1m(uboot),5m(kernel),128k(dtb),9
> CONFIG_OF_CONTROL=y
> CONFIG_DEFAULT_DEVICE_TREE="t2080qds"
> CONFIG_ENV_IS_IN_MMC=y
> +CONFIG_DM=y
> CONFIG_FSL_CAAM=y
> CONFIG_FSL_ESDHC=y
> CONFIG_MTD_NOR_FLASH=y
> @@ -53,6 +54,9 @@ CONFIG_PHYLIB=y
> CONFIG_PHY_AQUANTIA=y
> CONFIG_E1000=y
> CONFIG_MII=y
> +CONFIG_DM_PCI=y
> +CONFIG_DM_PCI_COMPAT=y
> +CONFIG_PCIE_FSL=y
> CONFIG_SYS_NS16550=y
> CONFIG_SPI=y
> CONFIG_FSL_ESPI=y
> diff --git a/configs/T2080QDS_SPIFLASH_defconfig b/configs/T2080QDS_SPIFLASH_defconfig
> index 0cccb3a85d..d85cc8d9d1 100644
> --- a/configs/T2080QDS_SPIFLASH_defconfig
> +++ b/configs/T2080QDS_SPIFLASH_defconfig
> @@ -39,6 +39,7 @@ CONFIG_MTDPARTS_DEFAULT="mtdparts=fe8000000.nor:1m(uboot),5m(kernel),128k(dtb),9
> CONFIG_OF_CONTROL=y
> CONFIG_DEFAULT_DEVICE_TREE="t2080qds"
> CONFIG_ENV_IS_IN_SPI_FLASH=y
> +CONFIG_DM=y
> CONFIG_FSL_CAAM=y
> CONFIG_FSL_ESDHC=y
> CONFIG_MTD_NOR_FLASH=y
> @@ -54,6 +55,9 @@ CONFIG_PHYLIB=y
> CONFIG_PHY_AQUANTIA=y
> CONFIG_E1000=y
> CONFIG_MII=y
> +CONFIG_DM_PCI=y
> +CONFIG_DM_PCI_COMPAT=y
> +CONFIG_PCIE_FSL=y
> CONFIG_SYS_NS16550=y
> CONFIG_SPI=y
> CONFIG_FSL_ESPI=y
> diff --git a/configs/T2080QDS_defconfig b/configs/T2080QDS_defconfig
> index 79cf4424b4..df1c5183e5 100644
> --- a/configs/T2080QDS_defconfig
> +++ b/configs/T2080QDS_defconfig
> @@ -28,6 +28,7 @@ CONFIG_MTDPARTS_DEFAULT="mtdparts=fe8000000.nor:1m(uboot),5m(kernel),128k(dtb),9
> CONFIG_OF_CONTROL=y
> CONFIG_DEFAULT_DEVICE_TREE="t2080qds"
> CONFIG_ENV_IS_IN_FLASH=y
> +CONFIG_DM=y
> CONFIG_FSL_CAAM=y
> CONFIG_FSL_ESDHC=y
> CONFIG_MTD_NOR_FLASH=y
> @@ -43,6 +44,9 @@ CONFIG_PHYLIB=y
> CONFIG_PHY_AQUANTIA=y
> CONFIG_E1000=y
> CONFIG_MII=y
> +CONFIG_DM_PCI=y
> +CONFIG_DM_PCI_COMPAT=y
> +CONFIG_PCIE_FSL=y
> CONFIG_SYS_NS16550=y
> CONFIG_SPI=y
> CONFIG_FSL_ESPI=y
> --
Regards,
Bin
^ permalink raw reply [flat|nested] 29+ messages in thread
* [U-Boot] [PATCHv2 8/8] configs: T2080QDS: Enable PCIe driver
2019-05-23 14:23 ` Bin Meng
@ 2019-05-23 16:04 ` Z.q. Hou
2019-05-24 0:10 ` Bin Meng
0 siblings, 1 reply; 29+ messages in thread
From: Z.q. Hou @ 2019-05-23 16:04 UTC (permalink / raw)
To: u-boot
Hi Bin,
Thanks a lot for your comments!
> -----Original Message-----
> From: Bin Meng [mailto:bmeng.cn at gmail.com]
> Sent: 2019年5月23日 22:23
> To: Z.q. Hou <zhiqiang.hou@nxp.com>
> Cc: u-boot at lists.denx.de; Prabhakar Kushwaha
> <prabhakar.kushwaha@nxp.com>; wd at denx.de; Shengzhou Liu
> <shengzhou.liu@nxp.com>; Jagdish Gediya <jagdish.gediya@nxp.com>;
> sjg at chromium.org; ley.foon.tan at intel.com; sr at denx.de; M.h. Lian
> <minghuan.lian@nxp.com>; Xiaowei Bao <xiaowei.bao@nxp.com>
> Subject: Re: [PATCHv2 8/8] configs: T2080QDS: Enable PCIe driver
>
> On Thu, May 23, 2019 at 8:22 PM Z.q. Hou <zhiqiang.hou@nxp.com> wrote:
> >
> > From: Hou Zhiqiang <Zhiqiang.Hou@nxp.com>
> >
> > Enable the DM PCIe driver in T2080QDS defconfig.
> >
> > Signed-off-by: Hou Zhiqiang <Zhiqiang.Hou@nxp.com>
> > ---
> > V2:
> > - Enable PCIe in NAND flash, SD card and SPI flash boot defconfig.
> >
> > configs/T2080QDS_NAND_defconfig | 4 ++++
> > configs/T2080QDS_SDCARD_defconfig | 4 ++++
> > configs/T2080QDS_SPIFLASH_defconfig | 4 ++++
> > configs/T2080QDS_defconfig | 4 ++++
> > 4 files changed, 16 insertions(+)
> >
> > diff --git a/configs/T2080QDS_NAND_defconfig
> > b/configs/T2080QDS_NAND_defconfig index 948a05af89..d04e223c23
> 100644
> > --- a/configs/T2080QDS_NAND_defconfig
> > +++ b/configs/T2080QDS_NAND_defconfig
> > @@ -38,6 +38,7 @@
> >
> CONFIG_MTDPARTS_DEFAULT="mtdparts=fe8000000.nor:1m(uboot),5m(kern
> el),1
> > 28k(dtb),9
> > CONFIG_OF_CONTROL=y
> > CONFIG_DEFAULT_DEVICE_TREE="t2080qds"
> > CONFIG_ENV_IS_IN_NAND=y
> > +CONFIG_DM=y
> > CONFIG_FSL_CAAM=y
> > CONFIG_FSL_ESDHC=y
> > CONFIG_MTD_NOR_FLASH=y
> > @@ -54,6 +55,9 @@ CONFIG_PHYLIB=y
> > CONFIG_PHY_AQUANTIA=y
> > CONFIG_E1000=y
> > CONFIG_MII=y
> > +CONFIG_DM_PCI=y
> > +CONFIG_DM_PCI_COMPAT=y
>
> I think we should avoid adding DM_PCI_COMPAT.
Without this macro the non-DM Endpoint driver (such as e1000) won't work.
> > +CONFIG_PCIE_FSL=y
> > CONFIG_SYS_NS16550=y
> > CONFIG_SPI=y
> > CONFIG_FSL_ESPI=y
> > diff --git a/configs/T2080QDS_SDCARD_defconfig
> > b/configs/T2080QDS_SDCARD_defconfig
> > index 30e87df3bb..7292de750b 100644
> > --- a/configs/T2080QDS_SDCARD_defconfig
> > +++ b/configs/T2080QDS_SDCARD_defconfig
> > @@ -38,6 +38,7 @@
> >
> CONFIG_MTDPARTS_DEFAULT="mtdparts=fe8000000.nor:1m(uboot),5m(kern
> el),1
> > 28k(dtb),9
> > CONFIG_OF_CONTROL=y
> > CONFIG_DEFAULT_DEVICE_TREE="t2080qds"
> > CONFIG_ENV_IS_IN_MMC=y
> > +CONFIG_DM=y
> > CONFIG_FSL_CAAM=y
> > CONFIG_FSL_ESDHC=y
> > CONFIG_MTD_NOR_FLASH=y
> > @@ -53,6 +54,9 @@ CONFIG_PHYLIB=y
> > CONFIG_PHY_AQUANTIA=y
> > CONFIG_E1000=y
> > CONFIG_MII=y
> > +CONFIG_DM_PCI=y
> > +CONFIG_DM_PCI_COMPAT=y
> > +CONFIG_PCIE_FSL=y
> > CONFIG_SYS_NS16550=y
> > CONFIG_SPI=y
> > CONFIG_FSL_ESPI=y
> > diff --git a/configs/T2080QDS_SPIFLASH_defconfig
> > b/configs/T2080QDS_SPIFLASH_defconfig
> > index 0cccb3a85d..d85cc8d9d1 100644
> > --- a/configs/T2080QDS_SPIFLASH_defconfig
> > +++ b/configs/T2080QDS_SPIFLASH_defconfig
> > @@ -39,6 +39,7 @@
> >
> CONFIG_MTDPARTS_DEFAULT="mtdparts=fe8000000.nor:1m(uboot),5m(kern
> el),1
> > 28k(dtb),9
> > CONFIG_OF_CONTROL=y
> > CONFIG_DEFAULT_DEVICE_TREE="t2080qds"
> > CONFIG_ENV_IS_IN_SPI_FLASH=y
> > +CONFIG_DM=y
> > CONFIG_FSL_CAAM=y
> > CONFIG_FSL_ESDHC=y
> > CONFIG_MTD_NOR_FLASH=y
> > @@ -54,6 +55,9 @@ CONFIG_PHYLIB=y
> > CONFIG_PHY_AQUANTIA=y
> > CONFIG_E1000=y
> > CONFIG_MII=y
> > +CONFIG_DM_PCI=y
> > +CONFIG_DM_PCI_COMPAT=y
> > +CONFIG_PCIE_FSL=y
> > CONFIG_SYS_NS16550=y
> > CONFIG_SPI=y
> > CONFIG_FSL_ESPI=y
> > diff --git a/configs/T2080QDS_defconfig b/configs/T2080QDS_defconfig
> > index 79cf4424b4..df1c5183e5 100644
> > --- a/configs/T2080QDS_defconfig
> > +++ b/configs/T2080QDS_defconfig
> > @@ -28,6 +28,7 @@
> >
> CONFIG_MTDPARTS_DEFAULT="mtdparts=fe8000000.nor:1m(uboot),5m(kern
> el),1
> > 28k(dtb),9
> > CONFIG_OF_CONTROL=y
> > CONFIG_DEFAULT_DEVICE_TREE="t2080qds"
> > CONFIG_ENV_IS_IN_FLASH=y
> > +CONFIG_DM=y
> > CONFIG_FSL_CAAM=y
> > CONFIG_FSL_ESDHC=y
> > CONFIG_MTD_NOR_FLASH=y
> > @@ -43,6 +44,9 @@ CONFIG_PHYLIB=y
> > CONFIG_PHY_AQUANTIA=y
> > CONFIG_E1000=y
> > CONFIG_MII=y
> > +CONFIG_DM_PCI=y
> > +CONFIG_DM_PCI_COMPAT=y
> > +CONFIG_PCIE_FSL=y
> > CONFIG_SYS_NS16550=y
> > CONFIG_SPI=y
> > CONFIG_FSL_ESPI=y
> > --
>
> Regards,
> Bin
Thanks,
Zhiqiang
^ permalink raw reply [flat|nested] 29+ messages in thread
* [U-Boot] [PATCHv2 8/8] configs: T2080QDS: Enable PCIe driver
2019-05-23 16:04 ` Z.q. Hou
@ 2019-05-24 0:10 ` Bin Meng
2019-05-24 2:10 ` [U-Boot] [EXT] " Xiaowei Bao
0 siblings, 1 reply; 29+ messages in thread
From: Bin Meng @ 2019-05-24 0:10 UTC (permalink / raw)
To: u-boot
Hi,
On Fri, May 24, 2019 at 12:04 AM Z.q. Hou <zhiqiang.hou@nxp.com> wrote:
>
> Hi Bin,
>
> Thanks a lot for your comments!
>
> > -----Original Message-----
> > From: Bin Meng [mailto:bmeng.cn at gmail.com]
> > Sent: 2019年5月23日 22:23
> > To: Z.q. Hou <zhiqiang.hou@nxp.com>
> > Cc: u-boot at lists.denx.de; Prabhakar Kushwaha
> > <prabhakar.kushwaha@nxp.com>; wd at denx.de; Shengzhou Liu
> > <shengzhou.liu@nxp.com>; Jagdish Gediya <jagdish.gediya@nxp.com>;
> > sjg at chromium.org; ley.foon.tan at intel.com; sr at denx.de; M.h. Lian
> > <minghuan.lian@nxp.com>; Xiaowei Bao <xiaowei.bao@nxp.com>
> > Subject: Re: [PATCHv2 8/8] configs: T2080QDS: Enable PCIe driver
> >
> > On Thu, May 23, 2019 at 8:22 PM Z.q. Hou <zhiqiang.hou@nxp.com> wrote:
> > >
> > > From: Hou Zhiqiang <Zhiqiang.Hou@nxp.com>
> > >
> > > Enable the DM PCIe driver in T2080QDS defconfig.
> > >
> > > Signed-off-by: Hou Zhiqiang <Zhiqiang.Hou@nxp.com>
> > > ---
> > > V2:
> > > - Enable PCIe in NAND flash, SD card and SPI flash boot defconfig.
> > >
> > > configs/T2080QDS_NAND_defconfig | 4 ++++
> > > configs/T2080QDS_SDCARD_defconfig | 4 ++++
> > > configs/T2080QDS_SPIFLASH_defconfig | 4 ++++
> > > configs/T2080QDS_defconfig | 4 ++++
> > > 4 files changed, 16 insertions(+)
> > >
> > > diff --git a/configs/T2080QDS_NAND_defconfig
> > > b/configs/T2080QDS_NAND_defconfig index 948a05af89..d04e223c23
> > 100644
> > > --- a/configs/T2080QDS_NAND_defconfig
> > > +++ b/configs/T2080QDS_NAND_defconfig
> > > @@ -38,6 +38,7 @@
> > >
> > CONFIG_MTDPARTS_DEFAULT="mtdparts=fe8000000.nor:1m(uboot),5m(kern
> > el),1
> > > 28k(dtb),9
> > > CONFIG_OF_CONTROL=y
> > > CONFIG_DEFAULT_DEVICE_TREE="t2080qds"
> > > CONFIG_ENV_IS_IN_NAND=y
> > > +CONFIG_DM=y
> > > CONFIG_FSL_CAAM=y
> > > CONFIG_FSL_ESDHC=y
> > > CONFIG_MTD_NOR_FLASH=y
> > > @@ -54,6 +55,9 @@ CONFIG_PHYLIB=y
> > > CONFIG_PHY_AQUANTIA=y
> > > CONFIG_E1000=y
> > > CONFIG_MII=y
> > > +CONFIG_DM_PCI=y
> > > +CONFIG_DM_PCI_COMPAT=y
> >
> > I think we should avoid adding DM_PCI_COMPAT.
>
> Without this macro the non-DM Endpoint driver (such as e1000) won't work.
e1000 is a fully DM driver now. Please redo the testing.
Regards,
Bin
^ permalink raw reply [flat|nested] 29+ messages in thread
* [U-Boot] [EXT] Re: [PATCHv2 8/8] configs: T2080QDS: Enable PCIe driver
2019-05-24 0:10 ` Bin Meng
@ 2019-05-24 2:10 ` Xiaowei Bao
2019-05-24 2:13 ` Bin Meng
0 siblings, 1 reply; 29+ messages in thread
From: Xiaowei Bao @ 2019-05-24 2:10 UTC (permalink / raw)
To: u-boot
-----Original Message-----
From: Bin Meng <bmeng.cn@gmail.com>
Sent: 2019年5月24日 8:11
To: Z.q. Hou <zhiqiang.hou@nxp.com>
Cc: u-boot at lists.denx.de; Prabhakar Kushwaha <prabhakar.kushwaha@nxp.com>; wd at denx.de; Shengzhou Liu <shengzhou.liu@nxp.com>; Jagdish Gediya <jagdish.gediya@nxp.com>; sjg at chromium.org; ley.foon.tan at intel.com; sr at denx.de; M.h. Lian <minghuan.lian@nxp.com>; Xiaowei Bao <xiaowei.bao@nxp.com>
Subject: [EXT] Re: [PATCHv2 8/8] configs: T2080QDS: Enable PCIe driver
Caution: EXT Email
Hi,
On Fri, May 24, 2019 at 12:04 AM Z.q. Hou <zhiqiang.hou@nxp.com> wrote:
>
> Hi Bin,
>
> Thanks a lot for your comments!
>
> > -----Original Message-----
> > From: Bin Meng [mailto:bmeng.cn at gmail.com]
> > Sent: 2019年5月23日 22:23
> > To: Z.q. Hou <zhiqiang.hou@nxp.com>
> > Cc: u-boot at lists.denx.de; Prabhakar Kushwaha
> > <prabhakar.kushwaha@nxp.com>; wd at denx.de; Shengzhou Liu
> > <shengzhou.liu@nxp.com>; Jagdish Gediya <jagdish.gediya@nxp.com>;
> > sjg at chromium.org; ley.foon.tan at intel.com; sr at denx.de; M.h. Lian
> > <minghuan.lian@nxp.com>; Xiaowei Bao <xiaowei.bao@nxp.com>
> > Subject: Re: [PATCHv2 8/8] configs: T2080QDS: Enable PCIe driver
> >
> > On Thu, May 23, 2019 at 8:22 PM Z.q. Hou <zhiqiang.hou@nxp.com> wrote:
> > >
> > > From: Hou Zhiqiang <Zhiqiang.Hou@nxp.com>
> > >
> > > Enable the DM PCIe driver in T2080QDS defconfig.
> > >
> > > Signed-off-by: Hou Zhiqiang <Zhiqiang.Hou@nxp.com>
> > > ---
> > > V2:
> > > - Enable PCIe in NAND flash, SD card and SPI flash boot defconfig.
> > >
> > > configs/T2080QDS_NAND_defconfig | 4 ++++
> > > configs/T2080QDS_SDCARD_defconfig | 4 ++++
> > > configs/T2080QDS_SPIFLASH_defconfig | 4 ++++
> > > configs/T2080QDS_defconfig | 4 ++++
> > > 4 files changed, 16 insertions(+)
> > >
> > > diff --git a/configs/T2080QDS_NAND_defconfig
> > > b/configs/T2080QDS_NAND_defconfig index 948a05af89..d04e223c23
> > 100644
> > > --- a/configs/T2080QDS_NAND_defconfig
> > > +++ b/configs/T2080QDS_NAND_defconfig
> > > @@ -38,6 +38,7 @@
> > >
> > CONFIG_MTDPARTS_DEFAULT="mtdparts=fe8000000.nor:1m(uboot),5m(kern
> > el),1
> > > 28k(dtb),9
> > > CONFIG_OF_CONTROL=y
> > > CONFIG_DEFAULT_DEVICE_TREE="t2080qds"
> > > CONFIG_ENV_IS_IN_NAND=y
> > > +CONFIG_DM=y
> > > CONFIG_FSL_CAAM=y
> > > CONFIG_FSL_ESDHC=y
> > > CONFIG_MTD_NOR_FLASH=y
> > > @@ -54,6 +55,9 @@ CONFIG_PHYLIB=y
> > > CONFIG_PHY_AQUANTIA=y
> > > CONFIG_E1000=y
> > > CONFIG_MII=y
> > > +CONFIG_DM_PCI=y
> > > +CONFIG_DM_PCI_COMPAT=y
> >
> > I think we should avoid adding DM_PCI_COMPAT.
>
> Without this macro the non-DM Endpoint driver (such as e1000) won't work.
e1000 is a fully DM driver now. Please redo the testing.
[Xiaowei Bao] I am not very clear, If someone want to implement the customized device driver(not DM) in u-boot, the driver will not work, yes?
Regards,
Bin
^ permalink raw reply [flat|nested] 29+ messages in thread
* [U-Boot] [EXT] Re: [PATCHv2 8/8] configs: T2080QDS: Enable PCIe driver
2019-05-24 2:10 ` [U-Boot] [EXT] " Xiaowei Bao
@ 2019-05-24 2:13 ` Bin Meng
2019-05-24 2:17 ` Xiaowei Bao
2019-05-24 8:17 ` Z.q. Hou
0 siblings, 2 replies; 29+ messages in thread
From: Bin Meng @ 2019-05-24 2:13 UTC (permalink / raw)
To: u-boot
On Fri, May 24, 2019 at 10:10 AM Xiaowei Bao <xiaowei.bao@nxp.com> wrote:
>
>
>
> -----Original Message-----
> From: Bin Meng <bmeng.cn@gmail.com>
> Sent: 2019年5月24日 8:11
> To: Z.q. Hou <zhiqiang.hou@nxp.com>
> Cc: u-boot at lists.denx.de; Prabhakar Kushwaha <prabhakar.kushwaha@nxp.com>; wd at denx.de; Shengzhou Liu <shengzhou.liu@nxp.com>; Jagdish Gediya <jagdish.gediya@nxp.com>; sjg at chromium.org; ley.foon.tan at intel.com; sr at denx.de; M.h. Lian <minghuan.lian@nxp.com>; Xiaowei Bao <xiaowei.bao@nxp.com>
> Subject: [EXT] Re: [PATCHv2 8/8] configs: T2080QDS: Enable PCIe driver
>
> Caution: EXT Email
>
> Hi,
>
> On Fri, May 24, 2019 at 12:04 AM Z.q. Hou <zhiqiang.hou@nxp.com> wrote:
> >
> > Hi Bin,
> >
> > Thanks a lot for your comments!
> >
> > > -----Original Message-----
> > > From: Bin Meng [mailto:bmeng.cn at gmail.com]
> > > Sent: 2019年5月23日 22:23
> > > To: Z.q. Hou <zhiqiang.hou@nxp.com>
> > > Cc: u-boot at lists.denx.de; Prabhakar Kushwaha
> > > <prabhakar.kushwaha@nxp.com>; wd at denx.de; Shengzhou Liu
> > > <shengzhou.liu@nxp.com>; Jagdish Gediya <jagdish.gediya@nxp.com>;
> > > sjg at chromium.org; ley.foon.tan at intel.com; sr at denx.de; M.h. Lian
> > > <minghuan.lian@nxp.com>; Xiaowei Bao <xiaowei.bao@nxp.com>
> > > Subject: Re: [PATCHv2 8/8] configs: T2080QDS: Enable PCIe driver
> > >
> > > On Thu, May 23, 2019 at 8:22 PM Z.q. Hou <zhiqiang.hou@nxp.com> wrote:
> > > >
> > > > From: Hou Zhiqiang <Zhiqiang.Hou@nxp.com>
> > > >
> > > > Enable the DM PCIe driver in T2080QDS defconfig.
> > > >
> > > > Signed-off-by: Hou Zhiqiang <Zhiqiang.Hou@nxp.com>
> > > > ---
> > > > V2:
> > > > - Enable PCIe in NAND flash, SD card and SPI flash boot defconfig.
> > > >
> > > > configs/T2080QDS_NAND_defconfig | 4 ++++
> > > > configs/T2080QDS_SDCARD_defconfig | 4 ++++
> > > > configs/T2080QDS_SPIFLASH_defconfig | 4 ++++
> > > > configs/T2080QDS_defconfig | 4 ++++
> > > > 4 files changed, 16 insertions(+)
> > > >
> > > > diff --git a/configs/T2080QDS_NAND_defconfig
> > > > b/configs/T2080QDS_NAND_defconfig index 948a05af89..d04e223c23
> > > 100644
> > > > --- a/configs/T2080QDS_NAND_defconfig
> > > > +++ b/configs/T2080QDS_NAND_defconfig
> > > > @@ -38,6 +38,7 @@
> > > >
> > > CONFIG_MTDPARTS_DEFAULT="mtdparts=fe8000000.nor:1m(uboot),5m(kern
> > > el),1
> > > > 28k(dtb),9
> > > > CONFIG_OF_CONTROL=y
> > > > CONFIG_DEFAULT_DEVICE_TREE="t2080qds"
> > > > CONFIG_ENV_IS_IN_NAND=y
> > > > +CONFIG_DM=y
> > > > CONFIG_FSL_CAAM=y
> > > > CONFIG_FSL_ESDHC=y
> > > > CONFIG_MTD_NOR_FLASH=y
> > > > @@ -54,6 +55,9 @@ CONFIG_PHYLIB=y
> > > > CONFIG_PHY_AQUANTIA=y
> > > > CONFIG_E1000=y
> > > > CONFIG_MII=y
> > > > +CONFIG_DM_PCI=y
> > > > +CONFIG_DM_PCI_COMPAT=y
> > >
> > > I think we should avoid adding DM_PCI_COMPAT.
> >
> > Without this macro the non-DM Endpoint driver (such as e1000) won't work.
>
> e1000 is a fully DM driver now. Please redo the testing.
> [Xiaowei Bao] I am not very clear, If someone want to implement the customized device driver(not DM) in u-boot, the driver will not work, yes?
We are going to convert all drivers to DM in U-Boot. There are some
deadlines to be set already. So you don't need consider such non-dm
compatibility.
I guess CONFIG_DM_PCI_COMPAT will be removed once all PCI drivers have
been converted.
Regards,
Bin
^ permalink raw reply [flat|nested] 29+ messages in thread
* [U-Boot] [EXT] Re: [PATCHv2 8/8] configs: T2080QDS: Enable PCIe driver
2019-05-24 2:13 ` Bin Meng
@ 2019-05-24 2:17 ` Xiaowei Bao
2019-05-24 5:23 ` Prabhakar Kushwaha
2019-05-24 8:17 ` Z.q. Hou
1 sibling, 1 reply; 29+ messages in thread
From: Xiaowei Bao @ 2019-05-24 2:17 UTC (permalink / raw)
To: u-boot
-----Original Message-----
From: Bin Meng <bmeng.cn@gmail.com>
Sent: 2019年5月24日 10:14
To: Xiaowei Bao <xiaowei.bao@nxp.com>; Simon Glass <sjg@chromium.org>
Cc: Z.q. Hou <zhiqiang.hou@nxp.com>; u-boot at lists.denx.de; Prabhakar Kushwaha <prabhakar.kushwaha@nxp.com>; wd at denx.de; Shengzhou Liu <shengzhou.liu@nxp.com>; Jagdish Gediya <jagdish.gediya@nxp.com>; ley.foon.tan at intel.com; sr at denx.de; M.h. Lian <minghuan.lian@nxp.com>
Subject: Re: [EXT] Re: [PATCHv2 8/8] configs: T2080QDS: Enable PCIe driver
Caution: EXT Email
On Fri, May 24, 2019 at 10:10 AM Xiaowei Bao <xiaowei.bao@nxp.com> wrote:
>
>
>
> -----Original Message-----
> From: Bin Meng <bmeng.cn@gmail.com>
> Sent: 2019年5月24日 8:11
> To: Z.q. Hou <zhiqiang.hou@nxp.com>
> Cc: u-boot at lists.denx.de; Prabhakar Kushwaha
> <prabhakar.kushwaha@nxp.com>; wd at denx.de; Shengzhou Liu
> <shengzhou.liu@nxp.com>; Jagdish Gediya <jagdish.gediya@nxp.com>;
> sjg at chromium.org; ley.foon.tan at intel.com; sr at denx.de; M.h. Lian
> <minghuan.lian@nxp.com>; Xiaowei Bao <xiaowei.bao@nxp.com>
> Subject: [EXT] Re: [PATCHv2 8/8] configs: T2080QDS: Enable PCIe driver
>
> Caution: EXT Email
>
> Hi,
>
> On Fri, May 24, 2019 at 12:04 AM Z.q. Hou <zhiqiang.hou@nxp.com> wrote:
> >
> > Hi Bin,
> >
> > Thanks a lot for your comments!
> >
> > > -----Original Message-----
> > > From: Bin Meng [mailto:bmeng.cn at gmail.com]
> > > Sent: 2019年5月23日 22:23
> > > To: Z.q. Hou <zhiqiang.hou@nxp.com>
> > > Cc: u-boot at lists.denx.de; Prabhakar Kushwaha
> > > <prabhakar.kushwaha@nxp.com>; wd at denx.de; Shengzhou Liu
> > > <shengzhou.liu@nxp.com>; Jagdish Gediya <jagdish.gediya@nxp.com>;
> > > sjg at chromium.org; ley.foon.tan at intel.com; sr at denx.de; M.h. Lian
> > > <minghuan.lian@nxp.com>; Xiaowei Bao <xiaowei.bao@nxp.com>
> > > Subject: Re: [PATCHv2 8/8] configs: T2080QDS: Enable PCIe driver
> > >
> > > On Thu, May 23, 2019 at 8:22 PM Z.q. Hou <zhiqiang.hou@nxp.com> wrote:
> > > >
> > > > From: Hou Zhiqiang <Zhiqiang.Hou@nxp.com>
> > > >
> > > > Enable the DM PCIe driver in T2080QDS defconfig.
> > > >
> > > > Signed-off-by: Hou Zhiqiang <Zhiqiang.Hou@nxp.com>
> > > > ---
> > > > V2:
> > > > - Enable PCIe in NAND flash, SD card and SPI flash boot defconfig.
> > > >
> > > > configs/T2080QDS_NAND_defconfig | 4 ++++
> > > > configs/T2080QDS_SDCARD_defconfig | 4 ++++
> > > > configs/T2080QDS_SPIFLASH_defconfig | 4 ++++
> > > > configs/T2080QDS_defconfig | 4 ++++
> > > > 4 files changed, 16 insertions(+)
> > > >
> > > > diff --git a/configs/T2080QDS_NAND_defconfig
> > > > b/configs/T2080QDS_NAND_defconfig index 948a05af89..d04e223c23
> > > 100644
> > > > --- a/configs/T2080QDS_NAND_defconfig
> > > > +++ b/configs/T2080QDS_NAND_defconfig
> > > > @@ -38,6 +38,7 @@
> > > >
> > > CONFIG_MTDPARTS_DEFAULT="mtdparts=fe8000000.nor:1m(uboot),5m(kern
> > > el),1
> > > > 28k(dtb),9
> > > > CONFIG_OF_CONTROL=y
> > > > CONFIG_DEFAULT_DEVICE_TREE="t2080qds"
> > > > CONFIG_ENV_IS_IN_NAND=y
> > > > +CONFIG_DM=y
> > > > CONFIG_FSL_CAAM=y
> > > > CONFIG_FSL_ESDHC=y
> > > > CONFIG_MTD_NOR_FLASH=y
> > > > @@ -54,6 +55,9 @@ CONFIG_PHYLIB=y CONFIG_PHY_AQUANTIA=y
> > > > CONFIG_E1000=y CONFIG_MII=y
> > > > +CONFIG_DM_PCI=y
> > > > +CONFIG_DM_PCI_COMPAT=y
> > >
> > > I think we should avoid adding DM_PCI_COMPAT.
> >
> > Without this macro the non-DM Endpoint driver (such as e1000) won't work.
>
> e1000 is a fully DM driver now. Please redo the testing.
> [Xiaowei Bao] I am not very clear, If someone want to implement the customized device driver(not DM) in u-boot, the driver will not work, yes?
We are going to convert all drivers to DM in U-Boot. There are some deadlines to be set already. So you don't need consider such non-dm compatibility.
[Xiaowei Bao] OK, got it, I will remove this macro from all Layerscap platform of NXP, and submit a new patch, thanks a lot.
I guess CONFIG_DM_PCI_COMPAT will be removed once all PCI drivers have been converted.
Regards,
Bin
^ permalink raw reply [flat|nested] 29+ messages in thread
* [U-Boot] [EXT] Re: [PATCHv2 8/8] configs: T2080QDS: Enable PCIe driver
2019-05-24 2:17 ` Xiaowei Bao
@ 2019-05-24 5:23 ` Prabhakar Kushwaha
2019-05-24 6:02 ` Xiaowei Bao
0 siblings, 1 reply; 29+ messages in thread
From: Prabhakar Kushwaha @ 2019-05-24 5:23 UTC (permalink / raw)
To: u-boot
Hi Xiaowei,
> -----Original Message-----
> From: Xiaowei Bao
> Sent: Friday, May 24, 2019 7:47 AM
> To: Bin Meng <bmeng.cn@gmail.com>; Simon Glass <sjg@chromium.org>
> Cc: Z.q. Hou <zhiqiang.hou@nxp.com>; u-boot at lists.denx.de; Prabhakar
> Kushwaha <prabhakar.kushwaha@nxp.com>; wd at denx.de; Shengzhou Liu
> <shengzhou.liu@nxp.com>; Jagdish Gediya <jagdish.gediya@nxp.com>;
> ley.foon.tan at intel.com; sr at denx.de; M.h. Lian <minghuan.lian@nxp.com>
> Subject: RE: [EXT] Re: [PATCHv2 8/8] configs: T2080QDS: Enable PCIe driver
>
>
>
> -----Original Message-----
> From: Bin Meng <bmeng.cn@gmail.com>
> Sent: 2019年5月24日 10:14
> To: Xiaowei Bao <xiaowei.bao@nxp.com>; Simon Glass <sjg@chromium.org>
> Cc: Z.q. Hou <zhiqiang.hou@nxp.com>; u-boot at lists.denx.de; Prabhakar
> Kushwaha <prabhakar.kushwaha@nxp.com>; wd at denx.de; Shengzhou Liu
> <shengzhou.liu@nxp.com>; Jagdish Gediya <jagdish.gediya@nxp.com>;
> ley.foon.tan at intel.com; sr at denx.de; M.h. Lian <minghuan.lian@nxp.com>
> Subject: Re: [EXT] Re: [PATCHv2 8/8] configs: T2080QDS: Enable PCIe driver
>
> Caution: EXT Email
>
> On Fri, May 24, 2019 at 10:10 AM Xiaowei Bao <xiaowei.bao@nxp.com> wrote:
> >
> >
> >
> > -----Original Message-----
> > From: Bin Meng <bmeng.cn@gmail.com>
> > Sent: 2019年5月24日 8:11
> > To: Z.q. Hou <zhiqiang.hou@nxp.com>
> > Cc: u-boot at lists.denx.de; Prabhakar Kushwaha
> > <prabhakar.kushwaha@nxp.com>; wd at denx.de; Shengzhou Liu
> > <shengzhou.liu@nxp.com>; Jagdish Gediya <jagdish.gediya@nxp.com>;
> > sjg at chromium.org; ley.foon.tan at intel.com; sr at denx.de; M.h. Lian
> > <minghuan.lian@nxp.com>; Xiaowei Bao <xiaowei.bao@nxp.com>
> > Subject: [EXT] Re: [PATCHv2 8/8] configs: T2080QDS: Enable PCIe driver
> >
> > Caution: EXT Email
> >
> > Hi,
> >
> > On Fri, May 24, 2019 at 12:04 AM Z.q. Hou <zhiqiang.hou@nxp.com> wrote:
> > >
> > > Hi Bin,
> > >
> > > Thanks a lot for your comments!
> > >
> > > > -----Original Message-----
> > > > From: Bin Meng [mailto:bmeng.cn at gmail.com]
> > > > Sent: 2019年5月23日 22:23
> > > > To: Z.q. Hou <zhiqiang.hou@nxp.com>
> > > > Cc: u-boot at lists.denx.de; Prabhakar Kushwaha
> > > > <prabhakar.kushwaha@nxp.com>; wd at denx.de; Shengzhou Liu
> > > > <shengzhou.liu@nxp.com>; Jagdish Gediya <jagdish.gediya@nxp.com>;
> > > > sjg at chromium.org; ley.foon.tan at intel.com; sr at denx.de; M.h. Lian
> > > > <minghuan.lian@nxp.com>; Xiaowei Bao <xiaowei.bao@nxp.com>
> > > > Subject: Re: [PATCHv2 8/8] configs: T2080QDS: Enable PCIe driver
> > > >
> > > > On Thu, May 23, 2019 at 8:22 PM Z.q. Hou <zhiqiang.hou@nxp.com>
> wrote:
> > > > >
> > > > > From: Hou Zhiqiang <Zhiqiang.Hou@nxp.com>
> > > > >
> > > > > Enable the DM PCIe driver in T2080QDS defconfig.
> > > > >
> > > > > Signed-off-by: Hou Zhiqiang <Zhiqiang.Hou@nxp.com>
> > > > > ---
> > > > > V2:
> > > > > - Enable PCIe in NAND flash, SD card and SPI flash boot defconfig.
> > > > >
> > > > > configs/T2080QDS_NAND_defconfig | 4 ++++
> > > > > configs/T2080QDS_SDCARD_defconfig | 4 ++++
> > > > > configs/T2080QDS_SPIFLASH_defconfig | 4 ++++
> > > > > configs/T2080QDS_defconfig | 4 ++++
> > > > > 4 files changed, 16 insertions(+)
> > > > >
> > > > > diff --git a/configs/T2080QDS_NAND_defconfig
> > > > > b/configs/T2080QDS_NAND_defconfig index 948a05af89..d04e223c23
> > > > 100644
> > > > > --- a/configs/T2080QDS_NAND_defconfig
> > > > > +++ b/configs/T2080QDS_NAND_defconfig
> > > > > @@ -38,6 +38,7 @@
> > > > >
> > > >
> CONFIG_MTDPARTS_DEFAULT="mtdparts=fe8000000.nor:1m(uboot),5m(kern
> > > > el),1
> > > > > 28k(dtb),9
> > > > > CONFIG_OF_CONTROL=y
> > > > > CONFIG_DEFAULT_DEVICE_TREE="t2080qds"
> > > > > CONFIG_ENV_IS_IN_NAND=y
> > > > > +CONFIG_DM=y
> > > > > CONFIG_FSL_CAAM=y
> > > > > CONFIG_FSL_ESDHC=y
> > > > > CONFIG_MTD_NOR_FLASH=y
> > > > > @@ -54,6 +55,9 @@ CONFIG_PHYLIB=y CONFIG_PHY_AQUANTIA=y
> > > > > CONFIG_E1000=y CONFIG_MII=y
> > > > > +CONFIG_DM_PCI=y
> > > > > +CONFIG_DM_PCI_COMPAT=y
> > > >
> > > > I think we should avoid adding DM_PCI_COMPAT.
> > >
> > > Without this macro the non-DM Endpoint driver (such as e1000) won't work.
> >
> > e1000 is a fully DM driver now. Please redo the testing.
> > [Xiaowei Bao] I am not very clear, If someone want to implement the
> customized device driver(not DM) in u-boot, the driver will not work, yes?
>
> We are going to convert all drivers to DM in U-Boot. There are some deadlines
> to be set already. So you don't need consider such non-dm compatibility.
> [Xiaowei Bao] OK, got it, I will remove this macro from all Layerscap platform of
> NXP, and submit a new patch, thanks a lot.
>
Please make sure to test e1000 before sending new patch-set.
--pk
^ permalink raw reply [flat|nested] 29+ messages in thread
* [U-Boot] [EXT] Re: [PATCHv2 8/8] configs: T2080QDS: Enable PCIe driver
2019-05-24 5:23 ` Prabhakar Kushwaha
@ 2019-05-24 6:02 ` Xiaowei Bao
0 siblings, 0 replies; 29+ messages in thread
From: Xiaowei Bao @ 2019-05-24 6:02 UTC (permalink / raw)
To: u-boot
Hi Prabhakar,
-----Original Message-----
From: Prabhakar Kushwaha
Sent: 2019年5月24日 13:23
To: Xiaowei Bao <xiaowei.bao@nxp.com>; Bin Meng <bmeng.cn@gmail.com>; Simon Glass <sjg@chromium.org>
Cc: Z.q. Hou <zhiqiang.hou@nxp.com>; u-boot at lists.denx.de; wd at denx.de; Shengzhou Liu <shengzhou.liu@nxp.com>; Jagdish Gediya <jagdish.gediya@nxp.com>; ley.foon.tan at intel.com; sr at denx.de; M.h. Lian <minghuan.lian@nxp.com>
Subject: RE: [EXT] Re: [PATCHv2 8/8] configs: T2080QDS: Enable PCIe driver
Hi Xiaowei,
> -----Original Message-----
> From: Xiaowei Bao
> Sent: Friday, May 24, 2019 7:47 AM
> To: Bin Meng <bmeng.cn@gmail.com>; Simon Glass <sjg@chromium.org>
> Cc: Z.q. Hou <zhiqiang.hou@nxp.com>; u-boot at lists.denx.de; Prabhakar
> Kushwaha <prabhakar.kushwaha@nxp.com>; wd at denx.de; Shengzhou Liu
> <shengzhou.liu@nxp.com>; Jagdish Gediya <jagdish.gediya@nxp.com>;
> ley.foon.tan at intel.com; sr at denx.de; M.h. Lian <minghuan.lian@nxp.com>
> Subject: RE: [EXT] Re: [PATCHv2 8/8] configs: T2080QDS: Enable PCIe
> driver
>
>
>
> -----Original Message-----
> From: Bin Meng <bmeng.cn@gmail.com>
> Sent: 2019年5月24日 10:14
> To: Xiaowei Bao <xiaowei.bao@nxp.com>; Simon Glass <sjg@chromium.org>
> Cc: Z.q. Hou <zhiqiang.hou@nxp.com>; u-boot at lists.denx.de; Prabhakar
> Kushwaha <prabhakar.kushwaha@nxp.com>; wd at denx.de; Shengzhou Liu
> <shengzhou.liu@nxp.com>; Jagdish Gediya <jagdish.gediya@nxp.com>;
> ley.foon.tan at intel.com; sr at denx.de; M.h. Lian <minghuan.lian@nxp.com>
> Subject: Re: [EXT] Re: [PATCHv2 8/8] configs: T2080QDS: Enable PCIe
> driver
>
> Caution: EXT Email
>
> On Fri, May 24, 2019 at 10:10 AM Xiaowei Bao <xiaowei.bao@nxp.com> wrote:
> >
> >
> >
> > -----Original Message-----
> > From: Bin Meng <bmeng.cn@gmail.com>
> > Sent: 2019年5月24日 8:11
> > To: Z.q. Hou <zhiqiang.hou@nxp.com>
> > Cc: u-boot at lists.denx.de; Prabhakar Kushwaha
> > <prabhakar.kushwaha@nxp.com>; wd at denx.de; Shengzhou Liu
> > <shengzhou.liu@nxp.com>; Jagdish Gediya <jagdish.gediya@nxp.com>;
> > sjg at chromium.org; ley.foon.tan at intel.com; sr at denx.de; M.h. Lian
> > <minghuan.lian@nxp.com>; Xiaowei Bao <xiaowei.bao@nxp.com>
> > Subject: [EXT] Re: [PATCHv2 8/8] configs: T2080QDS: Enable PCIe
> > driver
> >
> > Caution: EXT Email
> >
> > Hi,
> >
> > On Fri, May 24, 2019 at 12:04 AM Z.q. Hou <zhiqiang.hou@nxp.com> wrote:
> > >
> > > Hi Bin,
> > >
> > > Thanks a lot for your comments!
> > >
> > > > -----Original Message-----
> > > > From: Bin Meng [mailto:bmeng.cn at gmail.com]
> > > > Sent: 2019年5月23日 22:23
> > > > To: Z.q. Hou <zhiqiang.hou@nxp.com>
> > > > Cc: u-boot at lists.denx.de; Prabhakar Kushwaha
> > > > <prabhakar.kushwaha@nxp.com>; wd at denx.de; Shengzhou Liu
> > > > <shengzhou.liu@nxp.com>; Jagdish Gediya
> > > > <jagdish.gediya@nxp.com>; sjg at chromium.org;
> > > > ley.foon.tan at intel.com; sr at denx.de; M.h. Lian
> > > > <minghuan.lian@nxp.com>; Xiaowei Bao <xiaowei.bao@nxp.com>
> > > > Subject: Re: [PATCHv2 8/8] configs: T2080QDS: Enable PCIe driver
> > > >
> > > > On Thu, May 23, 2019 at 8:22 PM Z.q. Hou <zhiqiang.hou@nxp.com>
> wrote:
> > > > >
> > > > > From: Hou Zhiqiang <Zhiqiang.Hou@nxp.com>
> > > > >
> > > > > Enable the DM PCIe driver in T2080QDS defconfig.
> > > > >
> > > > > Signed-off-by: Hou Zhiqiang <Zhiqiang.Hou@nxp.com>
> > > > > ---
> > > > > V2:
> > > > > - Enable PCIe in NAND flash, SD card and SPI flash boot defconfig.
> > > > >
> > > > > configs/T2080QDS_NAND_defconfig | 4 ++++
> > > > > configs/T2080QDS_SDCARD_defconfig | 4 ++++
> > > > > configs/T2080QDS_SPIFLASH_defconfig | 4 ++++
> > > > > configs/T2080QDS_defconfig | 4 ++++
> > > > > 4 files changed, 16 insertions(+)
> > > > >
> > > > > diff --git a/configs/T2080QDS_NAND_defconfig
> > > > > b/configs/T2080QDS_NAND_defconfig index 948a05af89..d04e223c23
> > > > 100644
> > > > > --- a/configs/T2080QDS_NAND_defconfig
> > > > > +++ b/configs/T2080QDS_NAND_defconfig
> > > > > @@ -38,6 +38,7 @@
> > > > >
> > > >
> CONFIG_MTDPARTS_DEFAULT="mtdparts=fe8000000.nor:1m(uboot),5m(kern
> > > > el),1
> > > > > 28k(dtb),9
> > > > > CONFIG_OF_CONTROL=y
> > > > > CONFIG_DEFAULT_DEVICE_TREE="t2080qds"
> > > > > CONFIG_ENV_IS_IN_NAND=y
> > > > > +CONFIG_DM=y
> > > > > CONFIG_FSL_CAAM=y
> > > > > CONFIG_FSL_ESDHC=y
> > > > > CONFIG_MTD_NOR_FLASH=y
> > > > > @@ -54,6 +55,9 @@ CONFIG_PHYLIB=y CONFIG_PHY_AQUANTIA=y
> > > > > CONFIG_E1000=y CONFIG_MII=y
> > > > > +CONFIG_DM_PCI=y
> > > > > +CONFIG_DM_PCI_COMPAT=y
> > > >
> > > > I think we should avoid adding DM_PCI_COMPAT.
> > >
> > > Without this macro the non-DM Endpoint driver (such as e1000) won't work.
> >
> > e1000 is a fully DM driver now. Please redo the testing.
> > [Xiaowei Bao] I am not very clear, If someone want to implement the
> customized device driver(not DM) in u-boot, the driver will not work, yes?
>
> We are going to convert all drivers to DM in U-Boot. There are some
> deadlines to be set already. So you don't need consider such non-dm compatibility.
> [Xiaowei Bao] OK, got it, I will remove this macro from all Layerscap
> platform of NXP, and submit a new patch, thanks a lot.
>
Please make sure to test e1000 before sending new patch-set.
[Xiaowei Bao] OK
--pk
^ permalink raw reply [flat|nested] 29+ messages in thread
* [U-Boot] [EXT] Re: [PATCHv2 8/8] configs: T2080QDS: Enable PCIe driver
2019-05-24 2:13 ` Bin Meng
2019-05-24 2:17 ` Xiaowei Bao
@ 2019-05-24 8:17 ` Z.q. Hou
2019-05-24 9:38 ` Bin Meng
1 sibling, 1 reply; 29+ messages in thread
From: Z.q. Hou @ 2019-05-24 8:17 UTC (permalink / raw)
To: u-boot
Hi Bin and Prabhakar,
Thanks a lot for your comments!
> -----Original Message-----
> From: Bin Meng [mailto:bmeng.cn at gmail.com]
> Sent: 2019年5月24日 10:14
> To: Xiaowei Bao <xiaowei.bao@nxp.com>; Simon Glass <sjg@chromium.org>
> Cc: Z.q. Hou <zhiqiang.hou@nxp.com>; u-boot at lists.denx.de; Prabhakar
> Kushwaha <prabhakar.kushwaha@nxp.com>; wd at denx.de; Shengzhou Liu
> <shengzhou.liu@nxp.com>; Jagdish Gediya <jagdish.gediya@nxp.com>;
> ley.foon.tan at intel.com; sr at denx.de; M.h. Lian <minghuan.lian@nxp.com>
> Subject: Re: [EXT] Re: [PATCHv2 8/8] configs: T2080QDS: Enable PCIe driver
>
> On Fri, May 24, 2019 at 10:10 AM Xiaowei Bao <xiaowei.bao@nxp.com>
> wrote:
> >
> >
> >
> > -----Original Message-----
> > From: Bin Meng <bmeng.cn@gmail.com>
> > Sent: 2019年5月24日 8:11
> > To: Z.q. Hou <zhiqiang.hou@nxp.com>
> > Cc: u-boot at lists.denx.de; Prabhakar Kushwaha
> > <prabhakar.kushwaha@nxp.com>; wd at denx.de; Shengzhou Liu
> > <shengzhou.liu@nxp.com>; Jagdish Gediya <jagdish.gediya@nxp.com>;
> > sjg at chromium.org; ley.foon.tan at intel.com; sr at denx.de; M.h. Lian
> > <minghuan.lian@nxp.com>; Xiaowei Bao <xiaowei.bao@nxp.com>
> > Subject: [EXT] Re: [PATCHv2 8/8] configs: T2080QDS: Enable PCIe driver
> >
> > Caution: EXT Email
> >
> > Hi,
> >
> > On Fri, May 24, 2019 at 12:04 AM Z.q. Hou <zhiqiang.hou@nxp.com> wrote:
> > >
> > > Hi Bin,
> > >
> > > Thanks a lot for your comments!
> > >
> > > > -----Original Message-----
> > > > From: Bin Meng [mailto:bmeng.cn at gmail.com]
> > > > Sent: 2019年5月23日 22:23
> > > > To: Z.q. Hou <zhiqiang.hou@nxp.com>
> > > > Cc: u-boot at lists.denx.de; Prabhakar Kushwaha
> > > > <prabhakar.kushwaha@nxp.com>; wd at denx.de; Shengzhou Liu
> > > > <shengzhou.liu@nxp.com>; Jagdish Gediya <jagdish.gediya@nxp.com>;
> > > > sjg at chromium.org; ley.foon.tan at intel.com; sr at denx.de; M.h. Lian
> > > > <minghuan.lian@nxp.com>; Xiaowei Bao <xiaowei.bao@nxp.com>
> > > > Subject: Re: [PATCHv2 8/8] configs: T2080QDS: Enable PCIe driver
> > > >
> > > > On Thu, May 23, 2019 at 8:22 PM Z.q. Hou <zhiqiang.hou@nxp.com>
> wrote:
> > > > >
> > > > > From: Hou Zhiqiang <Zhiqiang.Hou@nxp.com>
> > > > >
> > > > > Enable the DM PCIe driver in T2080QDS defconfig.
> > > > >
> > > > > Signed-off-by: Hou Zhiqiang <Zhiqiang.Hou@nxp.com>
> > > > > ---
> > > > > V2:
> > > > > - Enable PCIe in NAND flash, SD card and SPI flash boot defconfig.
> > > > >
> > > > > configs/T2080QDS_NAND_defconfig | 4 ++++
> > > > > configs/T2080QDS_SDCARD_defconfig | 4 ++++
> > > > > configs/T2080QDS_SPIFLASH_defconfig | 4 ++++
> > > > > configs/T2080QDS_defconfig | 4 ++++
> > > > > 4 files changed, 16 insertions(+)
> > > > >
> > > > > diff --git a/configs/T2080QDS_NAND_defconfig
> > > > > b/configs/T2080QDS_NAND_defconfig index 948a05af89..d04e223c23
> > > > 100644
> > > > > --- a/configs/T2080QDS_NAND_defconfig
> > > > > +++ b/configs/T2080QDS_NAND_defconfig
> > > > > @@ -38,6 +38,7 @@
> > > > >
> > > >
> CONFIG_MTDPARTS_DEFAULT="mtdparts=fe8000000.nor:1m(uboot),5m(kern
> > > > el),1
> > > > > 28k(dtb),9
> > > > > CONFIG_OF_CONTROL=y
> > > > > CONFIG_DEFAULT_DEVICE_TREE="t2080qds"
> > > > > CONFIG_ENV_IS_IN_NAND=y
> > > > > +CONFIG_DM=y
> > > > > CONFIG_FSL_CAAM=y
> > > > > CONFIG_FSL_ESDHC=y
> > > > > CONFIG_MTD_NOR_FLASH=y
> > > > > @@ -54,6 +55,9 @@ CONFIG_PHYLIB=y
> CONFIG_PHY_AQUANTIA=y
> > > > > CONFIG_E1000=y CONFIG_MII=y
> > > > > +CONFIG_DM_PCI=y
> > > > > +CONFIG_DM_PCI_COMPAT=y
> > > >
> > > > I think we should avoid adding DM_PCI_COMPAT.
> > >
> > > Without this macro the non-DM Endpoint driver (such as e1000) won't
> work.
> >
> > e1000 is a fully DM driver now. Please redo the testing.
> > [Xiaowei Bao] I am not very clear, If someone want to implement the
> customized device driver(not DM) in u-boot, the driver will not work, yes?
>
> We are going to convert all drivers to DM in U-Boot. There are some
> deadlines to be set already. So you don't need consider such non-dm
> compatibility.
To enable e1000 DM driver, I added the DM_ETH to T2080QDS_defconfig, but some
build errors arose, seems FMan driver is not DM compatible.
The DM e1000 driver works well with DPAA FMan driver disabled.
> I guess CONFIG_DM_PCI_COMPAT will be removed once all PCI drivers have
> been converted.
I think we'd better keep it in defconfig currently, and will remove it when all EP
drivers have been converted.
>
> Regards,
> Bin
Thanks,
Zhiqiang
^ permalink raw reply [flat|nested] 29+ messages in thread
* [U-Boot] [EXT] Re: [PATCHv2 8/8] configs: T2080QDS: Enable PCIe driver
2019-05-24 8:17 ` Z.q. Hou
@ 2019-05-24 9:38 ` Bin Meng
2019-05-24 14:12 ` Z.q. Hou
0 siblings, 1 reply; 29+ messages in thread
From: Bin Meng @ 2019-05-24 9:38 UTC (permalink / raw)
To: u-boot
Hi Zhiqiang,
On Fri, May 24, 2019 at 4:17 PM Z.q. Hou <zhiqiang.hou@nxp.com> wrote:
>
> Hi Bin and Prabhakar,
>
> Thanks a lot for your comments!
>
> > -----Original Message-----
> > From: Bin Meng [mailto:bmeng.cn at gmail.com]
> > Sent: 2019年5月24日 10:14
> > To: Xiaowei Bao <xiaowei.bao@nxp.com>; Simon Glass <sjg@chromium.org>
> > Cc: Z.q. Hou <zhiqiang.hou@nxp.com>; u-boot at lists.denx.de; Prabhakar
> > Kushwaha <prabhakar.kushwaha@nxp.com>; wd at denx.de; Shengzhou Liu
> > <shengzhou.liu@nxp.com>; Jagdish Gediya <jagdish.gediya@nxp.com>;
> > ley.foon.tan at intel.com; sr at denx.de; M.h. Lian <minghuan.lian@nxp.com>
> > Subject: Re: [EXT] Re: [PATCHv2 8/8] configs: T2080QDS: Enable PCIe driver
> >
> > On Fri, May 24, 2019 at 10:10 AM Xiaowei Bao <xiaowei.bao@nxp.com>
> > wrote:
> > >
> > >
> > >
> > > -----Original Message-----
> > > From: Bin Meng <bmeng.cn@gmail.com>
> > > Sent: 2019年5月24日 8:11
> > > To: Z.q. Hou <zhiqiang.hou@nxp.com>
> > > Cc: u-boot at lists.denx.de; Prabhakar Kushwaha
> > > <prabhakar.kushwaha@nxp.com>; wd at denx.de; Shengzhou Liu
> > > <shengzhou.liu@nxp.com>; Jagdish Gediya <jagdish.gediya@nxp.com>;
> > > sjg at chromium.org; ley.foon.tan at intel.com; sr at denx.de; M.h. Lian
> > > <minghuan.lian@nxp.com>; Xiaowei Bao <xiaowei.bao@nxp.com>
> > > Subject: [EXT] Re: [PATCHv2 8/8] configs: T2080QDS: Enable PCIe driver
> > >
> > > Caution: EXT Email
> > >
> > > Hi,
> > >
> > > On Fri, May 24, 2019 at 12:04 AM Z.q. Hou <zhiqiang.hou@nxp.com> wrote:
> > > >
> > > > Hi Bin,
> > > >
> > > > Thanks a lot for your comments!
> > > >
> > > > > -----Original Message-----
> > > > > From: Bin Meng [mailto:bmeng.cn at gmail.com]
> > > > > Sent: 2019年5月23日 22:23
> > > > > To: Z.q. Hou <zhiqiang.hou@nxp.com>
> > > > > Cc: u-boot at lists.denx.de; Prabhakar Kushwaha
> > > > > <prabhakar.kushwaha@nxp.com>; wd at denx.de; Shengzhou Liu
> > > > > <shengzhou.liu@nxp.com>; Jagdish Gediya <jagdish.gediya@nxp.com>;
> > > > > sjg at chromium.org; ley.foon.tan at intel.com; sr at denx.de; M.h. Lian
> > > > > <minghuan.lian@nxp.com>; Xiaowei Bao <xiaowei.bao@nxp.com>
> > > > > Subject: Re: [PATCHv2 8/8] configs: T2080QDS: Enable PCIe driver
> > > > >
> > > > > On Thu, May 23, 2019 at 8:22 PM Z.q. Hou <zhiqiang.hou@nxp.com>
> > wrote:
> > > > > >
> > > > > > From: Hou Zhiqiang <Zhiqiang.Hou@nxp.com>
> > > > > >
> > > > > > Enable the DM PCIe driver in T2080QDS defconfig.
> > > > > >
> > > > > > Signed-off-by: Hou Zhiqiang <Zhiqiang.Hou@nxp.com>
> > > > > > ---
> > > > > > V2:
> > > > > > - Enable PCIe in NAND flash, SD card and SPI flash boot defconfig.
> > > > > >
> > > > > > configs/T2080QDS_NAND_defconfig | 4 ++++
> > > > > > configs/T2080QDS_SDCARD_defconfig | 4 ++++
> > > > > > configs/T2080QDS_SPIFLASH_defconfig | 4 ++++
> > > > > > configs/T2080QDS_defconfig | 4 ++++
> > > > > > 4 files changed, 16 insertions(+)
> > > > > >
> > > > > > diff --git a/configs/T2080QDS_NAND_defconfig
> > > > > > b/configs/T2080QDS_NAND_defconfig index 948a05af89..d04e223c23
> > > > > 100644
> > > > > > --- a/configs/T2080QDS_NAND_defconfig
> > > > > > +++ b/configs/T2080QDS_NAND_defconfig
> > > > > > @@ -38,6 +38,7 @@
> > > > > >
> > > > >
> > CONFIG_MTDPARTS_DEFAULT="mtdparts=fe8000000.nor:1m(uboot),5m(kern
> > > > > el),1
> > > > > > 28k(dtb),9
> > > > > > CONFIG_OF_CONTROL=y
> > > > > > CONFIG_DEFAULT_DEVICE_TREE="t2080qds"
> > > > > > CONFIG_ENV_IS_IN_NAND=y
> > > > > > +CONFIG_DM=y
> > > > > > CONFIG_FSL_CAAM=y
> > > > > > CONFIG_FSL_ESDHC=y
> > > > > > CONFIG_MTD_NOR_FLASH=y
> > > > > > @@ -54,6 +55,9 @@ CONFIG_PHYLIB=y
> > CONFIG_PHY_AQUANTIA=y
> > > > > > CONFIG_E1000=y CONFIG_MII=y
> > > > > > +CONFIG_DM_PCI=y
> > > > > > +CONFIG_DM_PCI_COMPAT=y
> > > > >
> > > > > I think we should avoid adding DM_PCI_COMPAT.
> > > >
> > > > Without this macro the non-DM Endpoint driver (such as e1000) won't
> > work.
> > >
> > > e1000 is a fully DM driver now. Please redo the testing.
> > > [Xiaowei Bao] I am not very clear, If someone want to implement the
> > customized device driver(not DM) in u-boot, the driver will not work, yes?
> >
> > We are going to convert all drivers to DM in U-Boot. There are some
> > deadlines to be set already. So you don't need consider such non-dm
> > compatibility.
>
> To enable e1000 DM driver, I added the DM_ETH to T2080QDS_defconfig, but some
> build errors arose, seems FMan driver is not DM compatible.
> The DM e1000 driver works well with DPAA FMan driver disabled.
>
This is expected. E1000 requires DM_ETH, that means all ethernet
drivers in your system have to be DM. So do you have plan to convert
the FMAN ethernet driver to DM?
> > I guess CONFIG_DM_PCI_COMPAT will be removed once all PCI drivers have
> > been converted.
>
> I think we'd better keep it in defconfig currently, and will remove it when all EP
> drivers have been converted.
Regards,
Bin
^ permalink raw reply [flat|nested] 29+ messages in thread
* [U-Boot] [EXT] Re: [PATCHv2 8/8] configs: T2080QDS: Enable PCIe driver
2019-05-24 9:38 ` Bin Meng
@ 2019-05-24 14:12 ` Z.q. Hou
2019-06-18 0:37 ` Prabhakar Kushwaha
0 siblings, 1 reply; 29+ messages in thread
From: Z.q. Hou @ 2019-05-24 14:12 UTC (permalink / raw)
To: u-boot
Hi Bin,
> -----Original Message-----
> From: Bin Meng [mailto:bmeng.cn at gmail.com]
> Sent: 2019年5月24日 17:38
> To: Z.q. Hou <zhiqiang.hou@nxp.com>
> Cc: Xiaowei Bao <xiaowei.bao@nxp.com>; Simon Glass <sjg@chromium.org>;
> u-boot at lists.denx.de; Prabhakar Kushwaha
> <prabhakar.kushwaha@nxp.com>; wd at denx.de; Shengzhou Liu
> <shengzhou.liu@nxp.com>; Jagdish Gediya <jagdish.gediya@nxp.com>;
> ley.foon.tan at intel.com; sr at denx.de; M.h. Lian <minghuan.lian@nxp.com>
> Subject: Re: [EXT] Re: [PATCHv2 8/8] configs: T2080QDS: Enable PCIe driver
>
> Hi Zhiqiang,
>
> On Fri, May 24, 2019 at 4:17 PM Z.q. Hou <zhiqiang.hou@nxp.com> wrote:
> >
> > Hi Bin and Prabhakar,
> >
> > Thanks a lot for your comments!
> >
> > > -----Original Message-----
> > > From: Bin Meng [mailto:bmeng.cn at gmail.com]
> > > Sent: 2019年5月24日 10:14
> > > To: Xiaowei Bao <xiaowei.bao@nxp.com>; Simon Glass
> > > <sjg@chromium.org>
> > > Cc: Z.q. Hou <zhiqiang.hou@nxp.com>; u-boot at lists.denx.de; Prabhakar
> > > Kushwaha <prabhakar.kushwaha@nxp.com>; wd at denx.de; Shengzhou
> Liu
> > > <shengzhou.liu@nxp.com>; Jagdish Gediya <jagdish.gediya@nxp.com>;
> > > ley.foon.tan at intel.com; sr at denx.de; M.h. Lian
> > > <minghuan.lian@nxp.com>
> > > Subject: Re: [EXT] Re: [PATCHv2 8/8] configs: T2080QDS: Enable PCIe
> > > driver
> > >
> > > On Fri, May 24, 2019 at 10:10 AM Xiaowei Bao <xiaowei.bao@nxp.com>
> > > wrote:
> > > >
> > > >
> > > >
> > > > -----Original Message-----
> > > > From: Bin Meng <bmeng.cn@gmail.com>
> > > > Sent: 2019年5月24日 8:11
> > > > To: Z.q. Hou <zhiqiang.hou@nxp.com>
> > > > Cc: u-boot at lists.denx.de; Prabhakar Kushwaha
> > > > <prabhakar.kushwaha@nxp.com>; wd at denx.de; Shengzhou Liu
> > > > <shengzhou.liu@nxp.com>; Jagdish Gediya <jagdish.gediya@nxp.com>;
> > > > sjg at chromium.org; ley.foon.tan at intel.com; sr at denx.de; M.h. Lian
> > > > <minghuan.lian@nxp.com>; Xiaowei Bao <xiaowei.bao@nxp.com>
> > > > Subject: [EXT] Re: [PATCHv2 8/8] configs: T2080QDS: Enable PCIe
> > > > driver
> > > >
> > > > Caution: EXT Email
> > > >
> > > > Hi,
> > > >
> > > > On Fri, May 24, 2019 at 12:04 AM Z.q. Hou <zhiqiang.hou@nxp.com>
> wrote:
> > > > >
> > > > > Hi Bin,
> > > > >
> > > > > Thanks a lot for your comments!
> > > > >
> > > > > > -----Original Message-----
> > > > > > From: Bin Meng [mailto:bmeng.cn at gmail.com]
> > > > > > Sent: 2019年5月23日 22:23
> > > > > > To: Z.q. Hou <zhiqiang.hou@nxp.com>
> > > > > > Cc: u-boot at lists.denx.de; Prabhakar Kushwaha
> > > > > > <prabhakar.kushwaha@nxp.com>; wd at denx.de; Shengzhou Liu
> > > > > > <shengzhou.liu@nxp.com>; Jagdish Gediya
> > > > > > <jagdish.gediya@nxp.com>; sjg at chromium.org;
> > > > > > ley.foon.tan at intel.com; sr at denx.de; M.h. Lian
> > > > > > <minghuan.lian@nxp.com>; Xiaowei Bao <xiaowei.bao@nxp.com>
> > > > > > Subject: Re: [PATCHv2 8/8] configs: T2080QDS: Enable PCIe
> > > > > > driver
> > > > > >
> > > > > > On Thu, May 23, 2019 at 8:22 PM Z.q. Hou
> > > > > > <zhiqiang.hou@nxp.com>
> > > wrote:
> > > > > > >
> > > > > > > From: Hou Zhiqiang <Zhiqiang.Hou@nxp.com>
> > > > > > >
> > > > > > > Enable the DM PCIe driver in T2080QDS defconfig.
> > > > > > >
> > > > > > > Signed-off-by: Hou Zhiqiang <Zhiqiang.Hou@nxp.com>
> > > > > > > ---
> > > > > > > V2:
> > > > > > > - Enable PCIe in NAND flash, SD card and SPI flash boot defconfig.
> > > > > > >
> > > > > > > configs/T2080QDS_NAND_defconfig | 4 ++++
> > > > > > > configs/T2080QDS_SDCARD_defconfig | 4 ++++
> > > > > > > configs/T2080QDS_SPIFLASH_defconfig | 4 ++++
> > > > > > > configs/T2080QDS_defconfig | 4 ++++
> > > > > > > 4 files changed, 16 insertions(+)
> > > > > > >
> > > > > > > diff --git a/configs/T2080QDS_NAND_defconfig
> > > > > > > b/configs/T2080QDS_NAND_defconfig index
> > > > > > > 948a05af89..d04e223c23
> > > > > > 100644
> > > > > > > --- a/configs/T2080QDS_NAND_defconfig
> > > > > > > +++ b/configs/T2080QDS_NAND_defconfig
> > > > > > > @@ -38,6 +38,7 @@
> > > > > > >
> > > > > >
> > >
> CONFIG_MTDPARTS_DEFAULT="mtdparts=fe8000000.nor:1m(uboot),5m(kern
> > > > > > el),1
> > > > > > > 28k(dtb),9
> > > > > > > CONFIG_OF_CONTROL=y
> > > > > > > CONFIG_DEFAULT_DEVICE_TREE="t2080qds"
> > > > > > > CONFIG_ENV_IS_IN_NAND=y
> > > > > > > +CONFIG_DM=y
> > > > > > > CONFIG_FSL_CAAM=y
> > > > > > > CONFIG_FSL_ESDHC=y
> > > > > > > CONFIG_MTD_NOR_FLASH=y
> > > > > > > @@ -54,6 +55,9 @@ CONFIG_PHYLIB=y
> > > CONFIG_PHY_AQUANTIA=y
> > > > > > > CONFIG_E1000=y CONFIG_MII=y
> > > > > > > +CONFIG_DM_PCI=y
> > > > > > > +CONFIG_DM_PCI_COMPAT=y
> > > > > >
> > > > > > I think we should avoid adding DM_PCI_COMPAT.
> > > > >
> > > > > Without this macro the non-DM Endpoint driver (such as e1000)
> > > > > won't
> > > work.
> > > >
> > > > e1000 is a fully DM driver now. Please redo the testing.
> > > > [Xiaowei Bao] I am not very clear, If someone want to implement
> > > > the
> > > customized device driver(not DM) in u-boot, the driver will not work, yes?
> > >
> > > We are going to convert all drivers to DM in U-Boot. There are some
> > > deadlines to be set already. So you don't need consider such non-dm
> > > compatibility.
> >
> > To enable e1000 DM driver, I added the DM_ETH to T2080QDS_defconfig,
> > but some build errors arose, seems FMan driver is not DM compatible.
> > The DM e1000 driver works well with DPAA FMan driver disabled.
> >
>
> This is expected. E1000 requires DM_ETH, that means all ethernet drivers in
> your system have to be DM. So do you have plan to convert the FMAN
> ethernet driver to DM?
No, I don't, perhaps my colleague is working on this, I will submit a patch to
remove the DM_PCI_COMPAT when all driver has been converted.
>
> > > I guess CONFIG_DM_PCI_COMPAT will be removed once all PCI drivers
> > > have been converted.
> >
> > I think we'd better keep it in defconfig currently, and will remove it
> > when all EP drivers have been converted.
>
> Regards,
> Bin
Thanks,
Zhiqiang
^ permalink raw reply [flat|nested] 29+ messages in thread
* [U-Boot] [EXT] Re: [PATCHv2 8/8] configs: T2080QDS: Enable PCIe driver
2019-05-24 14:12 ` Z.q. Hou
@ 2019-06-18 0:37 ` Prabhakar Kushwaha
0 siblings, 0 replies; 29+ messages in thread
From: Prabhakar Kushwaha @ 2019-06-18 0:37 UTC (permalink / raw)
To: u-boot
Hi Bin,
> -----Original Message-----
> From: Z.q. Hou
> Sent: Friday, May 24, 2019 7:42 PM
> To: Bin Meng <bmeng.cn@gmail.com>
> Cc: Xiaowei Bao <xiaowei.bao@nxp.com>; Simon Glass <sjg@chromium.org>;
> u-boot at lists.denx.de; Prabhakar Kushwaha <prabhakar.kushwaha@nxp.com>;
> wd at denx.de; Shengzhou Liu <shengzhou.liu@nxp.com>; Jagdish Gediya
> <jagdish.gediya@nxp.com>; ley.foon.tan at intel.com; sr at denx.de; M.h. Lian
> <minghuan.lian@nxp.com>
> Subject: RE: [EXT] Re: [PATCHv2 8/8] configs: T2080QDS: Enable PCIe driver
>
> Hi Bin,
>
> > -----Original Message-----
> > From: Bin Meng [mailto:bmeng.cn at gmail.com]
> > Sent: 2019年5月24日 17:38
> > To: Z.q. Hou <zhiqiang.hou@nxp.com>
> > Cc: Xiaowei Bao <xiaowei.bao@nxp.com>; Simon Glass <sjg@chromium.org>;
> > u-boot at lists.denx.de; Prabhakar Kushwaha
> <prabhakar.kushwaha@nxp.com>;
> > wd at denx.de; Shengzhou Liu <shengzhou.liu@nxp.com>; Jagdish Gediya
> > <jagdish.gediya@nxp.com>; ley.foon.tan at intel.com; sr at denx.de; M.h.
> > Lian <minghuan.lian@nxp.com>
> > Subject: Re: [EXT] Re: [PATCHv2 8/8] configs: T2080QDS: Enable PCIe
> > driver
> >
> > Hi Zhiqiang,
> >
> > On Fri, May 24, 2019 at 4:17 PM Z.q. Hou <zhiqiang.hou@nxp.com> wrote:
> > >
> > > Hi Bin and Prabhakar,
> > >
> > > Thanks a lot for your comments!
> > >
> > > > -----Original Message-----
> > > > From: Bin Meng [mailto:bmeng.cn at gmail.com]
> > > > Sent: 2019年5月24日 10:14
> > > > To: Xiaowei Bao <xiaowei.bao@nxp.com>; Simon Glass
> > > > <sjg@chromium.org>
> > > > Cc: Z.q. Hou <zhiqiang.hou@nxp.com>; u-boot at lists.denx.de;
> > > > Prabhakar Kushwaha <prabhakar.kushwaha@nxp.com>; wd at denx.de;
> > > > Shengzhou
> > Liu
> > > > <shengzhou.liu@nxp.com>; Jagdish Gediya <jagdish.gediya@nxp.com>;
> > > > ley.foon.tan at intel.com; sr at denx.de; M.h. Lian
> > > > <minghuan.lian@nxp.com>
> > > > Subject: Re: [EXT] Re: [PATCHv2 8/8] configs: T2080QDS: Enable
> > > > PCIe driver
> > > >
> > > > On Fri, May 24, 2019 at 10:10 AM Xiaowei Bao <xiaowei.bao@nxp.com>
> > > > wrote:
> > > > >
> > > > >
> > > > >
> > > > > -----Original Message-----
> > > > > From: Bin Meng <bmeng.cn@gmail.com>
> > > > > Sent: 2019年5月24日 8:11
> > > > > To: Z.q. Hou <zhiqiang.hou@nxp.com>
> > > > > Cc: u-boot at lists.denx.de; Prabhakar Kushwaha
> > > > > <prabhakar.kushwaha@nxp.com>; wd at denx.de; Shengzhou Liu
> > > > > <shengzhou.liu@nxp.com>; Jagdish Gediya
> > > > > <jagdish.gediya@nxp.com>; sjg at chromium.org;
> > > > > ley.foon.tan at intel.com; sr at denx.de; M.h. Lian
> > > > > <minghuan.lian@nxp.com>; Xiaowei Bao <xiaowei.bao@nxp.com>
> > > > > Subject: [EXT] Re: [PATCHv2 8/8] configs: T2080QDS: Enable PCIe
> > > > > driver
> > > > >
> > > > > Caution: EXT Email
> > > > >
> > > > > Hi,
> > > > >
> > > > > On Fri, May 24, 2019 at 12:04 AM Z.q. Hou <zhiqiang.hou@nxp.com>
> > wrote:
> > > > > >
> > > > > > Hi Bin,
> > > > > >
> > > > > > Thanks a lot for your comments!
> > > > > >
> > > > > > > -----Original Message-----
> > > > > > > From: Bin Meng [mailto:bmeng.cn at gmail.com]
> > > > > > > Sent: 2019年5月23日 22:23
> > > > > > > To: Z.q. Hou <zhiqiang.hou@nxp.com>
> > > > > > > Cc: u-boot at lists.denx.de; Prabhakar Kushwaha
> > > > > > > <prabhakar.kushwaha@nxp.com>; wd at denx.de; Shengzhou Liu
> > > > > > > <shengzhou.liu@nxp.com>; Jagdish Gediya
> > > > > > > <jagdish.gediya@nxp.com>; sjg at chromium.org;
> > > > > > > ley.foon.tan at intel.com; sr at denx.de; M.h. Lian
> > > > > > > <minghuan.lian@nxp.com>; Xiaowei Bao <xiaowei.bao@nxp.com>
> > > > > > > Subject: Re: [PATCHv2 8/8] configs: T2080QDS: Enable PCIe
> > > > > > > driver
> > > > > > >
> > > > > > > On Thu, May 23, 2019 at 8:22 PM Z.q. Hou
> > > > > > > <zhiqiang.hou@nxp.com>
> > > > wrote:
> > > > > > > >
> > > > > > > > From: Hou Zhiqiang <Zhiqiang.Hou@nxp.com>
> > > > > > > >
> > > > > > > > Enable the DM PCIe driver in T2080QDS defconfig.
> > > > > > > >
> > > > > > > > Signed-off-by: Hou Zhiqiang <Zhiqiang.Hou@nxp.com>
> > > > > > > > ---
> > > > > > > > V2:
> > > > > > > > - Enable PCIe in NAND flash, SD card and SPI flash boot defconfig.
> > > > > > > >
> > > > > > > > configs/T2080QDS_NAND_defconfig | 4 ++++
> > > > > > > > configs/T2080QDS_SDCARD_defconfig | 4 ++++
> > > > > > > > configs/T2080QDS_SPIFLASH_defconfig | 4 ++++
> > > > > > > > configs/T2080QDS_defconfig | 4 ++++
> > > > > > > > 4 files changed, 16 insertions(+)
> > > > > > > >
> > > > > > > > diff --git a/configs/T2080QDS_NAND_defconfig
> > > > > > > > b/configs/T2080QDS_NAND_defconfig index
> > > > > > > > 948a05af89..d04e223c23
> > > > > > > 100644
> > > > > > > > --- a/configs/T2080QDS_NAND_defconfig
> > > > > > > > +++ b/configs/T2080QDS_NAND_defconfig
> > > > > > > > @@ -38,6 +38,7 @@
> > > > > > > >
> > > > > > >
> > > >
> > CONFIG_MTDPARTS_DEFAULT="mtdparts=fe8000000.nor:1m(uboot),5m(kern
> > > > > > > el),1
> > > > > > > > 28k(dtb),9
> > > > > > > > CONFIG_OF_CONTROL=y
> > > > > > > > CONFIG_DEFAULT_DEVICE_TREE="t2080qds"
> > > > > > > > CONFIG_ENV_IS_IN_NAND=y
> > > > > > > > +CONFIG_DM=y
> > > > > > > > CONFIG_FSL_CAAM=y
> > > > > > > > CONFIG_FSL_ESDHC=y
> > > > > > > > CONFIG_MTD_NOR_FLASH=y
> > > > > > > > @@ -54,6 +55,9 @@ CONFIG_PHYLIB=y
> > > > CONFIG_PHY_AQUANTIA=y
> > > > > > > > CONFIG_E1000=y CONFIG_MII=y
> > > > > > > > +CONFIG_DM_PCI=y
> > > > > > > > +CONFIG_DM_PCI_COMPAT=y
> > > > > > >
> > > > > > > I think we should avoid adding DM_PCI_COMPAT.
> > > > > >
> > > > > > Without this macro the non-DM Endpoint driver (such as e1000)
> > > > > > won't
> > > > work.
> > > > >
> > > > > e1000 is a fully DM driver now. Please redo the testing.
> > > > > [Xiaowei Bao] I am not very clear, If someone want to implement
> > > > > the
> > > > customized device driver(not DM) in u-boot, the driver will not work, yes?
> > > >
> > > > We are going to convert all drivers to DM in U-Boot. There are
> > > > some deadlines to be set already. So you don't need consider such
> > > > non-dm compatibility.
> > >
> > > To enable e1000 DM driver, I added the DM_ETH to T2080QDS_defconfig,
> > > but some build errors arose, seems FMan driver is not DM compatible.
> > > The DM e1000 driver works well with DPAA FMan driver disabled.
> > >
> >
> > This is expected. E1000 requires DM_ETH, that means all ethernet
> > drivers in your system have to be DM. So do you have plan to convert
> > the FMAN ethernet driver to DM?
>
> No, I don't, perhaps my colleague is working on this, I will submit a patch to
> remove the DM_PCI_COMPAT when all driver has been converted.
> >
Considering there is no immediate plan to move FMAN ethernet to DM model.
There is no point of holding this patch series.
I am planning to accept this patch series.
Please do let us know, if you have any further review comments/concern..
--pk
^ permalink raw reply [flat|nested] 29+ messages in thread
* [U-Boot] [PATCHv2 1/8] powerpc: mpc85xx: Move CONFIG_FSL_PCIE_DISABLE_ASPM to Kconfig
2019-05-23 12:22 ` [U-Boot] [PATCHv2 1/8] powerpc: mpc85xx: Move CONFIG_FSL_PCIE_DISABLE_ASPM to Kconfig Z.q. Hou
@ 2019-06-20 9:44 ` Prabhakar Kushwaha
0 siblings, 0 replies; 29+ messages in thread
From: Prabhakar Kushwaha @ 2019-06-20 9:44 UTC (permalink / raw)
To: u-boot
> -----Original Message-----
> From: Z.q. Hou
> Sent: Thursday, May 23, 2019 5:52 PM
> To: u-boot at lists.denx.de; Prabhakar Kushwaha
> <prabhakar.kushwaha@nxp.com>; wd at denx.de; Shengzhou Liu
> <shengzhou.liu@nxp.com>; bmeng.cn at gmail.com; Jagdish Gediya
> <jagdish.gediya@nxp.com>; sjg at chromium.org; ley.foon.tan at intel.com;
> sr at denx.de
> Cc: M.h. Lian <minghuan.lian@nxp.com>; Xiaowei Bao
> <xiaowei.bao@nxp.com>; Z.q. Hou <zhiqiang.hou@nxp.com>
> Subject: [PATCHv2 1/8] powerpc: mpc85xx: Move
> CONFIG_FSL_PCIE_DISABLE_ASPM to Kconfig
>
> From: Hou Zhiqiang <Zhiqiang.Hou@nxp.com>
>
> Use the Kconfig option to select the PCIe ASPM errata.
>
> Signed-off-by: Hou Zhiqiang <Zhiqiang.Hou@nxp.com>
> ---
This patch has been applied to u-boot-mpc85xx, awaiting upstream.
--pk
^ permalink raw reply [flat|nested] 29+ messages in thread
* [U-Boot] [PATCHv2 2/8] powerpc: mpc85xx: Move CONFIG_FSL_PCIE_RESET to Kconfig
2019-05-23 12:22 ` [U-Boot] [PATCHv2 2/8] powerpc: mpc85xx: Move CONFIG_FSL_PCIE_RESET " Z.q. Hou
@ 2019-06-20 9:44 ` Prabhakar Kushwaha
0 siblings, 0 replies; 29+ messages in thread
From: Prabhakar Kushwaha @ 2019-06-20 9:44 UTC (permalink / raw)
To: u-boot
> -----Original Message-----
> From: Z.q. Hou
> Sent: Thursday, May 23, 2019 5:52 PM
> To: u-boot at lists.denx.de; Prabhakar Kushwaha
> <prabhakar.kushwaha@nxp.com>; wd at denx.de; Shengzhou Liu
> <shengzhou.liu@nxp.com>; bmeng.cn at gmail.com; Jagdish Gediya
> <jagdish.gediya@nxp.com>; sjg at chromium.org; ley.foon.tan at intel.com;
> sr at denx.de
> Cc: M.h. Lian <minghuan.lian@nxp.com>; Xiaowei Bao
> <xiaowei.bao@nxp.com>; Z.q. Hou <zhiqiang.hou@nxp.com>
> Subject: [PATCHv2 2/8] powerpc: mpc85xx: Move CONFIG_FSL_PCIE_RESET to
> Kconfig
>
> From: Hou Zhiqiang <Zhiqiang.Hou@nxp.com>
>
> Use the Kconfig option to select the PCIe reset errata.
>
> Signed-off-by: Hou Zhiqiang <Zhiqiang.Hou@nxp.com>
> ---
This patch has been applied to u-boot-mpc85xx, awaiting upstream.
--pk
^ permalink raw reply [flat|nested] 29+ messages in thread
* [U-Boot] [PATCHv2 3/8] powerpc: mpc85xx: Update the condition to compile PCI routines
2019-05-23 12:22 ` [U-Boot] [PATCHv2 3/8] powerpc: mpc85xx: Update the condition to compile PCI routines Z.q. Hou
@ 2019-06-20 9:44 ` Prabhakar Kushwaha
0 siblings, 0 replies; 29+ messages in thread
From: Prabhakar Kushwaha @ 2019-06-20 9:44 UTC (permalink / raw)
To: u-boot
> -----Original Message-----
> From: Z.q. Hou
> Sent: Thursday, May 23, 2019 5:52 PM
> To: u-boot at lists.denx.de; Prabhakar Kushwaha
> <prabhakar.kushwaha@nxp.com>; wd at denx.de; Shengzhou Liu
> <shengzhou.liu@nxp.com>; bmeng.cn at gmail.com; Jagdish Gediya
> <jagdish.gediya@nxp.com>; sjg at chromium.org; ley.foon.tan at intel.com;
> sr at denx.de
> Cc: M.h. Lian <minghuan.lian@nxp.com>; Xiaowei Bao
> <xiaowei.bao@nxp.com>; Z.q. Hou <zhiqiang.hou@nxp.com>
> Subject: [PATCHv2 3/8] powerpc: mpc85xx: Update the condition to compile PCI
> routines
>
> From: Hou Zhiqiang <Zhiqiang.Hou@nxp.com>
>
> Compile the routines of mpc85xx/pci.c when both FSL_PCI_INIT and DM_PCI are
> not enabled.
>
> Signed-off-by: Hou Zhiqiang <Zhiqiang.Hou@nxp.com>
> Reviewed-by: Simon Glass <sjg@chromium.org>
> ---
This patch has been applied to u-boot-mpc85xx, awaiting upstream.
--pk
^ permalink raw reply [flat|nested] 29+ messages in thread
* [U-Boot] [PATCHv2 4/8] powerpc: T208xQDS: Compile the legacy PCIe routines conditionally
2019-05-23 12:22 ` [U-Boot] [PATCHv2 4/8] powerpc: T208xQDS: Compile the legacy PCIe routines conditionally Z.q. Hou
@ 2019-06-20 9:44 ` Prabhakar Kushwaha
0 siblings, 0 replies; 29+ messages in thread
From: Prabhakar Kushwaha @ 2019-06-20 9:44 UTC (permalink / raw)
To: u-boot
> -----Original Message-----
> From: Z.q. Hou
> Sent: Thursday, May 23, 2019 5:52 PM
> To: u-boot at lists.denx.de; Prabhakar Kushwaha
> <prabhakar.kushwaha@nxp.com>; wd at denx.de; Shengzhou Liu
> <shengzhou.liu@nxp.com>; bmeng.cn at gmail.com; Jagdish Gediya
> <jagdish.gediya@nxp.com>; sjg at chromium.org; ley.foon.tan at intel.com;
> sr at denx.de
> Cc: M.h. Lian <minghuan.lian@nxp.com>; Xiaowei Bao
> <xiaowei.bao@nxp.com>; Z.q. Hou <zhiqiang.hou@nxp.com>
> Subject: [PATCHv2 4/8] powerpc: T208xQDS: Compile the legacy PCIe routines
> conditionally
>
> From: Hou Zhiqiang <Zhiqiang.Hou@nxp.com>
>
> Compile the legacy PCIe initialization reoutines only when DM_PCI is not
> enabled.
>
> Signed-off-by: Hou Zhiqiang <Zhiqiang.Hou@nxp.com>
> Reviewed-by: Simon Glass <sjg@chromium.org>
> ---
This patch has been applied to u-boot-mpc85xx, awaiting upstream.
--pk
^ permalink raw reply [flat|nested] 29+ messages in thread
* [U-Boot] [PATCHv2 5/8] t2080: dts: Added PCIe DT nodes
2019-05-23 12:22 ` [U-Boot] [PATCHv2 5/8] t2080: dts: Added PCIe DT nodes Z.q. Hou
@ 2019-06-20 9:44 ` Prabhakar Kushwaha
0 siblings, 0 replies; 29+ messages in thread
From: Prabhakar Kushwaha @ 2019-06-20 9:44 UTC (permalink / raw)
To: u-boot
> -----Original Message-----
> From: Z.q. Hou
> Sent: Thursday, May 23, 2019 5:52 PM
> To: u-boot at lists.denx.de; Prabhakar Kushwaha
> <prabhakar.kushwaha@nxp.com>; wd at denx.de; Shengzhou Liu
> <shengzhou.liu@nxp.com>; bmeng.cn at gmail.com; Jagdish Gediya
> <jagdish.gediya@nxp.com>; sjg at chromium.org; ley.foon.tan at intel.com;
> sr at denx.de
> Cc: M.h. Lian <minghuan.lian@nxp.com>; Xiaowei Bao
> <xiaowei.bao@nxp.com>; Z.q. Hou <zhiqiang.hou@nxp.com>
> Subject: [PATCHv2 5/8] t2080: dts: Added PCIe DT nodes
>
> From: Hou Zhiqiang <Zhiqiang.Hou@nxp.com>
>
> T2080 integrated 4 PCIe controllers, which is compatible with the PCI Expressâ„¢
> Base Specification, Revision 3.0, and this patch is to add DT node for each PCIe
> controller.
>
> Signed-off-by: Hou Zhiqiang <Zhiqiang.Hou@nxp.com>
> Reviewed-by: Simon Glass <sjg@chromium.org>
> ---
This patch has been applied to u-boot-mpc85xx, awaiting upstream.
--pk
^ permalink raw reply [flat|nested] 29+ messages in thread
* [U-Boot] [PATCHv2 6/8] dm: pci: add Freescale PowerPC PCIe driver
2019-05-23 12:22 ` [U-Boot] [PATCHv2 6/8] dm: pci: add Freescale PowerPC PCIe driver Z.q. Hou
@ 2019-06-20 9:44 ` Prabhakar Kushwaha
0 siblings, 0 replies; 29+ messages in thread
From: Prabhakar Kushwaha @ 2019-06-20 9:44 UTC (permalink / raw)
To: u-boot
> -----Original Message-----
> From: Z.q. Hou
> Sent: Thursday, May 23, 2019 5:52 PM
> To: u-boot at lists.denx.de; Prabhakar Kushwaha
> <prabhakar.kushwaha@nxp.com>; wd at denx.de; Shengzhou Liu
> <shengzhou.liu@nxp.com>; bmeng.cn at gmail.com; Jagdish Gediya
> <jagdish.gediya@nxp.com>; sjg at chromium.org; ley.foon.tan at intel.com;
> sr at denx.de
> Cc: M.h. Lian <minghuan.lian@nxp.com>; Xiaowei Bao
> <xiaowei.bao@nxp.com>; Z.q. Hou <zhiqiang.hou@nxp.com>
> Subject: [PATCHv2 6/8] dm: pci: add Freescale PowerPC PCIe driver
>
> From: Hou Zhiqiang <Zhiqiang.Hou@nxp.com>
>
> Add PCIe DM driver for Freescale PowerPC PCIe controllers.
>
> Signed-off-by: Hou Zhiqiang <Zhiqiang.Hou@nxp.com>
> ---
Updated copyright!!
This patch has been applied to u-boot-mpc85xx, awaiting upstream.
--pk
^ permalink raw reply [flat|nested] 29+ messages in thread
* [U-Boot] [PATCHv2 7/8] powerpc: T208xQDS: Disable legacy PCIe driver
2019-05-23 12:22 ` [U-Boot] [PATCHv2 7/8] powerpc: T208xQDS: Disable legacy " Z.q. Hou
@ 2019-06-20 9:44 ` Prabhakar Kushwaha
0 siblings, 0 replies; 29+ messages in thread
From: Prabhakar Kushwaha @ 2019-06-20 9:44 UTC (permalink / raw)
To: u-boot
> -----Original Message-----
> From: Z.q. Hou
> Sent: Thursday, May 23, 2019 5:52 PM
> To: u-boot at lists.denx.de; Prabhakar Kushwaha
> <prabhakar.kushwaha@nxp.com>; wd at denx.de; Shengzhou Liu
> <shengzhou.liu@nxp.com>; bmeng.cn at gmail.com; Jagdish Gediya
> <jagdish.gediya@nxp.com>; sjg at chromium.org; ley.foon.tan at intel.com;
> sr at denx.de
> Cc: M.h. Lian <minghuan.lian@nxp.com>; Xiaowei Bao
> <xiaowei.bao@nxp.com>; Z.q. Hou <zhiqiang.hou@nxp.com>
> Subject: [PATCHv2 7/8] powerpc: T208xQDS: Disable legacy PCIe driver
>
> From: Hou Zhiqiang <Zhiqiang.Hou@nxp.com>
>
> Disable legacy PCIe driver and remove unused PCIe macros.
>
> Signed-off-by: Hou Zhiqiang <Zhiqiang.Hou@nxp.com>
> Reviewed-by: Simon Glass <sjg@chromium.org>
> ---
Fixed for non PCIe DM build !!
This patch has been applied to u-boot-mpc85xx, awaiting upstream.
--pk
^ permalink raw reply [flat|nested] 29+ messages in thread
* [U-Boot] [PATCHv2 8/8] configs: T2080QDS: Enable PCIe driver
2019-05-23 12:22 ` [U-Boot] [PATCHv2 8/8] configs: T2080QDS: Enable " Z.q. Hou
2019-05-23 14:23 ` Bin Meng
@ 2019-06-20 9:44 ` Prabhakar Kushwaha
1 sibling, 0 replies; 29+ messages in thread
From: Prabhakar Kushwaha @ 2019-06-20 9:44 UTC (permalink / raw)
To: u-boot
> -----Original Message-----
> From: Z.q. Hou
> Sent: Thursday, May 23, 2019 5:53 PM
> To: u-boot at lists.denx.de; Prabhakar Kushwaha
> <prabhakar.kushwaha@nxp.com>; wd at denx.de; Shengzhou Liu
> <shengzhou.liu@nxp.com>; bmeng.cn at gmail.com; Jagdish Gediya
> <jagdish.gediya@nxp.com>; sjg at chromium.org; ley.foon.tan at intel.com;
> sr at denx.de
> Cc: M.h. Lian <minghuan.lian@nxp.com>; Xiaowei Bao
> <xiaowei.bao@nxp.com>; Z.q. Hou <zhiqiang.hou@nxp.com>
> Subject: [PATCHv2 8/8] configs: T2080QDS: Enable PCIe driver
>
> From: Hou Zhiqiang <Zhiqiang.Hou@nxp.com>
>
> Enable the DM PCIe driver in T2080QDS defconfig.
>
> Signed-off-by: Hou Zhiqiang <Zhiqiang.Hou@nxp.com>
> ---
Run savedefonfing and enabled for other 2 configs
This patch has been applied to u-boot-mpc85xx, awaiting upstream.
--pk
^ permalink raw reply [flat|nested] 29+ messages in thread
end of thread, other threads:[~2019-06-20 9:44 UTC | newest]
Thread overview: 29+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-05-23 12:21 [U-Boot] [PATCHv2 0/8] dm: pci: add Freescale PowerPC PCIe driver Z.q. Hou
2019-05-23 12:22 ` [U-Boot] [PATCHv2 1/8] powerpc: mpc85xx: Move CONFIG_FSL_PCIE_DISABLE_ASPM to Kconfig Z.q. Hou
2019-06-20 9:44 ` Prabhakar Kushwaha
2019-05-23 12:22 ` [U-Boot] [PATCHv2 2/8] powerpc: mpc85xx: Move CONFIG_FSL_PCIE_RESET " Z.q. Hou
2019-06-20 9:44 ` Prabhakar Kushwaha
2019-05-23 12:22 ` [U-Boot] [PATCHv2 3/8] powerpc: mpc85xx: Update the condition to compile PCI routines Z.q. Hou
2019-06-20 9:44 ` Prabhakar Kushwaha
2019-05-23 12:22 ` [U-Boot] [PATCHv2 4/8] powerpc: T208xQDS: Compile the legacy PCIe routines conditionally Z.q. Hou
2019-06-20 9:44 ` Prabhakar Kushwaha
2019-05-23 12:22 ` [U-Boot] [PATCHv2 5/8] t2080: dts: Added PCIe DT nodes Z.q. Hou
2019-06-20 9:44 ` Prabhakar Kushwaha
2019-05-23 12:22 ` [U-Boot] [PATCHv2 6/8] dm: pci: add Freescale PowerPC PCIe driver Z.q. Hou
2019-06-20 9:44 ` Prabhakar Kushwaha
2019-05-23 12:22 ` [U-Boot] [PATCHv2 7/8] powerpc: T208xQDS: Disable legacy " Z.q. Hou
2019-06-20 9:44 ` Prabhakar Kushwaha
2019-05-23 12:22 ` [U-Boot] [PATCHv2 8/8] configs: T2080QDS: Enable " Z.q. Hou
2019-05-23 14:23 ` Bin Meng
2019-05-23 16:04 ` Z.q. Hou
2019-05-24 0:10 ` Bin Meng
2019-05-24 2:10 ` [U-Boot] [EXT] " Xiaowei Bao
2019-05-24 2:13 ` Bin Meng
2019-05-24 2:17 ` Xiaowei Bao
2019-05-24 5:23 ` Prabhakar Kushwaha
2019-05-24 6:02 ` Xiaowei Bao
2019-05-24 8:17 ` Z.q. Hou
2019-05-24 9:38 ` Bin Meng
2019-05-24 14:12 ` Z.q. Hou
2019-06-18 0:37 ` Prabhakar Kushwaha
2019-06-20 9:44 ` [U-Boot] " Prabhakar Kushwaha
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.