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. > --- > 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; -- Regards, Laurent Pinchart -- 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