* Tweak I2C SDA hold time on GemniLake to make touchpad work @ 2019-09-03 8:10 Chris Chiu 2019-09-03 8:18 ` Mika Westerberg 0 siblings, 1 reply; 6+ messages in thread From: Chris Chiu @ 2019-09-03 8:10 UTC (permalink / raw) To: Andy Shevchenko, lee.jones, Mika Westerberg, Linux Kernel, Linux Upstreaming Team Hi, We're working on the acer Gemnilake laptop TravelMate B118-M for touchpad not working issue. The touchpad fails to bring up and the i2c-hid ouput the message as follows [ 8.317293] i2c_hid i2c-ELAN0502:00: hid_descr_cmd failed We tried on latest linux kernel 5.3.0-rc6 and it reports the same. We then look into I2C signal level measurement to find out why. The following is the signal output from LA for the SCL/SDA. https://imgur.com/sKcpvdo The SCL frequency is ~400kHz from the SCL period, but the SDA transition is quite weird. Per the I2C spec, the data on the SDA line must be stable during the high period of the clock. The HIGH or LOW state of the data line can only change when the clock signal on the SCL line is LOW. The SDA period span across 2 SCL high, I think that's the reason why the I2C read the wrong data and fail to initialize. Thus, we treak the SDA hold time by the following modification. --- a/drivers/mfd/intel-lpss-pci.c +++ b/drivers/mfd/intel-lpss-pci.c @@ -97,7 +97,8 @@ static const struct intel_lpss_platform_info bxt_uart_info = { }; static struct property_entry bxt_i2c_properties[] = { - PROPERTY_ENTRY_U32("i2c-sda-hold-time-ns", 42), + PROPERTY_ENTRY_U32("i2c-sda-hold-time-ns", 230), PROPERTY_ENTRY_U32("i2c-sda-falling-time-ns", 171), PROPERTY_ENTRY_U32("i2c-scl-falling-time-ns", 208), { }, The reason why I choose sda hold time is by the Table 10 of https://www.nxp.com/docs/en/user-guide/UM10204.pdf, the device must provide a hold time at lease 300ns and and 42 here is relatively too small. The signal measurement result for the same pin on Windows is as follows. https://imgur.com/BtKUIZB Comparing to the same result running Linux https://imgur.com/N4fPTYN After applying the sda hold time tweak patch above, the touchpad can be correctly initialized and work. The LA signal is shown as down below. https://imgur.com/B3PmnIp The chart which has yellow mark is the tweak version, the red marks the original version. I need suggestions about whether if the hold time is the value I can tweak? Or I should modify sda falling time? Please help if any better idea. Thanks ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: Tweak I2C SDA hold time on GemniLake to make touchpad work 2019-09-03 8:10 Tweak I2C SDA hold time on GemniLake to make touchpad work Chris Chiu @ 2019-09-03 8:18 ` Mika Westerberg 2019-09-03 12:03 ` Jarkko Nikula 0 siblings, 1 reply; 6+ messages in thread From: Mika Westerberg @ 2019-09-03 8:18 UTC (permalink / raw) To: Chris Chiu Cc: Andy Shevchenko, lee.jones, Linux Kernel, Linux Upstreaming Team, Jarkko Nikula +Jarkko On Tue, Sep 03, 2019 at 04:10:27PM +0800, Chris Chiu wrote: > Hi, > > We're working on the acer Gemnilake laptop TravelMate B118-M for > touchpad not working issue. The touchpad fails to bring up and the > i2c-hid ouput the message as follows > [ 8.317293] i2c_hid i2c-ELAN0502:00: hid_descr_cmd failed > We tried on latest linux kernel 5.3.0-rc6 and it reports the same. > > We then look into I2C signal level measurement to find out why. > The following is the signal output from LA for the SCL/SDA. > https://imgur.com/sKcpvdo > The SCL frequency is ~400kHz from the SCL period, but the SDA > transition is quite weird. Per the I2C spec, the data on the SDA line > must be stable during the high period of the clock. The HIGH or LOW > state of the data line can only change when the clock signal on the > SCL line is LOW. The SDA period span across 2 SCL high, I think > that's the reason why the I2C read the wrong data and fail to initialize. > > Thus, we treak the SDA hold time by the following modification. > > --- a/drivers/mfd/intel-lpss-pci.c > +++ b/drivers/mfd/intel-lpss-pci.c > @@ -97,7 +97,8 @@ static const struct intel_lpss_platform_info bxt_uart_info = { > }; > > static struct property_entry bxt_i2c_properties[] = { > - PROPERTY_ENTRY_U32("i2c-sda-hold-time-ns", 42), > + PROPERTY_ENTRY_U32("i2c-sda-hold-time-ns", 230), > PROPERTY_ENTRY_U32("i2c-sda-falling-time-ns", 171), > PROPERTY_ENTRY_U32("i2c-scl-falling-time-ns", 208), > { }, > > The reason why I choose sda hold time is by the Table 10 of > https://www.nxp.com/docs/en/user-guide/UM10204.pdf, the device > must provide a hold time at lease 300ns and and 42 here is relatively > too small. The signal measurement result for the same pin on Windows > is as follows. > https://imgur.com/BtKUIZB > Comparing to the same result running Linux > https://imgur.com/N4fPTYN > > After applying the sda hold time tweak patch above, the touchpad can > be correctly initialized and work. The LA signal is shown as down below. > https://imgur.com/B3PmnIp > > The chart which has yellow mark is the tweak version, the red marks > the original version. > > I need suggestions about whether if the hold time is the value I can tweak? > Or I should modify sda falling time? Please help if any better idea. > > Thanks ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: Tweak I2C SDA hold time on GemniLake to make touchpad work 2019-09-03 8:18 ` Mika Westerberg @ 2019-09-03 12:03 ` Jarkko Nikula 2019-09-04 4:38 ` Chris Chiu 0 siblings, 1 reply; 6+ messages in thread From: Jarkko Nikula @ 2019-09-03 12:03 UTC (permalink / raw) To: Mika Westerberg, Chris Chiu Cc: Andy Shevchenko, lee.jones, Linux Kernel, Linux Upstreaming Team [-- Attachment #1: Type: text/plain, Size: 2741 bytes --] Hi Chris On 9/3/19 11:18 AM, Mika Westerberg wrote: > +Jarkko > > On Tue, Sep 03, 2019 at 04:10:27PM +0800, Chris Chiu wrote: >> Hi, >> >> We're working on the acer Gemnilake laptop TravelMate B118-M for >> touchpad not working issue. The touchpad fails to bring up and the >> i2c-hid ouput the message as follows >> [ 8.317293] i2c_hid i2c-ELAN0502:00: hid_descr_cmd failed >> We tried on latest linux kernel 5.3.0-rc6 and it reports the same. >> >> We then look into I2C signal level measurement to find out why. >> The following is the signal output from LA for the SCL/SDA. >> https://imgur.com/sKcpvdo >> The SCL frequency is ~400kHz from the SCL period, but the SDA >> transition is quite weird. Per the I2C spec, the data on the SDA line >> must be stable during the high period of the clock. The HIGH or LOW >> state of the data line can only change when the clock signal on the >> SCL line is LOW. The SDA period span across 2 SCL high, I think >> that's the reason why the I2C read the wrong data and fail to initialize. >> >> Thus, we treak the SDA hold time by the following modification. >> >> --- a/drivers/mfd/intel-lpss-pci.c >> +++ b/drivers/mfd/intel-lpss-pci.c >> @@ -97,7 +97,8 @@ static const struct intel_lpss_platform_info bxt_uart_info = { >> }; >> >> static struct property_entry bxt_i2c_properties[] = { >> - PROPERTY_ENTRY_U32("i2c-sda-hold-time-ns", 42), >> + PROPERTY_ENTRY_U32("i2c-sda-hold-time-ns", 230), >> PROPERTY_ENTRY_U32("i2c-sda-falling-time-ns", 171), >> PROPERTY_ENTRY_U32("i2c-scl-falling-time-ns", 208), >> { }, >> >> The reason why I choose sda hold time is by the Table 10 of >> https://www.nxp.com/docs/en/user-guide/UM10204.pdf, the device >> must provide a hold time at lease 300ns and and 42 here is relatively >> too small. The signal measurement result for the same pin on Windows >> is as follows. >> https://imgur.com/BtKUIZB >> Comparing to the same result running Linux >> https://imgur.com/N4fPTYN >> >> After applying the sda hold time tweak patch above, the touchpad can >> be correctly initialized and work. The LA signal is shown as down below. >> https://imgur.com/B3PmnIp >> Could you try does attached patch work for you? It's from last year for another related issue but there platform was actually Apollo Lake instead of Gemini Lake but anyway it was found out that Windows uses different timing parameters than Linux on Gemini Lake. I didn't take patch forward back then due known Gemini Lake machines were working with the Broxton I2C timing parameters but now it's time if attached patch fixes the issue on your machine. Patch is from top of v5.3-rc7 but should probably apply also to older kernels. -- Jarkko [-- Attachment #2: 0001-mfd-intel-lpss-Add-default-I2C-device-properties-for.patch --] [-- Type: text/x-patch, Size: 3268 bytes --] From c1b1aa445838a67617bda5e8de47a0f25bea16df Mon Sep 17 00:00:00 2001 From: Jarkko Nikula <jarkko.nikula@linux.intel.com> Date: Wed, 14 Nov 2018 13:37:16 +0200 Subject: [PATCH] mfd: intel-lpss: Add default I2C device properties for Gemini Lake It turned out Intel Gemini Lake doesn't use the same I2C timing parameters as Broxton. I got confirmation from the Windows team that Gemini Lake systems should use updated timing parameters that differ from those used in Broxton based systems. Fixes: f80e78aa11ad ("mfd: intel-lpss: Add Intel Gemini Lake PCI IDs") Signed-off-by: Jarkko Nikula <jarkko.nikula@linux.intel.com> --- This is not immediate stable material since there is no regression related to this. Those machines that need updated parameters have obviously never worked and I don't want this to cause regression either so better to let this get some test coverage first. --- drivers/mfd/intel-lpss-pci.c | 28 ++++++++++++++++++++-------- 1 file changed, 20 insertions(+), 8 deletions(-) diff --git a/drivers/mfd/intel-lpss-pci.c b/drivers/mfd/intel-lpss-pci.c index ade6e1ce5a98..269cb851a596 100644 --- a/drivers/mfd/intel-lpss-pci.c +++ b/drivers/mfd/intel-lpss-pci.c @@ -120,6 +120,18 @@ static const struct intel_lpss_platform_info apl_i2c_info = { .properties = apl_i2c_properties, }; +static struct property_entry glk_i2c_properties[] = { + PROPERTY_ENTRY_U32("i2c-sda-hold-time-ns", 313), + PROPERTY_ENTRY_U32("i2c-sda-falling-time-ns", 171), + PROPERTY_ENTRY_U32("i2c-scl-falling-time-ns", 290), + { }, +}; + +static const struct intel_lpss_platform_info glk_i2c_info = { + .clk_rate = 133000000, + .properties = glk_i2c_properties, +}; + static const struct intel_lpss_platform_info cnl_i2c_info = { .clk_rate = 216000000, .properties = spt_i2c_properties, @@ -172,14 +184,14 @@ static const struct pci_device_id intel_lpss_pci_ids[] = { { PCI_VDEVICE(INTEL, 0x1ac6), (kernel_ulong_t)&bxt_info }, { PCI_VDEVICE(INTEL, 0x1aee), (kernel_ulong_t)&bxt_uart_info }, /* GLK */ - { PCI_VDEVICE(INTEL, 0x31ac), (kernel_ulong_t)&bxt_i2c_info }, - { PCI_VDEVICE(INTEL, 0x31ae), (kernel_ulong_t)&bxt_i2c_info }, - { PCI_VDEVICE(INTEL, 0x31b0), (kernel_ulong_t)&bxt_i2c_info }, - { PCI_VDEVICE(INTEL, 0x31b2), (kernel_ulong_t)&bxt_i2c_info }, - { PCI_VDEVICE(INTEL, 0x31b4), (kernel_ulong_t)&bxt_i2c_info }, - { PCI_VDEVICE(INTEL, 0x31b6), (kernel_ulong_t)&bxt_i2c_info }, - { PCI_VDEVICE(INTEL, 0x31b8), (kernel_ulong_t)&bxt_i2c_info }, - { PCI_VDEVICE(INTEL, 0x31ba), (kernel_ulong_t)&bxt_i2c_info }, + { PCI_VDEVICE(INTEL, 0x31ac), (kernel_ulong_t)&glk_i2c_info }, + { PCI_VDEVICE(INTEL, 0x31ae), (kernel_ulong_t)&glk_i2c_info }, + { PCI_VDEVICE(INTEL, 0x31b0), (kernel_ulong_t)&glk_i2c_info }, + { PCI_VDEVICE(INTEL, 0x31b2), (kernel_ulong_t)&glk_i2c_info }, + { PCI_VDEVICE(INTEL, 0x31b4), (kernel_ulong_t)&glk_i2c_info }, + { PCI_VDEVICE(INTEL, 0x31b6), (kernel_ulong_t)&glk_i2c_info }, + { PCI_VDEVICE(INTEL, 0x31b8), (kernel_ulong_t)&glk_i2c_info }, + { PCI_VDEVICE(INTEL, 0x31ba), (kernel_ulong_t)&glk_i2c_info }, { PCI_VDEVICE(INTEL, 0x31bc), (kernel_ulong_t)&bxt_uart_info }, { PCI_VDEVICE(INTEL, 0x31be), (kernel_ulong_t)&bxt_uart_info }, { PCI_VDEVICE(INTEL, 0x31c0), (kernel_ulong_t)&bxt_uart_info }, -- 2.23.0.rc1 ^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: Tweak I2C SDA hold time on GemniLake to make touchpad work 2019-09-03 12:03 ` Jarkko Nikula @ 2019-09-04 4:38 ` Chris Chiu 2019-09-04 5:54 ` Jarkko Nikula 0 siblings, 1 reply; 6+ messages in thread From: Chris Chiu @ 2019-09-04 4:38 UTC (permalink / raw) To: Jarkko Nikula Cc: Mika Westerberg, Andy Shevchenko, lee.jones, Linux Kernel, Linux Upstreaming Team On Tue, Sep 3, 2019 at 8:03 PM Jarkko Nikula <jarkko.nikula@linux.intel.com> wrote: > > Hi Chris > > On 9/3/19 11:18 AM, Mika Westerberg wrote: > > +Jarkko > > > > On Tue, Sep 03, 2019 at 04:10:27PM +0800, Chris Chiu wrote: > >> Hi, > >> > >> We're working on the acer Gemnilake laptop TravelMate B118-M for > >> touchpad not working issue. The touchpad fails to bring up and the > >> i2c-hid ouput the message as follows > >> [ 8.317293] i2c_hid i2c-ELAN0502:00: hid_descr_cmd failed > >> We tried on latest linux kernel 5.3.0-rc6 and it reports the same. > >> > >> We then look into I2C signal level measurement to find out why. > >> The following is the signal output from LA for the SCL/SDA. > >> https://imgur.com/sKcpvdo > >> The SCL frequency is ~400kHz from the SCL period, but the SDA > >> transition is quite weird. Per the I2C spec, the data on the SDA line > >> must be stable during the high period of the clock. The HIGH or LOW > >> state of the data line can only change when the clock signal on the > >> SCL line is LOW. The SDA period span across 2 SCL high, I think > >> that's the reason why the I2C read the wrong data and fail to initialize. > >> > >> Thus, we treak the SDA hold time by the following modification. > >> > >> --- a/drivers/mfd/intel-lpss-pci.c > >> +++ b/drivers/mfd/intel-lpss-pci.c > >> @@ -97,7 +97,8 @@ static const struct intel_lpss_platform_info bxt_uart_info = { > >> }; > >> > >> static struct property_entry bxt_i2c_properties[] = { > >> - PROPERTY_ENTRY_U32("i2c-sda-hold-time-ns", 42), > >> + PROPERTY_ENTRY_U32("i2c-sda-hold-time-ns", 230), > >> PROPERTY_ENTRY_U32("i2c-sda-falling-time-ns", 171), > >> PROPERTY_ENTRY_U32("i2c-scl-falling-time-ns", 208), > >> { }, > >> > >> The reason why I choose sda hold time is by the Table 10 of > >> https://www.nxp.com/docs/en/user-guide/UM10204.pdf, the device > >> must provide a hold time at lease 300ns and and 42 here is relatively > >> too small. The signal measurement result for the same pin on Windows > >> is as follows. > >> https://imgur.com/BtKUIZB > >> Comparing to the same result running Linux > >> https://imgur.com/N4fPTYN > >> > >> After applying the sda hold time tweak patch above, the touchpad can > >> be correctly initialized and work. The LA signal is shown as down below. > >> https://imgur.com/B3PmnIp > >> > Could you try does attached patch work for you? > > It's from last year for another related issue but there platform was > actually Apollo Lake instead of Gemini Lake but anyway it was found out > that Windows uses different timing parameters than Linux on Gemini Lake. > > I didn't take patch forward back then due known Gemini Lake machines > were working with the Broxton I2C timing parameters but now it's time if > attached patch fixes the issue on your machine. > > Patch is from top of v5.3-rc7 but should probably apply also to older > kernels. > > -- > Jarkko Thanks, Jarkko, the patche works on my acer laptops. Chris ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: Tweak I2C SDA hold time on GemniLake to make touchpad work 2019-09-04 4:38 ` Chris Chiu @ 2019-09-04 5:54 ` Jarkko Nikula 2019-09-04 6:24 ` Chris Chiu 0 siblings, 1 reply; 6+ messages in thread From: Jarkko Nikula @ 2019-09-04 5:54 UTC (permalink / raw) To: Chris Chiu Cc: Mika Westerberg, Andy Shevchenko, lee.jones, Linux Kernel, Linux Upstreaming Team On 9/4/19 7:38 AM, Chris Chiu wrote: > On Tue, Sep 3, 2019 at 8:03 PM Jarkko Nikula > <jarkko.nikula@linux.intel.com> wrote: >> >> Hi Chris >> >> On 9/3/19 11:18 AM, Mika Westerberg wrote: >>> +Jarkko >>> >>> On Tue, Sep 03, 2019 at 04:10:27PM +0800, Chris Chiu wrote: >>>> Hi, >>>> >>>> We're working on the acer Gemnilake laptop TravelMate B118-M for >>>> touchpad not working issue. The touchpad fails to bring up and the >>>> i2c-hid ouput the message as follows >>>> [ 8.317293] i2c_hid i2c-ELAN0502:00: hid_descr_cmd failed >>>> We tried on latest linux kernel 5.3.0-rc6 and it reports the same. >>>> >>>> We then look into I2C signal level measurement to find out why. >>>> The following is the signal output from LA for the SCL/SDA. >>>> https://imgur.com/sKcpvdo >>>> The SCL frequency is ~400kHz from the SCL period, but the SDA >>>> transition is quite weird. Per the I2C spec, the data on the SDA line >>>> must be stable during the high period of the clock. The HIGH or LOW >>>> state of the data line can only change when the clock signal on the >>>> SCL line is LOW. The SDA period span across 2 SCL high, I think >>>> that's the reason why the I2C read the wrong data and fail to initialize. >>>> >>>> Thus, we treak the SDA hold time by the following modification. >>>> >>>> --- a/drivers/mfd/intel-lpss-pci.c >>>> +++ b/drivers/mfd/intel-lpss-pci.c >>>> @@ -97,7 +97,8 @@ static const struct intel_lpss_platform_info bxt_uart_info = { >>>> }; >>>> >>>> static struct property_entry bxt_i2c_properties[] = { >>>> - PROPERTY_ENTRY_U32("i2c-sda-hold-time-ns", 42), >>>> + PROPERTY_ENTRY_U32("i2c-sda-hold-time-ns", 230), >>>> PROPERTY_ENTRY_U32("i2c-sda-falling-time-ns", 171), >>>> PROPERTY_ENTRY_U32("i2c-scl-falling-time-ns", 208), >>>> { }, >>>> >>>> The reason why I choose sda hold time is by the Table 10 of >>>> https://www.nxp.com/docs/en/user-guide/UM10204.pdf, the device >>>> must provide a hold time at lease 300ns and and 42 here is relatively >>>> too small. The signal measurement result for the same pin on Windows >>>> is as follows. >>>> https://imgur.com/BtKUIZB >>>> Comparing to the same result running Linux >>>> https://imgur.com/N4fPTYN >>>> >>>> After applying the sda hold time tweak patch above, the touchpad can >>>> be correctly initialized and work. The LA signal is shown as down below. >>>> https://imgur.com/B3PmnIp >>>> >> Could you try does attached patch work for you? >> >> It's from last year for another related issue but there platform was >> actually Apollo Lake instead of Gemini Lake but anyway it was found out >> that Windows uses different timing parameters than Linux on Gemini Lake. >> >> I didn't take patch forward back then due known Gemini Lake machines >> were working with the Broxton I2C timing parameters but now it's time if >> attached patch fixes the issue on your machine. >> >> Patch is from top of v5.3-rc7 but should probably apply also to older >> kernels. >> >> -- >> Jarkko > > Thanks, Jarkko, the patche works on my acer laptops. > Thanks. I'll send the patch out with Cc'ing you. I took the freedom to add your Tested-by tag if you don't mind :-) -- Jarkko ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: Tweak I2C SDA hold time on GemniLake to make touchpad work 2019-09-04 5:54 ` Jarkko Nikula @ 2019-09-04 6:24 ` Chris Chiu 0 siblings, 0 replies; 6+ messages in thread From: Chris Chiu @ 2019-09-04 6:24 UTC (permalink / raw) To: Jarkko Nikula Cc: Mika Westerberg, Andy Shevchenko, lee.jones, Linux Kernel, Linux Upstreaming Team On Wed, Sep 4, 2019 at 1:54 PM Jarkko Nikula <jarkko.nikula@linux.intel.com> wrote: > > On 9/4/19 7:38 AM, Chris Chiu wrote: > > On Tue, Sep 3, 2019 at 8:03 PM Jarkko Nikula > > <jarkko.nikula@linux.intel.com> wrote: > >> > >> Hi Chris > >> > >> On 9/3/19 11:18 AM, Mika Westerberg wrote: > >>> +Jarkko > >>> > >>> On Tue, Sep 03, 2019 at 04:10:27PM +0800, Chris Chiu wrote: > >>>> Hi, > >>>> > >>>> We're working on the acer Gemnilake laptop TravelMate B118-M for > >>>> touchpad not working issue. The touchpad fails to bring up and the > >>>> i2c-hid ouput the message as follows > >>>> [ 8.317293] i2c_hid i2c-ELAN0502:00: hid_descr_cmd failed > >>>> We tried on latest linux kernel 5.3.0-rc6 and it reports the same. > >>>> > >>>> We then look into I2C signal level measurement to find out why. > >>>> The following is the signal output from LA for the SCL/SDA. > >>>> https://imgur.com/sKcpvdo > >>>> The SCL frequency is ~400kHz from the SCL period, but the SDA > >>>> transition is quite weird. Per the I2C spec, the data on the SDA line > >>>> must be stable during the high period of the clock. The HIGH or LOW > >>>> state of the data line can only change when the clock signal on the > >>>> SCL line is LOW. The SDA period span across 2 SCL high, I think > >>>> that's the reason why the I2C read the wrong data and fail to initialize. > >>>> > >>>> Thus, we treak the SDA hold time by the following modification. > >>>> > >>>> --- a/drivers/mfd/intel-lpss-pci.c > >>>> +++ b/drivers/mfd/intel-lpss-pci.c > >>>> @@ -97,7 +97,8 @@ static const struct intel_lpss_platform_info bxt_uart_info = { > >>>> }; > >>>> > >>>> static struct property_entry bxt_i2c_properties[] = { > >>>> - PROPERTY_ENTRY_U32("i2c-sda-hold-time-ns", 42), > >>>> + PROPERTY_ENTRY_U32("i2c-sda-hold-time-ns", 230), > >>>> PROPERTY_ENTRY_U32("i2c-sda-falling-time-ns", 171), > >>>> PROPERTY_ENTRY_U32("i2c-scl-falling-time-ns", 208), > >>>> { }, > >>>> > >>>> The reason why I choose sda hold time is by the Table 10 of > >>>> https://www.nxp.com/docs/en/user-guide/UM10204.pdf, the device > >>>> must provide a hold time at lease 300ns and and 42 here is relatively > >>>> too small. The signal measurement result for the same pin on Windows > >>>> is as follows. > >>>> https://imgur.com/BtKUIZB > >>>> Comparing to the same result running Linux > >>>> https://imgur.com/N4fPTYN > >>>> > >>>> After applying the sda hold time tweak patch above, the touchpad can > >>>> be correctly initialized and work. The LA signal is shown as down below. > >>>> https://imgur.com/B3PmnIp > >>>> > >> Could you try does attached patch work for you? > >> > >> It's from last year for another related issue but there platform was > >> actually Apollo Lake instead of Gemini Lake but anyway it was found out > >> that Windows uses different timing parameters than Linux on Gemini Lake. > >> > >> I didn't take patch forward back then due known Gemini Lake machines > >> were working with the Broxton I2C timing parameters but now it's time if > >> attached patch fixes the issue on your machine. > >> > >> Patch is from top of v5.3-rc7 but should probably apply also to older > >> kernels. > >> > >> -- > >> Jarkko > > > > Thanks, Jarkko, the patche works on my acer laptops. > > > Thanks. I'll send the patch out with Cc'ing you. I took the freedom to > add your Tested-by tag if you don't mind :-) > > -- > Jarkko No problem. Thanks ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2019-09-04 6:24 UTC | newest] Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2019-09-03 8:10 Tweak I2C SDA hold time on GemniLake to make touchpad work Chris Chiu 2019-09-03 8:18 ` Mika Westerberg 2019-09-03 12:03 ` Jarkko Nikula 2019-09-04 4:38 ` Chris Chiu 2019-09-04 5:54 ` Jarkko Nikula 2019-09-04 6:24 ` Chris Chiu
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).