]> git.dujemihanovic.xyz Git - u-boot.git/commitdiff
expo: cedit: Support writing settings to a file
authorSimon Glass <sjg@chromium.org>
Mon, 14 Aug 2023 22:40:33 +0000 (16:40 -0600)
committerTom Rini <trini@konsulko.com>
Fri, 25 Aug 2023 17:54:33 +0000 (13:54 -0400)
Support writing settings from an expo into a file in FDT format. It
consists of a single node with a two properties for each sceneitem,
one with tag ID chosen by the user and another for its text value.

Signed-off-by: Simon Glass <sjg@chromium.org>
boot/cedit.c
cmd/cedit.c
doc/usage/cmd/cedit.rst
include/cedit.h
test/boot/cedit.c

index 6c10b21145418b628a2b8dd6fcb2582ce558ec98..4dd79a2263d6366ec1ff5d22ba1e4cd069bc2b41 100644 (file)
@@ -9,6 +9,7 @@
 #define LOG_CATEGORY LOGC_EXPO
 
 #include <common.h>
+#include <abuf.h>
 #include <cedit.h>
 #include <cli.h>
 #include <dm.h>
 #include <linux/delay.h>
 #include "scene_internal.h"
 
+/**
+ * struct cedit_iter_priv - private data for cedit operations
+ *
+ * @buf: Buffer to use when writing settings to the devicetree
+ */
+struct cedit_iter_priv {
+       struct abuf *buf;
+};
+
 int cedit_arange(struct expo *exp, struct video_priv *vpriv, uint scene_id)
 {
        struct scene_obj_txt *txt;
@@ -182,3 +192,129 @@ int cedit_run(struct expo *exp)
 
        return 0;
 }
+
+static int check_space(int ret, struct abuf *buf)
+{
+       if (ret == -FDT_ERR_NOSPACE) {
+               if (!abuf_realloc_inc(buf, CEDIT_SIZE_INC))
+                       return log_msg_ret("spc", -ENOMEM);
+               ret = fdt_resize(abuf_data(buf), abuf_data(buf),
+                                abuf_size(buf));
+               if (ret)
+                       return log_msg_ret("res", -EFAULT);
+       }
+
+       return 0;
+}
+
+static int h_write_settings(struct scene_obj *obj, void *vpriv)
+{
+       struct cedit_iter_priv *priv = vpriv;
+       struct abuf *buf = priv->buf;
+
+       switch (obj->type) {
+       case SCENEOBJT_NONE:
+       case SCENEOBJT_IMAGE:
+       case SCENEOBJT_TEXT:
+               break;
+       case SCENEOBJT_MENU: {
+               const struct scene_obj_menu *menu;
+               const struct scene_obj_txt *txt;
+               struct scene *scn = obj->scene;
+               const struct scene_menitem *mi;
+               const char *str;
+               char name[80];
+               int ret, i;
+
+               menu = (struct scene_obj_menu *)obj;
+               ret = -EAGAIN;
+               for (i = 0; ret && i < 2; i++) {
+                       ret = fdt_property_u32(abuf_data(buf), obj->name,
+                                              menu->cur_item_id);
+                       if (!i) {
+                               ret = check_space(ret, buf);
+                               if (ret)
+                                       return log_msg_ret("res", -ENOMEM);
+                       }
+               }
+               /* this should not happen */
+               if (ret)
+                       return log_msg_ret("wrt", -EFAULT);
+
+               mi = scene_menuitem_find(menu, menu->cur_item_id);
+               if (!mi)
+                       return log_msg_ret("mi", -ENOENT);
+
+               txt = scene_obj_find(scn, mi->label_id, SCENEOBJT_TEXT);
+               if (!txt)
+                       return log_msg_ret("txt", -ENOENT);
+
+               str = expo_get_str(scn->expo, txt->str_id);
+               if (!str)
+                       return log_msg_ret("str", -ENOENT);
+
+               snprintf(name, sizeof(name), "%s-str", obj->name);
+               ret = -EAGAIN;
+               for (i = 0; ret && i < 2; i++) {
+                       ret = fdt_property_string(abuf_data(buf), name, str);
+                       if (!i) {
+                               ret = check_space(ret, buf);
+                               if (ret)
+                                       return log_msg_ret("rs2", -ENOMEM);
+                       }
+               }
+
+               /* this should not happen */
+               if (ret)
+                       return log_msg_ret("wr2", -EFAULT);
+
+               break;
+       }
+       }
+
+       return 0;
+}
+
+int cedit_write_settings(struct expo *exp, struct abuf *buf)
+{
+       struct cedit_iter_priv priv;
+       void *fdt;
+       int ret;
+
+       abuf_init(buf);
+       if (!abuf_realloc(buf, CEDIT_SIZE_INC))
+               return log_msg_ret("buf", -ENOMEM);
+
+       fdt = abuf_data(buf);
+       ret = fdt_create(fdt, abuf_size(buf));
+       if (!ret)
+               ret = fdt_finish_reservemap(fdt);
+       if (!ret)
+               ret = fdt_begin_node(fdt, "");
+       if (!ret)
+               ret = fdt_begin_node(fdt, CEDIT_NODE_NAME);
+       if (ret) {
+               log_debug("Failed to start FDT (err=%d)\n", ret);
+               return log_msg_ret("sta", -EINVAL);
+       }
+
+       /* write out the items */
+       priv.buf = buf;
+       ret = expo_iter_scene_objs(exp, h_write_settings, &priv);
+       if (ret) {
+               log_debug("Failed to write settings (err=%d)\n", ret);
+               return log_msg_ret("set", ret);
+       }
+
+       ret = fdt_end_node(fdt);
+       if (!ret)
+               ret = fdt_end_node(fdt);
+       if (!ret)
+               ret = fdt_finish(fdt);
+       if (ret) {
+               log_debug("Failed to finish FDT (err=%d)\n", ret);
+               return log_msg_ret("fin", -EINVAL);
+       }
+
+       return 0;
+}
index e98121b067b2a889d47e4936fc5c62f03f64607e..18cc8ba191bf99815b6b4afd4d842a7d1c03358b 100644 (file)
@@ -7,15 +7,28 @@
  */
 
 #include <common.h>
