Re: [PATCH 4/5] HID: autoload hid-multitouch as needed

From: Jiri Kosina
Date: Fri Mar 16 2012 - 07:26:32 EST


On Tue, 6 Mar 2012, benjamin.tissoires wrote:

> From: Benjamin Tissoires <benjamin.tissoires@xxxxxxx>
>
> The code is inspired from the one present in the bttv module.
>
> Signed-off-by: Benjamin Tissoires <benjamin.tissoires@xxxxxxx>
> ---
>
> As I mentioned in the mail 0/5, I'd really like to have your opinion on this
> one. I copied the code from bttv, but it forces us to change hid_device which
> is not very good for ABI changes reasons.

It's indeed probably the best and most "lightweight" solution before we
come up with more robust attitude (with userspace involvement).

Could you please resend the patch with proper changelog entry, and perhaps
add some more commentary into the code to better explain the reasons for
this gymnastics?

Thanks,

--
Jiri Kosina
SUSE Labs
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/