video: fbdev: radeonfb: remove set but not used variable 'hSyncPol'
authoryu kuai <yukuai3@huawei.com>
Sun, 19 Jan 2020 12:17:27 +0000 (20:17 +0800)
committerBartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>
Mon, 2 Mar 2020 15:32:07 +0000 (16:32 +0100)
Fixes gcc '-Wunused-but-set-variable' warning:

drivers/video/fbdev/aty/radeon_base.c: In function
‘radeonfb_set_par’:
drivers/video/fbdev/aty/radeon_base.c:1653:6: warning: variable
‘hSyncPol’ set but not used [-Wunused-but-set-variable]

It is never used, and so can be removed.

Signed-off-by: yu kuai <yukuai3@huawei.com>
Cc: Benjamin Herrenschmidt <benh@kernel.crashing.org>
Cc: zhengbin <zhengbin13@huawei.com>
Cc: yi.zhang <yi.zhang@huawei.com>
[b.zolnierkie: minor patch summary fixup]
Signed-off-by: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20200119121730.10701-2-yukuai3@huawei.com
drivers/video/fbdev/aty/radeon_base.c

index 3af00e3..d2e68ec 100644 (file)
@@ -1650,7 +1650,7 @@ static int radeonfb_set_par(struct fb_info *info)
        struct fb_var_screeninfo *mode = &info->var;
        struct radeon_regs *newmode;
        int hTotal, vTotal, hSyncStart, hSyncEnd,
-           hSyncPol, vSyncStart, vSyncEnd, vSyncPol, cSync;
+           vSyncStart, vSyncEnd, vSyncPol, cSync;
        u8 hsync_adj_tab[] = {0, 0x12, 9, 9, 6, 5};
        u8 hsync_fudge_fp[] = {2, 2, 0, 0, 5, 5};
        u32 sync, h_sync_pol, v_sync_pol, dotClock, pixClock;
@@ -1730,7 +1730,6 @@ static int radeonfb_set_par(struct fb_info *info)
        else if (vsync_wid > 0x1f)      /* max */
                vsync_wid = 0x1f;
 
-       hSyncPol = mode->sync & FB_SYNC_HOR_HIGH_ACT ? 0 : 1;
        vSyncPol = mode->sync & FB_SYNC_VERT_HIGH_ACT ? 0 : 1;
 
        cSync = mode->sync & FB_SYNC_COMP_HIGH_ACT ? (1 << 4) : 0;