linux-kernel - Re: [PATCH V5 1/5] PM: domains: Allow devices attached to genpd to be managed by HW

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 for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20240318060312.z5pk4roz6uqhwosc@dhruva>
Date: Mon, 18 Mar 2024 11:33:12 +0530
From: Dhruva Gole <d-gole@...com>
To: Jagadeesh Kona <quic_jkona@...cinc.com>
CC: "Rafael J . Wysocki" <rafael@...nel.org>,
        Kevin Hilman
	<khilman@...nel.org>,
        Ulf Hansson <ulf.hansson@...aro.org>, Pavel Machek
	<pavel@....cz>,
        Len Brown <len.brown@...el.com>,
        Greg Kroah-Hartman
	<gregkh@...uxfoundation.org>,
        Bjorn Andersson <andersson@...nel.org>,
        Konrad
 Dybcio <konrad.dybcio@...aro.org>,
        Michael Turquette
	<mturquette@...libre.com>,
        Andy Gross <agross@...nel.org>, Stephen Boyd
	<sboyd@...nel.org>,
        Stanimir Varbanov <stanimir.k.varbanov@...il.com>,
        Vikash
 Garodia <quic_vgarodia@...cinc.com>,
        "Bryan O'Donoghue"
	<bryan.odonoghue@...aro.org>,
        Mauro Carvalho Chehab <mchehab@...nel.org>,
        Dmitry Baryshkov <dmitry.baryshkov@...aro.org>,
        Abel Vesa
	<abel.vesa@...aro.org>, Taniya Das <quic_tdas@...cinc.com>,
        <linux-arm-msm@...r.kernel.org>, <linux-clk@...r.kernel.org>,
        <linux-kernel@...r.kernel.org>, <linux-media@...r.kernel.org>,
        <linux-pm@...r.kernel.org>, Imran Shaik <quic_imrashai@...cinc.com>,
        Ajit
 Pandey <quic_ajipan@...cinc.com>
Subject: Re: [PATCH V5 1/5] PM: domains: Allow devices attached to genpd to
 be managed by HW

On Mar 15, 2024 at 16:40:42 +0530, Jagadeesh Kona wrote:
> From: Ulf Hansson <ulf.hansson@...aro.org>
> 
> Some power-domains may be capable of relying on the HW to control the power
> for a device that's hooked up to it. Typically, for these kinds of
> configurations the consumer driver should be able to change the behavior of
> power domain at runtime, control the power domain in SW mode for certain
> configurations and handover the control to HW mode for other usecases.
> 
> To allow a consumer driver to change the behaviour of the PM domain for its
> device, let's provide a new function, dev_pm_genpd_set_hwmode(). Moreover,
> let's add a corresponding optional genpd callback, ->set_hwmode_dev(),
> which the genpd provider should implement if it can support switching
> between HW controlled mode and SW controlled mode. Similarly, add the
> dev_pm_genpd_get_hwmode() to allow consumers to read the current mode and
> its corresponding optional genpd callback, ->get_hwmode_dev(), which the
> genpd provider can also implement to synchronize the initial HW mode
> state in genpd_add_device() by reading back the mode from the hardware.
> 
> Signed-off-by: Ulf Hansson <ulf.hansson@...aro.org>
> Signed-off-by: Abel Vesa <abel.vesa@...aro.org>
> Signed-off-by: Jagadeesh Kona <quic_jkona@...cinc.com>
> Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@...aro.org>
> ---
>  drivers/pmdomain/core.c   | 64 +++++++++++++++++++++++++++++++++++++++
>  include/linux/pm_domain.h | 17 +++++++++++
>  2 files changed, 81 insertions(+)
> 

LGTM!

Reviewed-by: Dhruva Gole <d-gole@...com>

-- 
Best regards,
Dhruva