From mboxrd@z Thu Jan 1 00:00:00 1970 From: Lukasz Majewski Date: Fri, 17 Nov 2017 10:14:17 +0100 Subject: [U-Boot] [RFC PATCH 03/10] env: Rename env_driver_lookup_default and env_get_default_location In-Reply-To: <20171116092231.27740-4-maxime.ripard@free-electrons.com> References: <20171116092231.27740-1-maxime.ripard@free-electrons.com> <20171116092231.27740-4-maxime.ripard@free-electrons.com> Message-ID: <20171117101417.37b8dc99@jawa> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On Thu, 16 Nov 2017 10:22:24 +0100 Maxime Ripard wrote: > The env_driver_lookup_default and env_get_default_location functions > are about to get refactored to support loading from multiple > environment. > > The name is therefore not really well suited anymore. Drop the default > part to be a bit more relevant. > > Signed-off-by: Maxime Ripard > --- > env/env.c | 18 +++++++++--------- > 1 file changed, 9 insertions(+), 9 deletions(-) > > diff --git a/env/env.c b/env/env.c > index 094538ff5b62..97ada5b5a6fd 100644 > --- a/env/env.c > +++ b/env/env.c > @@ -10,7 +10,7 @@ > > DECLARE_GLOBAL_DATA_PTR; > > -static struct env_driver *env_driver_lookup(enum env_location loc) > +static struct env_driver *_env_driver_lookup(enum env_location loc) > { > struct env_driver *drv; > const int n_ents = ll_entry_count(struct env_driver, > env_driver); @@ -26,7 +26,7 @@ static struct env_driver > *env_driver_lookup(enum env_location loc) return NULL; > } > > -static enum env_location env_get_default_location(void) > +static enum env_location env_get_location(void) > { > if IS_ENABLED(CONFIG_ENV_IS_IN_EEPROM) > return ENVL_EEPROM; > @@ -52,12 +52,12 @@ static enum env_location > env_get_default_location(void) return ENVL_UNKNOWN; > } > > -static struct env_driver *env_driver_lookup_default(void) > +static struct env_driver *env_driver_lookup(void) > { > - enum env_location loc = env_get_default_location(); > + enum env_location loc = env_get_location(); > struct env_driver *drv; > > - drv = env_driver_lookup(loc); > + drv = _env_driver_lookup(loc); > if (!drv) { > debug("%s: No environment driver for location %d\n", > __func__, loc); > @@ -69,7 +69,7 @@ static struct env_driver > *env_driver_lookup_default(void) > int env_get_char(int index) > { > - struct env_driver *drv = env_driver_lookup_default(); > + struct env_driver *drv = env_driver_lookup(); > int ret; > > if (gd->env_valid == ENV_INVALID) > @@ -89,7 +89,7 @@ int env_get_char(int index) > > int env_load(void) > { > - struct env_driver *drv = env_driver_lookup_default(); > + struct env_driver *drv = env_driver_lookup(); > int ret = 0; > > if (!drv) > @@ -108,7 +108,7 @@ int env_load(void) > > int env_save(void) > { > - struct env_driver *drv = env_driver_lookup_default(); > + struct env_driver *drv = env_driver_lookup(); > int ret; > > if (!drv) > @@ -129,7 +129,7 @@ int env_save(void) > > int env_init(void) > { > - struct env_driver *drv = env_driver_lookup_default(); > + struct env_driver *drv = env_driver_lookup(); > int ret = -ENOENT; > > if (!drv) Reviewed-by: Lukasz Majewski Best regards, Lukasz Majewski -- DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de -------------- next part -------------- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 488 bytes Desc: OpenPGP digital signature URL: