From: Antti Palosaari Date: Sat, 9 Aug 2014 18:02:14 +0000 (-0300) Subject: [media] it913x: make checkpatch.pl happy X-Git-Tag: v5.15~16478^2~595 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=d11132411538761ca79ecded63cd16621b6c40ee;p=platform%2Fkernel%2Flinux-starfive.git [media] it913x: make checkpatch.pl happy Correct issues reported by checkpatch.pl Signed-off-by: Antti Palosaari Signed-off-by: Mauro Carvalho Chehab --- diff --git a/drivers/media/tuners/tuner_it913x.c b/drivers/media/tuners/tuner_it913x.c index 281d8c5..b92d599 100644 --- a/drivers/media/tuners/tuner_it913x.c +++ b/drivers/media/tuners/tuner_it913x.c @@ -46,6 +46,7 @@ static int it913x_rd_regs(struct it913x_state *state, { .addr = state->i2c_addr, .flags = I2C_M_RD, .buf = data, .len = count } }; + b[0] = (u8)(reg >> 16) & 0xff; b[1] = (u8)(reg >> 8) & 0xff; b[2] = (u8) reg & 0xff; @@ -61,6 +62,7 @@ static int it913x_rd_reg(struct it913x_state *state, u32 reg) { int ret; u8 b[1]; + ret = it913x_rd_regs(state, reg, &b[0], sizeof(b)); return (ret < 0) ? -ENODEV : b[0]; } @@ -75,6 +77,7 @@ static int it913x_wr_regs(struct it913x_state *state, .buf = b, .len = 3 + count } }; int ret; + b[0] = (u8)(reg >> 16) & 0xff; b[1] = (u8)(reg >> 8) & 0xff; b[2] = (u8) reg & 0xff; @@ -122,6 +125,7 @@ static int it913x_script_loader(struct it913x_state *state, struct it913xset *loadscript) { int ret, i; + if (loadscript == NULL) return -EINVAL;