linux-kernel - Re: [PATCH] ARM: dts: add 533 MHz step to p4note GPU opp table

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]

Message-ID: <131c9bd9-b035-8a75-ca27-f9c2b3993566@canonical.com>
Date:   Mon, 31 Jan 2022 09:54:00 +0100
From:   Krzysztof Kozlowski <krzysztof.kozlowski@...onical.com>
To:     Martin Jücker <martin.juecker@...il.com>,
        Rob Herring <robh+dt@...nel.org>, devicetree@...r.kernel.org,
        linux-arm-kernel@...ts.infradead.org,
        linux-samsung-soc@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] ARM: dts: add 533 MHz step to p4note GPU opp table

On 29/01/2022 23:14, Martin Jücker wrote:
> The p4note devices support 533 MHz GPU speed, so let's make use of it.
> 

If there is going to be resend, please add prefix "exynos:" in subject.
Just like other commits (git log --oneline arch/arm/boot/dts/exynos*).

> Signed-off-by: Martin Jücker <martin.juecker@...il.com>
> ---
>  arch/arm/boot/dts/exynos4412-p4note.dtsi | 7 +++++++
>  1 file changed, 7 insertions(+)
> 
> diff --git a/arch/arm/boot/dts/exynos4412-p4note.dtsi b/arch/arm/boot/dts/exynos4412-p4note.dtsi
> index 4ae6eb3091c4..d277b2337ae0 100644
> --- a/arch/arm/boot/dts/exynos4412-p4note.dtsi
> +++ b/arch/arm/boot/dts/exynos4412-p4note.dtsi
> @@ -355,6 +355,13 @@ &gpu {
>  	status = "okay";
>  };
>  
> +&gpu_opp_table {
> +	opp-533000000 {
> +		opp-hz = /bits/ 64 <533000000>;
> +		opp-microvolt = <1075000>;
> +	};
> +};

This looks like Exynos4412 Prime. Please check also CPU frequencies and
if they match, just include exynos4412-prime.dtsi (like Odroid U3).

> +
>  &hsotg {
>  	vusb_a-supply = <&ldo12_reg>;
>  	dr_mode = "peripheral";


Best regards,
Krzysztof