Message ID | 20190723130938.47805-35-Zhiqiang.Hou@nxp.com |
---|---|
State | Superseded |
Delegated to: | Prabhakar Kushwaha |
Headers | show |
Series | powerpc: Enable PCIe DM drvier for some platforms | expand |
On Tue, Jul 23, 2019 at 9:29 PM Hou Zhiqiang <Zhiqiang.Hou@nxp.com> wrote: > > Disable legacy PCIe driver and unused PCIe macros when DM_PCI enabled. > > Signed-off-by: Hou Zhiqiang <Zhiqiang.Hou@nxp.com> > --- > include/configs/corenet_ds.h | 63 +++++++++++++------------------------------- > 1 file changed, 19 insertions(+), 44 deletions(-) > > diff --git a/include/configs/corenet_ds.h b/include/configs/corenet_ds.h > index f974291..07844c1 100644 > --- a/include/configs/corenet_ds.h > +++ b/include/configs/corenet_ds.h > @@ -54,7 +54,6 @@ > #define CONFIG_SYS_NUM_CPC CONFIG_SYS_NUM_DDR_CTLRS > #define CONFIG_PCIE1 /* PCIE controller 1 */ > #define CONFIG_PCIE2 /* PCIE controller 2 */ > -#define CONFIG_FSL_PCI_INIT /* Use common FSL init code */ > #define CONFIG_SYS_PCI_64BIT /* enable 64-bit PCI resources */ > > #define CONFIG_ENV_OVERWRITE > @@ -362,68 +361,25 @@ > > /* controller 1, direct to uli, tgtid 3, Base address 20000 */ > #define CONFIG_SYS_PCIE1_MEM_VIRT 0x80000000 > -#ifdef CONFIG_PHYS_64BIT > -#define CONFIG_SYS_PCIE1_MEM_BUS 0xe0000000 > #define CONFIG_SYS_PCIE1_MEM_PHYS 0xc00000000ull > -#else > -#define CONFIG_SYS_PCIE1_MEM_BUS 0x80000000 > -#define CONFIG_SYS_PCIE1_MEM_PHYS 0x80000000 > -#endif > -#define CONFIG_SYS_PCIE1_MEM_SIZE 0x20000000 /* 512M */ > #define CONFIG_SYS_PCIE1_IO_VIRT 0xf8000000 > -#define CONFIG_SYS_PCIE1_IO_BUS 0x00000000 > -#ifdef CONFIG_PHYS_64BIT > #define CONFIG_SYS_PCIE1_IO_PHYS 0xff8000000ull > -#else > -#define CONFIG_SYS_PCIE1_IO_PHYS 0xf8000000 > -#endif > -#define CONFIG_SYS_PCIE1_IO_SIZE 0x00010000 /* 64k */ > > /* controller 2, Slot 2, tgtid 2, Base address 201000 */ > #define CONFIG_SYS_PCIE2_MEM_VIRT 0xa0000000 > -#ifdef CONFIG_PHYS_64BIT > -#define CONFIG_SYS_PCIE2_MEM_BUS 0xe0000000 > #define CONFIG_SYS_PCIE2_MEM_PHYS 0xc20000000ull > -#else > -#define CONFIG_SYS_PCIE2_MEM_BUS 0xa0000000 > -#define CONFIG_SYS_PCIE2_MEM_PHYS 0xa0000000 > -#endif > -#define CONFIG_SYS_PCIE2_MEM_SIZE 0x20000000 /* 512M */ > #define CONFIG_SYS_PCIE2_IO_VIRT 0xf8010000 > -#define CONFIG_SYS_PCIE2_IO_BUS 0x00000000 > -#ifdef CONFIG_PHYS_64BIT > #define CONFIG_SYS_PCIE2_IO_PHYS 0xff8010000ull > -#else > -#define CONFIG_SYS_PCIE2_IO_PHYS 0xf8010000 > -#endif > -#define CONFIG_SYS_PCIE2_IO_SIZE 0x00010000 /* 64k */ > > /* controller 3, Slot 1, tgtid 1, Base address 202000 */ > #define CONFIG_SYS_PCIE3_MEM_VIRT 0xc0000000 > -#ifdef CONFIG_PHYS_64BIT > -#define CONFIG_SYS_PCIE3_MEM_BUS 0xe0000000 > #define CONFIG_SYS_PCIE3_MEM_PHYS 0xc40000000ull > -#else > -#define CONFIG_SYS_PCIE3_MEM_BUS 0xc0000000 > -#define CONFIG_SYS_PCIE3_MEM_PHYS 0xc0000000 > -#endif > -#define CONFIG_SYS_PCIE3_MEM_SIZE 0x20000000 /* 512M */ > #define CONFIG_SYS_PCIE3_IO_VIRT 0xf8020000 > -#define CONFIG_SYS_PCIE3_IO_BUS 0x00000000 > -#ifdef CONFIG_PHYS_64BIT > #define CONFIG_SYS_PCIE3_IO_PHYS 0xff8020000ull > -#else > -#define CONFIG_SYS_PCIE3_IO_PHYS 0xf8020000 > -#endif > -#define CONFIG_SYS_PCIE3_IO_SIZE 0x00010000 /* 64k */ > > /* controller 4, Base address 203000 */ > -#define CONFIG_SYS_PCIE4_MEM_BUS 0xe0000000 > #define CONFIG_SYS_PCIE4_MEM_PHYS 0xc60000000ull > -#define CONFIG_SYS_PCIE4_MEM_SIZE 0x20000000 /* 512M */ > -#define CONFIG_SYS_PCIE4_IO_BUS 0x00000000 > #define CONFIG_SYS_PCIE4_IO_PHYS 0xff8030000ull > -#define CONFIG_SYS_PCIE4_IO_SIZE 0x00010000 /* 64k */ > > /* Qman/Bman */ > #define CONFIG_SYS_BMAN_NUM_PORTALS 10 > @@ -505,7 +461,26 @@ > #endif > > #ifdef CONFIG_PCI > +#if !defined(CONFIG_DM_PCI) > +#define CONFIG_FSL_PCI_INIT /* Use common FSL init code */ > #define CONFIG_PCI_INDIRECT_BRIDGE > +#define CONFIG_SYS_PCIE1_MEM_BUS 0xe0000000 CONFIG_PHYS_64BIT? > +#define CONFIG_SYS_PCIE1_MEM_SIZE 0x20000000 /* 512M */ > +#define CONFIG_SYS_PCIE1_IO_BUS 0x00000000 > +#define CONFIG_SYS_PCIE1_IO_SIZE 0x00010000 /* 64k */ > +#define CONFIG_SYS_PCIE2_MEM_BUS 0xe0000000 > +#define CONFIG_SYS_PCIE2_MEM_SIZE 0x20000000 /* 512M */ > +#define CONFIG_SYS_PCIE2_IO_BUS 0x00000000 > +#define CONFIG_SYS_PCIE2_IO_SIZE 0x00010000 /* 64k */ > +#define CONFIG_SYS_PCIE3_MEM_BUS 0xe0000000 > +#define CONFIG_SYS_PCIE3_MEM_SIZE 0x20000000 /* 512M */ > +#define CONFIG_SYS_PCIE3_IO_BUS 0x00000000 > +#define CONFIG_SYS_PCIE3_IO_SIZE 0x00010000 /* 64k */ > +#define CONFIG_SYS_PCIE4_MEM_BUS 0xe0000000 > +#define CONFIG_SYS_PCIE4_MEM_SIZE 0x20000000 /* 512M */ > +#define CONFIG_SYS_PCIE4_IO_BUS 0x00000000 > +#define CONFIG_SYS_PCIE4_IO_SIZE 0x00010000 /* 64k */ > +#endif > > #define CONFIG_PCI_SCAN_SHOW /* show pci devices on startup */ > #endif /* CONFIG_PCI */ > -- Regards, Bin
Hi Bin, Thanks a lot for your comments! > -----Original Message----- > From: Bin Meng <bmeng.cn@gmail.com> > Sent: 2019年8月26日 22:50 > To: Z.q. Hou <zhiqiang.hou@nxp.com> > Cc: U-Boot Mailing List <u-boot@lists.denx.de>; Prabhakar Kushwaha > <prabhakar.kushwaha@nxp.com>; Wolfgang Denk <wd@denx.de>; Priyanka > Jain <priyanka.jain@nxp.com>; Shengzhou Liu <shengzhou.liu@nxp.com> > Subject: Re: [U-Boot] [PATCH 34/47] powerpc: corenet_ds: Disable legacy > PCIe driver when DM_PCI is enabled > > On Tue, Jul 23, 2019 at 9:29 PM Hou Zhiqiang <Zhiqiang.Hou@nxp.com> > wrote: > > > > Disable legacy PCIe driver and unused PCIe macros when DM_PCI enabled. > > > > Signed-off-by: Hou Zhiqiang <Zhiqiang.Hou@nxp.com> > > --- > > include/configs/corenet_ds.h | 63 > > +++++++++++++------------------------------- > > 1 file changed, 19 insertions(+), 44 deletions(-) > > > > diff --git a/include/configs/corenet_ds.h > > b/include/configs/corenet_ds.h index f974291..07844c1 100644 > > --- a/include/configs/corenet_ds.h > > +++ b/include/configs/corenet_ds.h > > @@ -54,7 +54,6 @@ > > #define CONFIG_SYS_NUM_CPC > CONFIG_SYS_NUM_DDR_CTLRS > > #define CONFIG_PCIE1 /* PCIE controller 1 */ > > #define CONFIG_PCIE2 /* PCIE controller 2 */ > > -#define CONFIG_FSL_PCI_INIT /* Use common FSL init > code */ > > #define CONFIG_SYS_PCI_64BIT /* enable 64-bit PCI > resources */ > > > > #define CONFIG_ENV_OVERWRITE > > @@ -362,68 +361,25 @@ > > > > /* controller 1, direct to uli, tgtid 3, Base address 20000 */ > > #define CONFIG_SYS_PCIE1_MEM_VIRT 0x80000000 > > -#ifdef CONFIG_PHYS_64BIT > > -#define CONFIG_SYS_PCIE1_MEM_BUS 0xe0000000 > > #define CONFIG_SYS_PCIE1_MEM_PHYS 0xc00000000ull > > -#else > > -#define CONFIG_SYS_PCIE1_MEM_BUS 0x80000000 > > -#define CONFIG_SYS_PCIE1_MEM_PHYS 0x80000000 > > -#endif > > -#define CONFIG_SYS_PCIE1_MEM_SIZE 0x20000000 /* > 512M */ > > #define CONFIG_SYS_PCIE1_IO_VIRT 0xf8000000 > > -#define CONFIG_SYS_PCIE1_IO_BUS 0x00000000 > > -#ifdef CONFIG_PHYS_64BIT > > #define CONFIG_SYS_PCIE1_IO_PHYS 0xff8000000ull > > -#else > > -#define CONFIG_SYS_PCIE1_IO_PHYS 0xf8000000 > > -#endif > > -#define CONFIG_SYS_PCIE1_IO_SIZE 0x00010000 /* 64k */ > > > > /* controller 2, Slot 2, tgtid 2, Base address 201000 */ > > #define CONFIG_SYS_PCIE2_MEM_VIRT 0xa0000000 > > -#ifdef CONFIG_PHYS_64BIT > > -#define CONFIG_SYS_PCIE2_MEM_BUS 0xe0000000 > > #define CONFIG_SYS_PCIE2_MEM_PHYS 0xc20000000ull > > -#else > > -#define CONFIG_SYS_PCIE2_MEM_BUS 0xa0000000 > > -#define CONFIG_SYS_PCIE2_MEM_PHYS 0xa0000000 > > -#endif > > -#define CONFIG_SYS_PCIE2_MEM_SIZE 0x20000000 /* > 512M */ > > #define CONFIG_SYS_PCIE2_IO_VIRT 0xf8010000 > > -#define CONFIG_SYS_PCIE2_IO_BUS 0x00000000 > > -#ifdef CONFIG_PHYS_64BIT > > #define CONFIG_SYS_PCIE2_IO_PHYS 0xff8010000ull > > -#else > > -#define CONFIG_SYS_PCIE2_IO_PHYS 0xf8010000 > > -#endif > > -#define CONFIG_SYS_PCIE2_IO_SIZE 0x00010000 /* 64k */ > > > > /* controller 3, Slot 1, tgtid 1, Base address 202000 */ > > #define CONFIG_SYS_PCIE3_MEM_VIRT 0xc0000000 > > -#ifdef CONFIG_PHYS_64BIT > > -#define CONFIG_SYS_PCIE3_MEM_BUS 0xe0000000 > > #define CONFIG_SYS_PCIE3_MEM_PHYS 0xc40000000ull > > -#else > > -#define CONFIG_SYS_PCIE3_MEM_BUS 0xc0000000 > > -#define CONFIG_SYS_PCIE3_MEM_PHYS 0xc0000000 > > -#endif > > -#define CONFIG_SYS_PCIE3_MEM_SIZE 0x20000000 /* > 512M */ > > #define CONFIG_SYS_PCIE3_IO_VIRT 0xf8020000 > > -#define CONFIG_SYS_PCIE3_IO_BUS 0x00000000 > > -#ifdef CONFIG_PHYS_64BIT > > #define CONFIG_SYS_PCIE3_IO_PHYS 0xff8020000ull > > -#else > > -#define CONFIG_SYS_PCIE3_IO_PHYS 0xf8020000 > > -#endif > > -#define CONFIG_SYS_PCIE3_IO_SIZE 0x00010000 /* 64k */ > > > > /* controller 4, Base address 203000 */ > > -#define CONFIG_SYS_PCIE4_MEM_BUS 0xe0000000 > > #define CONFIG_SYS_PCIE4_MEM_PHYS 0xc60000000ull > > -#define CONFIG_SYS_PCIE4_MEM_SIZE 0x20000000 /* > 512M */ > > -#define CONFIG_SYS_PCIE4_IO_BUS 0x00000000 > > #define CONFIG_SYS_PCIE4_IO_PHYS 0xff8030000ull > > -#define CONFIG_SYS_PCIE4_IO_SIZE 0x00010000 /* 64k */ > > > > /* Qman/Bman */ > > #define CONFIG_SYS_BMAN_NUM_PORTALS 10 > > @@ -505,7 +461,26 @@ > > #endif > > > > #ifdef CONFIG_PCI > > +#if !defined(CONFIG_DM_PCI) > > +#define CONFIG_FSL_PCI_INIT /* Use common FSL init code */ > > #define CONFIG_PCI_INDIRECT_BRIDGE > > +#define CONFIG_SYS_PCIE1_MEM_BUS 0xe0000000 > > CONFIG_PHYS_64BIT? All CORENET DS boards selected PHYS_64BIT, so removed these unused macros. Thanks, Zhiqiang > > > +#define CONFIG_SYS_PCIE1_MEM_SIZE 0x20000000 /* > 512M */ > > +#define CONFIG_SYS_PCIE1_IO_BUS 0x00000000 > > +#define CONFIG_SYS_PCIE1_IO_SIZE 0x00010000 /* 64k */ > > +#define CONFIG_SYS_PCIE2_MEM_BUS 0xe0000000 > > +#define CONFIG_SYS_PCIE2_MEM_SIZE 0x20000000 /* > 512M */ > > +#define CONFIG_SYS_PCIE2_IO_BUS 0x00000000 > > +#define CONFIG_SYS_PCIE2_IO_SIZE 0x00010000 /* 64k */ > > +#define CONFIG_SYS_PCIE3_MEM_BUS 0xe0000000 > > +#define CONFIG_SYS_PCIE3_MEM_SIZE 0x20000000 /* > 512M */ > > +#define CONFIG_SYS_PCIE3_IO_BUS 0x00000000 > > +#define CONFIG_SYS_PCIE3_IO_SIZE 0x00010000 /* 64k */ > > +#define CONFIG_SYS_PCIE4_MEM_BUS 0xe0000000 > > +#define CONFIG_SYS_PCIE4_MEM_SIZE 0x20000000 /* > 512M */ > > +#define CONFIG_SYS_PCIE4_IO_BUS 0x00000000 > > +#define CONFIG_SYS_PCIE4_IO_SIZE 0x00010000 /* 64k */ > > +#endif > > > > #define CONFIG_PCI_SCAN_SHOW /* show pci devices on > startup */ > > #endif /* CONFIG_PCI */ > > -- > > Regards, > Bin
On Tue, Jul 23, 2019 at 9:29 PM Hou Zhiqiang <Zhiqiang.Hou@nxp.com> wrote: > > Disable legacy PCIe driver and unused PCIe macros when DM_PCI enabled. > > Signed-off-by: Hou Zhiqiang <Zhiqiang.Hou@nxp.com> > --- > include/configs/corenet_ds.h | 63 +++++++++++++------------------------------- > 1 file changed, 19 insertions(+), 44 deletions(-) > Reviewed-by: Bin Meng <bmeng.cn@gmail.com>
diff --git a/include/configs/corenet_ds.h b/include/configs/corenet_ds.h index f974291..07844c1 100644 --- a/include/configs/corenet_ds.h +++ b/include/configs/corenet_ds.h @@ -54,7 +54,6 @@ #define CONFIG_SYS_NUM_CPC CONFIG_SYS_NUM_DDR_CTLRS #define CONFIG_PCIE1 /* PCIE controller 1 */ #define CONFIG_PCIE2 /* PCIE controller 2 */ -#define CONFIG_FSL_PCI_INIT /* Use common FSL init code */ #define CONFIG_SYS_PCI_64BIT /* enable 64-bit PCI resources */ #define CONFIG_ENV_OVERWRITE @@ -362,68 +361,25 @@ /* controller 1, direct to uli, tgtid 3, Base address 20000 */ #define CONFIG_SYS_PCIE1_MEM_VIRT 0x80000000 -#ifdef CONFIG_PHYS_64BIT -#define CONFIG_SYS_PCIE1_MEM_BUS 0xe0000000 #define CONFIG_SYS_PCIE1_MEM_PHYS 0xc00000000ull -#else -#define CONFIG_SYS_PCIE1_MEM_BUS 0x80000000 -#define CONFIG_SYS_PCIE1_MEM_PHYS 0x80000000 -#endif -#define CONFIG_SYS_PCIE1_MEM_SIZE 0x20000000 /* 512M */ #define CONFIG_SYS_PCIE1_IO_VIRT 0xf8000000 -#define CONFIG_SYS_PCIE1_IO_BUS 0x00000000 -#ifdef CONFIG_PHYS_64BIT #define CONFIG_SYS_PCIE1_IO_PHYS 0xff8000000ull -#else -#define CONFIG_SYS_PCIE1_IO_PHYS 0xf8000000 -#endif -#define CONFIG_SYS_PCIE1_IO_SIZE 0x00010000 /* 64k */ /* controller 2, Slot 2, tgtid 2, Base address 201000 */ #define CONFIG_SYS_PCIE2_MEM_VIRT 0xa0000000 -#ifdef CONFIG_PHYS_64BIT -#define CONFIG_SYS_PCIE2_MEM_BUS 0xe0000000 #define CONFIG_SYS_PCIE2_MEM_PHYS 0xc20000000ull -#else -#define CONFIG_SYS_PCIE2_MEM_BUS 0xa0000000 -#define CONFIG_SYS_PCIE2_MEM_PHYS 0xa0000000 -#endif -#define CONFIG_SYS_PCIE2_MEM_SIZE 0x20000000 /* 512M */ #define CONFIG_SYS_PCIE2_IO_VIRT 0xf8010000 -#define CONFIG_SYS_PCIE2_IO_BUS 0x00000000 -#ifdef CONFIG_PHYS_64BIT #define CONFIG_SYS_PCIE2_IO_PHYS 0xff8010000ull -#else -#define CONFIG_SYS_PCIE2_IO_PHYS 0xf8010000 -#endif -#define CONFIG_SYS_PCIE2_IO_SIZE 0x00010000 /* 64k */ /* controller 3, Slot 1, tgtid 1, Base address 202000 */ #define CONFIG_SYS_PCIE3_MEM_VIRT 0xc0000000 -#ifdef CONFIG_PHYS_64BIT -#define CONFIG_SYS_PCIE3_MEM_BUS 0xe0000000 #define CONFIG_SYS_PCIE3_MEM_PHYS 0xc40000000ull -#else -#define CONFIG_SYS_PCIE3_MEM_BUS 0xc0000000 -#define CONFIG_SYS_PCIE3_MEM_PHYS 0xc0000000 -#endif -#define CONFIG_SYS_PCIE3_MEM_SIZE 0x20000000 /* 512M */ #define CONFIG_SYS_PCIE3_IO_VIRT 0xf8020000 -#define CONFIG_SYS_PCIE3_IO_BUS 0x00000000 -#ifdef CONFIG_PHYS_64BIT #define CONFIG_SYS_PCIE3_IO_PHYS 0xff8020000ull -#else -#define CONFIG_SYS_PCIE3_IO_PHYS 0xf8020000 -#endif -#define CONFIG_SYS_PCIE3_IO_SIZE 0x00010000 /* 64k */ /* controller 4, Base address 203000 */ -#define CONFIG_SYS_PCIE4_MEM_BUS 0xe0000000 #define CONFIG_SYS_PCIE4_MEM_PHYS 0xc60000000ull -#define CONFIG_SYS_PCIE4_MEM_SIZE 0x20000000 /* 512M */ -#define CONFIG_SYS_PCIE4_IO_BUS 0x00000000 #define CONFIG_SYS_PCIE4_IO_PHYS 0xff8030000ull -#define CONFIG_SYS_PCIE4_IO_SIZE 0x00010000 /* 64k */ /* Qman/Bman */ #define CONFIG_SYS_BMAN_NUM_PORTALS 10 @@ -505,7 +461,26 @@ #endif #ifdef CONFIG_PCI +#if !defined(CONFIG_DM_PCI) +#define CONFIG_FSL_PCI_INIT /* Use common FSL init code */ #define CONFIG_PCI_INDIRECT_BRIDGE +#define CONFIG_SYS_PCIE1_MEM_BUS 0xe0000000 +#define CONFIG_SYS_PCIE1_MEM_SIZE 0x20000000 /* 512M */ +#define CONFIG_SYS_PCIE1_IO_BUS 0x00000000 +#define CONFIG_SYS_PCIE1_IO_SIZE 0x00010000 /* 64k */ +#define CONFIG_SYS_PCIE2_MEM_BUS 0xe0000000 +#define CONFIG_SYS_PCIE2_MEM_SIZE 0x20000000 /* 512M */ +#define CONFIG_SYS_PCIE2_IO_BUS 0x00000000 +#define CONFIG_SYS_PCIE2_IO_SIZE 0x00010000 /* 64k */ +#define CONFIG_SYS_PCIE3_MEM_BUS 0xe0000000 +#define CONFIG_SYS_PCIE3_MEM_SIZE 0x20000000 /* 512M */ +#define CONFIG_SYS_PCIE3_IO_BUS 0x00000000 +#define CONFIG_SYS_PCIE3_IO_SIZE 0x00010000 /* 64k */ +#define CONFIG_SYS_PCIE4_MEM_BUS 0xe0000000 +#define CONFIG_SYS_PCIE4_MEM_SIZE 0x20000000 /* 512M */ +#define CONFIG_SYS_PCIE4_IO_BUS 0x00000000 +#define CONFIG_SYS_PCIE4_IO_SIZE 0x00010000 /* 64k */ +#endif #define CONFIG_PCI_SCAN_SHOW /* show pci devices on startup */ #endif /* CONFIG_PCI */
Disable legacy PCIe driver and unused PCIe macros when DM_PCI enabled. Signed-off-by: Hou Zhiqiang <Zhiqiang.Hou@nxp.com> --- include/configs/corenet_ds.h | 63 +++++++++++++------------------------------- 1 file changed, 19 insertions(+), 44 deletions(-)