diff options
| author | Sam Ravnborg <sam@ravnborg.org> | 2020-12-06 20:02:38 +0100 |
|---|---|---|
| committer | Sam Ravnborg <sam@ravnborg.org> | 2020-12-08 18:33:48 +0100 |
| commit | 7143c92be85b82f9018f293ab550d61cd4215ee5 (patch) | |
| tree | 5bacc50895a0252c6b7a02738485aee674f004b3 /drivers/video | |
| parent | video: fbdev: omapfb: Fix set but not used warnings in hdmi*_core (diff) | |
| download | linux-7143c92be85b82f9018f293ab550d61cd4215ee5.tar.xz linux-7143c92be85b82f9018f293ab550d61cd4215ee5.zip | |
video: fbdev: uvesafb: Fix set but not used warning
Fix W=1 warning by deleting unused local variable.
v2:
- Updated subject (Lee)
v3:
- Return early in case of an error (Thomas)
Signed-off-by: Sam Ravnborg <sam@ravnborg.org>
Acked-by: Thomas Zimmermann <tzimmermann@suse.de>
Cc: Michal Januszewski <spock@gentoo.org>
Cc: linux-fbdev@vger.kernel.org
Cc: Lee Jones <lee.jones@linaro.org>
Link: https://patchwork.freedesktop.org/patch/msgid/20201206190247.1861316-5-sam@ravnborg.org
Diffstat (limited to 'drivers/video')
| -rw-r--r-- | drivers/video/fbdev/uvesafb.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/drivers/video/fbdev/uvesafb.c b/drivers/video/fbdev/uvesafb.c index 6c9cfab39313..4df6772802d7 100644 --- a/drivers/video/fbdev/uvesafb.c +++ b/drivers/video/fbdev/uvesafb.c @@ -560,6 +560,8 @@ static int uvesafb_vbe_getpmi(struct uvesafb_ktask *task, task->t.regs.eax = 0x4f0a; task->t.regs.ebx = 0x0; err = uvesafb_exec(task); + if (err) + return err; if ((task->t.regs.eax & 0xffff) != 0x4f || task->t.regs.es < 0xc000) { par->pmi_setpal = par->ypan = 0; |