From: "Rafael J. Wysocki" <rjw@rjwysocki.net>
To: Dmitry Torokhov <dmitry.torokhov@gmail.com>
Cc: Ulf Hansson <ulf.hansson@linaro.org>,
Len Brown <len.brown@intel.com>, Pavel Machek <pavel@ucw.cz>,
Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
linux-pm@vger.kernel.org, devicetree@vger.kernel.org,
Kevin Hilman <khilman@linaro.org>,
Russell King <linux@arm.linux.org.uk>,
Philipp Zabel <philipp.zabel@gmail.com>,
Geert Uytterhoeven <geert+renesas@glider.be>,
Wolfram Sang <wsa@the-dreams.de>,
Stephen Boyd <sboyd@codeaurora.org>,
Linus Walleij <linus.walleij@linaro.org>,
Daniel Lezcano <daniel.lezcano@linaro.org>,
Magnus Damm <magnus.damm@gmail.com>,
Tomasz Figa <tomasz.figa@gmail.com>,
Chris Ball <chris@printf.net>,
linux-acpi@vger.kernel.org, Simon Horman <horms@verge.net.au>,
Alan Stern <stern@rowland.harvard.edu>,
Mark Brown <broonie@kernel.org>, Ben Dooks <ben-linux@fluff.org>,
Kukjin Kim <kgene.kim@samsung.com>,
linux-arm-kernel@lists.infradead.org
Subject: Re: [v4,1/9] ACPI / PM: Let acpi_dev_pm_detach() return an error code
Date: Sun, 14 Sep 2014 18:38:58 +0200 [thread overview]
Message-ID: <6318611.DnX9cEgMjk@vostro.rjw.lan> (raw)
In-Reply-To: <20140912210553.GA27924@core.coreip.homeip.net>
On Friday, September 12, 2014 02:05:53 PM Dmitry Torokhov wrote:
> Hi Ulf,
>
> On Tue, Sep 09, 2014 at 01:36:02PM +0200, Ulf Hansson wrote:
> > To give callers the option of acting on a errors while removing the
> > pm_domain ops for the device in the ACPI PM domain, let
> > acpi_dev_pm_detach() return an int to provide the error code.
>
> So how would callers handle the errors? As far as I can see
> acpi_dev_pm_detach() is called from ->remove() and ->shutdown() methods, where
> there is no meaningful strategy to handle errors as you are past the point of
> no return and you keep on tearing down the device.
This is specifically for what patch [3/9] is doing AFAICS.
The existing callers don't need to worry about this.
Rafael
WARNING: multiple messages have this Message-ID (diff)
From: rjw@rjwysocki.net (Rafael J. Wysocki)
To: linux-arm-kernel@lists.infradead.org
Subject: [v4, 1/9] ACPI / PM: Let acpi_dev_pm_detach() return an error code
Date: Sun, 14 Sep 2014 18:38:58 +0200 [thread overview]
Message-ID: <6318611.DnX9cEgMjk@vostro.rjw.lan> (raw)
In-Reply-To: <20140912210553.GA27924@core.coreip.homeip.net>
On Friday, September 12, 2014 02:05:53 PM Dmitry Torokhov wrote:
> Hi Ulf,
>
> On Tue, Sep 09, 2014 at 01:36:02PM +0200, Ulf Hansson wrote:
> > To give callers the option of acting on a errors while removing the
> > pm_domain ops for the device in the ACPI PM domain, let
> > acpi_dev_pm_detach() return an int to provide the error code.
>
> So how would callers handle the errors? As far as I can see
> acpi_dev_pm_detach() is called from ->remove() and ->shutdown() methods, where
> there is no meaningful strategy to handle errors as you are past the point of
> no return and you keep on tearing down the device.
This is specifically for what patch [3/9] is doing AFAICS.
The existing callers don't need to worry about this.
Rafael
next prev parent reply other threads:[~2014-09-14 16:19 UTC|newest]
Thread overview: 96+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-09-04 13:52 [PATCH v3 0/9] PM / Domains: Generic OF-based support Ulf Hansson
2014-09-04 13:52 ` Ulf Hansson
2014-09-04 13:52 ` [PATCH v3 1/9] ACPI / PM: Let acpi_dev_pm_detach() return an error code Ulf Hansson
2014-09-04 13:52 ` Ulf Hansson
2014-08-23 22:45 ` Pavel Machek
2014-08-23 22:45 ` Pavel Machek
2014-09-07 22:12 ` Rafael J. Wysocki
2014-09-07 22:12 ` Rafael J. Wysocki
2014-09-04 13:52 ` [PATCH v3 2/9] PM / Domains: Add generic OF-based PM domain look-up Ulf Hansson
2014-09-04 13:52 ` Ulf Hansson
2014-09-07 22:13 ` Rafael J. Wysocki
2014-09-07 22:13 ` Rafael J. Wysocki
2014-09-08 7:26 ` Ulf Hansson
2014-09-08 7:26 ` Ulf Hansson
2014-09-08 21:04 ` Rafael J. Wysocki
2014-09-08 21:04 ` Rafael J. Wysocki
2014-09-08 21:08 ` Rafael J. Wysocki
2014-09-08 21:08 ` Rafael J. Wysocki
2014-09-08 20:53 ` Geert Uytterhoeven
2014-09-08 20:53 ` Geert Uytterhoeven
[not found] ` <CAMuHMdURbp8a0Ps2sZUZn7UHcWXZvRgS-4ubk1r=wGgXZ1PHmA-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2014-09-08 21:26 ` Rafael J. Wysocki
2014-09-08 21:26 ` Rafael J. Wysocki
2014-09-09 11:40 ` Mark Brown
2014-09-09 11:40 ` Mark Brown
2014-09-09 13:45 ` Rafael J. Wysocki
2014-09-09 13:45 ` Rafael J. Wysocki
2014-09-08 20:53 ` Tomasz Figa
2014-09-08 20:53 ` Tomasz Figa
2014-09-09 7:04 ` Ulf Hansson
2014-09-09 7:04 ` Ulf Hansson
2014-09-04 13:52 ` [PATCH v3 3/9] PM / Domains: Add APIs to attach/detach a PM domain for a device Ulf Hansson
2014-09-04 13:52 ` Ulf Hansson
2014-09-08 22:11 ` Rafael J. Wysocki
2014-09-08 22:11 ` Rafael J. Wysocki
2014-09-04 13:52 ` [PATCH v3 4/9] drivercore / platform: Convert to dev_pm_domain_attach|detach() Ulf Hansson
2014-09-04 13:52 ` Ulf Hansson
2014-09-04 13:52 ` [PATCH v3 5/9] i2c: core: " Ulf Hansson
2014-09-04 13:52 ` Ulf Hansson
2014-09-04 13:52 ` [PATCH v3 6/9] mmc: sdio: " Ulf Hansson
2014-09-04 13:52 ` Ulf Hansson
2014-09-04 13:52 ` [PATCH v3 7/9] spi: core: " Ulf Hansson
2014-09-04 13:52 ` Ulf Hansson
2014-09-04 13:52 ` [PATCH v3 8/9] amba: Add support for attach/detach of PM domains Ulf Hansson
2014-09-04 13:52 ` Ulf Hansson
2014-09-04 13:52 ` [PATCH v3 9/9] ARM: exynos: Move to generic PM domain DT bindings Ulf Hansson
2014-09-04 13:52 ` Ulf Hansson
2014-09-05 15:39 ` [PATCH v3 0/9] PM / Domains: Generic OF-based support Kevin Hilman
2014-09-05 15:39 ` Kevin Hilman
2014-09-09 11:36 ` [PATCH v4 " Ulf Hansson
2014-09-09 11:36 ` Ulf Hansson
2014-09-09 11:36 ` [PATCH v4 1/9] ACPI / PM: Let acpi_dev_pm_detach() return an error code Ulf Hansson
2014-09-09 11:36 ` Ulf Hansson
2014-09-12 21:05 ` [v4,1/9] " Dmitry Torokhov
2014-09-12 21:05 ` [v4, 1/9] " Dmitry Torokhov
2014-09-14 16:38 ` Rafael J. Wysocki [this message]
2014-09-14 16:38 ` Rafael J. Wysocki
2014-09-15 16:53 ` [v4,1/9] " Dmitry Torokhov
2014-09-15 16:53 ` [v4, 1/9] " Dmitry Torokhov
2014-09-15 23:36 ` [v4,1/9] " Rafael J. Wysocki
2014-09-15 23:36 ` [v4, 1/9] " Rafael J. Wysocki
2014-09-17 18:25 ` [v4,1/9] " Ulf Hansson
2014-09-17 18:25 ` [v4, 1/9] " Ulf Hansson
2014-09-17 20:10 ` [v4,1/9] " Dmitry Torokhov
2014-09-17 20:10 ` [v4, 1/9] " Dmitry Torokhov
2014-09-17 23:20 ` [v4,1/9] " Ulf Hansson
2014-09-17 23:20 ` [v4, 1/9] " Ulf Hansson
2014-09-17 23:43 ` [v4,1/9] " Dmitry Torokhov
2014-09-17 23:43 ` [v4, 1/9] " Dmitry Torokhov
[not found] ` <20140917234331.GA28771-WlK9ik9hQGAhIp7JRqBPierSzoNAToWh@public.gmane.org>
2014-09-18 0:35 ` [v4,1/9] " Ulf Hansson
2014-09-18 0:35 ` [v4, 1/9] " Ulf Hansson
2014-09-18 23:13 ` [v4,1/9] " Rafael J. Wysocki
2014-09-18 23:13 ` [v4, 1/9] " Rafael J. Wysocki
2014-09-09 11:36 ` [PATCH v4 2/9] PM / Domains: Add generic OF-based PM domain look-up Ulf Hansson
2014-09-09 11:36 ` Ulf Hansson
2014-09-09 11:36 ` [PATCH v4 3/9] PM / Domains: Add APIs to attach/detach a PM domain for a device Ulf Hansson
2014-09-09 11:36 ` Ulf Hansson
2014-09-09 11:36 ` [PATCH v4 4/9] drivercore / platform: Convert to dev_pm_domain_attach|detach() Ulf Hansson
2014-09-09 11:36 ` Ulf Hansson
2014-09-09 13:46 ` Rafael J. Wysocki
2014-09-09 13:46 ` Rafael J. Wysocki
2014-09-09 11:36 ` [PATCH v4 5/9] i2c: core: " Ulf Hansson
2014-09-09 11:36 ` Ulf Hansson
2014-09-09 11:36 ` [PATCH v4 6/9] mmc: sdio: " Ulf Hansson
2014-09-09 11:36 ` Ulf Hansson
2014-09-09 11:36 ` [PATCH v4 7/9] spi: core: " Ulf Hansson
2014-09-09 11:36 ` Ulf Hansson
2014-09-09 11:36 ` [PATCH v4 8/9] amba: Add support for attach/detach of PM domains Ulf Hansson
2014-09-09 11:36 ` Ulf Hansson
2014-09-09 11:36 ` [PATCH v4 9/9] ARM: exynos: Move to generic PM domain DT bindings Ulf Hansson
2014-09-09 11:36 ` Ulf Hansson
2014-09-09 11:43 ` [PATCH v4 0/9] PM / Domains: Generic OF-based support Tomasz Figa
2014-09-09 11:43 ` Tomasz Figa
2014-09-09 11:54 ` Mark Brown
2014-09-09 11:54 ` Mark Brown
2014-09-09 12:45 ` Ulf Hansson
2014-09-09 12:45 ` Ulf Hansson
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=6318611.DnX9cEgMjk@vostro.rjw.lan \
--to=rjw@rjwysocki.net \
--cc=ben-linux@fluff.org \
--cc=broonie@kernel.org \
--cc=chris@printf.net \
--cc=daniel.lezcano@linaro.org \
--cc=devicetree@vger.kernel.org \
--cc=dmitry.torokhov@gmail.com \
--cc=geert+renesas@glider.be \
--cc=gregkh@linuxfoundation.org \
--cc=horms@verge.net.au \
--cc=kgene.kim@samsung.com \
--cc=khilman@linaro.org \
--cc=len.brown@intel.com \
--cc=linus.walleij@linaro.org \
--cc=linux-acpi@vger.kernel.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-pm@vger.kernel.org \
--cc=linux@arm.linux.org.uk \
--cc=magnus.damm@gmail.com \
--cc=pavel@ucw.cz \
--cc=philipp.zabel@gmail.com \
--cc=sboyd@codeaurora.org \
--cc=stern@rowland.harvard.edu \
--cc=tomasz.figa@gmail.com \
--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.