[PATCH v1 27/28] video: fbdev: cirrusfb: Fix W=1 warnings

[Sam Ravnborg]

Fix warnings:
- fix kernel-doc
- delete unused code

Signed-off-by: Sam Ravnborg <s...@ravnborg.org>
Cc: Thomas Zimemrmann <tzimmerm...@suse.de>
Cc: Sam Ravnborg <s...@ravnborg.org>
Cc: "Gustavo A. R. Silva" <gustavo...@kernel.org>
Cc: Daniel Vetter <daniel.vet...@ffwll.ch>
Cc: Saeed Mirzamohammadi <saeed.mirzamohamm...@oracle.com>
Cc: Jani Nikula <jani.nik...@intel.com>
Cc: Mike Rapoport <r...@kernel.org>
---
 drivers/video/fbdev/cirrusfb.c | 20 +++++++++-----------
 1 file changed, 9 insertions(+), 11 deletions(-)

diff --git a/drivers/video/fbdev/cirrusfb.c b/drivers/video/fbdev/cirrusfb.c
index e9027172c0f5..93802abbbc72 100644
--- a/drivers/video/fbdev/cirrusfb.c
+++ b/drivers/video/fbdev/cirrusfb.c
@@ -2463,8 +2463,6 @@ static void AttrOn(const struct cirrusfb_info *cinfo)
  */
 static void WHDR(const struct cirrusfb_info *cinfo, unsigned char val)
 {
-       unsigned char dummy;
-
        if (is_laguna(cinfo))
                return;
        if (cinfo->btype == BT_PICASSO) {
@@ -2473,18 +2471,18 @@ static void WHDR(const struct cirrusfb_info *cinfo, 
unsigned char val)
                WGen(cinfo, VGA_PEL_MSK, 0x00);
                udelay(200);
                /* next read dummy from pixel address (3c8) */
-               dummy = RGen(cinfo, VGA_PEL_IW);
+               RGen(cinfo, VGA_PEL_IW);
                udelay(200);
        }
        /* now do the usual stuff to access the HDR */
 
-       dummy = RGen(cinfo, VGA_PEL_MSK);
+       RGen(cinfo, VGA_PEL_MSK);
        udelay(200);
-       dummy = RGen(cinfo, VGA_PEL_MSK);
+       RGen(cinfo, VGA_PEL_MSK);
        udelay(200);
-       dummy = RGen(cinfo, VGA_PEL_MSK);
+       RGen(cinfo, VGA_PEL_MSK);
        udelay(200);
-       dummy = RGen(cinfo, VGA_PEL_MSK);
+       RGen(cinfo, VGA_PEL_MSK);
        udelay(200);
 
        WGen(cinfo, VGA_PEL_MSK, val);
@@ -2492,7 +2490,7 @@ static void WHDR(const struct cirrusfb_info *cinfo, 
unsigned char val)
 
        if (cinfo->btype == BT_PICASSO) {
                /* now first reset HDR access counter */
-               dummy = RGen(cinfo, VGA_PEL_IW);
+               RGen(cinfo, VGA_PEL_IW);
                udelay(200);
 
                /* and at the end, restore the mask value */
@@ -2800,9 +2798,9 @@ static void bestclock(long freq, int *nom, int *den, int 
*div)
 
 #ifdef CIRRUSFB_DEBUG
 
-/**
+/*
  * cirrusfb_dbg_print_regs
- * @base: If using newmmio, the newmmio base address, otherwise %NULL
+ * @regbase: If using newmmio, the newmmio base address, otherwise %NULL
  * @reg_class: type of registers to read: %CRT, or %SEQ
  *
  * DESCRIPTION:
@@ -2847,7 +2845,7 @@ static void cirrusfb_dbg_print_regs(struct fb_info *info,
        va_end(list);
 }
 
-/**
+/*
  * cirrusfb_dbg_reg_dump
  * @base: If using newmmio, the newmmio base address, otherwise %NULL
  *
-- 
2.27.0

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel