From d11132411538761ca79ecded63cd16621b6c40ee Mon Sep 17 00:00:00 2001 From: Antti Palosaari Date: Sat, 9 Aug 2014 15:02:14 -0300 Subject: [PATCH] [media] it913x: make checkpatch.pl happy Correct issues reported by checkpatch.pl Signed-off-by: Antti Palosaari Signed-off-by: Mauro Carvalho Chehab --- drivers/media/tuners/tuner_it913x.c | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/drivers/media/tuners/tuner_it913x.c b/drivers/media/tuners/tuner_it913x.c index 281d8c5cc0a9..b92d59979437 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; -- 2.20.1