Hi Niklas,

On Wed, Sep 26, 2018 at 11:40:06PM +0200, Niklas Söderlund wrote:
> When adding support for parallel subdev for Gen3 it was missed that the
> symbol 'i' in rvin_group_link_notify() was already declare, remove the
> dupe as it's only used as a loop variable this have no functional
> change. This fixes warning:
>
>     rcar-core.c:117:52: originally declared here
>     rcar-core.c:173:30: warning: symbol 'i' shadows an earlier one
>
> Fixes: 1284605dc821cebd ("media: rcar-vin: Handle parallel subdev in 
> link_notify")
> Signed-off-by: Niklas Söderlund <[email protected]>

Acked-by: Jacopo Mondi <[email protected]>

Thanks
  j

> ---
>  drivers/media/platform/rcar-vin/rcar-core.c | 1 -
>  1 file changed, 1 deletion(-)
>
> diff --git a/drivers/media/platform/rcar-vin/rcar-core.c 
> b/drivers/media/platform/rcar-vin/rcar-core.c
> index 5dd16af3625c333b..01e418c2d4c6792e 100644
> --- a/drivers/media/platform/rcar-vin/rcar-core.c
> +++ b/drivers/media/platform/rcar-vin/rcar-core.c
> @@ -170,7 +170,6 @@ static int rvin_group_link_notify(struct media_link 
> *link, u32 flags,
>
>       if (csi_id == -ENODEV) {
>               struct v4l2_subdev *sd;
> -             unsigned int i;
>
>               /*
>                * Make sure the source entity subdevice is registered as
> --
> 2.19.0
>

Attachment: signature.asc
Description: PGP signature

Reply via email to