video: fbdev: tdfx: Fix set but not used warning in att_outb()
authorSam Ravnborg <sam@ravnborg.org>
Sat, 28 Nov 2020 22:40:58 +0000 (23:40 +0100)
committerSam Ravnborg <sam@ravnborg.org>
Mon, 30 Nov 2020 19:04:30 +0000 (20:04 +0100)
The tmp variable was assigned but the result was never used,
so delete the tmp variable.

v2:
  - Update subject (Lee)

Signed-off-by: Sam Ravnborg <sam@ravnborg.org>
Acked-by: Thomas Zimmermann <tzimmermann@suse.de>
Cc: "Gustavo A. R. Silva" <gustavoars@kernel.org>
Cc: Sam Ravnborg <sam@ravnborg.org>
Cc: Jani Nikula <jani.nikula@intel.com>
Cc: Daniel Vetter <daniel.vetter@ffwll.ch>
Cc: Arnd Bergmann <arnd@arndb.de>
Cc: Lee Jones <lee.jones@linaro.org>
Link: https://patchwork.freedesktop.org/patch/msgid/20201128224114.1033617-13-sam@ravnborg.org
drivers/video/fbdev/tdfxfb.c

index f056d80..67e37a6 100644 (file)
@@ -206,9 +206,7 @@ static inline u8 crt_inb(struct tdfx_par *par, u32 idx)
 
 static inline void att_outb(struct tdfx_par *par, u32 idx, u8 val)
 {
-       unsigned char tmp;
-
-       tmp = vga_inb(par, IS1_R);
+       vga_inb(par, IS1_R);
        vga_outb(par, ATT_IW, idx);
        vga_outb(par, ATT_IW, val);
 }