cmd: Migrate from_env() from pxe.c to nvedit.c

Migrate from_env() from pxe.c to nvedit.c as it's not pxe specific.

Signed-off-by: Patrice Chotard <patrice.chotard@st.com>

diff --git a/cmd/nvedit.c b/cmd/nvedit.c
index b5da37591336ff96e810ec2d50da2bc42fbf0591..3420e0b985b984fd50a581666f787592d10c5d06 100644 (file)
--- a/cmd/nvedit.c
+++ b/cmd/nvedit.c
@@ -681,6 +681,23 @@ char *env_get(const char *name)
        return NULL;
 }
 
+/*
+ * Like env_get, but prints an error if envvar isn't defined in the
+ * environment.  It always returns what env_get does, so it can be used in
+ * place of env_get without changing error handling otherwise.
+ */
+char *from_env(const char *envvar)
+{
+       char *ret;
+
+       ret = env_get(envvar);
+
+       if (!ret)
+               printf("missing environment variable: %s\n", envvar);
+
+       return ret;
+}
+
 /*
  * Look up variable from environment for restricted C runtime env.
  */

diff --git a/cmd/pxe.c b/cmd/pxe.c
index 768e50aba610ed6a60336597b6b85f0e8d3abd94..ed553953b37d823ed769f18af3210a16abe01c24 100644 (file)
--- a/cmd/pxe.c
+++ b/cmd/pxe.c
@@ -37,23 +37,6 @@ const char *pxe_default_paths[] = {
 
 static bool is_pxe;
 
-/*
- * Like env_get, but prints an error if envvar isn't defined in the
- * environment.  It always returns what env_get does, so it can be used in
- * place of env_get without changing error handling otherwise.
- */
-static char *from_env(const char *envvar)
-{
-       char *ret;
-
-       ret = env_get(envvar);
-
-       if (!ret)
-               printf("missing environment variable: %s\n", envvar);
-
-       return ret;
-}
-
 #ifdef CONFIG_CMD_NET
 /*
  * Convert an ethaddr from the environment to the format used by pxelinux

diff --git a/include/env.h b/include/env.h
index b72239f6a58c915e0d133107be8108fbec15adf5..d6c2d751d62b6cc7eb3bbf9a108b36be14a8a486 100644 (file)
--- a/include/env.h
+++ b/include/env.h
@@ -113,6 +113,16 @@ int env_match(unsigned char *name, int index);
  */
 char *env_get(const char *varname);
 
+/*
+ * Like env_get, but prints an error if envvar isn't defined in the
+ * environment.  It always returns what env_get does, so it can be used in
+ * place of env_get without changing error handling otherwise.
+ *
+ * @varname:   Variable to look up
+ * @return value of variable, or NULL if not found
+ */
+char *from_env(const char *envvar);
+
 /**
  * env_get_f() - Look up the value of an environment variable (early)
  *