Re: [PATCH] output human readable form of the .status field from VIDIOC_ENUMINPUT

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

 



On Fri, Oct 23, 2009 at 9:35 AM, Sigmund Augdal <sigmund@xxxxxxx> wrote:
> On Fri, Oct 23, 2009 at 12:10 AM, Hans Verkuil <hverkuil@xxxxxxxxx> wrote:
>>
>>> The attach patch modifies v4l2-ctl -I to also output signal status as
>>> detected by the driver/hardware. This info is available in the status
>>> field of the data returned by VIDIOC_ENUMINPUT which v4l2-ctl -I
>>> already calls. The strings are copied from the v4l2 api specification
>>> and could perhaps be modified a bit to fit the application.
>>>
>>> Best regards
>>>
>>> Sigmund Augdal
>>>
>>
>> Hi Sigmund,
>>
>> This doesn't work right: the status field is a bitmask, so multiple bits
>> can be set at the same time. So a switch is not the right choice for that.
>> Look at some of the other functions to print bitmasks in v4l2-ctl.cpp for
>> ideas on how to implement this properly.
>>
>> But it will be nice to have this in v4l2-ctl!
> Right, I realized this shortly after sending. I'll take a look at this
> today. However, I'm unsure how to handle the value 0. It seems this is
> used both for "signal detected and everything is ok" and "driver has
> no clue if there is a signal or not". Any feedback welcome.

And again, this time with the attachment.

Best regards

Sigmund Augdal
>
> Best regards
>
> Sigmund Augdal
>>
>> Regards,
>>
>>      Hans
>>
>> --
>> Hans Verkuil - video4linux developer - sponsored by TANDBERG Telecom
>>
>>
>

Attachment: v4l2-ctl-enuminput2.patch
Description: Binary data


[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