-----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@lists.denx.de; Prabhakar Kushwaha prabhakar.kushwaha@nxp.com; wd@denx.de; Shengzhou Liu shengzhou.liu@nxp.com; Jagdish Gediya jagdish.gediya@nxp.com; ley.foon.tan@intel.com; sr@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@lists.denx.de; Prabhakar Kushwaha prabhakar.kushwaha@nxp.com; wd@denx.de; Shengzhou Liu shengzhou.liu@nxp.com; Jagdish Gediya jagdish.gediya@nxp.com; sjg@chromium.org; ley.foon.tan@intel.com; sr@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@gmail.com] Sent: 2019年5月23日 22:23 To: Z.q. Hou zhiqiang.hou@nxp.com Cc: u-boot@lists.denx.de; Prabhakar Kushwaha prabhakar.kushwaha@nxp.com; wd@denx.de; Shengzhou Liu shengzhou.liu@nxp.com; Jagdish Gediya jagdish.gediya@nxp.com; sjg@chromium.org; ley.foon.tan@intel.com; sr@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