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 <ma...@denx.de> > Cc: Adam Ford <aford...@gmail.com> > Cc: Fabio Estevam <feste...@denx.de> > Cc: Marek BehĂșn <marek.be...@nic.cz> > Cc: Peng Fan <peng....@nxp.com> > Cc: Simon Glass <s...@chromium.org> > Cc: Stefano Babic <sba...@denx.de> > Cc: Tim Harvey <thar...@gateworks.com> > Cc: Tom Rini <tr...@konsulko.com> > Cc: Tommaso Merciai <tomm.merc...@gmail.com> > Cc: Vladimir Oltean <vladimir.olt...@nxp.com>
Applied to u-boot/master, thanks! -- Tom
signature.asc
Description: PGP signature