summaryrefslogtreecommitdiff
path: root/drivers/video
diff options
context:
space:
mode:
authorSam Ravnborg <sam@ravnborg.org>2020-12-06 22:02:38 +0300
committerSam Ravnborg <sam@ravnborg.org>2020-12-08 20:33:48 +0300
commit7143c92be85b82f9018f293ab550d61cd4215ee5 (patch)
tree5bacc50895a0252c6b7a02738485aee674f004b3 /drivers/video
parente72ce1c4d1cf20d7d681ae670f707c6e29976a2b (diff)
downloadlinux-7143c92be85b82f9018f293ab550d61cd4215ee5.tar.xz
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.c2
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;