CC linux-media. It's linux-media@xxxxxxxxxxxxxxx not linux-media@xxxxxxxxxxxxxx, sorry for the noise. On 2018-03-07 23:58:16 +0100, Niklas Söderlund wrote: > This fixes the following sparse warning: > > drivers/media/i2c/adv748x/adv748x-afe.c:294:34: expected unsigned int [usertype] *signal > drivers/media/i2c/adv748x/adv748x-afe.c:294:34: got int *<noident> > drivers/media/i2c/adv748x/adv748x-afe.c:294:34: warning: incorrect type in argument 2 (different signedness) > > Signed-off-by: Niklas Söderlund <niklas.soderlund+renesas@xxxxxxxxxxxx> > > --- > > * Changes since v1 > - Use u32 instead of unsigned int as suggested by Geert. > --- > drivers/media/i2c/adv748x/adv748x-afe.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/drivers/media/i2c/adv748x/adv748x-afe.c b/drivers/media/i2c/adv748x/adv748x-afe.c > index 5188178588c9067d..61514bae7e5ceb42 100644 > --- a/drivers/media/i2c/adv748x/adv748x-afe.c > +++ b/drivers/media/i2c/adv748x/adv748x-afe.c > @@ -275,7 +275,8 @@ static int adv748x_afe_s_stream(struct v4l2_subdev *sd, int enable) > { > struct adv748x_afe *afe = adv748x_sd_to_afe(sd); > struct adv748x_state *state = adv748x_afe_to_state(afe); > - int ret, signal = V4L2_IN_ST_NO_SIGNAL; > + u32 signal = V4L2_IN_ST_NO_SIGNAL; > + int ret; > > mutex_lock(&state->mutex); > > -- > 2.16.2 > -- Regards, Niklas Söderlund