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
next prev parent 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.