> +static int adv7511_edid_rd(struct v4l2_subdev *sd, uint16_t len, uint8_t 
> *buf)
>  {
>       struct adv7511_state *state = get_adv7511_state(sd);
> +     s32 len;

And 'len' here shadows the function argument :( Ok, I need to resend
this patch. Still, looking forward to a comment if an approach like this
is acceptable.

Attachment: signature.asc
Description: PGP signature

Reply via email to