Hi Lokesh, On Thu, Jan 10, 2019 at 4:47 AM Lokesh Vutla <lokeshvu...@ti.com> 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 <prane...@ti.com> > > --- > > 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
