env: the ops driver load becomes mandatory in struct env_driver
authorPatrick Delaunay <patrick.delaunay@st.com>
Tue, 28 Jul 2020 09:51:19 +0000 (11:51 +0200)
committerTom Rini <trini@konsulko.com>
Fri, 31 Jul 2020 14:13:00 +0000 (10:13 -0400)
The ops driver load becomes mandatory in struct env_drive,
change the comment for this ops and remove unnecessary test.

Signed-off-by: Patrick Delaunay <patrick.delaunay@st.com>
env/env.c
include/env_internal.h

index bcc68c6..5cf5bd2 100644 (file)
--- a/env/env.c
+++ b/env/env.c
@@ -187,9 +187,6 @@ int env_load(void)
        for (prio = 0; (drv = env_driver_lookup(ENVOP_LOAD, prio)); prio++) {
                int ret;
 
-               if (!drv->load)
-                       continue;
-
                if (!env_has_inited(drv->location))
                        continue;
 
index b9459f9..b26dc62 100644 (file)
@@ -154,8 +154,7 @@ struct env_driver {
        /**
         * load() - Load the environment from storage
         *
-        * This method is optional. If not provided, no environment will be
-        * loaded.
+        * This method is required for loading environment
         *
         * @return 0 if OK, -ve on error
         */