Re: [PATCH] Input: elan_i2c - enable ELAN0600 acpi panels

[Dmitry Torokhov <dmitry.torokhov@gmail.com>]

On Thu, Jun 11, 2015 at 04:19:48PM -0400, Benjamin Tissoires wrote:
> On Apr 09 2015 or thereabouts, duson wrote:
> > Hi Dmitry,
> > 
> > Sorry later for reply. If touchpad is designed for Window OS, 
> > the firmware will support standard HID mouse format (non-driver) and elan raw data format (with driver).
> > 
> > So, if user change OS from Window to Linux, touchpad will work fine under HID format.
> > 
> 
> Well, Jurgen reported a ELAN0100 (04F3:0401) which is similar but does
> not work with the below patch (when adapted to match his PID).
> And like under Windows, the HID mouse report format is just not enough under
> Linux. We need to have access to the raw touches to be able to provide a
> consistent experience. So we need to get his device access to the
> proprietary raw format.
> 
> I think what makes Jurgen's device unhappy is that the length of the
> report 0x5d is 27 instead of 31 (according to the report descriptors).
> But elan_i2c_i2c.c checks for the length of the report and fails (only
> suppositions).
> 
> I would be happy to help here, but it looks like it is going to be
> difficult to remotely debug this and switch the whole driver into HID,
> especially given that elan_i2c_i2c uses a non-HID command to set the
> touchpad in the absolute mode.
> 
> So basically, I don't know what to do for Jurgen, but we could still
> enable Alessio's touchpad (the ELAN0600, below). Jiri, Dmitry?

Yes, if the controller indeed has basic HID mode + advanced proprietary
protocol we should blacklist it in HID and let elan driver handle the
device. Devices that speak proper HID (both basic and multitouch) should
be handled by HID.

Thanks.

-- 
Dmitry
--
To unsubscribe from this list: send the line "unsubscribe linux-input" in