]> git.dujemihanovic.xyz Git - u-boot.git/commitdiff
cmd: Fixup DT to pass PStore Ramoops parameters
authorFrédéric Danis <frederic.danis@collabora.com>
Fri, 20 Mar 2020 09:59:24 +0000 (10:59 +0100)
committerTom Rini <trini@konsulko.com>
Wed, 14 Oct 2020 15:16:34 +0000 (11:16 -0400)
To simplify configuration and keep synchronized the PStore/Ramoops between
U-Boot and the Linux kernel, this commit dynamically adds the Ramoops
parameters defined in the U-Boot session to the Device Tree.

Signed-off-by: Frédéric Danis <frederic.danis@collabora.com>
Cc: Tom Rini <trini@konsulko.com>
Cc: Heinrich Schuchardt <xypron.glpk@gmx.de>
Cc: Wolfgang Denk <wd@denx.de>
Cc: Heiko Schocher <hs@denx.de>
cmd/pstore.c
common/image-fdt.c
doc/pstore.rst
include/fdt_support.h

index 530eb03b182ca377e35512a15f8531a000aa7230..9a8b38c7f26ef3aff169253b2c9df6474c25409c 100644 (file)
@@ -480,6 +480,44 @@ static int do_pstore(struct cmd_tbl *cmdtp, int flag, int argc, char * const arg
        return c->cmd(cmdtp, flag, argc, argv);
 }
 
+void fdt_fixup_pstore(void *blob)
+{
+       char node[32];
+       int  nodeoffset;        /* node offset from libfdt */
+
+       nodeoffset = fdt_path_offset(blob, "/");
+       if (nodeoffset < 0) {
+               /* Not found or something else bad happened. */
+               log_err("fdt_path_offset() returned %s\n", fdt_strerror(nodeoffset));
+               return;
+       }
+
+       nodeoffset = fdt_add_subnode(blob, nodeoffset, "reserved-memory");
+       if (nodeoffset < 0) {
+               log_err("Add 'reserved-memory' node failed: %s\n",
+                               fdt_strerror(nodeoffset));
+               return;
+       }
+       fdt_setprop_u32(blob, nodeoffset, "#address-cells", 2);
+       fdt_setprop_u32(blob, nodeoffset, "#size-cells", 2);
+       fdt_setprop_empty(blob, nodeoffset, "ranges");
+
+       sprintf(node, "ramoops@%llx", (unsigned long long)pstore_addr);
+       nodeoffset = fdt_add_subnode(blob, nodeoffset, node);
+       if (nodeoffset < 0) {
+               log_err("Add '%s' node failed: %s\n", node, fdt_strerror(nodeoffset));
+               return;
+       }
+       fdt_setprop_string(blob, nodeoffset, "compatible", "ramoops");
+       fdt_setprop_u64(blob, nodeoffset, "reg", pstore_addr);
+       fdt_appendprop_u64(blob, nodeoffset, "reg", pstore_length);
+       fdt_setprop_u32(blob, nodeoffset, "record-size", pstore_record_size);
+       fdt_setprop_u32(blob, nodeoffset, "console-size", pstore_console_size);
+       fdt_setprop_u32(blob, nodeoffset, "ftrace-size", pstore_ftrace_size);
+       fdt_setprop_u32(blob, nodeoffset, "pmsg-size", pstore_pmsg_size);
+       fdt_setprop_u32(blob, nodeoffset, "ecc-size", pstore_ecc_size);
+}
+
 U_BOOT_CMD(pstore, 10, 0, do_pstore,
           "Manage Linux Persistent Storage",
           "set <addr> <len> [record-size] [console-size] [ftrace-size] [pmsg_size] [ecc-size]\n"
index 3d6935ad40b6661aaa1d0989ad5a296cf7d54947..327a8c4c39591ba5867bd79d610e91fed3c81332 100644 (file)
@@ -567,6 +567,10 @@ int image_setup_libfdt(bootm_headers_t *images, void *blob,
 
        /* Update ethernet nodes */
        fdt_fixup_ethernet(blob);
+#if CONFIG_IS_ENABLED(CMD_PSTORE)
+       /* Append PStore configuration */
+       fdt_fixup_pstore(blob);
+#endif
        if (IMAGE_OF_BOARD_SETUP) {
                fdt_ret = ft_board_setup(blob, gd->bd);
                if (fdt_ret) {
index 4fceb2dd9907c237bfdb806399f2f4a08b871c0a..8427d8fd97a9e8af9ff21e5f518205b6db0d6a49 100644 (file)
@@ -25,6 +25,8 @@ i.e.::
 
 The same values should be set in U-Boot to be able to retrieve the records.
 This values can be set at build time in U-Boot configuration file, or at runtime.
+U-Boot automatically patches the Device Tree to pass the Ramoops parameters to
+the kernel.
 
 The PStore configuration parameters are:
 
index 9684cffe80b2788ffd113a7c2ef71430fc9657f5..dbbac0fb6a3d161fbb4c68fad73a62d4ca02ff40 100644 (file)
@@ -359,4 +359,7 @@ int fdt_update_ethernet_dt(void *blob);
 #ifdef CONFIG_FSL_MC_ENET
 void fdt_fixup_board_enet(void *blob);
 #endif
+#ifdef CONFIG_CMD_PSTORE
+void fdt_fixup_pstore(void *blob);
+#endif
 #endif /* ifndef __FDT_SUPPORT_H */