video: fbdev: radeonfb: remove set but not used variable '‘cSync’'
authoryu kuai <yukuai3@huawei.com>
Sun, 19 Jan 2020 12:17:29 +0000 (20:17 +0800)
committerBartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>
Mon, 2 Mar 2020 15:32:09 +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:48: warning: variable
‘cSync’ 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: patch summary fixups]
Signed-off-by: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20200119121730.10701-4-yukuai3@huawei.com
drivers/video/fbdev/aty/radeon_base.c

index 0666f848bf70a5b769045a64d1532662760dc100..7d2ee889ffcd22cf61e244ac64bbff9ea3a48b93 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,
-           vSyncStart, vSyncEnd, cSync;
+           vSyncStart, vSyncEnd;
        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,8 +1730,6 @@ static int radeonfb_set_par(struct fb_info *info)
        else if (vsync_wid > 0x1f)      /* max */
                vsync_wid = 0x1f;
 
-       cSync = mode->sync & FB_SYNC_COMP_HIGH_ACT ? (1 << 4) : 0;
-
        format = radeon_get_dstbpp(depth);
        bytpp = mode->bits_per_pixel >> 3;