Re: [GIT PULL FOR 2.6.39] Media controller and OMAP3 ISP driver

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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


[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux