Re: [PATCH] arm: dts: fix ast2500-evb inclusion for correct target
On Sat, Aug 22, 2020 at 10:24:58PM -0700, Thirupathaiah Annapureddy wrote: > > > On 8/20/2020 5:03 PM, Tom Rini wrote: > > On Thu, Aug 20, 2020 at 02:09:55PM -0700, Thirupathaiah Annapureddy wrote: > >> Hi Ryan, > >> > >> Thanks for reviewing the patch. Please see my comment(s): > >> > >> On 8/19/2020 7:06 PM, Ryan Chen wrote: > >>> Hi > >>>> -Original Message- > >>>> From: Thirupathaiah Annapureddy [mailto:thir...@linux.microsoft.com] > >>>> Sent: Thursday, August 20, 2020 8:16 AM > >>>> To: u-boot@lists.denx.de > >>>> Cc: Maxim Sloyko ; Marek Vasut ; > >>>> ChiaWei Wang ; Ryan Chen > >>>> > >>>> Subject: Re: [PATCH] arm: dts: fix ast2500-evb inclusion for correct > >>>> target > >>>> > >>>> Adding Ryan and Chiawei to the list. > >>>> > >>>> On 8/17/2020 5:53 PM, Thirupathaiah Annapureddy wrote: > >>>>> Include ast2500-evb.dtb for CONFIG_TARGET_EVB_AST2500 instead of for > >>>>> all aspeed targets. > >>> > >>> There should not have to many Kconfig for ASPEED platform. > >> When you build U-Boot, you have to provide a platform specific defconfig > >> as the target. ex: evb-ast2500_defconfig. > >> defconfig specifies the platform specific device tree file. > >> ex: CONFIG_DEFAULT_DEVICE_TREE="ast2500-evb" > >> > >> I do not see any reason to make other device trees (ex: ast2600a0-evb.dtb) > >> when we are building for evb-ast2500. > > > > It keeps the Makefile logic clearer and makes future moves towards more > > platforms in a single binary easier if we just build all of the dtb > > files. > > > ast2400 is based on ARM926EJ-S processor (ARMv5-architecture). > ast2500 is based on ARM1176JZS processor (ARMv6-architecture). > ast2600 is based on Cortex A7 processor (ARMv7-A architecture). > > Each of the above SOC is using a different ARM CPU(s) with different ARM > architecture revision. Is it possible for single binary to support multiple > platforms based on above SOCs? Thanks for the details. It depends on thee goals and needs. There's probably some hurdles to supporting v5/v6 and v7 in a single binary. But yes, if we aren't supporting all 3 of those families yet, we should include and build the families we can support. But that would still mean all ast2500 DTBs for example still. -- Tom signature.asc Description: PGP signature
Re: [PATCH] arm: dts: fix ast2500-evb inclusion for correct target
On 8/20/2020 5:03 PM, Tom Rini wrote: > On Thu, Aug 20, 2020 at 02:09:55PM -0700, Thirupathaiah Annapureddy wrote: >> Hi Ryan, >> >> Thanks for reviewing the patch. Please see my comment(s): >> >> On 8/19/2020 7:06 PM, Ryan Chen wrote: >>> Hi >>>> -Original Message- >>>> From: Thirupathaiah Annapureddy [mailto:thir...@linux.microsoft.com] >>>> Sent: Thursday, August 20, 2020 8:16 AM >>>> To: u-boot@lists.denx.de >>>> Cc: Maxim Sloyko ; Marek Vasut ; >>>> ChiaWei Wang ; Ryan Chen >>>> >>>> Subject: Re: [PATCH] arm: dts: fix ast2500-evb inclusion for correct target >>>> >>>> Adding Ryan and Chiawei to the list. >>>> >>>> On 8/17/2020 5:53 PM, Thirupathaiah Annapureddy wrote: >>>>> Include ast2500-evb.dtb for CONFIG_TARGET_EVB_AST2500 instead of for >>>>> all aspeed targets. >>> >>> There should not have to many Kconfig for ASPEED platform. >> When you build U-Boot, you have to provide a platform specific defconfig >> as the target. ex: evb-ast2500_defconfig. >> defconfig specifies the platform specific device tree file. >> ex: CONFIG_DEFAULT_DEVICE_TREE="ast2500-evb" >> >> I do not see any reason to make other device trees (ex: ast2600a0-evb.dtb) >> when we are building for evb-ast2500. > > It keeps the Makefile logic clearer and makes future moves towards more > platforms in a single binary easier if we just build all of the dtb > files. > ast2400 is based on ARM926EJ-S processor (ARMv5-architecture). ast2500 is based on ARM1176JZS processor (ARMv6-architecture). ast2600 is based on Cortex A7 processor (ARMv7-A architecture). Each of the above SOC is using a different ARM CPU(s) with different ARM architecture revision. Is it possible for single binary to support multiple platforms based on above SOCs?
Re: [PATCH] arm: dts: fix ast2500-evb inclusion for correct target
On Thu, Aug 20, 2020 at 02:09:55PM -0700, Thirupathaiah Annapureddy wrote: > Hi Ryan, > > Thanks for reviewing the patch. Please see my comment(s): > > On 8/19/2020 7:06 PM, Ryan Chen wrote: > > Hi > >> -Original Message- > >> From: Thirupathaiah Annapureddy [mailto:thir...@linux.microsoft.com] > >> Sent: Thursday, August 20, 2020 8:16 AM > >> To: u-boot@lists.denx.de > >> Cc: Maxim Sloyko ; Marek Vasut ; > >> ChiaWei Wang ; Ryan Chen > >> > >> Subject: Re: [PATCH] arm: dts: fix ast2500-evb inclusion for correct target > >> > >> Adding Ryan and Chiawei to the list. > >> > >> On 8/17/2020 5:53 PM, Thirupathaiah Annapureddy wrote: > >>> Include ast2500-evb.dtb for CONFIG_TARGET_EVB_AST2500 instead of for > >>> all aspeed targets. > > > > There should not have to many Kconfig for ASPEED platform. > When you build U-Boot, you have to provide a platform specific defconfig > as the target. ex: evb-ast2500_defconfig. > defconfig specifies the platform specific device tree file. > ex: CONFIG_DEFAULT_DEVICE_TREE="ast2500-evb" > > I do not see any reason to make other device trees (ex: ast2600a0-evb.dtb) > when we are building for evb-ast2500. It keeps the Makefile logic clearer and makes future moves towards more platforms in a single binary easier if we just build all of the dtb files. -- Tom signature.asc Description: PGP signature
Re: [PATCH] arm: dts: fix ast2500-evb inclusion for correct target
Hi Ryan, Thanks for reviewing the patch. Please see my comment(s): On 8/19/2020 7:06 PM, Ryan Chen wrote: > Hi >> -Original Message- >> From: Thirupathaiah Annapureddy [mailto:thir...@linux.microsoft.com] >> Sent: Thursday, August 20, 2020 8:16 AM >> To: u-boot@lists.denx.de >> Cc: Maxim Sloyko ; Marek Vasut ; >> ChiaWei Wang ; Ryan Chen >> >> Subject: Re: [PATCH] arm: dts: fix ast2500-evb inclusion for correct target >> >> Adding Ryan and Chiawei to the list. >> >> On 8/17/2020 5:53 PM, Thirupathaiah Annapureddy wrote: >>> Include ast2500-evb.dtb for CONFIG_TARGET_EVB_AST2500 instead of for >>> all aspeed targets. > > There should not have to many Kconfig for ASPEED platform. When you build U-Boot, you have to provide a platform specific defconfig as the target. ex: evb-ast2500_defconfig. defconfig specifies the platform specific device tree file. ex: CONFIG_DEFAULT_DEVICE_TREE="ast2500-evb" I do not see any reason to make other device trees (ex: ast2600a0-evb.dtb) when we are building for evb-ast2500. > I prefer use following to build all all ASPEED platform. Like following. > dtb-$(CONFIG_ARCH_ASPEED) += \ > ast2400-evb.dtb \ > ast2500-evb.dtb \ > ast2600a0-evb.dtb \ > ast2600a0-ncsi.dtb \ > ast2600a1-evb.dtb \ > ast2600a1-ncsi.dtb \ > ast2600-fpga.dtb \ > ast2600-rainier.dtb \ > ast2600-slt.dtb \ > ast2600-tacoma.dtb > >>> >>> Signed-off-by: Thirupathaiah Annapureddy >> --- >>> arch/arm/dts/Makefile | 2 +- >>> 1 file changed, 1 insertion(+), 1 deletion(-) >>> >>> diff --git a/arch/arm/dts/Makefile b/arch/arm/dts/Makefile index >>> 7e29b9096b..d019f26983 100644 >>> --- a/arch/arm/dts/Makefile >>> +++ b/arch/arm/dts/Makefile >>> @@ -938,7 +938,7 @@ dtb-$(CONFIG_ARCH_BCM6858) += \ >>> >>> dtb-$(CONFIG_TARGET_BCMNS3) += ns3-board.dtb >>> >>> -dtb-$(CONFIG_ARCH_ASPEED) += ast2500-evb.dtb >>> +dtb-$(CONFIG_TARGET_EVB_AST2500) += ast2500-evb.dtb >>> >>> dtb-$(CONFIG_ARCH_STI) += stih410-b2260.dtb >>> >>> > Best Regards, Thiru
RE: [PATCH] arm: dts: fix ast2500-evb inclusion for correct target
Hi > -Original Message- > From: Thirupathaiah Annapureddy [mailto:thir...@linux.microsoft.com] > Sent: Thursday, August 20, 2020 8:16 AM > To: u-boot@lists.denx.de > Cc: Maxim Sloyko ; Marek Vasut ; > ChiaWei Wang ; Ryan Chen > > Subject: Re: [PATCH] arm: dts: fix ast2500-evb inclusion for correct target > > Adding Ryan and Chiawei to the list. > > On 8/17/2020 5:53 PM, Thirupathaiah Annapureddy wrote: > > Include ast2500-evb.dtb for CONFIG_TARGET_EVB_AST2500 instead of for > > all aspeed targets. There should not have to many Kconfig for ASPEED platform. I prefer use following to build all all ASPEED platform. Like following. dtb-$(CONFIG_ARCH_ASPEED) += \ ast2400-evb.dtb \ ast2500-evb.dtb \ ast2600a0-evb.dtb \ ast2600a0-ncsi.dtb \ ast2600a1-evb.dtb \ ast2600a1-ncsi.dtb \ ast2600-fpga.dtb \ ast2600-rainier.dtb \ ast2600-slt.dtb \ ast2600-tacoma.dtb > > > > Signed-off-by: Thirupathaiah Annapureddy > --- > > arch/arm/dts/Makefile | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/arch/arm/dts/Makefile b/arch/arm/dts/Makefile index > > 7e29b9096b..d019f26983 100644 > > --- a/arch/arm/dts/Makefile > > +++ b/arch/arm/dts/Makefile > > @@ -938,7 +938,7 @@ dtb-$(CONFIG_ARCH_BCM6858) += \ > > > > dtb-$(CONFIG_TARGET_BCMNS3) += ns3-board.dtb > > > > -dtb-$(CONFIG_ARCH_ASPEED) += ast2500-evb.dtb > > +dtb-$(CONFIG_TARGET_EVB_AST2500) += ast2500-evb.dtb > > > > dtb-$(CONFIG_ARCH_STI) += stih410-b2260.dtb > > > >
Re: [PATCH] arm: dts: fix ast2500-evb inclusion for correct target
Adding Ryan and Chiawei to the list. On 8/17/2020 5:53 PM, Thirupathaiah Annapureddy wrote: > Include ast2500-evb.dtb for CONFIG_TARGET_EVB_AST2500 instead of > for all aspeed targets. > > Signed-off-by: Thirupathaiah Annapureddy --- > arch/arm/dts/Makefile | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/arch/arm/dts/Makefile b/arch/arm/dts/Makefile > index 7e29b9096b..d019f26983 100644 > --- a/arch/arm/dts/Makefile > +++ b/arch/arm/dts/Makefile > @@ -938,7 +938,7 @@ dtb-$(CONFIG_ARCH_BCM6858) += \ > > dtb-$(CONFIG_TARGET_BCMNS3) += ns3-board.dtb > > -dtb-$(CONFIG_ARCH_ASPEED) += ast2500-evb.dtb > +dtb-$(CONFIG_TARGET_EVB_AST2500) += ast2500-evb.dtb > > dtb-$(CONFIG_ARCH_STI) += stih410-b2260.dtb > >
[PATCH] arm: dts: fix ast2500-evb inclusion for correct target
Include ast2500-evb.dtb for CONFIG_TARGET_EVB_AST2500 instead of for all aspeed targets. Signed-off-by: Thirupathaiah Annapureddy --- arch/arm/dts/Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/arm/dts/Makefile b/arch/arm/dts/Makefile index 7e29b9096b..d019f26983 100644 --- a/arch/arm/dts/Makefile +++ b/arch/arm/dts/Makefile @@ -938,7 +938,7 @@ dtb-$(CONFIG_ARCH_BCM6858) += \ dtb-$(CONFIG_TARGET_BCMNS3) += ns3-board.dtb -dtb-$(CONFIG_ARCH_ASPEED) += ast2500-evb.dtb +dtb-$(CONFIG_TARGET_EVB_AST2500) += ast2500-evb.dtb dtb-$(CONFIG_ARCH_STI) += stih410-b2260.dtb -- 2.25.2