[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20240320074431.6yzao3jlyaxuii7c@dhruva>
Date: Wed, 20 Mar 2024 13:14:31 +0530
From: Dhruva Gole <d-gole@...com>
To: Thomas Richard <thomas.richard@...tlin.com>
CC: Linus Walleij <linus.walleij@...aro.org>,
Bartosz Golaszewski
<brgl@...ev.pl>,
Andy Shevchenko <andy@...nel.org>, Tony Lindgren
<tony@...mide.com>,
Haojian Zhuang <haojian.zhuang@...aro.org>,
Vignesh R
<vigneshr@...com>, Aaro Koskinen <aaro.koskinen@....fi>,
Janusz Krzysztofik
<jmkrzyszt@...il.com>,
Andi Shyti <andi.shyti@...nel.org>, Peter Rosin
<peda@...ntia.se>,
Vinod Koul <vkoul@...nel.org>,
Kishon Vijay Abraham I
<kishon@...nel.org>,
Philipp Zabel <p.zabel@...gutronix.de>,
Lorenzo
Pieralisi <lpieralisi@...nel.org>,
Krzysztof WilczyĆski
<kw@...ux.com>,
Rob Herring <robh@...nel.org>, Bjorn Helgaas
<bhelgaas@...gle.com>,
<linux-gpio@...r.kernel.org>, <linux-kernel@...r.kernel.org>,
<linux-arm-kernel@...ts.infradead.org>, <linux-omap@...r.kernel.org>,
<linux-i2c@...r.kernel.org>, <linux-phy@...ts.infradead.org>,
<linux-pci@...r.kernel.org>, <gregory.clement@...tlin.com>,
<theo.lebrun@...tlin.com>, <thomas.petazzoni@...tlin.com>,
<u-kumar1@...com>, Andy Shevchenko
<andy.shevchenko@...il.com>
Subject: Re: [PATCH v4 02/18] pinctrl: pinctrl-single: move
suspend()/resume() callbacks to noirq
Hi,
On Mar 04, 2024 at 16:35:45 +0100, Thomas Richard wrote:
> The goal is to extend the active period of pinctrl.
> Some devices may need active pinctrl after suspend() and/or before
> resume().
> So move suspend()/resume() to suspend_noirq()/resume_noirq() in order to
> have active pinctrl until suspend_noirq() (included), and from
> resume_noirq() (included).
>
> The deprecated API has been removed to use the new one (dev_pm_ops struct).
>
> No need to check the pointer returned by dev_get_drvdata(), as
> platform_set_drvdata() is called during the probe.
>
> Reviewed-by: Linus Walleij <linus.walleij@...aro.org>
> Reviewed-by: Andy Shevchenko <andy.shevchenko@...il.com>
> Signed-off-by: Thomas Richard <thomas.richard@...tlin.com>
> ---
I was planning to do this but didn't see particular benefit to it. Do
you see the benefit on your specific device? Can you help me understand
how? Not against the patch, just curious.
Reviewed-by: Dhruva Gole <d-gole@...com>
--
Best regards,
Dhruva
Powered by blists - more mailing lists