state: disable load command
Explicitly loading environment is not required as it will be loaded if available during device probing Signed-off-by: Enrico Jorns <ejo@pengutronix.de> Signed-off-by: Marc Kleine-Budde <mkl@pengutronix.de> Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
This commit is contained in:
parent
7098d7121e
commit
446f3d07dd
|
@ -21,26 +21,20 @@ static int do_state(int argc, char *argv[])
|
|||
{
|
||||
int opt, ret = 0;
|
||||
struct state *state = NULL;
|
||||
int do_save = 0, do_load = 0;
|
||||
int do_save = 0;
|
||||
const char *statename = "state";
|
||||
|
||||
while ((opt = getopt(argc, argv, "sl")) > 0) {
|
||||
while ((opt = getopt(argc, argv, "s")) > 0) {
|
||||
switch (opt) {
|
||||
case 's':
|
||||
do_save = 1;
|
||||
break;
|
||||
case 'l':
|
||||
do_load = 1;
|
||||
break;
|
||||
default:
|
||||
return COMMAND_ERROR_USAGE;
|
||||
}
|
||||
}
|
||||
|
||||
if (do_save && do_load)
|
||||
return COMMAND_ERROR_USAGE;
|
||||
|
||||
if (!do_save && !do_load) {
|
||||
if (!do_save) {
|
||||
state_info();
|
||||
return 0;
|
||||
}
|
||||
|
@ -56,8 +50,6 @@ static int do_state(int argc, char *argv[])
|
|||
|
||||
if (do_save)
|
||||
ret = state_save(state);
|
||||
else if (do_load)
|
||||
ret = state_load(state);
|
||||
|
||||
return ret;
|
||||
}
|
||||
|
@ -67,13 +59,12 @@ BAREBOX_CMD_HELP_TEXT("Usage: state [OPTIONS] [STATENAME]")
|
|||
BAREBOX_CMD_HELP_TEXT("")
|
||||
BAREBOX_CMD_HELP_TEXT("options:")
|
||||
BAREBOX_CMD_HELP_OPT ("-s", "save state")
|
||||
BAREBOX_CMD_HELP_OPT ("-l", "load state")
|
||||
BAREBOX_CMD_HELP_END
|
||||
|
||||
BAREBOX_CMD_START(state)
|
||||
.cmd = do_state,
|
||||
BAREBOX_CMD_DESC("load and save state information")
|
||||
BAREBOX_CMD_OPTS("[-sl] [STATENAME]")
|
||||
BAREBOX_CMD_DESC("save state information")
|
||||
BAREBOX_CMD_OPTS("[-s] [STATENAME]")
|
||||
BAREBOX_CMD_GROUP(CMD_GRP_MISC)
|
||||
BAREBOX_CMD_HELP(cmd_state_help)
|
||||
BAREBOX_CMD_END
|
||||
|
|
|
@ -51,7 +51,6 @@ struct state {
|
|||
};
|
||||
|
||||
struct state_backend {
|
||||
int (*load)(struct state_backend *backend, struct state *state);
|
||||
int (*save)(struct state_backend *backend, struct state *state);
|
||||
const char *name;
|
||||
const char *of_path;
|
||||
|
@ -1048,30 +1047,6 @@ int state_get_name(const struct state *state, char const **name)
|
|||
return 0;
|
||||
}
|
||||
|
||||
/*
|
||||
* state_load - load a state from the backing store
|
||||
*
|
||||
* @state The state instance to load
|
||||
*/
|
||||
int state_load(struct state *state)
|
||||
{
|
||||
int ret;
|
||||
|
||||
if (!state->backend)
|
||||
return -ENOSYS;
|
||||
|
||||
ret = state->backend->load(state->backend, state);
|
||||
if (ret) {
|
||||
dev_warn(&state->dev, "load failed\n");
|
||||
state->dirty = 1;
|
||||
} else {
|
||||
dev_info(&state->dev, "load successful\n");
|
||||
state->dirty = 0;
|
||||
}
|
||||
|
||||
return ret;
|
||||
}
|
||||
|
||||
/*
|
||||
* state_save - save a state to the backing store
|
||||
*
|
||||
|
@ -1226,7 +1201,6 @@ int state_backend_dtb_file(struct state *state, const char *of_path, const char
|
|||
backend_dtb = xzalloc(sizeof(*backend_dtb));
|
||||
backend = &backend_dtb->backend;
|
||||
|
||||
backend->load = state_backend_dtb_load;
|
||||
backend->save = state_backend_dtb_save;
|
||||
backend->of_path = xstrdup(of_path);
|
||||
backend->path = xstrdup(path);
|
||||
|
@ -1238,6 +1212,15 @@ int state_backend_dtb_file(struct state *state, const char *of_path, const char
|
|||
if (!ret && !(meminfo.flags & MTD_NO_ERASE))
|
||||
backend_dtb->need_erase = true;
|
||||
|
||||
ret = state_backend_dtb_load(backend, state);
|
||||
if (ret) {
|
||||
dev_warn(&state->dev, "load failed - using defaults\n");
|
||||
} else {
|
||||
dev_info(&state->dev, "load successful\n");
|
||||
state->dirty = 0;
|
||||
}
|
||||
|
||||
/* ignore return value of load() */
|
||||
return 0;
|
||||
}
|
||||
|
||||
|
@ -1548,7 +1531,6 @@ int state_backend_raw_file(struct state *state, const char *of_path,
|
|||
backend_raw = xzalloc(sizeof(*backend_raw));
|
||||
backend = &backend_raw->backend;
|
||||
|
||||
backend->load = state_backend_raw_load;
|
||||
backend->save = state_backend_raw_save;
|
||||
backend->of_path = xstrdup(of_path);
|
||||
backend->path = xstrdup(path);
|
||||
|
@ -1583,6 +1565,15 @@ int state_backend_raw_file(struct state *state, const char *of_path,
|
|||
goto err;
|
||||
}
|
||||
|
||||
ret = state_backend_raw_load(backend, state);
|
||||
if (ret) {
|
||||
dev_warn(&state->dev, "load failed - using defaults\n");
|
||||
} else {
|
||||
dev_info(&state->dev, "load successful\n");
|
||||
state->dirty = 0;
|
||||
}
|
||||
|
||||
/* ignore return value of load() */
|
||||
return 0;
|
||||
err:
|
||||
free(backend_raw);
|
||||
|
|
|
@ -86,7 +86,6 @@ static int state_probe(struct device_d *dev)
|
|||
dev_info(dev, "backend: %s, path: %s, of_path: %s\n", backend_type, path, of_path);
|
||||
free(path);
|
||||
|
||||
state_load(state);
|
||||
return 0;
|
||||
|
||||
out_free:
|
||||
|
|
|
@ -17,7 +17,6 @@ struct state *state_by_name(const char *name);
|
|||
struct state *state_by_node(const struct device_node *node);
|
||||
int state_get_name(const struct state *state, char const **name);
|
||||
|
||||
int state_load(struct state *state);
|
||||
int state_save(struct state *state);
|
||||
void state_info(void);
|
||||
|
||||
|
|
Loading…
Reference in New Issue