diff options
| author | Marcel Ziswiler <marcel.ziswiler@toradex.com> | 2020-11-19 12:12:45 +0100 |
|---|---|---|
| committer | Denys Drozdov <denys.drozdov@toradex.com> | 2021-01-08 19:42:09 +0200 |
| commit | f2c35562c77c0d224417199a44a128d831b027e5 (patch) | |
| tree | 57c3070ecda7c9c4b69c86efd3cd6bb24deed422 | |
| parent | 5eeefc7e4c6db58743cc4028d62ec82448b1a5bf (diff) | |
fastboot: fix build warnings
Fix the following annoying build warnings:
drivers/fastboot/fb_fsl/fb_fsl_common.c: In function ‘fastboot_setup’:
drivers/fastboot/fb_fsl/fb_fsl_common.c:359:2: warning: ‘serialnr.high’ is used uninitialized in this function [-Wuninitialized]
359 | sprintf(serial, "%08x%08x", serialnr.high, serialnr.low);
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/fastboot/fb_fsl/fb_fsl_common.c:359:2: warning: ‘serialnr.low’ is used uninitialized in this function [-Wuninitialized]
drivers/fastboot/fb_fsl/fb_fsl_getvar.c:117:13: warning: ‘serial’ defined but not used [-Wunused-variable]
117 | static char serial[IMX_SERIAL_LEN];
| ^~~~~~
Signed-off-by: Marcel Ziswiler <marcel.ziswiler@toradex.com>
| -rw-r--r-- | drivers/fastboot/fb_fsl/fb_fsl_common.c | 5 | ||||
| -rw-r--r-- | drivers/fastboot/fb_fsl/fb_fsl_getvar.c | 2 |
2 files changed, 6 insertions, 1 deletions
diff --git a/drivers/fastboot/fb_fsl/fb_fsl_common.c b/drivers/fastboot/fb_fsl/fb_fsl_common.c index 8d798502bc..b992560ff9 100644 --- a/drivers/fastboot/fb_fsl/fb_fsl_common.c +++ b/drivers/fastboot/fb_fsl/fb_fsl_common.c @@ -352,7 +352,10 @@ static int _fastboot_setup_dev(int *switched) void fastboot_setup(void) { int sw, ret; - struct tag_serialnr serialnr; + struct tag_serialnr serialnr = { + .high = 0, + .low = 0, + }; char serial[17]; get_board_serial(&serialnr); diff --git a/drivers/fastboot/fb_fsl/fb_fsl_getvar.c b/drivers/fastboot/fb_fsl/fb_fsl_getvar.c index 457b5ae123..fec4971eee 100644 --- a/drivers/fastboot/fb_fsl/fb_fsl_getvar.c +++ b/drivers/fastboot/fb_fsl/fb_fsl_getvar.c @@ -118,7 +118,9 @@ static bool is_slotvar(char *cmd) return false; } +#ifdef CONFIG_SERIAL_TAG static char serial[IMX_SERIAL_LEN]; +#endif char *get_serial(void) { |