Re: [PATCH v4 2/9] [media] v4l2-core: Add VFL_TYPE_TOUCH_SENSOR

From: Florian Echtler
Date: Wed Jun 22 2016 - 08:05:58 EST


On 20.06.2016 14:00, Hans Verkuil wrote:
> On 06/17/2016 04:16 PM, Nick Dyer wrote:
>> Some touch controllers send out raw touch data in a similar way to a
>> greyscale frame grabber. Add a new device type for these devices.
>>
>> Use a new device prefix v4l-touch for these devices, to stop generic
>> capture software from treating them as webcams.
>>
>> Signed-off-by: Nick Dyer <nick.dyer@xxxxxxxxxxx>
>> ---
>> drivers/input/touchscreen/sur40.c | 4 ++--
>> drivers/media/v4l2-core/v4l2-dev.c | 13 ++++++++++---
>> drivers/media/v4l2-core/v4l2-ioctl.c | 15 ++++++++++-----
>> include/media/v4l2-dev.h | 3 ++-
>> include/uapi/linux/videodev2.h | 1 +

Generally a good idea in my opinion, but I think the SUR40 is a special
case: the whole point of putting in a V4L2 driver was that software like
reacTIVision, which already has a V4L2 interface, can then use that
device like any other camera.

Come to think of it, wouldn't it make sense to expose the other touch
devices as generic frame grabbers, too, so you can easily view the debug
output with any generic tool like cheese?

Best, Florian
--
SENT FROM MY DEC VT50 TERMINAL

Attachment: signature.asc
Description: OpenPGP digital signature