linux-kernel - Re: [PATCH] pinctrl: actions: use devm_clk_get

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] [day] [month] [year] [list]

Message-ID: <b0463cf0-9fbe-4b7a-b1fc-bdfdbb6bc780@kernel.org>
Date: Tue, 24 Jun 2025 10:15:22 +0200
From: Krzysztof Kozlowski <krzk@...nel.org>
To: Qi Han <hanqi@...o.com>, linus.walleij@...aro.org, afaerber@...e.de,
 mani@...nel.org
Cc: linux-gpio@...r.kernel.org, linux-arm-kernel@...ts.infradead.org,
 linux-actions@...ts.infradead.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] pinctrl: actions: use devm_clk_get_enabled()

On 24/06/2025 08:22, Qi Han wrote:
> Use devm_clk_get_enabled() to simplify the code.
> 
> Change-Id: I0902c50e50db565381c65e8d8a7f1481e82b271a

Please run scripts/checkpatch.pl on the patches and fix reported
warnings. After that, run also 'scripts/checkpatch.pl --strict' on the
patches and (probably) fix more warnings. Some warnings can be ignored,
especially from --strict run, but the code here looks like it needs a
fix. Feel free to get in touch if the warning is not clear.

> Signed-off-by: Qi Han <hanqi@...o.com>
> ---
>  drivers/pinctrl/actions/pinctrl-owl.c | 11 ++---------
>  1 file changed, 2 insertions(+), 9 deletions(-)
> 
> diff --git a/drivers/pinctrl/actions/pinctrl-owl.c b/drivers/pinctrl/actions/pinctrl-owl.c
> index 86f3d5c69e36..a66739a27489 100644
> --- a/drivers/pinctrl/actions/pinctrl-owl.c
> +++ b/drivers/pinctrl/actions/pinctrl-owl.c
> @@ -941,18 +941,13 @@ int owl_pinctrl_probe(struct platform_device *pdev,
>  		return PTR_ERR(pctrl->base);
>  
>  	/* enable GPIO/MFP clock */
> -	pctrl->clk = devm_clk_get(&pdev->dev, NULL);
> +	pctrl->clk = devm_clk_get_enabled(&pdev->dev, NULL);
> +

Why are you introducing whitespace changes?

Are you sure that you are not introducing same bugs as other vivo
patches? Do you understand the issue with this scripting work?

Best regards,
Krzysztof