From mboxrd@z Thu Jan 1 00:00:00 1970 From: Mario Limonciello Subject: Re: [PATCH] Add a quirk for the Dell XPS 13 (2015) when in PS/2 mode. Date: Fri, 10 Apr 2015 18:07:03 -0500 Message-ID: <55285797.1080303@dell.com> References: <1424310180-2512-1-git-send-email-mario_limonciello@dell.com> <54EE1917.4020805@dell.com> <201502252145.22577@pali> <201504110039.40761@pali> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: QUOTED-PRINTABLE Return-path: Received: from ausc60pc101.us.dell.com ([143.166.85.206]:18067 "EHLO ausc60pc101.us.dell.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932193AbbDJXHF (ORCPT ); Fri, 10 Apr 2015 19:07:05 -0400 In-Reply-To: <201504110039.40761@pali> Sender: linux-input-owner@vger.kernel.org List-Id: linux-input@vger.kernel.org To: =?UTF-8?B?UGFsaSBSb2jDoXI=?= Cc: LKML , "linux-input@vger.kernel.org" On 04/10/2015 05:39 PM, Pali Roh=C3=A1r wrote: > On Wednesday 25 February 2015 21:45:22 Pali Roh=C3=A1r 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 g= o into the details about it publicly. I think it's better to not bother including my proposed solution, howev= er. 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 I2= C 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 op= tion acpi_osi=3D"!Windows 2013". This was required with BIOS A01 due t= o 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 comm= and 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 pr= operly 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=3D= for-next&id=3De1e62b98ebddc3234f3259019d3236f66fc667f8 and https://git.kernel.org/cgit/linux/kernel/git/tiwai/sound.git/commit/?h=3D= for-next&id=3Df3b703326541d0c1ce85f5e570f6d2b6bd4296ec 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 w= ill end up in PS2 mode with the current development focus kernels. Thanks, -- To unsubscribe from this list: send the line "unsubscribe linux-input" = in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933344AbbDJXHN (ORCPT ); Fri, 10 Apr 2015 19:07:13 -0400 Received: from ausc60pc101.us.dell.com ([143.166.85.206]:18067 "EHLO ausc60pc101.us.dell.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932193AbbDJXHF (ORCPT ); Fri, 10 Apr 2015 19:07:05 -0400 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=; X-LoopCount0: from 10.208.46.70 X-IronPort-AV: E=Sophos;i="5.11,559,1422943200"; d="scan'208";a="776558728" Message-ID: <55285797.1080303@dell.com> Date: Fri, 10 Apr 2015 18:07:03 -0500 From: Mario Limonciello 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=?= CC: LKML , "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 List-ID: X-Mailing-List: linux-kernel@vger.kernel.org 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,