linux-kernel - Re: [PATCH 11/11] arm64: dts: qcom: msm8996-oneplus: do not enable incomplete nodes

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: <4667f133-8084-3358-f230-5e7ddd76ee0e@linaro.org>
Date:   Mon, 6 Mar 2023 14:55:53 +0100
From:   Konrad Dybcio <konrad.dybcio@...aro.org>
To:     Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>,
        Andy Gross <agross@...nel.org>,
        Bjorn Andersson <andersson@...nel.org>,
        Rob Herring <robh+dt@...nel.org>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@...aro.org>,
        Melody Olvera <quic_molvera@...cinc.com>,
        Souradeep Chowdhury <quic_schowdhu@...cinc.com>,
        Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
        linux-arm-msm@...r.kernel.org, devicetree@...r.kernel.org,
        linux-kernel@...r.kernel.org,
        Harry Austen <hpausten@...tonmail.com>
Subject: Re: [PATCH 11/11] arm64: dts: qcom: msm8996-oneplus: do not enable
 incomplete nodes



On 5.03.2023 13:59, Krzysztof Kozlowski wrote:
> status=okay should appear in final place where all required properties
> are provided, because that makes the code the easiest to read.  Move the
> status from common OnePlus DTSI to board DTS.  No functional changes.
> 
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>
> ---
+CC Harry

Are all of these firmwares actually different on 3/3T?

My educated guess would be that at least ZAP and Venus could
be shared. Could you doublecheck?

For the patch, I suppose this does make sense and it looks like we're
already doing it, so:

Reviewed-by: Konrad Dybcio <konrad.dybcio@...aro.org>

Konrad
>  .../boot/dts/qcom/msm8996-oneplus-common.dtsi   | 17 -----------------
>  arch/arm64/boot/dts/qcom/msm8996-oneplus3.dts   |  6 ++++++
>  arch/arm64/boot/dts/qcom/msm8996-oneplus3t.dts  |  6 ++++++
>  3 files changed, 12 insertions(+), 17 deletions(-)
> 
> diff --git a/arch/arm64/boot/dts/qcom/msm8996-oneplus-common.dtsi b/arch/arm64/boot/dts/qcom/msm8996-oneplus-common.dtsi
> index 104deaee74e5..2adadc1e5b7c 100644
> --- a/arch/arm64/boot/dts/qcom/msm8996-oneplus-common.dtsi
> +++ b/arch/arm64/boot/dts/qcom/msm8996-oneplus-common.dtsi
> @@ -85,10 +85,6 @@ wlan_en: wlan-en-regulator {
>  	};
>  };
>  
> -&adsp_pil {
> -	status = "okay";
> -};
> -
>  &blsp1_i2c3 {
>  	status = "okay";
>  
> @@ -183,10 +179,6 @@ &dsi0_phy {
>  	status = "okay";
>  };
>  
> -&gpu {
> -	status = "okay";
> -};
> -
>  &hsusb_phy1 {
>  	vdd-supply = <&vreg_l28a_0p925>;
>  	vdda-pll-supply = <&vreg_l12a_1p8>;
> @@ -215,7 +207,6 @@ &mmcc {
>  
>  &mss_pil {
>  	pll-supply = <&vreg_l12a_1p8>;
> -	status = "okay";
>  };
>  
>  &pcie0 {
> @@ -548,10 +539,6 @@ wcd9335: codec@1,0 {
>  	};
>  };
>  
> -&slpi_pil {
> -	status = "okay";
> -};
> -
>  &sound {
>  	compatible = "qcom,apq8096-sndcard";
>  	model = "OnePlus3";
> @@ -812,7 +799,3 @@ &usb3_dwc3 {
>  
>  	maximum-speed = "high-speed";
>  };
> -
> -&venus {
> -	status = "okay";
> -};
> diff --git a/arch/arm64/boot/dts/qcom/msm8996-oneplus3.dts b/arch/arm64/boot/dts/qcom/msm8996-oneplus3.dts
> index 1bdc1b134305..dfe75119b8d2 100644
> --- a/arch/arm64/boot/dts/qcom/msm8996-oneplus3.dts
> +++ b/arch/arm64/boot/dts/qcom/msm8996-oneplus3.dts
> @@ -17,6 +17,7 @@ / {
>  
>  &adsp_pil {
>  	firmware-name = "qcom/msm8996/oneplus3/adsp.mbn";
> +	status = "okay";
>  };
>  
>  &battery {
> @@ -25,6 +26,8 @@ &battery {
>  };
>  
>  &gpu {
> +	status = "okay";
> +
>  	zap-shader {
>  		firmware-name = "qcom/msm8996/oneplus3/a530_zap.mbn";
>  	};
> @@ -33,12 +36,15 @@ zap-shader {
>  &mss_pil {
>  	firmware-name = "qcom/msm8996/oneplus3/mba.mbn",
>  			"qcom/msm8996/oneplus3/modem.mbn";
> +	status = "okay";
>  };
>  
>  &slpi_pil {
>  	firmware-name = "qcom/msm8996/oneplus3/slpi.mbn";
> +	status = "okay";
>  };
>  
>  &venus {
>  	firmware-name = "qcom/msm8996/oneplus3/venus.mbn";
> +	status = "okay";
>  };
> diff --git a/arch/arm64/boot/dts/qcom/msm8996-oneplus3t.dts b/arch/arm64/boot/dts/qcom/msm8996-oneplus3t.dts
> index 34f837dd0c12..51fce65e89f1 100644
> --- a/arch/arm64/boot/dts/qcom/msm8996-oneplus3t.dts
> +++ b/arch/arm64/boot/dts/qcom/msm8996-oneplus3t.dts
> @@ -18,6 +18,7 @@ / {
>  
>  &adsp_pil {
>  	firmware-name = "qcom/msm8996/oneplus3t/adsp.mbn";
> +	status = "okay";
>  };
>  
>  &battery {
> @@ -26,6 +27,8 @@ &battery {
>  };
>  
>  &gpu {
> +	status = "okay";
> +
>  	zap-shader {
>  		firmware-name = "qcom/msm8996/oneplus3t/a530_zap.mbn";
>  	};
> @@ -34,12 +37,15 @@ zap-shader {
>  &mss_pil {
>  	firmware-name = "qcom/msm8996/oneplus3t/mba.mbn",
>  			"qcom/msm8996/oneplus3t/modem.mbn";
> +	status = "okay";
>  };
>  
>  &slpi_pil {
>  	firmware-name = "qcom/msm8996/oneplus3t/slpi.mbn";
> +	status = "okay";
>  };
>  
>  &venus {
>  	firmware-name = "qcom/msm8996/oneplus3t/venus.mbn";
> +	status = "okay";
>  };