Linux kernel mirror (for testing) git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
kernel os linux

[media] ov2659: get rid of unused values

Why to store the chosed values for prediv, postdiv and mult if
those won't be used?

drivers/media/i2c/ov2659.c: In function 'ov2659_pll_calc_params':
drivers/media/i2c/ov2659.c:912:35: warning: variable 's_mult' set but not used [-Wunused-but-set-variable]
u32 s_prediv = 1, s_postdiv = 1, s_mult = 1;
^
drivers/media/i2c/ov2659.c:912:20: warning: variable 's_postdiv' set but not used [-Wunused-but-set-variable]
u32 s_prediv = 1, s_postdiv = 1, s_mult = 1;
^
drivers/media/i2c/ov2659.c:912:6: warning: variable 's_prediv' set but not used [-Wunused-but-set-variable]
u32 s_prediv = 1, s_postdiv = 1, s_mult = 1;
^

This is likely some leftover from some past change.

Signed-off-by: Mauro Carvalho Chehab <mchehab@osg.samsung.com>

-4
-4
drivers/media/i2c/ov2659.c
··· 909 909 u8 ctrl1_reg = 0, ctrl2_reg = 0, ctrl3_reg = 0; 910 910 struct i2c_client *client = ov2659->client; 911 911 unsigned int desired = pdata->link_frequency; 912 - u32 s_prediv = 1, s_postdiv = 1, s_mult = 1; 913 912 u32 prediv, postdiv, mult; 914 913 u32 bestdelta = -1; 915 914 u32 delta, actual; ··· 928 929 929 930 if ((delta < bestdelta) || (bestdelta == -1)) { 930 931 bestdelta = delta; 931 - s_mult = mult; 932 - s_prediv = prediv; 933 - s_postdiv = postdiv; 934 932 ctrl1_reg = ctrl1[i].reg; 935 933 ctrl2_reg = mult; 936 934 ctrl3_reg = ctrl3[j].reg;