[media] af9033: make checkpatch.pl happy
Correct issues reported by checkpatch.pl. Signed-off-by: Antti Palosaari <crope@iki.fi> Signed-off-by: Mauro Carvalho Chehab <mchehab@osg.samsung.com>
This commit is contained in:
Родитель
c10989a2a8
Коммит
24e419a0f3
|
@ -69,8 +69,9 @@ static int af9033_wr_regs(struct af9033_state *state, u32 reg, const u8 *val,
|
||||||
if (ret == 1) {
|
if (ret == 1) {
|
||||||
ret = 0;
|
ret = 0;
|
||||||
} else {
|
} else {
|
||||||
dev_warn(&state->i2c->dev, "%s: i2c wr failed=%d reg=%06x " \
|
dev_warn(&state->i2c->dev,
|
||||||
"len=%d\n", KBUILD_MODNAME, ret, reg, len);
|
"%s: i2c wr failed=%d reg=%06x len=%d\n",
|
||||||
|
KBUILD_MODNAME, ret, reg, len);
|
||||||
ret = -EREMOTEIO;
|
ret = -EREMOTEIO;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -101,8 +102,9 @@ static int af9033_rd_regs(struct af9033_state *state, u32 reg, u8 *val, int len)
|
||||||
if (ret == 2) {
|
if (ret == 2) {
|
||||||
ret = 0;
|
ret = 0;
|
||||||
} else {
|
} else {
|
||||||
dev_warn(&state->i2c->dev, "%s: i2c rd failed=%d reg=%06x " \
|
dev_warn(&state->i2c->dev,
|
||||||
"len=%d\n", KBUILD_MODNAME, ret, reg, len);
|
"%s: i2c rd failed=%d reg=%06x len=%d\n",
|
||||||
|
KBUILD_MODNAME, ret, reg, len);
|
||||||
ret = -EREMOTEIO;
|
ret = -EREMOTEIO;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -835,7 +837,7 @@ static int af9033_read_snr(struct dvb_frontend *fe, u16 *snr)
|
||||||
int ret, i, len;
|
int ret, i, len;
|
||||||
u8 buf[3], tmp;
|
u8 buf[3], tmp;
|
||||||
u32 snr_val;
|
u32 snr_val;
|
||||||
const struct val_snr *uninitialized_var(snr_lut);
|
const struct val_snr *snr_lut;
|
||||||
|
|
||||||
/* read value */
|
/* read value */
|
||||||
ret = af9033_rd_regs(state, 0x80002c, buf, 3);
|
ret = af9033_rd_regs(state, 0x80002c, buf, 3);
|
||||||
|
@ -928,7 +930,9 @@ static int af9033_update_ch_stat(struct af9033_state *state)
|
||||||
abort_cnt = 1000;
|
abort_cnt = 1000;
|
||||||
state->ber = 0xffffffff;
|
state->ber = 0xffffffff;
|
||||||
} else {
|
} else {
|
||||||
/* 8 byte packets, that have not been rejected already */
|
/*
|
||||||
|
* 8 byte packets, that have not been rejected already
|
||||||
|
*/
|
||||||
bit_cnt -= (u32)abort_cnt;
|
bit_cnt -= (u32)abort_cnt;
|
||||||
if (bit_cnt == 0) {
|
if (bit_cnt == 0) {
|
||||||
state->ber = 0xffffffff;
|
state->ber = 0xffffffff;
|
||||||
|
@ -1015,7 +1019,8 @@ err:
|
||||||
return ret;
|
return ret;
|
||||||
}
|
}
|
||||||
|
|
||||||
static int af9033_pid_filter(struct dvb_frontend *fe, int index, u16 pid, int onoff)
|
static int af9033_pid_filter(struct dvb_frontend *fe, int index, u16 pid,
|
||||||
|
int onoff)
|
||||||
{
|
{
|
||||||
struct af9033_state *state = fe->demodulator_priv;
|
struct af9033_state *state = fe->demodulator_priv;
|
||||||
int ret;
|
int ret;
|
||||||
|
@ -1069,8 +1074,8 @@ struct dvb_frontend *af9033_attach(const struct af9033_config *config,
|
||||||
memcpy(&state->cfg, config, sizeof(struct af9033_config));
|
memcpy(&state->cfg, config, sizeof(struct af9033_config));
|
||||||
|
|
||||||
if (state->cfg.clock != 12000000) {
|
if (state->cfg.clock != 12000000) {
|
||||||
dev_err(&state->i2c->dev, "%s: af9033: unsupported clock=%d, " \
|
dev_err(&state->i2c->dev,
|
||||||
"only 12000000 Hz is supported currently\n",
|
"%s: af9033: unsupported clock=%d, only 12000000 Hz is supported currently\n",
|
||||||
KBUILD_MODNAME, state->cfg.clock);
|
KBUILD_MODNAME, state->cfg.clock);
|
||||||
goto err;
|
goto err;
|
||||||
}
|
}
|
||||||
|
@ -1084,9 +1089,10 @@ struct dvb_frontend *af9033_attach(const struct af9033_config *config,
|
||||||
if (ret < 0)
|
if (ret < 0)
|
||||||
goto err;
|
goto err;
|
||||||
|
|
||||||
dev_info(&state->i2c->dev, "%s: firmware version: LINK=%d.%d.%d.%d " \
|
dev_info(&state->i2c->dev,
|
||||||
"OFDM=%d.%d.%d.%d\n", KBUILD_MODNAME, buf[0], buf[1],
|
"%s: firmware version: LINK=%d.%d.%d.%d OFDM=%d.%d.%d.%d\n",
|
||||||
buf[2], buf[3], buf[4], buf[5], buf[6], buf[7]);
|
KBUILD_MODNAME, buf[0], buf[1], buf[2], buf[3], buf[4],
|
||||||
|
buf[5], buf[6], buf[7]);
|
||||||
|
|
||||||
/* sleep */
|
/* sleep */
|
||||||
switch (state->cfg.tuner) {
|
switch (state->cfg.tuner) {
|
||||||
|
|
Загрузка…
Ссылка в новой задаче