From: Jarkko Nikula <jarkko.nikula@linux.intel.com>
To: "Shevchenko, Andriy" <andriy.shevchenko@intel.com>,
"rjw@rjwysocki.net" <rjw@rjwysocki.net>
Cc: "Koul, Vinod" <vinod.koul@intel.com>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
"tglx@linutronix.de" <tglx@linutronix.de>,
"dmaengine@vger.kernel.org" <dmaengine@vger.kernel.org>,
"mika.westerberg@linux.intel.com"
<mika.westerberg@linux.intel.com>,
"gregkh@linuxfoundation.org" <gregkh@linuxfoundation.org>,
"linux-acpi@vger.kernel.org" <linux-acpi@vger.kernel.org>
Subject: Re: [PATCH v2 2/7] ACPI / LPSS: allow to use specific PM domain during ->probe()
Date: Fri, 4 Dec 2015 15:04:07 +0200 [thread overview]
Message-ID: <56618F47.9080205@linux.intel.com> (raw)
In-Reply-To: <1449171008.15393.158.camel@intel.com>
On 12/03/2015 09:29 PM, Shevchenko, Andriy wrote:
> I briefly checked this for DMA issue. It will not help anyhow, so we
> *have to* move a power domain assignment to the BIND stage.
>
> For I2C and rest LPSS devices this might help (though didn't look
> deeply). My understanding that we assign those callbacks in the LPSS
> custom PM domain and call them explicitly in acpi_lpss.c.
>
> The code will be the same as we are using now to bring device from
> runtime suspend resume. This means whenever we call probe for e.g. I2C
> we end up in a sequence similar to:
> pm_runtime_resume(I2C);
> ->probe(I2C);
> pm_runtime_suspend(I2C);
>
> I will try to mock up this and check if it will work, though have no
> idea what to do if I2C during probe calls pm_runtime_forbid().
>
> Jarkko, what do you think?
>
I suppose device core will handle it. If the runtime PM is forbidden or
not initialized at all the device shouldn't idle.
--
Jarkko
next prev parent reply other threads:[~2015-12-04 13:06 UTC|newest]
Thread overview: 31+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-11-26 15:19 [PATCH v2 0/7] ACPI / LPSS: fix system hangup on BYT/BSW/CHT Andy Shevchenko
2015-11-26 15:19 ` [PATCH v2 1/7] device core: add BUS_NOTIFY_BIND_DRIVER_ERROR notification Andy Shevchenko
2015-11-26 23:09 ` Rafael J. Wysocki
2015-11-27 9:46 ` Andy Shevchenko
2015-11-27 9:46 ` Andy Shevchenko
2015-11-26 15:19 ` [PATCH v2 2/7] ACPI / LPSS: allow to use specific PM domain during ->probe() Andy Shevchenko
2015-11-26 16:30 ` Jarkko Nikula
2015-11-26 16:45 ` Andy Shevchenko
2015-11-26 16:45 ` Andy Shevchenko
2015-11-26 23:15 ` Rafael J. Wysocki
2015-11-27 9:56 ` Andy Shevchenko
2015-11-27 9:56 ` Andy Shevchenko
2015-12-03 19:29 ` Shevchenko, Andriy
2015-12-03 19:29 ` Shevchenko, Andriy
2015-12-04 13:04 ` Jarkko Nikula [this message]
2015-11-27 7:05 ` Jarkko Nikula
2015-11-27 10:01 ` Andy Shevchenko
2015-11-26 15:19 ` [PATCH v2 3/7] ACPI / LPSS: do delay for all LPSS devices when D3->D0 Andy Shevchenko
2015-11-26 15:19 ` [PATCH v2 4/7] ACPI / LPSS: override power state for LPSS DMA device Andy Shevchenko
2015-11-26 15:19 ` [PATCH v2 5/7] dmaengine: dw: platform: power on device on shutdown Andy Shevchenko
2015-11-26 17:01 ` Vinod Koul
2015-11-26 17:24 ` Andy Shevchenko
2015-11-26 17:24 ` Andy Shevchenko
2015-11-26 17:41 ` Vinod Koul
2015-11-26 17:41 ` Vinod Koul
2015-11-26 17:58 ` Andy Shevchenko
2015-11-26 17:58 ` Andy Shevchenko
2015-11-26 18:11 ` Shevchenko, Andriy
2015-11-26 18:11 ` Shevchenko, Andriy
2015-11-26 15:19 ` [PATCH v2 6/7] dmaengine: dw: return immediately from IRQ when DMA isn't in use Andy Shevchenko
2015-11-26 15:19 ` [PATCH v2 7/7] Revert "dmaengine: dw: platform: provide platform data for Intel" 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=56618F47.9080205@linux.intel.com \
--to=jarkko.nikula@linux.intel.com \
--cc=andriy.shevchenko@intel.com \
--cc=dmaengine@vger.kernel.org \
--cc=gregkh@linuxfoundation.org \
--cc=linux-acpi@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=mika.westerberg@linux.intel.com \
--cc=rjw@rjwysocki.net \
--cc=tglx@linutronix.de \
--cc=vinod.koul@intel.com \
/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.