From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andy Shevchenko Subject: Re: [PATCH] i2c: designware: Do nothing in system suspend/resume when RT suspended Date: Tue, 25 Apr 2017 14:36:04 +0300 Message-ID: <1493120164.24567.194.camel@linux.intel.com> References: <20170330120444.12499-1-jarkko.nikula@linux.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit Return-path: Received: from mga02.intel.com ([134.134.136.20]:25191 "EHLO mga02.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1946957AbdDYLgJ (ORCPT ); Tue, 25 Apr 2017 07:36:09 -0400 In-Reply-To: Sender: linux-pm-owner@vger.kernel.org List-Id: linux-pm@vger.kernel.org To: Ulf Hansson , Jarkko Nikula Cc: "linux-i2c@vger.kernel.org" , "linux-pm@vger.kernel.org" , Wolfram Sang , "Rafael J . Wysocki" , John Stultz , Mika Westerberg On Tue, 2017-04-25 at 13:12 +0200, Ulf Hansson wrote: > On 25 April 2017 at 13:08, Jarkko Nikula m> wrote: > > On 04/25/2017 12:24 PM, Ulf Hansson wrote: > > > To me, the proper solution is to use the > > > pm_runtime_force_suspend|resume() helpers to deal with system > > > suspend/resume. However I understand that the behavior of the ACPI > > > PM > > > domain currently prevents us from doing this. That said, perhaps > > > we > > > should instead try to make the ACPI PM domain to better > > > collaborate > > > with drivers using pm_runtime_force_suspend|resume()? I have been > > > investigating that and started to cook some patches, although I > > > have > > > not yet been able to post something. If you think it could make > > > sense, > > > I can pick it up. > > > > > > > That's a good idea. I didn't think about it at all. > > Okay, thanks! I will continue to look into this and try to submit > something within a reasonable time frame, keep you posted. But please keep in mind that ACPI PM domain for BayTrail / CherryTrail is used by several IPs including most nasty GPDMA case. If something breaks that it would be show stopper. -- Andy Shevchenko Intel Finland Oy