linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: "Pali Rohár" <pali.rohar@gmail.com>
To: Dmitry Torokhov <dmitry.torokhov@gmail.com>
Cc: Mario Limonciello <mario_limonciello@dell.com>,
	LKML <linux-kernel@vger.kernel.org>,
	"linux-input@vger.kernel.org" <linux-input@vger.kernel.org>
Subject: Re: [PATCH] Add a quirk for the Dell XPS 13 (2015) when in PS/2 mode.
Date: Fri, 20 Feb 2015 20:24:20 +0100	[thread overview]
Message-ID: <201502202024.20741@pali> (raw)
In-Reply-To: <20150220184717.GC20060@dtor-glaptop>

[-- Attachment #1: Type: Text/Plain, Size: 3296 bytes --]

On Friday 20 February 2015 19:47:17 Dmitry Torokhov wrote:
> Hi Mario,
> 
> On Thu, Feb 19, 2015 at 12:16:51PM -0600, Mario Limonciello 
wrote:
> > Hi Dmitry,
> > 
> > On 02/19/2015 11:16 AM, Dmitry Torokhov wrote:
> > >What kind of glitch is this? Is there a certain pattern to
> > >it? Even if we do not reset the mouse the logs will be
> > >full of error messages.
> > >
> > >Thanks.
> > 
> > From waveform capture data leaving the touchpad is valid,
> > but when it is resent through the EC it's getting
> > corrupted.  This isn't exclusive to Linux setting it up
> > wrong or anything, it also happens on Windows.  On Windows
> > 7 the touchpad driver does not issue a reset from the bad
> > data however.
> > 
> > The glitch is more prevalent as the machine is turned on but
> > seems to go away after it's been running a while.  We
> > currently don't believe it to be a problem with the EC
> > firmware, but we are still exploring other firmware based
> > solutions.
> 
> Can it be related to ther Dell models (Latitudes with ALPS
> touchpad) also sending junk data under certain conditions
> (adding Pali to teh CC as he was looking at this issue)?
> 

Dell Latitude Exx40 models (with ALPS touchpads) have similar 
problems. Linux psmouse.ko/alps.c driver receive invalid packets 
which cause lot of problems... ALPS people told me those packets 
which was found on i8042 bus are really invalid ALPS packets and 
do not come from ALPS touchpad. Unless there is invisible bug in 
ALPS touchpad firmware (which was not discovered yet), problem is 
either in Dell EmeddedController where is connected ALPS touchpad 
or in Dell BIOS/UEFI (which I believe can modify any such data).

If Dell share EC firmware code in more models (Latitude and XPS) 
or share some BIOS parts, then problem can be really there.

> > Yes, the logs do fill up with error messages about the bad
> > data within the first few minutes of usage.  In my opinion
> > not freezing but getting errors in the log is better than
> > freezing and getting errors in the log, so we're at least
> > trying to provide a workaround for the problem.  If we come
> > up with a firmware based solution I'd be happy to adjust or
> > remove this later.
> 
> I am not saying we do not need the solution, I am wondering if
> we can suppress errors altogether. I am also curious why
> reset does not work as it should reinitialize the driver
> completely.
> 
> And even if you do fix the firmware majority of users will
> still need the software solution as updating BIOS is not
> something that happens often.
> 
> Thanks.

I do not know what can kernel do when it receive invalid PS/2 
data from i8042 bus. If bogus packets are total random we can 
just try to ignore them and try be not out-of-sync. No idea how 
working solution it would be for new XPS model. Looks like for 
Latitude Exx40 models with their problems it is working...

Of course correct way is to fix firmware or BIOS or which part of 
HW is buggy. Ideally distribute that firmware fix to users. I 
heard that synaptics touchpad supports something like on-the-fly 
firmware load (without flashing it) which will be active until 
notebook shutdown.

-- 
Pali Rohár
pali.rohar@gmail.com

[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 198 bytes --]

  reply	other threads:[~2015-02-20 19:24 UTC|newest]

Thread overview: 37+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-02-19  1:43 [PATCH] Add a quirk for the Dell XPS 13 (2015) when in PS/2 mode Mario Limonciello
2015-02-19 17:16 ` Dmitry Torokhov
2015-02-19 18:16   ` Mario Limonciello
2015-02-20 18:47     ` Dmitry Torokhov
2015-02-20 19:24       ` Pali Rohár [this message]
2015-02-20 19:56         ` Mario Limonciello
2015-02-20 20:41           ` Pali Rohár
2015-02-20 21:21             ` Mario Limonciello
2015-02-20 21:31               ` Benjamin Tissoires
2015-02-20 21:40                 ` Mario Limonciello
2015-02-20 21:46                   ` Pali Rohár
2015-02-20 21:54                     ` Mario Limonciello
2015-02-22 16:55               ` Pali Rohár
2015-02-23 23:31                 ` Mario Limonciello
2015-02-24  0:01                   ` Pali Rohár
2015-02-25 18:16                     ` [SUSPECT SPAM] " Mario Limonciello
2015-02-24  0:51                 ` Dmitry Torokhov
2015-02-25 18:26                   ` Mario Limonciello
2015-02-25 18:48             ` Mario Limonciello
2015-02-25 20:45               ` Pali Rohár
2015-03-14 19:17                 ` Benjamin Tissoires
2015-03-16 14:29                   ` Mario Limonciello
2015-03-16 14:40                     ` Benjamin Tissoires
2015-03-16 17:10                     ` Jason Ekstrand
2015-03-16 18:50                       ` Mario Limonciello
2015-03-16 20:42                         ` Jason Ekstrand
2015-03-16 20:50                           ` Mario Limonciello
2015-03-16 20:57                             ` Jason Ekstrand
2015-03-16 21:07                               ` Benjamin Tissoires
2015-03-17  0:45                                 ` Mario Limonciello
2015-04-10 22:39                 ` Pali Rohár
2015-04-10 23:07                   ` Mario Limonciello
2015-04-10 23:14                     ` Pali Rohár
2015-04-10 23:32                       ` Mario Limonciello
2015-04-11  2:19                         ` Ben Gamari
2015-04-13 18:55                           ` Mario Limonciello
2015-03-21 17:21           ` Ben Gamari

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=201502202024.20741@pali \
    --to=pali.rohar@gmail.com \
    --cc=dmitry.torokhov@gmail.com \
    --cc=linux-input@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mario_limonciello@dell.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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).