Hi Simon,

On Wed, Jan 17, 2018 at 03:07:58PM -0700, Simon Glass wrote:
> On 16 January 2018 at 01:16, Maxime Ripard
> <maxime.rip...@free-electrons.com> wrote:
> > Allow boards and architectures to override the default environment lookup
> > code by overriding env_get_location.
> >
> > Reviewed-by: Andre Przywara <andre.przyw...@arm.com>
> > Reviewed-by: Lukasz Majewski <lu...@denx.de>
> > Signed-off-by: Maxime Ripard <maxime.rip...@free-electrons.com>
> > ---
> >  env/env.c | 20 +++++++++++++++++++-
> >  1 file changed, 19 insertions(+), 1 deletion(-)
> >
> 
> I still don't really understand why this needs to be a weak function.
> If the board knows the priority order, can it not put it into
> global_data? We could have a little u8 array of 4 items with a
> terminator?

Sure that would be simpler, but that would also prevent us from doing
"smart" things based on data other than just whether the previous
environment is usable. Things based for example on a GPIO state, or a
custom algorithm to transition (or duplicate) the environment.

Maxime

-- 
Maxime Ripard, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com

Attachment: signature.asc
Description: PGP signature

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Reply via email to