diff options
author | Heinrich Schuchardt | 2022-08-23 10:31:56 +0200 |
---|---|---|
committer | Simon Glass | 2022-08-26 16:32:59 -0600 |
commit | 49740e02555f72394fb319a8221bad833d8207db (patch) | |
tree | ff51888002e1acb78ef111fa00bc50b6a924a63a | |
parent | d81eeacd48f760516abf12e69aba4ce5dd44545f (diff) |
boot: simplify bootmeth_vbe_simple_ft_fixup()
Don't assign a value to a variable if it is not used afterwards.
Move variables to the code fragment where they are used.
Addresses-Coverity: CID 356243 ("Code maintainability issues (UNUSED_VALUE)")
Signed-off-by: Heinrich Schuchardt <heinrich.schuchardt@canonical.com>
Reviewed-by: Simon Glass <sjg@chromium.org>
-rw-r--r-- | boot/vbe_simple.c | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/boot/vbe_simple.c b/boot/vbe_simple.c index a395bc20a60..0fc57388f35 100644 --- a/boot/vbe_simple.c +++ b/boot/vbe_simple.c @@ -225,17 +225,16 @@ static int bootmeth_vbe_simple_ft_fixup(void *ctx, struct event *event) { oftree tree = event->data.ft_fixup.tree; struct udevice *dev; - ofnode node; - int ret; /* * Ideally we would have driver model support for fixups, but that does * not exist yet. It is a step too far to try to do this before VBE is * in place. */ - for (ret = vbe_find_first_device(&dev); dev; - ret = vbe_find_next_device(&dev)) { + for (vbe_find_first_device(&dev); dev; vbe_find_next_device(&dev)) { struct simple_state state; + ofnode node; + int ret; if (strcmp("vbe_simple", dev->driver->name)) continue; |