From mboxrd@z Thu Jan 1 00:00:00 1970 From: Maxime Ripard Date: Fri, 5 Jan 2018 10:29:33 +0100 Subject: [U-Boot] [PATCH 12/14] env: Mark env_get_location as weak In-Reply-To: References: <537a3581d52c026ce964762a38cf65faed7c175c.1511864667.git-series.maxime.ripard@free-electrons.com> Message-ID: <20180105092933.2muofkfh3opbmvqf@flea.lan> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hi Simon, On Thu, Dec 28, 2017 at 08:13:42PM -0700, Simon Glass wrote: > Hi Maxime, > > On 28 November 2017 at 03:24, Maxime Ripard > wrote: > > Allow boards and architectures to override the default environment lookup > > code by overriding env_get_location. > > > > Reviewed-by: Lukasz Majewski > > Signed-off-by: Maxime Ripard > > --- > > env/env.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/env/env.c b/env/env.c > > index b4d8886e7a69..9b8b38cf3c2b 100644 > > --- a/env/env.c > > +++ b/env/env.c > > @@ -62,7 +62,7 @@ static enum env_location env_locations[] = { > > > > static enum env_location env_load_location; > > > > -static enum env_location env_get_location(enum env_operation op, int prio) > > +__weak enum env_location env_get_location(enum env_operation op, int prio) > > Is it possible instead to make this deterministic, so that the board > sets up the location during boot? I'm not really sure what you mean here. The default implementation is deterministic, and the board implementations should make sure that it is if it's of any value to them. Do you want me to add a comment to make that clearer? Thanks! Maxime -- Maxime Ripard, Free Electrons Embedded Linux and Kernel engineering http://free-electrons.com -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 833 bytes Desc: not available URL: