Commit 2f60f13c authored by Mauro Carvalho Chehab's avatar Mauro Carvalho Chehab

[media] drxk: better handle errors

As reported by smatch:
	drivers/media/dvb-frontends/drxk_hard.c:3277 dvbt_sc_command() warn: missing break? reassigning 'status'

This is basically because the error handling logic there was crappy.
Signed-off-by: default avatarMauro Carvalho Chehab <mchehab@osg.samsung.com>
parent 84cb7be4
...@@ -3262,6 +3262,7 @@ static int dvbt_sc_command(struct drxk_state *state, ...@@ -3262,6 +3262,7 @@ static int dvbt_sc_command(struct drxk_state *state,
} }
/* Write needed parameters and the command */ /* Write needed parameters and the command */
status = 0;
switch (cmd) { switch (cmd) {
/* All commands using 5 parameters */ /* All commands using 5 parameters */
/* All commands using 4 parameters */ /* All commands using 4 parameters */
...@@ -3270,16 +3271,16 @@ static int dvbt_sc_command(struct drxk_state *state, ...@@ -3270,16 +3271,16 @@ static int dvbt_sc_command(struct drxk_state *state,
case OFDM_SC_RA_RAM_CMD_PROC_START: case OFDM_SC_RA_RAM_CMD_PROC_START:
case OFDM_SC_RA_RAM_CMD_SET_PREF_PARAM: case OFDM_SC_RA_RAM_CMD_SET_PREF_PARAM:
case OFDM_SC_RA_RAM_CMD_PROGRAM_PARAM: case OFDM_SC_RA_RAM_CMD_PROGRAM_PARAM:
status = write16(state, OFDM_SC_RA_RAM_PARAM1__A, param1); status |= write16(state, OFDM_SC_RA_RAM_PARAM1__A, param1);
/* All commands using 1 parameters */ /* All commands using 1 parameters */
case OFDM_SC_RA_RAM_CMD_SET_ECHO_TIMING: case OFDM_SC_RA_RAM_CMD_SET_ECHO_TIMING:
case OFDM_SC_RA_RAM_CMD_USER_IO: case OFDM_SC_RA_RAM_CMD_USER_IO:
status = write16(state, OFDM_SC_RA_RAM_PARAM0__A, param0); status |= write16(state, OFDM_SC_RA_RAM_PARAM0__A, param0);
/* All commands using 0 parameters */ /* All commands using 0 parameters */
case OFDM_SC_RA_RAM_CMD_GET_OP_PARAM: case OFDM_SC_RA_RAM_CMD_GET_OP_PARAM:
case OFDM_SC_RA_RAM_CMD_NULL: case OFDM_SC_RA_RAM_CMD_NULL:
/* Write command */ /* Write command */
status = write16(state, OFDM_SC_RA_RAM_CMD__A, cmd); status |= write16(state, OFDM_SC_RA_RAM_CMD__A, cmd);
break; break;
default: default:
/* Unknown command */ /* Unknown command */
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment