Hi Mauro, On Wed, Oct 28, 2015 at 10:30:30AM +0900, Mauro Carvalho Chehab wrote: > Em Tue, 27 Oct 2015 01:01:45 +0200 > Sakari Ailus <sakari.ailus@xxxxxx> escreveu: > > > From: Sakari Ailus <sakari.ailus@xxxxxxxxxxxxxxx> > > > > Signed-off-by: Sakari Ailus <sakari.ailus@xxxxxxxxxxxxxxx> > > --- > > drivers/media/platform/omap3isp/isp.c | 21 +++++++++++++-------- > > drivers/media/platform/omap3isp/isp.h | 5 +++-- > > drivers/media/platform/omap3isp/ispccdc.c | 2 +- > > drivers/media/platform/omap3isp/ispvideo.c | 20 ++++++++++++++------ > > drivers/media/platform/omap3isp/ispvideo.h | 4 ++-- > > 5 files changed, 33 insertions(+), 19 deletions(-) > > > > diff --git a/drivers/media/platform/omap3isp/isp.c b/drivers/media/platform/omap3isp/isp.c > > index 4a01a36..61c128e 100644 > > --- a/drivers/media/platform/omap3isp/isp.c > > +++ b/drivers/media/platform/omap3isp/isp.c > > @@ -896,7 +896,7 @@ static int isp_pipeline_enable(struct isp_pipeline *pipe, > > * starting entities if the pipeline won't start anyway (those entities > > * would then likely fail to stop, making the problem worse). > > */ > > - if (pipe->entities & isp->crashed) > > + if (media_entity_enum_intersects(&pipe->entities, &isp->crashed)) > > return -EIO; > > If the size of entities/crashed enums is different, it should be > returning an error, I guess, as this would be a driver's problem, and the > graph traversal on OMAP3 would likely be wrong. They should always have the same size. The omap3isp does not support dynamic entity (un)registration. Both enums are initialised once all the entities have been registered. -- Kind regards, Sakari Ailus e-mail: sakari.ailus@xxxxxx XMPP: sailus@xxxxxxxxxxxxxx -- 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
