All of lore.kernel.org
 help / color / mirror / Atom feed
From: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
To: Ulf Hansson <ulf.hansson@linaro.org>,
	Jarkko Nikula <jarkko.nikula@linux.intel.com>
Cc: "linux-i2c@vger.kernel.org" <linux-i2c@vger.kernel.org>,
	"linux-pm@vger.kernel.org" <linux-pm@vger.kernel.org>,
	Wolfram Sang <wsa@the-dreams.de>,
	"Rafael J . Wysocki" <rjw@rjwysocki.net>,
	John Stultz <john.stultz@linaro.org>,
	Mika Westerberg <mika.westerberg@linux.intel.com>
Subject: Re: [PATCH] i2c: designware: Do nothing in system suspend/resume when RT suspended
Date: Tue, 25 Apr 2017 14:36:04 +0300	[thread overview]
Message-ID: <1493120164.24567.194.camel@linux.intel.com> (raw)
In-Reply-To: <CAPDyKFqg3TDMwBO7a-1zuKYeLjE-BQ4RPmtXFKzA0s2a50jQqg@mail.gmail.com>

On Tue, 2017-04-25 at 13:12 +0200, Ulf Hansson wrote:
> On 25 April 2017 at 13:08, Jarkko Nikula <jarkko.nikula@linux.intel.co
> 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 <andriy.shevchenko@linux.intel.com>
Intel Finland Oy

  reply	other threads:[~2017-04-25 11:36 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-03-30 12:04 [PATCH] i2c: designware: Do nothing in system suspend/resume when RT suspended Jarkko Nikula
2017-04-04  2:55 ` John Stultz
2017-04-19 18:59 ` Wolfram Sang
2017-04-19 20:24   ` Rafael J. Wysocki
2017-04-19 20:24 ` Rafael J. Wysocki
2017-04-20  7:25   ` Jarkko Nikula
2017-04-20 10:33     ` Rafael J. Wysocki
2017-04-24 14:27       ` Jarkko Nikula
2017-04-25  9:24 ` Ulf Hansson
2017-04-25 11:08   ` Jarkko Nikula
2017-04-25 11:12     ` Ulf Hansson
2017-04-25 11:36       ` Andy Shevchenko [this message]
2017-04-25 12:04         ` Ulf Hansson
2017-06-16 13:49       ` Ulf Hansson
2017-06-20 13:07         ` Ulf Hansson
2017-06-20 16:08           ` Andy Shevchenko
2017-06-21 14:40             ` Ulf Hansson
2017-06-27 14:34               ` Andy Shevchenko

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1493120164.24567.194.camel@linux.intel.com \
    --to=andriy.shevchenko@linux.intel.com \
    --cc=jarkko.nikula@linux.intel.com \
    --cc=john.stultz@linaro.org \
    --cc=linux-i2c@vger.kernel.org \
    --cc=linux-pm@vger.kernel.org \
    --cc=mika.westerberg@linux.intel.com \
    --cc=rjw@rjwysocki.net \
    --cc=ulf.hansson@linaro.org \
    --cc=wsa@the-dreams.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.