linux-kernel - Re: [PATCH v2 1/5] arm64: dts: qcom: sc7280: Fix gmu unit address

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: <01aac7b1-88e9-32e3-9b3c-625a2b03bdce@quicinc.com>
Date:   Tue, 1 Feb 2022 12:57:30 +0530
From:   Akhil P Oommen <quic_akhilpo@...cinc.com>
To:     Douglas Anderson <dianders@...omium.org>,
        Bjorn Andersson <bjorn.andersson@...aro.org>
CC:     <konrad.dybcio@...ainline.org>, <swboyd@...omium.org>,
        <kgodara@...eaurora.org>, <mka@...omium.org>,
        <sibis@...eaurora.org>, <pmaliset@...eaurora.org>,
        <quic_rjendra@...cinc.com>, Andy Gross <agross@...nel.org>,
        Rob Herring <robh+dt@...nel.org>, <devicetree@...r.kernel.org>,
        <linux-arm-msm@...r.kernel.org>, <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH v2 1/5] arm64: dts: qcom: sc7280: Fix gmu unit address

On 1/26/2022 4:14 AM, Douglas Anderson wrote:
> When processing sc7280 device trees, I can see:
>
>    Warning (simple_bus_reg): /soc@...mu@...9000:
>      simple-bus unit address format error, expected "3d6a000"
>
> There's a clear typo in the node name. Fix it.
>
> Fixes: 96c471970b7b ("arm64: dts: qcom: sc7280: Add gpu support")
> Signed-off-by: Douglas Anderson <dianders@...omium.org>
> Reviewed-by: Stephen Boyd <swboyd@...omium.org>
> Reviewed-by: Matthias Kaehlcke <mka@...omium.org>
> ---
>
> (no changes since v1)
>
>   arch/arm64/boot/dts/qcom/sc7280.dtsi | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm64/boot/dts/qcom/sc7280.dtsi b/arch/arm64/boot/dts/qcom/sc7280.dtsi
> index 937c2e0e93eb..eab7a8505053 100644
> --- a/arch/arm64/boot/dts/qcom/sc7280.dtsi
> +++ b/arch/arm64/boot/dts/qcom/sc7280.dtsi
> @@ -1790,7 +1790,7 @@ opp-550000000 {
>   			};
>   		};
>   
> -		gmu: gmu@...9000 {
> +		gmu: gmu@...a000 {
>   			compatible="qcom,adreno-gmu-635.0", "qcom,adreno-gmu";
>   			reg = <0 0x03d6a000 0 0x34000>,
>   				<0 0x3de0000 0 0x10000>,

My bad! Thanks for the fix.

fwiw, Reviewed-by: Akhil P Oommen <quic_akhilpo@...cinc.com>

-Akhil.