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=heXs3dpyQnE+LmuU0zZ8J8Ox1EjfeHL63yeMcvQ1W6w1iBaWS/AX2P/Q
   6xMQ8JNXaceEHt3OuTLLS2zxhxAr4cMNxTkplmprAwi9wg+WdBL2b2W71
   JVl85AnDvSG+GcFT/z8Kd8jFaIoRFvU8nsl+iH/X5Y7WT9vJClWanVOaF
   s=;
Message-ID: <55285797.1080303@dell.com>
Date: Fri, 10 Apr 2015 18:07:03 -0500
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: 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.
References: <1424310180-2512-1-git-send-email-mario_limonciello@dell.com> <54EE1917.4020805@dell.com> <201502252145.22577@pali> <201504110039.40761@pali>
In-Reply-To: <201504110039.40761@pali>
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Transfer-Encoding: 8bit
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 2113
Lines: 35


On 04/10/2015 05:39 PM, Pali Rohár wrote:
> On Wednesday 25 February 2015 21:45:22 Pali Rohár wrote:
> Hello Mario,
> have you patched synaptics driver with some resetafter parameter?
> And have some team in dell found reason for invalid packets?
>
Hi Pali,

The reason was found for the invalid packets in PS2 mode, but I can't go into the details about it publicly.

I think it's better to not bother including my proposed solution, however.  The vector of users in PS2 mode should be very small now.
Here's why:
With no kernel command line options applied the touchpad will run in I2C mode with the current kernel.  The only reason that users were being exposed to the PS2 mode was if they ran with the kernel command line option acpi_osi="!Windows 2013".  This was required with BIOS A01 due to the problem with I2S audio.  This I2S audio problem has been resolved for BIOS A02 and BIOS A03.  With no options applied on the kernel command line the system will run in I2C touchpad and HDA audio.
All new machines will be coming with A03 or later.  Users with earlier versions have been advised to upgrade to fix this among other issues.

The HDA audio will work correctly for output with 4.0.  It will work properly for input when these two patches are applied (which will land in 4.1)

https://git.kernel.org/cgit/linux/kernel/git/tiwai/sound.git/commit/?h=for-next&id=e1e62b98ebddc3234f3259019d3236f66fc667f8

and

https://git.kernel.org/cgit/linux/kernel/git/tiwai/sound.git/commit/?h=for-next&id=f3b703326541d0c1ce85f5e570f6d2b6bd4296ec


If the patch to disambiguate the value of _REV() lands in 4.1 the audio will begin to operate fully in I2S mode and the necessary support will be included in 4.1 as well.

So with all those factors I feel that it's a very low risk that users will end up in PS2 mode with the current development focus kernels.

Thanks,
--
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/