DomainKey-Signature: s=smtpout; d=dell.com; c=nofws; q=dns;
  h=X-LoopCount0:X-IronPort-AV:Message-ID:Date:From:
   Organization:User-Agent:MIME-Version:To:CC:Subject:
   References:In-Reply-To:Content-Type:
   Content-Transfer-Encoding;
  b=gT0A8lvu+Hl/dECHskoJw6VyDI32qAgfqxSil5OWnC5s+t9id15B61oZ
   +JmrTtNtxR9y3BBQDVzRsy3ATlrU9Otd0cCp3R6YhM2C6YCYF3AmfXK2y
   kpUw9gqwCVwhSC7M2VTyLfc/iex4nZM57iTrFvMsDqHTTE8s3x8LE1qfH
   I=;
Message-ID: <54EE1917.4020805@dell.com>
Date: Wed, 25 Feb 2015 12:48:55 -0600
From: Mario Limonciello <mario_limonciello@dell.com>
Organization: Dell Inc.
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.4.0
MIME-Version: 1.0
To: =?UTF-8?B?UGFsaSBSb2jDoXI=?= <pali.rohar@gmail.com>
CC: Dmitry Torokhov <dmitry.torokhov@gmail.com>,
        LKML <linux-kernel@vger.kernel.org>,
        "linux-input@vger.kernel.org" <linux-input@vger.kernel.org>,
        Rob <robr.bensson@gmail.com>
Subject: Re: [PATCH] Add a quirk for the Dell XPS 13 (2015) when in PS/2 mode.
References: <1424310180-2512-1-git-send-email-mario_limonciello@dell.com> <201502202024.20741@pali> <54E79167.6070701@dell.com> <201502202141.16017@pali>
In-Reply-To: <201502202141.16017@pali>
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Transfer-Encoding: 8bit
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1372
Lines: 24


On 02/20/2015 02:41 PM, Pali Rohár wrote:
> On Friday 20 February 2015 20:56:23 Mario Limonciello wrote:
>
> resetafter=0 means to never reset (even if driver receive e.g
> thousand invalid packets). I think this is very dangerous if
> there will be other bugs either in linux driver or some other HW
> problems.
>
> For ALPS issue I added resetafter = pktsize * 2 (Allow 2 invalid
> packets without resetting device). Cannot you find something
> similar for synaptics touchpads on XPS? (pktsize for ALPS is 6,
> no idea how big are synaptics packets).
>
Pali,

I've done some experimentation with increasing the size to resetafter to up to pktsize * 4.  It will decrease the number of occurrences of this problem, but the problem still occurs eventually.  pktsize for synaptics is 6 as well.  Would you recommend to continue to go higher than that?  Since out_of_sync_cnt is reset when a full packet gets received, some arbitrarily high number should likely fix it to.

That being said, if you try to more closely follow what Windows does for the mouse, it's not issuing a reconnect no matter how much bad data is received.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/