Re: [PATCH 3/3 v2] [media] sta2x11_vip: convert to videobuf2 and control framework

From: Federico Vaga
Date: Mon Aug 06 2012 - 04:51:25 EST


> > + vip->video_dev->flags |= V4L2_FL_USES_V4L2_FH |
> > V4L2_FL_USE_FH_PRIO;
> Been there, done that :-)
>
> V4L2_FL_USE_FH_PRIO is a bit number, not a bit mask. Use set_bit
> instead:
>
> set_bit(V4L2_FL_USE_FH_PRIO, &vip->video_dev->flags);
>
> No need to set V4L2_FL_USES_V4L2_FH, BTW. That will be set
> automatically as soon as v4l2_fh_open is called.

I saw "unsigned long flags;" in the header but without reading the
comment :) Thank you. I will test it in these days but I think it's all
done.

--
Federico Vaga
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/