MIPS: BCM47xx: Fix nvram_getenv return value.
Hauke Mehrtens [Mon, 2 Aug 2010 21:56:22 +0000 (23:56 +0200)]
Nvram_getenv should behave like cfe_getenv. cfe_getenv returns 0 on
success and -9 if the value was not found. If the input was wrong -8
will be returned by cfe_getenv.  Change nvram_getenv to do the same.

Signed-off-by: Hauke Mehrtens <hauke@hauke-m.de>
Cc: Waldemar Brodkorb <wbx@openadk.org>
Cc: linux-mips@linux-mips.org
Patchwork: https://patchwork.linux-mips.org/patch/1520/
Signed-off-by: Ralf Baechle <ralf@linux-mips.org>

arch/mips/bcm47xx/nvram.c
arch/mips/include/asm/mach-bcm47xx/nvram.h

index 06e03b2..e5b6615 100644 (file)
@@ -69,7 +69,7 @@ int nvram_getenv(char *name, char *val, size_t val_len)
        char *var, *value, *end, *eq;
 
        if (!name)
-               return 1;
+               return NVRAM_ERR_INV_PARAM;
 
        if (!nvram_buf[0])
                early_nvram_init();
@@ -89,6 +89,6 @@ int nvram_getenv(char *name, char *val, size_t val_len)
                        return 0;
                }
        }
-       return 1;
+       return NVRAM_ERR_ENVNOTFOUND;
 }
 EXPORT_SYMBOL(nvram_getenv);
index 0d8cc14..c58ebd8 100644 (file)
@@ -31,6 +31,9 @@ struct nvram_header {
 #define NVRAM_MAX_VALUE_LEN 255
 #define NVRAM_MAX_PARAM_LEN 64
 
+#define NVRAM_ERR_INV_PARAM    -8
+#define NVRAM_ERR_ENVNOTFOUND  -9
+
 extern int nvram_getenv(char *name, char *val, size_t val_len);
 
 #endif