[lkml]   [2019]   [Jan]   [17]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
SubjectRe: [PATCH] HID: i2c-hid: Disable runtime PM on Goodix touchpad

> On Jan 17, 2019, at 16:06, Benjamin Tissoires <> wrote:
> On Thu, Jan 17, 2019 at 6:02 AM Kai-Heng Feng
> <> wrote:
>> Goodix says the firmware needs at least 60ms to fully respond ON and
>> SLEEP command.
> I was about to say that this is not conformant to the specification.
> But looking at 7.2.8 SET_POWER of
> They say:
> "The DEVICE must ensure that it transitions to the HOST specified
> Power State in under 1 second. "
> But in the RESPONSE paragraph: "The DEVICE shall not respond back
> after receiving the command. The DEVICE is mandated to enter that
> power state imminently."
> My interpretation was always that the device has to be in a ready
> state for new commands "immediately".
> However, the first sentence may suggest that the driver would block
> any command to the device before the 1 second timestamp.
>> I’ll see if an 200ms autosuspend can solve all Goodix, LG and Raydium
>> touchpanels.
> We already have a I2C_HID_QUIRK_DELAY_AFTER_SLEEP quirk that delay
> operations after sleep by 20ms. Maybe we can keep the runtime PM but
> use the same timers to not confuse the hardware.

Yes, but wouldn’t use pm_*_autosuspend() helpers can both solve the issue
and make the code more cleaner?


> Cheers,
> Benjamin

 \ /
  Last update: 2019-01-17 12:41    [W:0.107 / U:2.600 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site