[PATCH 1/3] env: Implement lower priority arch_env_get_location()

Tom Rini trini at konsulko.com
Thu Apr 21 17:43:38 CEST 2022


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 <marex at denx.de>
> Cc: Adam Ford <aford173 at gmail.com>
> Cc: Fabio Estevam <festevam at denx.de>
> Cc: Marek Behún <marek.behun at nic.cz>
> Cc: Peng Fan <peng.fan at nxp.com>
> Cc: Simon Glass <sjg at chromium.org>
> Cc: Stefano Babic <sbabic at denx.de>
> Cc: Tim Harvey <tharvey at gateworks.com>
> Cc: Tom Rini <trini at konsulko.com>
> Cc: Tommaso Merciai <tomm.merciai at gmail.com>
> Cc: Vladimir Oltean <vladimir.oltean at nxp.com>

Applied to u-boot/master, thanks!

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20220421/9ea122c3/attachment.sig>


More information about the U-Boot mailing list