Remove initialization of the verbose struct member, it will be
overwritten by the call to bootm_data_init_defaults(). Also remove
members which are initialized in the call.
Verbose should be enabled if either the argument or variable is set, so
|| both.

Signed-off-by: Rouven Czerwinski <[email protected]>
---
 common/blspec.c | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/common/blspec.c b/common/blspec.c
index 14c98d4f7b..7858df685f 100644
--- a/common/blspec.c
+++ b/common/blspec.c
@@ -130,9 +130,6 @@ static int blspec_boot(struct bootentry *be, int verbose, 
int dryrun)
        const char *overlays;
        const char *appendroot;
        struct bootm_data data = {
-               .initrd_address = UIMAGE_INVALID_ADDRESS,
-               .os_address = UIMAGE_SOME_ADDRESS,
-               .verbose = verbose,
                .dryrun = dryrun,
        };
 
@@ -143,6 +140,8 @@ static int blspec_boot(struct bootentry *be, int verbose, 
int dryrun)
 
        bootm_data_init_defaults(&data);
 
+       data.verbose = verbose || data.verbose;
+
        devicetree = blspec_entry_var_get(entry, "devicetree");
        initrd = blspec_entry_var_get(entry, "initrd");
        options = blspec_entry_var_get(entry, "options");
-- 
2.28.0


_______________________________________________
barebox mailing list
[email protected]
http://lists.infradead.org/mailman/listinfo/barebox

Reply via email to