mvebu: 5.15: fix compilation warning for mangle bootargs patch
Fix compilation warning for mangle bootargs patch. Fix compilation warning: arch/arm/boot/compressed/atags_to_fdt.c: In function 'append_rootblock': arch/arm/boot/compressed/atags_to_fdt.c:84:13: error: assignment discards 'const' qualifier from pointer target type [-Werror=discarded-qualifiers] 84 | ptr = str - 1; | ^ arch/arm/boot/compressed/atags_to_fdt.c: In function 'atags_to_fdt': arch/arm/boot/compressed/atags_to_fdt.c:213:18: error: unused variable 'memsize' [-Werror=unused-variable] 213 | int ret, memsize; | ^~~~~~~ arch/arm/boot/compressed/atags_to_fdt.c:212:13: error: unused variable 'memcount' [-Werror=unused-variable] 212 | int memcount = 0; | ^~~~~~~~ arch/arm/boot/compressed/atags_to_fdt.c:211:16: error: unused variable 'mem_reg_property' [-Werror=unused-variable] 211 | __be32 mem_reg_property[2 * 2 * NR_BANKS]; | ^~~~~~~~~~~~~~~~ arch/arm/boot/compressed/atags_to_fdt.c: At top level: arch/arm/boot/compressed/atags_to_fdt.c:180:13: error: 'hex_str' defined but not used [-Werror=unused-function] 180 | static void hex_str(char *out, uint32_t value) | ^~~~~~~ arch/arm/boot/compressed/atags_to_fdt.c:63:17: error: 'get_cell_size' defined but not used [-Werror=unused-function] 63 | static uint32_t get_cell_size(const void *fdt) | ^~~~~~~~~~~~~ arch/arm/boot/compressed/atags_to_fdt.c:43:12: error: 'setprop_cell' defined but not used [-Werror=unused-function] 43 | static int setprop_cell(void *fdt, const char *node_path, | ^~~~~~~~~~~~ arch/arm/boot/compressed/atags_to_fdt.c:25:12: error: 'setprop' defined but not used [-Werror=unused-function] 25 | static int setprop(void *fdt, const char *node_path, const char *property, | ^~~~~~~ cc1: all warnings being treated as errors Signed-off-by: Christian Marangi <ansuelsmth@gmail.com>
This commit is contained in:
parent
bec927c70f
commit
2f9bc747cb
@ -57,16 +57,58 @@ Signed-off-by: Michael Gray <michael.gray@lantisproject.com>
|
|||||||
#else
|
#else
|
||||||
#define do_extend_cmdline 0
|
#define do_extend_cmdline 0
|
||||||
#endif
|
#endif
|
||||||
@@ -69,6 +71,72 @@ static uint32_t get_cell_size(const void
|
@@ -20,6 +22,7 @@ static int node_offset(void *fdt, const
|
||||||
|
return offset;
|
||||||
|
}
|
||||||
|
|
||||||
|
+#ifndef CONFIG_ARM_ATAG_DTB_COMPAT_CMDLINE_MANGLE
|
||||||
|
static int setprop(void *fdt, const char *node_path, const char *property,
|
||||||
|
void *val_array, int size)
|
||||||
|
{
|
||||||
|
@@ -28,6 +31,7 @@ static int setprop(void *fdt, const char
|
||||||
|
return offset;
|
||||||
|
return fdt_setprop(fdt, offset, property, val_array, size);
|
||||||
|
}
|
||||||
|
+#endif
|
||||||
|
|
||||||
|
static int setprop_string(void *fdt, const char *node_path,
|
||||||
|
const char *property, const char *string)
|
||||||
|
@@ -38,6 +42,7 @@ static int setprop_string(void *fdt, con
|
||||||
|
return fdt_setprop_string(fdt, offset, property, string);
|
||||||
|
}
|
||||||
|
|
||||||
|
+#ifndef CONFIG_ARM_ATAG_DTB_COMPAT_CMDLINE_MANGLE
|
||||||
|
static int setprop_cell(void *fdt, const char *node_path,
|
||||||
|
const char *property, uint32_t val)
|
||||||
|
{
|
||||||
|
@@ -46,6 +51,7 @@ static int setprop_cell(void *fdt, const
|
||||||
|
return offset;
|
||||||
|
return fdt_setprop_cell(fdt, offset, property, val);
|
||||||
|
}
|
||||||
|
+#endif
|
||||||
|
|
||||||
|
static const void *getprop(const void *fdt, const char *node_path,
|
||||||
|
const char *property, int *len)
|
||||||
|
@@ -58,6 +64,7 @@ static const void *getprop(const void *f
|
||||||
|
return fdt_getprop(fdt, offset, property, len);
|
||||||
|
}
|
||||||
|
|
||||||
|
+#ifndef CONFIG_ARM_ATAG_DTB_COMPAT_CMDLINE_MANGLE
|
||||||
|
static uint32_t get_cell_size(const void *fdt)
|
||||||
|
{
|
||||||
|
int len;
|
||||||
|
@@ -69,6 +76,74 @@ static uint32_t get_cell_size(const void
|
||||||
return cell_size;
|
return cell_size;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
+#endif
|
||||||
|
+
|
||||||
+#if defined(CONFIG_ARM_ATAG_DTB_COMPAT_CMDLINE_MANGLE)
|
+#if defined(CONFIG_ARM_ATAG_DTB_COMPAT_CMDLINE_MANGLE)
|
||||||
+
|
+
|
||||||
+static char *append_rootblock(char *dest, const char *str, int len, void *fdt)
|
+static char *append_rootblock(char *dest, const char *str, int len, void *fdt)
|
||||||
+{
|
+{
|
||||||
+ char *ptr, *end;
|
+ const char *ptr, *end;
|
||||||
+ char *root="root=";
|
+ const char *root="root=";
|
||||||
+ int i, l;
|
+ int i, l;
|
||||||
+ const char *rootblock;
|
+ const char *rootblock;
|
||||||
+
|
+
|
||||||
@ -130,7 +172,7 @@ Signed-off-by: Michael Gray <michael.gray@lantisproject.com>
|
|||||||
static void merge_fdt_bootargs(void *fdt, const char *fdt_cmdline)
|
static void merge_fdt_bootargs(void *fdt, const char *fdt_cmdline)
|
||||||
{
|
{
|
||||||
char cmdline[COMMAND_LINE_SIZE];
|
char cmdline[COMMAND_LINE_SIZE];
|
||||||
@@ -88,12 +156,21 @@ static void merge_fdt_bootargs(void *fdt
|
@@ -88,18 +163,28 @@ static void merge_fdt_bootargs(void *fdt
|
||||||
|
|
||||||
/* and append the ATAG_CMDLINE */
|
/* and append the ATAG_CMDLINE */
|
||||||
if (fdt_cmdline) {
|
if (fdt_cmdline) {
|
||||||
@ -152,7 +194,36 @@ Signed-off-by: Michael Gray <michael.gray@lantisproject.com>
|
|||||||
}
|
}
|
||||||
*ptr = '\0';
|
*ptr = '\0';
|
||||||
|
|
||||||
@@ -168,7 +245,9 @@ int atags_to_fdt(void *atag_list, void *
|
setprop_string(fdt, "/chosen", "bootargs", cmdline);
|
||||||
|
}
|
||||||
|
|
||||||
|
+#ifndef CONFIG_ARM_ATAG_DTB_COMPAT_CMDLINE_MANGLE
|
||||||
|
static void hex_str(char *out, uint32_t value)
|
||||||
|
{
|
||||||
|
uint32_t digit;
|
||||||
|
@@ -117,6 +202,7 @@ static void hex_str(char *out, uint32_t
|
||||||
|
}
|
||||||
|
*out = '\0';
|
||||||
|
}
|
||||||
|
+#endif
|
||||||
|
|
||||||
|
/*
|
||||||
|
* Convert and fold provided ATAGs into the provided FDT.
|
||||||
|
@@ -131,9 +217,11 @@ int atags_to_fdt(void *atag_list, void *
|
||||||
|
struct tag *atag = atag_list;
|
||||||
|
/* In the case of 64 bits memory size, need to reserve 2 cells for
|
||||||
|
* address and size for each bank */
|
||||||
|
+#ifndef CONFIG_ARM_ATAG_DTB_COMPAT_CMDLINE_MANGLE
|
||||||
|
__be32 mem_reg_property[2 * 2 * NR_BANKS];
|
||||||
|
- int memcount = 0;
|
||||||
|
- int ret, memsize;
|
||||||
|
+ int memsize, memcount = 0;
|
||||||
|
+#endif
|
||||||
|
+ int ret;
|
||||||
|
|
||||||
|
/* make sure we've got an aligned pointer */
|
||||||
|
if ((u32)atag_list & 0x3)
|
||||||
|
@@ -168,7 +256,9 @@ int atags_to_fdt(void *atag_list, void *
|
||||||
else
|
else
|
||||||
setprop_string(fdt, "/chosen", "bootargs",
|
setprop_string(fdt, "/chosen", "bootargs",
|
||||||
atag->u.cmdline.cmdline);
|
atag->u.cmdline.cmdline);
|
||||||
@ -163,7 +234,7 @@ Signed-off-by: Michael Gray <michael.gray@lantisproject.com>
|
|||||||
if (memcount >= sizeof(mem_reg_property)/4)
|
if (memcount >= sizeof(mem_reg_property)/4)
|
||||||
continue;
|
continue;
|
||||||
if (!atag->u.mem.size)
|
if (!atag->u.mem.size)
|
||||||
@@ -212,6 +291,10 @@ int atags_to_fdt(void *atag_list, void *
|
@@ -212,6 +302,10 @@ int atags_to_fdt(void *atag_list, void *
|
||||||
setprop(fdt, "/memory", "reg", mem_reg_property,
|
setprop(fdt, "/memory", "reg", mem_reg_property,
|
||||||
4 * memcount * memsize);
|
4 * memcount * memsize);
|
||||||
}
|
}
|
||||||
|
Loading…
x
Reference in New Issue
Block a user