On 22/06/2016 12:48, Florian Echtler wrote: > 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. Thanks. I see that reactivision definitely uses this already (https://github.com/mkalten/reacTIVision/issues/3 ) and we don't want to break it - I've split the sur40.c change out of this patch now so it can be considered separately. > 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? While I like the idea of being able to use the generic tools, I think we needed to do something to stop these devices turning up in e.g. video conferencing software - it would cause a lot of confusion. There's nothing stopping particular tools adding the necessary code to handle touch devices if they feel their users want it. Also, the RMI4 and Atmel mXT touchscreens output signed data, which unfortunately would confuse the generic tools. -- 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
