Re: [PATCH 2/3] hid: Multitouch support for the N-Trig touchscreen

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

 




Le 19 mai 09 à 17:20, Stéphane Chatty a écrit :


Le 19 mai 09 à 16:46, Jiri Kosina a écrit :

On Tue, 19 May 2009, Stéphane Chatty wrote:

Exported hidinput_hid_event, so as to be able to call it from device-specific
drivers.

Why is this needed? ->event() callback is called for every HID driver
which has this callback registered, and fallback to hidinput_hid_event() is performed in cases the driver-specific ->event() handler returns return value > 0 , signalizing that the event hasn't been fully processed and
processing by generic layer is needed.

Correct me if I am wrong, but if ->event() is called and returns 1, then hiddev_hid_event gets no chance to be called from hid_process_event. Is hidinput exclusive of hiddev?

OK, actually there's a way of doing it without calling hidinput_hid_event() at all, even if we want to make sure that hiddev- hid-event() gets a chance to be called. Then there's no use for a patch to hid-input.c. New version coming for the patch to hid- ntrig.c, taking this into account.


St.

--
To unsubscribe from this list: send the line "unsubscribe linux-input" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Media Devel]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Linux Wireless Networking]     [Linux Omap]

  Powered by Linux