Re: [PATCH] drivers/media: vsp1_video: fix compile error

From: Mauro Carvalho Chehab
Date: Mon Jan 25 2016 - 06:14:35 EST


Em Mon, 25 Jan 2016 00:15:28 +0200
Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx> escreveu:

> Hi Anders,
>
> Thank you for the patch.
>
> On Friday 15 January 2016 01:09:43 Anders Roxell wrote:
> > This was found with the -RT patch enabled, but the fix should apply to
> > non-RT also.
> >
> > Compilation error without this fix:
> > ../drivers/media/platform/vsp1/vsp1_video.c: In function
> > 'vsp1_pipeline_stopped':
> > ../drivers/media/platform/vsp1/vsp1_video.c:524:2: error: expected
> > expression before 'do'
> > spin_unlock_irqrestore(&pipe->irqlock, flags);
> > ^
> >
> > Signed-off-by: Anders Roxell <anders.roxell@xxxxxxxxxx>
>
> Reviewed-by: Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx>
>
> and applied to my tree.

Better to apply this to the fixes branch and send it together with
the next set of patches upstream.

>
> > ---
> > drivers/media/platform/vsp1/vsp1_video.c | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/drivers/media/platform/vsp1/vsp1_video.c
> > b/drivers/media/platform/vsp1/vsp1_video.c index 637d0d6..b4dca57 100644
> > --- a/drivers/media/platform/vsp1/vsp1_video.c
> > +++ b/drivers/media/platform/vsp1/vsp1_video.c
> > @@ -515,7 +515,7 @@ static bool vsp1_pipeline_stopped(struct vsp1_pipeline
> > *pipe) bool stopped;
> >
> > spin_lock_irqsave(&pipe->irqlock, flags);
> > - stopped = pipe->state == VSP1_PIPELINE_STOPPED,
> > + stopped = pipe->state == VSP1_PIPELINE_STOPPED;
> > spin_unlock_irqrestore(&pipe->irqlock, flags);
> >
> > return stopped;
>