On Fri, Oct 4, 2013 at 10:44 AM, Antti Palosaari <crope@xxxxxx> wrote: > On 04.10.2013 17:01, Hans Verkuil wrote: >> >> From: Hans Verkuil <hans.verkuil@xxxxxxxxx> >> >> drivers/media/dvb-frontends/drxd_hard.c:1017:70: warning: Using plain >> integer as NULL pointer >> drivers/media/dvb-frontends/drxd_hard.c:1038:69: warning: Using plain >> integer as NULL pointer >> drivers/media/dvb-frontends/drxd_hard.c:2836:33: warning: Using plain >> integer as NULL pointer >> drivers/media/dvb-frontends/drxd_hard.c:2972:30: warning: Using plain >> integer as NULL pointer >> >> Signed-off-by: Hans Verkuil <hans.verkuil@xxxxxxxxx> > > > Reviewed-by: Antti Palosaari <crope@xxxxxx> > > > >> Cc: Antti Palosaari <crope@xxxxxx> >> --- >> drivers/media/dvb-frontends/drxd_hard.c | 8 ++++---- >> 1 file changed, 4 insertions(+), 4 deletions(-) >> >> diff --git a/drivers/media/dvb-frontends/drxd_hard.c >> b/drivers/media/dvb-frontends/drxd_hard.c >> index cbd7c92..959ae36 100644 >> --- a/drivers/media/dvb-frontends/drxd_hard.c >> +++ b/drivers/media/dvb-frontends/drxd_hard.c >> @@ -1014,7 +1014,7 @@ static int HI_CfgCommand(struct drxd_state *state) >> status = Write16(state, HI_RA_RAM_SRV_CMD__A, >> HI_RA_RAM_SRV_CMD_CONFIG, 0); >> else >> - status = HI_Command(state, HI_RA_RAM_SRV_CMD_CONFIG, 0); >> + status = HI_Command(state, HI_RA_RAM_SRV_CMD_CONFIG, >> NULL); >> mutex_unlock(&state->mutex); >> return status; >> } >> @@ -1035,7 +1035,7 @@ static int HI_ResetCommand(struct drxd_state *state) >> status = Write16(state, HI_RA_RAM_SRV_RST_KEY__A, >> HI_RA_RAM_SRV_RST_KEY_ACT, 0); >> if (status == 0) >> - status = HI_Command(state, HI_RA_RAM_SRV_CMD_RESET, 0); >> + status = HI_Command(state, HI_RA_RAM_SRV_CMD_RESET, NULL); >> mutex_unlock(&state->mutex); >> msleep(1); >> return status; >> @@ -2833,7 +2833,7 @@ static int drxd_init(struct dvb_frontend *fe) >> int err = 0; >> >> /* if (request_firmware(&state->fw, "drxd.fw", state->dev)<0) */ >> - return DRXD_init(state, 0, 0); >> + return DRXD_init(state, NULL, 0); >> >> err = DRXD_init(state, state->fw->data, state->fw->size); >> release_firmware(state->fw); >> @@ -2969,7 +2969,7 @@ struct dvb_frontend *drxd_attach(const struct >> drxd_config *config, >> >> mutex_init(&state->mutex); >> >> - if (Read16(state, 0, 0, 0) < 0) >> + if (Read16(state, 0, NULL, 0) < 0) >> goto error; >> >> state->frontend.ops = drxd_ops; >> > > > -- > http://palosaari.fi/ > > -- > To unsubscribe from this list: send the line "unsubscribe linux-media" in > the body of a message to majordomo@xxxxxxxxxxxxxxx > More majordomo info at http://vger.kernel.org/majordomo-info.html Reviewed-by: Michael Krufky <mkrufky@xxxxxxxxxxx> -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
