Hi Hans, On Sat, Mar 5, 2011 at 1:52 PM, Hans Verkuil <hverkuil@xxxxxxxxx> wrote: > On Friday, March 04, 2011 21:10:05 Mauro Carvalho Chehab wrote: >> Em 03-03-2011 07:25, Laurent Pinchart escreveu: >> > Hi Mauro, >> > >> > The following changes since commit 88a763df226facb74fdb254563e30e9efb64275c: >> > >> > Â [media] dw2102: prof 1100 corrected (2011-03-02 16:56:54 -0300) >> > >> > are available in the git repository at: >> > Â git://linuxtv.org/pinchartl/media.git media-2.6.39-0005-omap3isp >> > >> > The branch has been rebased on top of the latest for_v2.6.39 branch, with the >> > v4l2-ioctl.c conflict resolved. >> > >> > Antti Koskipaa (1): >> > Â Â Â v4l: v4l2_subdev userspace crop API >> > >> > David Cohen (1): >> > Â Â Â omap3isp: Statistics >> > >> > Laurent Pinchart (36): >> > Â Â Â v4l: Share code between video_usercopy and video_ioctl2 >> > Â Â Â v4l: subdev: Don't require core operations >> > Â Â Â v4l: subdev: Add device node support >> > Â Â Â v4l: subdev: Uninline the v4l2_subdev_init function >> > Â Â Â v4l: subdev: Control ioctls support >> > Â Â Â media: Media device node support >> > Â Â Â media: Media device >> > Â Â Â media: Entities, pads and links >> > Â Â Â media: Entity use count >> > Â Â Â media: Media device information query >> > Â Â Â media: Entities, pads and links enumeration >> > Â Â Â media: Links setup >> > Â Â Â media: Pipelines and media streams >> > Â Â Â v4l: Add a media_device pointer to the v4l2_device structure >> > Â Â Â v4l: Make video_device inherit from media_entity >> > Â Â Â v4l: Make v4l2_subdev inherit from media_entity >> > Â Â Â v4l: Move the media/v4l2-mediabus.h header to include/linux >> > Â Â Â v4l: Replace enums with fixed-sized fields in public structure >> > Â Â Â v4l: Rename V4L2_MBUS_FMT_GREY8_1X8 to V4L2_MBUS_FMT_Y8_1X8 >> > Â Â Â v4l: Group media bus pixel codes by types and sort them alphabetically >> >> The presence of those mediabus names against the traditional fourcc codes >> at the API adds some mess to the media controller. Not sure how to solve, >> but maybe the best way is to add a table at the V4L2 API associating each >> media bus format to the corresponding V4L2 fourcc codes. > > You can't do that in general. Only for specific hardware platforms. If you > could do it, then we would have never bothered creating these mediabus fourccs. > > How a mediabus fourcc translates to a pixelcode (== memory format) depends > entirely on the hardware capabilities (mostly that of the DMA engine). May I ask you one question here? (not entirely related to this patch set). Why pixelcode != mediabus fourcc? e.g. OMAP2 camera driver talks to sensor through subdev interface and sets its own output pixelformat depending on sensor's mediabus fourcc. So it needs a translation table mbus_pixelcode -> pixelformat. Why can't it be pixelformat -> pixelformat ? Regards, David > > A generic V4L2 application will never use mediabus fourcc codes. It's only used > by drivers and applications written specifically for that hardware and using > /dev/v4l-subdevX devices. > > Regards, > > Â Â Â ÂHans > > -- > Hans Verkuil - video4linux developer - sponsored by Cisco > -- > 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 > -- 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