From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jarkko Nikula Subject: Re: [PATCH 1/9] i2c: designware: Fix system suspend Date: Tue, 27 Jun 2017 10:55:12 +0300 Message-ID: References: <1498072888-14782-1-git-send-email-ulf.hansson@linaro.org> <1498072888-14782-2-git-send-email-ulf.hansson@linaro.org> <20170622104933.GY629@lahna.fi.intel.com> <5540bae1-b1bf-0e15-1fa0-977076fa0753@linux.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <5540bae1-b1bf-0e15-1fa0-977076fa0753@linux.intel.com> Sender: linux-acpi-owner@vger.kernel.org To: Mika Westerberg , "Rafael J. Wysocki" Cc: Ulf Hansson , Wolfram Sang , "Rafael J . Wysocki" , Len Brown , ACPI Devel Maling List , Linux PM , Kevin Hilman , Andy Shevchenko , Jisheng Zhang , John Stultz , Guodong Xu , Sumit Semwal , Haojian Zhuang , "linux-arm-kernel@lists.infradead.org" , linux-i2c , stable@vger.linux.org List-Id: linux-i2c@vger.kernel.org On 06/22/2017 02:16 PM, Jarkko Nikula wrote: > On 06/22/2017 01:49 PM, Mika Westerberg wrote: >> On Thu, Jun 22, 2017 at 01:31:51AM +0200, Rafael J. Wysocki wrote: >>> On Wed, Jun 21, 2017 at 9:21 PM, Ulf Hansson >>> wrote: >> Thanks Ulf for taking care of this! >> > Indeed! > >> I tested this series on Dell XPS 9350 which has touch panel connected to >> I2C and suspend/resume still works fine and I can see the controller >> going to D3 when the touch panel is idle. >> >> I can perform more comprehensive testing next week. >> > Unfortunately I'm seeing interrupt storm during suspend/resume on > platform using PM domain from drivers/acpi/acpi_lpss.c straight after > this patch. Maybe some timing related as I see it only if I have debug > messages on (i2c_designware_core.dyndbg=+p). But it occurs only after > this patch. > Sorry the noise, this was bogus. That platform is doing this interrupt storm randomly and it occurs also without the patch. -- Jarkko