There is this code: if (v >= 0x20) { ... } else if (v < 0x20) { ... } else if (v > 0x30) { /* this branch is impossible */ } It would be sensibly for last branch to be on the top. Signed-off-by: Ivan Bornyakov <brnkv.i1@xxxxxxxxx> --- drivers/media/dvb-frontends/stv090x.c | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/media/dvb-frontends/stv090x.c b/drivers/media/dvb-frontends/stv090x.c index 9133f65d4623..d70eb311ebaf 100644 --- a/drivers/media/dvb-frontends/stv090x.c +++ b/drivers/media/dvb-frontends/stv090x.c @@ -4841,7 +4841,11 @@ static int stv090x_setup(struct dvb_frontend *fe) } state->internal->dev_ver = stv090x_read_reg(state, STV090x_MID); - if (state->internal->dev_ver >= 0x20) { + if (state->internal->dev_ver > 0x30) { + /* we shouldn't bail out from here */ + dprintk(FE_ERROR, 1, "INFO: Cut: 0x%02x probably incomplete support!", + state->internal->dev_ver); + } else if (state->internal->dev_ver >= 0x20) { if (stv090x_write_reg(state, STV090x_TSGENERAL, 0x0c) < 0) goto err; @@ -4857,10 +4861,6 @@ static int stv090x_setup(struct dvb_frontend *fe) state->internal->dev_ver); goto err; - } else if (state->internal->dev_ver > 0x30) { - /* we shouldn't bail out from here */ - dprintk(FE_ERROR, 1, "INFO: Cut: 0x%02x probably incomplete support!", - state->internal->dev_ver); } /* ADC1 range */ -- 2.16.4