[U-Boot] [EXT] Re: [PATCHv2 8/8] configs: T2080QDS: Enable PCIe driver

From: Xiaowei Bao <xiaowei.bao@nxp.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [EXT] Re: [PATCHv2 8/8] configs: T2080QDS: Enable PCIe driver
Date: Fri, 24 May 2019 06:02:32 +0000	[thread overview]
Message-ID: <AM5PR04MB32999C410B82E4D7D3D99AB1F5020@AM5PR04MB3299.eurprd04.prod.outlook.com> (raw)
In-Reply-To: <VI1PR0402MB3854B6CD5F99C9747794526997020@VI1PR0402MB3854.eurprd04.prod.outlook.com>

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