From mboxrd@z Thu Jan 1 00:00:00 1970 From: Bin Meng Date: Fri, 24 May 2019 10:13:56 +0800 Subject: [U-Boot] [EXT] Re: [PATCHv2 8/8] configs: T2080QDS: Enable PCIe driver In-Reply-To: References: <20190523122327.37264-1-Zhiqiang.Hou@nxp.com> <20190523122327.37264-9-Zhiqiang.Hou@nxp.com> Message-ID: List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 8bit To: u-boot@lists.denx.de On Fri, May 24, 2019 at 10:10 AM Xiaowei Bao wrote: > > > > -----Original Message----- > From: Bin Meng > Sent: 2019年5月24日 8:11 > To: Z.q. Hou > Cc: u-boot at lists.denx.de; Prabhakar Kushwaha ; wd at denx.de; Shengzhou Liu ; Jagdish Gediya ; sjg at chromium.org; ley.foon.tan at intel.com; sr at denx.de; M.h. Lian ; Xiaowei Bao > 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 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 > > > Cc: u-boot at lists.denx.de; Prabhakar Kushwaha > > > ; wd at denx.de; Shengzhou Liu > > > ; Jagdish Gediya ; > > > sjg at chromium.org; ley.foon.tan at intel.com; sr at denx.de; M.h. Lian > > > ; Xiaowei Bao > > > Subject: Re: [PATCHv2 8/8] configs: T2080QDS: Enable PCIe driver > > > > > > On Thu, May 23, 2019 at 8:22 PM Z.q. Hou wrote: > > > > > > > > From: Hou Zhiqiang > > > > > > > > Enable the DM PCIe driver in T2080QDS defconfig. > > > > > > > > Signed-off-by: Hou Zhiqiang > > > > --- > > > > 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