lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
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

Powered by Openwall GNU/*/Linux Powered by OpenVZ