On Fri, Mar 31, 2023 at 12:19:14PM -0500, Andrew Davis wrote: > On 3/31/23 12:04 PM, Nishanth Menon wrote: > > On 16:06-20230330, Ryan Eatmon wrote: > > > The oe-core class for assembling the fitImage includes the vendor > > > sub-directory (with the / changed to _) in the config sections of > > > the fitImage. Our env var settings for chosing which section to > > > boot from needs to be updated to agree with the fitImage. > > > > > > Signed-off-by: Ryan Eatmon <reat...@ti.com> > > > --- > > > include/configs/am64x_evm.h | 4 ++-- > > > include/configs/am65x_evm.h | 2 +- > > > include/configs/j721e_evm.h | 6 +++--- > > > include/configs/j721s2_evm.h | 6 +++--- > > > 4 files changed, 9 insertions(+), 9 deletions(-) > > > > > > diff --git a/include/configs/am64x_evm.h b/include/configs/am64x_evm.h > > > index 26a7f2521e..456a44730c 100644 > > > --- a/include/configs/am64x_evm.h > > > +++ b/include/configs/am64x_evm.h > > > @@ -26,9 +26,9 @@ > > > #define EXTRA_ENV_AM642_BOARD_SETTINGS > > > \ > > > "findfdt=" > > > \ > > > "if test $board_name = am64x_gpevm; then " \ > > > - "setenv fdtfile k3-am642-evm.dtb; fi; " \ > > > + "setenv fdtfile ti_k3-am642-evm.dtb; fi; " \ > > > "if test $board_name = am64x_skevm; then " \ > > > - "setenv fdtfile k3-am642-sk.dtb; fi;" \ > > > + "setenv fdtfile ti_k3-am642-sk.dtb; fi;" \ > > > > > > Kernel.org builds k3-am642-sk.dtb > > > > Nope, kernel.org build ti/k3-am642-sk.dtb > > > Why would we want to go and fix it with non-standard? > > > > The dtbs need to have their dir prefixed or there could be collisions
The key here is we're talking about what's used in FIT images, and not just loaded directly. All of the cases where fdtfile is used on arm64 platforms in tree today use "/" because it's loading a literal. In this case, why again is OE mangling the name, and what are we using it for again here? In other words, why can't it be treated as "/" to match the load a file case? And if it _must_ be mangled, must we still use fdtfile and not something else? -- Tom
signature.asc
Description: PGP signature