On Wed, Apr 06, 2022 at 02:21:32AM +0200, Marek Vasut wrote: > Currently there is only one way to override desired environment location, > by implementing env_get_location(). This is increasingly being conflated > both on board level and architecture level, which leads to a problem on > boards where this function is already implemented on architecture level, > since those boards have no way to override this environment location on > board level anymore. > > Implement arch_env_get_location() function which is architecture specific > and should only ever be implemented in architecture code. This function > has lower priority than env_get_location(), which should only ever be > implemented in board code, and which overrides the arch_env_get_location() > architecture environment selection. > > This way, architecture can define its default environment chooser, while > board can now override it as needed at all times. > > There is no functional change, since env_get_location() simply returns > arch_env_get_location(), and arch_env_get_location() implements the > current env_get_location() default content. > > Signed-off-by: Marek Vasut <[email protected]> > Cc: Adam Ford <[email protected]> > Cc: Fabio Estevam <[email protected]> > Cc: Marek Behún <[email protected]> > Cc: Peng Fan <[email protected]> > Cc: Simon Glass <[email protected]> > Cc: Stefano Babic <[email protected]> > Cc: Tim Harvey <[email protected]> > Cc: Tom Rini <[email protected]> > Cc: Tommaso Merciai <[email protected]> > Cc: Vladimir Oltean <[email protected]>
Applied to u-boot/master, thanks! -- Tom
signature.asc
Description: PGP signature
