Re: [PATCHv2] New V4L2 ioctls for OMAP class of Devices

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

 



On Thursday 29 January 2009 22:59:13 Trent Piepho wrote:
> On Thu, 29 Jan 2009, Hans Verkuil wrote:
> > On Thursday 29 January 2009 08:44:20 DongSoo Kim wrote:
> > > Hello.
> > >
> > > > +#define VIDIOC_S_COLOR_SPACE_CONV      _IOW('V', 83, struct
> > > > v4l2_color_space_conversion) +#define VIDIOC_G_COLOR_SPACE_CONV
> > > > _IOR('V', 84, struct v4l2_color_space_conversion)
> > >
> > > Do you mind if I ask a question about those ioctls?
> > > Because as far as I understand, we can use VIDIOC_S_FMT ioctl to
> > > convert colorspaces. Setting through colorspace member in
> > > v4l2_pix_format, we could change output colorspace.
> >
> > Colorspace is read-only, you cannot set it. It just gives you the
> > colorspace that the hardware uses by default.
>
> Is there a reason it must be read-only?

The V4L2 spec. And I've no idea if we can safely change it to a read/write 
property or if there are apps out their that do not set this field.

> > If you want to fully control the colorspace, then you need these
> > ioctls.
>
> I don't know about "fully".  I don't see anything about gamma correction.
> Since there is no documentation, it's also not clear if it can describe
> the full range luma the bt848 and cx88 can produce.

I'm also waiting for full documentation on this. I do know that this type of 
matrix is common in other devices as well.

I'm not sure if gamma correction can be done with CSC, but I think that 
should be done through a separate control anyway. However, I'm no expert.

Regards,

	Hans

-- 
Hans Verkuil - video4linux developer - sponsored by TANDBERG
--
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