linux-kernel - Re: [PATCH 2/2] ARM: dts: qcom: minor whitespace cleanup around '='

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: <2587c8d6-52d5-bc3b-a053-9502dd4239e7@linaro.org>
Date:   Mon, 3 Jul 2023 11:31:37 +0200
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>,
        Conor Dooley <conor+dt@...nel.org>,
        cros-qcom-dts-watchers@...omium.org, linux-arm-msm@...r.kernel.org,
        devicetree@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH 2/2] ARM: dts: qcom: minor whitespace cleanup around '='

On 2.07.2023 20:50, Krzysztof Kozlowski wrote:
> The DTS code coding style expects exactly one space before and after '='
> sign.
> 
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>
> ---
Reviewed-by: Konrad Dybcio <konrad.dybcio@...aro.org>

Konrad
>  arch/arm/boot/dts/qcom/qcom-ipq4019.dtsi      | 20 +++++++++----------
>  .../qcom/qcom-msm8974pro-fairphone-fp2.dts    |  2 +-
>  2 files changed, 11 insertions(+), 11 deletions(-)
> 
> diff --git a/arch/arm/boot/dts/qcom/qcom-ipq4019.dtsi b/arch/arm/boot/dts/qcom/qcom-ipq4019.dtsi
> index f0ef86fadc9d..5f0ff61017d1 100644
> --- a/arch/arm/boot/dts/qcom/qcom-ipq4019.dtsi
> +++ b/arch/arm/boot/dts/qcom/qcom-ipq4019.dtsi
> @@ -416,10 +416,10 @@ restart@...000 {
>  
>  		pcie0: pci@...00000 {
>  			compatible = "qcom,pcie-ipq4019";
> -			reg =  <0x40000000 0xf1d
> -				0x40000f20 0xa8
> -				0x80000 0x2000
> -				0x40100000 0x1000>;
> +			reg = <0x40000000 0xf1d>,
> +			      <0x40000f20 0xa8>,
> +			      <0x80000 0x2000>,
> +			      <0x40100000 0x1000>;
>  			reg-names = "dbi", "elbi", "parf", "config";
>  			device_type = "pci";
>  			linux,pci-domain = <0>;
> @@ -543,9 +543,9 @@ wifi0: wifi@...0000 {
>  				     <GIC_SPI 46 IRQ_TYPE_EDGE_RISING>,
>  				     <GIC_SPI 47 IRQ_TYPE_EDGE_RISING>,
>  				     <GIC_SPI 168 IRQ_TYPE_LEVEL_HIGH>;
> -			interrupt-names =  "msi0",  "msi1",  "msi2",  "msi3",
> -					   "msi4",  "msi5",  "msi6",  "msi7",
> -					   "msi8",  "msi9", "msi10", "msi11",
> +			interrupt-names = "msi0",  "msi1",  "msi2",  "msi3",
> +					  "msi4",  "msi5",  "msi6",  "msi7",
> +					  "msi8",  "msi9", "msi10", "msi11",
>  					  "msi12", "msi13", "msi14", "msi15",
>  					  "legacy";
>  			status = "disabled";
> @@ -585,9 +585,9 @@ wifi1: wifi@...0000 {
>  				     <GIC_SPI 62 IRQ_TYPE_EDGE_RISING>,
>  				     <GIC_SPI 63 IRQ_TYPE_EDGE_RISING>,
>  				     <GIC_SPI 169 IRQ_TYPE_LEVEL_HIGH>;
> -			interrupt-names =  "msi0",  "msi1",  "msi2",  "msi3",
> -					   "msi4",  "msi5",  "msi6",  "msi7",
> -					   "msi8",  "msi9", "msi10", "msi11",
> +			interrupt-names = "msi0",  "msi1",  "msi2",  "msi3",
> +					  "msi4",  "msi5",  "msi6",  "msi7",
> +					  "msi8",  "msi9", "msi10", "msi11",
>  					  "msi12", "msi13", "msi14", "msi15",
>  					  "legacy";
>  			status = "disabled";
> diff --git a/arch/arm/boot/dts/qcom/qcom-msm8974pro-fairphone-fp2.dts b/arch/arm/boot/dts/qcom/qcom-msm8974pro-fairphone-fp2.dts
> index f531d2679f6c..42d253b75dad 100644
> --- a/arch/arm/boot/dts/qcom/qcom-msm8974pro-fairphone-fp2.dts
> +++ b/arch/arm/boot/dts/qcom/qcom-msm8974pro-fairphone-fp2.dts
> @@ -414,7 +414,7 @@ cmd-data-pins {
>  
>  	wcnss_pin_a: wcnss-pin-active-state {
>  		wlan-pins {
> -			pins =  "gpio36", "gpio37", "gpio38", "gpio39", "gpio40";
> +			pins = "gpio36", "gpio37", "gpio38", "gpio39", "gpio40";
>  			function = "wlan";
>  
>  			drive-strength = <6>;