[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <df6b1f7a-3dd4-496b-bb6d-572aa8e08257@somainline.org>
Date: Mon, 19 Sep 2022 21:19:58 +0200
From: Konrad Dybcio <konrad.dybcio@...ainline.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>,
linux-arm-msm@...r.kernel.org, devicetree@...r.kernel.org,
linux-kernel@...r.kernel.org
Subject: Re: [PATCH] arm64: dts: qcom: correct white-space before {
On 19.09.2022 18:33, Krzysztof Kozlowski wrote:
> Add missing space or remove redundant one before opening {.
>
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>
> ---
Very nice!
Reviewed-by: Konrad Dybcio <konrad.dybcio@...ainline.org>
Konrad
> arch/arm64/boot/dts/qcom/msm8996.dtsi | 2 +-
> arch/arm64/boot/dts/qcom/msm8998.dtsi | 8 ++++----
> arch/arm64/boot/dts/qcom/qrb5165-rb5.dts | 4 ++--
> arch/arm64/boot/dts/qcom/sc7180-trogdor-coachz.dtsi | 2 +-
> arch/arm64/boot/dts/qcom/sc7180-trogdor-homestar.dtsi | 2 +-
> arch/arm64/boot/dts/qcom/sc7180-trogdor.dtsi | 8 ++++----
> arch/arm64/boot/dts/qcom/sdm845-cheza.dtsi | 6 +++---
> arch/arm64/boot/dts/qcom/sdm845-db845c.dts | 8 ++++----
> arch/arm64/boot/dts/qcom/sdm850-lenovo-yoga-c630.dts | 4 ++--
> arch/arm64/boot/dts/qcom/sdm850-samsung-w737.dts | 6 +++---
> 10 files changed, 25 insertions(+), 25 deletions(-)
>
> diff --git a/arch/arm64/boot/dts/qcom/msm8996.dtsi b/arch/arm64/boot/dts/qcom/msm8996.dtsi
> index 1e6b70582866..6a0420c079f5 100644
> --- a/arch/arm64/boot/dts/qcom/msm8996.dtsi
> +++ b/arch/arm64/boot/dts/qcom/msm8996.dtsi
> @@ -1498,7 +1498,7 @@ blsp2_i2c3_sleep: blsp2-i2c3-sleep {
> bias-disable;
> };
>
> - wcd_intr_default: wcd-intr-default{
> + wcd_intr_default: wcd-intr-default {
> pins = "gpio54";
> function = "gpio";
> drive-strength = <2>;
> diff --git a/arch/arm64/boot/dts/qcom/msm8998.dtsi b/arch/arm64/boot/dts/qcom/msm8998.dtsi
> index d463a66715ea..7e2a8b168d91 100644
> --- a/arch/arm64/boot/dts/qcom/msm8998.dtsi
> +++ b/arch/arm64/boot/dts/qcom/msm8998.dtsi
> @@ -1903,7 +1903,7 @@ etm5: etm@...0000 {
>
> cpu = <&CPU4>;
>
> - port{
> + port {
> etm4_out: endpoint {
> remote-endpoint = <&apss_funnel_in4>;
> };
> @@ -1920,7 +1920,7 @@ etm6: etm@...0000 {
>
> cpu = <&CPU5>;
>
> - port{
> + port {
> etm5_out: endpoint {
> remote-endpoint = <&apss_funnel_in5>;
> };
> @@ -1937,7 +1937,7 @@ etm7: etm@...0000 {
>
> cpu = <&CPU6>;
>
> - port{
> + port {
> etm6_out: endpoint {
> remote-endpoint = <&apss_funnel_in6>;
> };
> @@ -1954,7 +1954,7 @@ etm8: etm@...0000 {
>
> cpu = <&CPU7>;
>
> - port{
> + port {
> etm7_out: endpoint {
> remote-endpoint = <&apss_funnel_in7>;
> };
> diff --git a/arch/arm64/boot/dts/qcom/qrb5165-rb5.dts b/arch/arm64/boot/dts/qcom/qrb5165-rb5.dts
> index bf8077a1cf9a..3d1bd8e3fc73 100644
> --- a/arch/arm64/boot/dts/qcom/qrb5165-rb5.dts
> +++ b/arch/arm64/boot/dts/qcom/qrb5165-rb5.dts
> @@ -1007,7 +1007,7 @@ can@0 {
> };
>
> &swr0 {
> - left_spkr: wsa8810-left{
> + left_spkr: wsa8810-left {
> compatible = "sdw10217211000";
> reg = <0 3>;
> powerdown-gpios = <&tlmm 130 GPIO_ACTIVE_HIGH>;
> @@ -1016,7 +1016,7 @@ left_spkr: wsa8810-left{
> #sound-dai-cells = <0>;
> };
>
> - right_spkr: wsa8810-right{
> + right_spkr: wsa8810-right {
> compatible = "sdw10217211000";
> reg = <0 4>;
> powerdown-gpios = <&tlmm 130 GPIO_ACTIVE_HIGH>;
> diff --git a/arch/arm64/boot/dts/qcom/sc7180-trogdor-coachz.dtsi b/arch/arm64/boot/dts/qcom/sc7180-trogdor-coachz.dtsi
> index 7ee407f7b6bb..1ce73187a562 100644
> --- a/arch/arm64/boot/dts/qcom/sc7180-trogdor-coachz.dtsi
> +++ b/arch/arm64/boot/dts/qcom/sc7180-trogdor-coachz.dtsi
> @@ -181,7 +181,7 @@ &sound_multimedia0_codec {
> /* PINCTRL - modifications to sc7180-trogdor.dtsi */
>
> &en_pp3300_dx_edp {
> - pinmux {
> + pinmux {
> pins = "gpio67";
> };
>
> diff --git a/arch/arm64/boot/dts/qcom/sc7180-trogdor-homestar.dtsi b/arch/arm64/boot/dts/qcom/sc7180-trogdor-homestar.dtsi
> index 1bd6c7dcd9e9..7fcff4eddd3a 100644
> --- a/arch/arm64/boot/dts/qcom/sc7180-trogdor-homestar.dtsi
> +++ b/arch/arm64/boot/dts/qcom/sc7180-trogdor-homestar.dtsi
> @@ -189,7 +189,7 @@ pinconf {
> };
> };
>
> -&sec_mi2s_active{
> +&sec_mi2s_active {
> pinmux {
> pins = "gpio49", "gpio50", "gpio51", "gpio52";
> function = "mi2s_1";
> diff --git a/arch/arm64/boot/dts/qcom/sc7180-trogdor.dtsi b/arch/arm64/boot/dts/qcom/sc7180-trogdor.dtsi
> index b5f534db135a..6ea62ce138ce 100644
> --- a/arch/arm64/boot/dts/qcom/sc7180-trogdor.dtsi
> +++ b/arch/arm64/boot/dts/qcom/sc7180-trogdor.dtsi
> @@ -1220,7 +1220,7 @@ pinconf {
> };
>
> ap_suspend_l_neuter: ap-suspend-l-neuter {
> - pinmux {
> + pinmux {
> pins = "gpio27";
> function = "gpio";
> };
> @@ -1365,7 +1365,7 @@ pinconf {
> };
>
> pen_rst_odl: pen-rst-odl {
> - pinmux {
> + pinmux {
> pins = "gpio18";
> function = "gpio";
> };
> @@ -1489,7 +1489,7 @@ pinconf {
> };
>
> ts_int_l: ts-int-l {
> - pinmux {
> + pinmux {
> pins = "gpio9";
> function = "gpio";
> };
> @@ -1501,7 +1501,7 @@ pinconf {
> };
>
> ts_reset_l: ts-reset-l {
> - pinmux {
> + pinmux {
> pins = "gpio8";
> function = "gpio";
> };
> diff --git a/arch/arm64/boot/dts/qcom/sdm845-cheza.dtsi b/arch/arm64/boot/dts/qcom/sdm845-cheza.dtsi
> index b5eb8f7eca1d..d46a6d2a7912 100644
> --- a/arch/arm64/boot/dts/qcom/sdm845-cheza.dtsi
> +++ b/arch/arm64/boot/dts/qcom/sdm845-cheza.dtsi
> @@ -1341,7 +1341,7 @@ pinconf {
> };
>
> pen_rst_l: pen-rst-l {
> - pinmux {
> + pinmux {
> pins = "gpio23";
> function = "gpio";
> };
> @@ -1408,7 +1408,7 @@ pinconf {
> };
>
> ts_int_l: ts-int-l {
> - pinmux {
> + pinmux {
> pins = "gpio125";
> function = "gpio";
> };
> @@ -1420,7 +1420,7 @@ pinconf {
> };
>
> ts_reset_l: ts-reset-l {
> - pinmux {
> + pinmux {
> pins = "gpio118";
> function = "gpio";
> };
> diff --git a/arch/arm64/boot/dts/qcom/sdm845-db845c.dts b/arch/arm64/boot/dts/qcom/sdm845-db845c.dts
> index 132417e2d11e..feb1396fadb6 100644
> --- a/arch/arm64/boot/dts/qcom/sdm845-db845c.dts
> +++ b/arch/arm64/boot/dts/qcom/sdm845-db845c.dts
> @@ -1076,7 +1076,7 @@ &venus {
> status = "okay";
> };
>
> -&wcd9340{
> +&wcd9340 {
> pinctrl-0 = <&wcd_intr_default>;
> pinctrl-names = "default";
> clock-names = "extclk";
> @@ -1089,7 +1089,7 @@ &wcd9340{
> vdd-io-supply = <&vreg_s4a_1p8>;
>
> swm: swm@c85 {
> - left_spkr: wsa8810-left{
> + left_spkr: wsa8810-left {
> compatible = "sdw10217201000";
> reg = <0 1>;
> powerdown-gpios = <&wcdgpio 1 GPIO_ACTIVE_HIGH>;
> @@ -1098,7 +1098,7 @@ left_spkr: wsa8810-left{
> #sound-dai-cells = <0>;
> };
>
> - right_spkr: wsa8810-right{
> + right_spkr: wsa8810-right {
> compatible = "sdw10217201000";
> powerdown-gpios = <&wcdgpio 1 GPIO_ACTIVE_HIGH>;
> reg = <0 2>;
> @@ -1126,7 +1126,7 @@ &qup_spi2_default {
> drive-strength = <16>;
> };
>
> -&qup_uart3_default{
> +&qup_uart3_default {
> pinmux {
> pins = "gpio41", "gpio42", "gpio43", "gpio44";
> function = "qup3";
> diff --git a/arch/arm64/boot/dts/qcom/sdm850-lenovo-yoga-c630.dts b/arch/arm64/boot/dts/qcom/sdm850-lenovo-yoga-c630.dts
> index be59a8ba9c1f..761927535778 100644
> --- a/arch/arm64/boot/dts/qcom/sdm850-lenovo-yoga-c630.dts
> +++ b/arch/arm64/boot/dts/qcom/sdm850-lenovo-yoga-c630.dts
> @@ -783,7 +783,7 @@ &wcd9340{
> qcom,mbhc-headphone-vthreshold-microvolt = <50000>;
>
> swm: swm@c85 {
> - left_spkr: wsa8810-left{
> + left_spkr: wsa8810-left {
> compatible = "sdw10217211000";
> reg = <0 3>;
> powerdown-gpios = <&wcdgpio 1 GPIO_ACTIVE_HIGH>;
> @@ -792,7 +792,7 @@ left_spkr: wsa8810-left{
> #sound-dai-cells = <0>;
> };
>
> - right_spkr: wsa8810-right{
> + right_spkr: wsa8810-right {
> compatible = "sdw10217211000";
> powerdown-gpios = <&wcdgpio 2 GPIO_ACTIVE_HIGH>;
> reg = <0 4>;
> diff --git a/arch/arm64/boot/dts/qcom/sdm850-samsung-w737.dts b/arch/arm64/boot/dts/qcom/sdm850-samsung-w737.dts
> index f954fe5cb61a..7e92438034e8 100644
> --- a/arch/arm64/boot/dts/qcom/sdm850-samsung-w737.dts
> +++ b/arch/arm64/boot/dts/qcom/sdm850-samsung-w737.dts
> @@ -574,7 +574,7 @@ pinconf {
> };
>
> pen_rst_l: pen-rst-l {
> - pinmux {
> + pinmux {
> pins = "gpio21";
> function = "gpio";
> };
> @@ -715,7 +715,7 @@ &wcd9340{
> qcom,mbhc-headphone-vthreshold-microvolt = <50000>;
>
> swm: swm@c85 {
> - left_spkr: wsa8810-left{
> + left_spkr: wsa8810-left {
> compatible = "sdw10217211000";
> reg = <0 3>;
> powerdown-gpios = <&wcdgpio 1 GPIO_ACTIVE_HIGH>;
> @@ -724,7 +724,7 @@ left_spkr: wsa8810-left{
> #sound-dai-cells = <0>;
> };
>
> - right_spkr: wsa8810-right{
> + right_spkr: wsa8810-right {
> compatible = "sdw10217211000";
> powerdown-gpios = <&wcdgpio 2 GPIO_ACTIVE_HIGH>;
> reg = <0 4>;
Powered by blists - more mailing lists