Re: [U-Boot] [PATCH] arm: ti: boot: Fix am57xx evm fdtfile name
Hi Lokesh, On Thu, Jan 10, 2019 at 4:47 AM Lokesh Vutla wrote: > > > > On 1/10/2019 1:55 AM, Praneeth Bajjuri wrote: > > am57xx evm uses am57xx-evm-reva3.dtb. > > update findfdt to pick the correct default dtb > > > > Signed-off-by: Praneeth Bajjuri > > --- > > include/environment/ti/boot.h | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/include/environment/ti/boot.h b/include/environment/ti/boot.h > > index 86ff6d3ea7c6..c9ad3046acaa 100644 > > --- a/include/environment/ti/boot.h > > +++ b/include/environment/ti/boot.h > > @@ -125,7 +125,7 @@ > > "if test $board_name = am57xx_evm; then " \ > > "setenv fdtfile am57xx-beagle-x15.dtb; fi;" \ > > "if test $board_name = am57xx_evm_reva3; then " \ > > - "setenv fdtfile am57xx-beagle-x15.dtb; fi;" \ > > + "setenv fdtfile am57xx-evm-reva3.dtb; fi;" \ > > I do not see this dts in kernel. Either it is not upstream or we are > suposed to use overlays in this case. > Agreed. We should target upstream kernel in upstream U-Boot. So we should either: 1. Add corresponding dtb/dtbo files for AM57x EVM in upstream kernel first 2. Or drop this patch and continue using X15 dtb for EVM. As I understand, (1) is not going to happen any time soon (though I would prefer this solution, as EVM already happened and being used). So we should probably stick to (2). > Thanks and regards, > Lokesh > > > "if test $board_name = am571x_idk; then " \ > > "setenv fdtfile am571x-idk.dtb; fi;" \ > > "if test $fdtfile = undefined; then " \ > > ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
Re: [U-Boot] [PATCH] arm: ti: boot: Fix am57xx evm fdtfile name
On 1/10/2019 1:55 AM, Praneeth Bajjuri wrote: > am57xx evm uses am57xx-evm-reva3.dtb. > update findfdt to pick the correct default dtb > > Signed-off-by: Praneeth Bajjuri > --- > include/environment/ti/boot.h | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/include/environment/ti/boot.h b/include/environment/ti/boot.h > index 86ff6d3ea7c6..c9ad3046acaa 100644 > --- a/include/environment/ti/boot.h > +++ b/include/environment/ti/boot.h > @@ -125,7 +125,7 @@ > "if test $board_name = am57xx_evm; then " \ > "setenv fdtfile am57xx-beagle-x15.dtb; fi;" \ > "if test $board_name = am57xx_evm_reva3; then " \ > - "setenv fdtfile am57xx-beagle-x15.dtb; fi;" \ > + "setenv fdtfile am57xx-evm-reva3.dtb; fi;" \ I do not see this dts in kernel. Either it is not upstream or we are suposed to use overlays in this case. Thanks and regards, Lokesh > "if test $board_name = am571x_idk; then " \ > "setenv fdtfile am571x-idk.dtb; fi;" \ > "if test $fdtfile = undefined; then " \ > ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
[U-Boot] [PATCH] arm: ti: boot: Fix am57xx evm fdtfile name
am57xx evm uses am57xx-evm-reva3.dtb. update findfdt to pick the correct default dtb Signed-off-by: Praneeth Bajjuri --- include/environment/ti/boot.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/include/environment/ti/boot.h b/include/environment/ti/boot.h index 86ff6d3ea7c6..c9ad3046acaa 100644 --- a/include/environment/ti/boot.h +++ b/include/environment/ti/boot.h @@ -125,7 +125,7 @@ "if test $board_name = am57xx_evm; then " \ "setenv fdtfile am57xx-beagle-x15.dtb; fi;" \ "if test $board_name = am57xx_evm_reva3; then " \ - "setenv fdtfile am57xx-beagle-x15.dtb; fi;" \ + "setenv fdtfile am57xx-evm-reva3.dtb; fi;" \ "if test $board_name = am571x_idk; then " \ "setenv fdtfile am571x-idk.dtb; fi;" \ "if test $fdtfile = undefined; then " \ -- 2.7.4 ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot