From: Minghao Chi Date: Tue, 4 Jan 2022 10:43:36 +0000 (+0000) Subject: video: fbdev: mach64_ct: remove redundant res variable X-Git-Tag: v6.1-rc5~1764^2~53 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=97db79f705055624bb30c147b2d695ca7b6d710e;p=platform%2Fkernel%2Flinux-starfive.git video: fbdev: mach64_ct: remove redundant res variable Return value from aty_ld_8() directly instead of taking this in another redundant variable. Reported-by: Zeal Robot Signed-off-by: Minghao Chi Signed-off-by: CGEL ZTE Signed-off-by: Helge Deller --- diff --git a/drivers/video/fbdev/aty/mach64_ct.c b/drivers/video/fbdev/aty/mach64_ct.c index 011b07e..e967536 100644 --- a/drivers/video/fbdev/aty/mach64_ct.c +++ b/drivers/video/fbdev/aty/mach64_ct.c @@ -22,13 +22,11 @@ static u32 aty_pll_to_var_ct(const struct fb_info *info, const union aty_pll *pl u8 aty_ld_pll_ct(int offset, const struct atyfb_par *par) { - u8 res; /* write addr byte */ aty_st_8(CLOCK_CNTL_ADDR, (offset << 2) & PLL_ADDR, par); /* read the register value */ - res = aty_ld_8(CLOCK_CNTL_DATA, par); - return res; + return aty_ld_8(CLOCK_CNTL_DATA, par); } static void aty_st_pll_ct(int offset, u8 val, const struct atyfb_par *par)