+#include <abuf.h>
 #include <cedit.h>
 #include <command.h>
 #include <expo.h>
 #include <fs.h>
+#include <malloc.h>
+#include <mapmem.h>
 #include <dm/ofnode.h>
 #include <linux/sizes.h>
 
 struct expo *cur_exp;
 
+static int check_cur_expo(void)
+{
+       if (!cur_exp) {
+               printf("No expo loaded\n");
+               return -ENOENT;
+       }
+
+       return 0;
+}
+
 static int do_cedit_load(struct cmd_tbl *cmdtp, int flag, int argc,
                         char *const argv[])
 {
@@ -54,16 +67,46 @@ static int do_cedit_load(struct cmd_tbl *cmdtp, int flag, int argc,
        return 0;
 }
 
+static int do_cedit_write_fdt(struct cmd_tbl *cmdtp, int flag, int argc,
+                             char *const argv[])
+{
+       const char *fname;
+       struct abuf buf;
+       loff_t bytes;
+       int ret;
+
+       if (argc < 4)
+               return CMD_RET_USAGE;
+       fname = argv[3];
+
+       if (check_cur_expo())
+               return CMD_RET_FAILURE;
+
+       ret = cedit_write_settings(cur_exp, &buf);
+       if (ret) {
+               printf("Failed to write settings: %dE\n", ret);
+               return CMD_RET_FAILURE;
+       }
+
+       if (fs_set_blk_dev(argv[1], argv[2], FS_TYPE_ANY))
+               return CMD_RET_FAILURE;
+
+       ret = fs_write(fname, map_to_sysmem(abuf_data(&buf)), 0,
+                      abuf_size(&buf), &bytes);
+       if (ret)
+               return CMD_RET_FAILURE;
+
+       return 0;
+}
+
 static int do_cedit_run(struct cmd_tbl *cmdtp, int flag, int argc,
                        char *const argv[])
 {
        ofnode node;
        int ret;
 
-       if (!cur_exp) {
-               printf("No expo loaded\n");
+       if (check_cur_expo())
                return CMD_RET_FAILURE;
-       }
 
        node = ofnode_path("/bootstd/cedit-theme");
        if (ofnode_valid(node)) {
@@ -85,10 +128,12 @@ static int do_cedit_run(struct cmd_tbl *cmdtp, int flag, int argc,
 #ifdef CONFIG_SYS_LONGHELP
 static char cedit_help_text[] =
        "load <interface> <dev[:part]> <filename>   - load config editor\n"
+       "cedit write_fdt <i/f> <dev[:part]> <filename>    - write settings\n"
        "cedit run                                        - run config editor";
 #endif /* CONFIG_SYS_LONGHELP */
 
 U_BOOT_CMD_WITH_SUBCMDS(cedit, "Configuration editor", cedit_help_text,
        U_BOOT_SUBCMD_MKENT(load, 5, 1, do_cedit_load),
+       U_BOOT_SUBCMD_MKENT(write_fdt, 5, 1, do_cedit_write_fdt),
        U_BOOT_SUBCMD_MKENT(run, 1, 1, do_cedit_run),
 );
index d34a220797ee1529d02553d60eaa3c98361844ad..0581594831fafa4cba3f522a18d3b9aee6003d11 100644 (file)
@@ -10,6 +10,7 @@ Synopis
 
     cedit load <interface> <dev[:part]> <filename>
     cedit run
+    cedit write_fdt <dev[:part]> <filename>
 
 Description
 -----------
@@ -38,6 +39,12 @@ Runs the default configuration-editor event loop. This is very simple, just
 accepting character input and moving through the objects under user control.
 The implementation is at `cedit_run()`.
 
+cedit write_fdt
+~~~~~~~~~~~~~~~
+
+Writes the current user settings to a devicetree file. For each menu item the
+selected ID and its text string are written.
+
 
 Example
 -------
@@ -46,3 +53,15 @@ Example
 
     => cedit load hostfs - fred.dtb
     => cedit run
+    => cedit write_fdt hostfs - settings.dtb
+
+That results in::
+
+    / {
+        cedit-values {
+            cpu-speed = <0x00000006>;
+            cpu-speed-str = "2 GHz";
+            power-loss = <0x0000000a>;
+            power-loss-str = "Always Off";
+        };
+    }
index 851d8e83564a9933af7442b109d7a00e5226330c..6086e302006279559cd2d54942781eb48878b086 100644 (file)
@@ -7,10 +7,21 @@
 #ifndef __CEDIT_H
 #define __CEDIT_H
 
+struct abuf;
 struct expo;
 struct scene;
 struct video_priv;
 
+enum {
+       /* size increment for writing FDT */
+       CEDIT_SIZE_INC  = 1024,
+};
+
+/* Name of the cedit node in the devicetree */
+#define CEDIT_NODE_NAME                "cedit-values"
+
+extern struct expo *cur_exp;
+
 /**
  * cedit_arange() - Arrange objects in a configuration-editor scene
  *
@@ -45,4 +56,15 @@ int cedit_run(struct expo *exp);
 int cedit_prepare(struct expo *exp, struct video_priv **vid_privp,
                  struct scene **scnp);
 
+/**
+ * cedit_write_settings() - Write settings in FDT format
+ *
+ * Sets up an FDT with the settings
+ *
+ * @exp: Expo to write settings from
+ * @buf: Returns abuf containing the settings FDT (inited by this function)
+ * Return: 0 if OK, -ve on error
+ */
+int cedit_write_settings(struct expo *exp, struct abuf *buf);
+
 #endif /* __CEDIT_H */
index f3411f734fa26df0c94038d53e89ed1ccfb55efe..1dd78c64158dd4375905a996b0c70bd05fbb39d1 100644 (file)
@@ -7,6 +7,8 @@
 #include <common.h>
 #include <cedit.h>
 #include <expo.h>
+#include <mapmem.h>
+#include <dm/ofnode.h>
 #include <test/ut.h>
 #include "bootstd_common.h"
 #include <test/cedit-test.h>
@@ -51,3 +53,46 @@ static int cedit_base(struct unit_test_state *uts)
        return 0;
 }
 BOOTSTD_TEST(cedit_base, 0);
+
+/* Check the cedit write_fdt commands */
+static int cedit_fdt(struct unit_test_state *uts)
+{
+       struct video_priv *vid_priv;
+       extern struct expo *cur_exp;
+       ulong addr = 0x1000;
+       struct ofprop prop;
+       struct scene *scn;
+       oftree tree;
+       ofnode node;
+       void *fdt;
+       int i;
+
+       console_record_reset_enable();
+       ut_assertok(run_command("cedit load hostfs - cedit.dtb", 0));
+
+       ut_asserteq(ID_SCENE1, cedit_prepare(cur_exp, &vid_priv, &scn));
+
+       ut_assertok(run_command("cedit write_fdt hostfs - settings.dtb", 0));
+       ut_assertok(run_commandf("load hostfs - %lx settings.dtb", addr));
+       ut_assert_nextlinen("1024 bytes read");
+
+       fdt = map_sysmem(addr, 1024);
+       tree = oftree_from_fdt(fdt);
+       node = ofnode_find_subnode(oftree_root(tree), CEDIT_NODE_NAME);
+
+       ut_asserteq(ID_CPU_SPEED_1,
+                   ofnode_read_u32_default(node, "cpu-speed", 0));
+       ut_asserteq_str("2 GHz", ofnode_read_string(node, "cpu-speed-str"));
+       ut_assert(ofnode_valid(node));
+
+       /* There should only be 4 properties */
+       for (i = 0, ofnode_first_property(node, &prop); ofprop_valid(&prop);
+            i++, ofnode_next_property(&prop))
+               ;
+       ut_asserteq(4, i);
+
+       ut_assert_console_end();
+
+       return 0;
+}
+BOOTSTD_TEST(cedit_fdt, 0);