Skip to content

Commit

Permalink
video: fbdev: mach64_ct: remove redundant res variable
Browse files Browse the repository at this point in the history
Return value from aty_ld_8() directly instead
of taking this in another redundant variable.

Reported-by: Zeal Robot <zealci@zte.com.cn>
Signed-off-by: Minghao Chi <chi.minghao@zte.com.cn>
Signed-off-by: CGEL ZTE <cgel.zte@gmail.com>
Signed-off-by: Helge Deller <deller@gmx.de>
  • Loading branch information
Minghao Chi authored and Helge Deller committed Jan 29, 2022
1 parent 19d10a8 commit 97db79f
Showing 1 changed file with 1 addition and 3 deletions.
4 changes: 1 addition & 3 deletions drivers/video/fbdev/aty/mach64_ct.c
Original file line number Diff line number Diff line change
Expand Up @@ -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)
Expand Down

0 comments on commit 97db79f

Please sign in to comment.