linux-kernel - Re: [PATCH] arm64: dts: qcom: Fix hwlock index for SMEM for IPQ platforms

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: <64808e4c-f835-487e-a608-d00e81820f47@quicinc.com>
Date:   Mon, 4 Sep 2023 12:11:54 +0530
From:   Kathiravan Thirumoorthy <quic_kathirav@...cinc.com>
To:     Vignesh Viswanathan <quic_viswanat@...cinc.com>,
        <agross@...nel.org>, <andersson@...nel.org>,
        <konrad.dybcio@...aro.org>, <robh+dt@...nel.org>,
        <krzysztof.kozlowski+dt@...aro.org>, <conor+dt@...nel.org>,
        <linux-arm-msm@...r.kernel.org>, <devicetree@...r.kernel.org>,
        <linux-kernel@...r.kernel.org>
CC:     <quic_anusha@...cinc.com>, <quic_sjaganat@...cinc.com>,
        <quic_srichara@...cinc.com>, <quic_varada@...cinc.com>
Subject: Re: [PATCH] arm64: dts: qcom: Fix hwlock index for SMEM for IPQ
 platforms


On 9/4/2023 12:03 PM, Vignesh Viswanathan wrote:
> SMEM uses lock index 3 of the TCSR Mutex hwlock for allocations
> in SMEM region shared by the Host and FW.
>
> Fix the SMEM hwlock index to 3 for IPQ5332, IPQ6018, IPQ8074 and IPQ9574.


Fixes tag please?


>
> Signed-off-by: Vignesh Viswanathan <quic_viswanat@...cinc.com>
> ---
>   arch/arm64/boot/dts/qcom/ipq5332.dtsi | 2 +-
>   arch/arm64/boot/dts/qcom/ipq6018.dtsi | 2 +-
>   arch/arm64/boot/dts/qcom/ipq8074.dtsi | 2 +-
>   arch/arm64/boot/dts/qcom/ipq9574.dtsi | 2 +-
>   4 files changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/arch/arm64/boot/dts/qcom/ipq5332.dtsi b/arch/arm64/boot/dts/qcom/ipq5332.dtsi
> index 1abc992ede31..a47c79c3d5f0 100644
> --- a/arch/arm64/boot/dts/qcom/ipq5332.dtsi
> +++ b/arch/arm64/boot/dts/qcom/ipq5332.dtsi
> @@ -135,7 +135,7 @@ smem@...00000 {
>   			reg = <0x0 0x4a800000 0x0 0x100000>;
>   			no-map;
>   
> -			hwlocks = <&tcsr_mutex 0>;
> +			hwlocks = <&tcsr_mutex 3>;
>   		};
>   
>   		q6_region: wcnss@...00000 {
> diff --git a/arch/arm64/boot/dts/qcom/ipq6018.dtsi b/arch/arm64/boot/dts/qcom/ipq6018.dtsi
> index 56537cf8243f..9577341592c3 100644
> --- a/arch/arm64/boot/dts/qcom/ipq6018.dtsi
> +++ b/arch/arm64/boot/dts/qcom/ipq6018.dtsi
> @@ -211,7 +211,7 @@ q6_region: memory@...00000 {
>   	smem {
>   		compatible = "qcom,smem";
>   		memory-region = <&smem_region>;
> -		hwlocks = <&tcsr_mutex 0>;
> +		hwlocks = <&tcsr_mutex 3>;
>   	};
>   
>   	soc: soc@0 {
> diff --git a/arch/arm64/boot/dts/qcom/ipq8074.dtsi b/arch/arm64/boot/dts/qcom/ipq8074.dtsi
> index 00ed71936b47..92fd924bbdbe 100644
> --- a/arch/arm64/boot/dts/qcom/ipq8074.dtsi
> +++ b/arch/arm64/boot/dts/qcom/ipq8074.dtsi
> @@ -101,7 +101,7 @@ smem@...00000 {
>   			reg = <0x0 0x4ab00000 0x0 0x100000>;
>   			no-map;
>   
> -			hwlocks = <&tcsr_mutex 0>;
> +			hwlocks = <&tcsr_mutex 3>;
>   		};
>   
>   		memory@...00000 {
> diff --git a/arch/arm64/boot/dts/qcom/ipq9574.dtsi b/arch/arm64/boot/dts/qcom/ipq9574.dtsi
> index f22f7e78850d..cc0e4a2b67d2 100644
> --- a/arch/arm64/boot/dts/qcom/ipq9574.dtsi
> +++ b/arch/arm64/boot/dts/qcom/ipq9574.dtsi
> @@ -195,7 +195,7 @@ tz_region: tz@...00000 {
>   		smem@...00000 {
>   			compatible = "qcom,smem";
>   			reg = <0x0 0x4aa00000 0x0 0x100000>;
> -			hwlocks = <&tcsr_mutex 0>;
> +			hwlocks = <&tcsr_mutex 3>;
>   			no-map;
>   		};
>