* [PATCH v2] arm64: dts: qcom: sc7280: Remove unused sleep pin control nodes
@ 2022-11-16 9:33 Srinivasa Rao Mandadapu
2022-11-16 9:42 ` Krzysztof Kozlowski
` (2 more replies)
0 siblings, 3 replies; 5+ messages in thread
From: Srinivasa Rao Mandadapu @ 2022-11-16 9:33 UTC (permalink / raw)
To: vkoul, agross, andersson, robh+dt, broonie,
krzysztof.kozlowski+dt, linux-arm-msm, devicetree, linux-kernel,
quic_rohkumar, srinivas.kandagatla, dianders, swboyd, judyhsiao,
alsa-devel, quic_rjendra, konrad.dybcio, mka
Cc: Srinivasa Rao Mandadapu
Remove unused and redundant sleep pin control entries as they are
not referenced anywhere in sc7280 based platform's device tree variants.
Signed-off-by: Srinivasa Rao Mandadapu <quic_srivasam@quicinc.com>
---
Chnges Since v1:
-- Update subject prefixes and commit message.
.../qcom/sc7280-herobrine-audio-rt5682-3mic.dtsi | 8 -----
.../dts/qcom/sc7280-herobrine-audio-wcd9385.dtsi | 20 -----------
arch/arm64/boot/dts/qcom/sc7280-idp.dtsi | 20 -----------
arch/arm64/boot/dts/qcom/sc7280.dtsi | 40 ----------------------
4 files changed, 88 deletions(-)
diff --git a/arch/arm64/boot/dts/qcom/sc7280-herobrine-audio-rt5682-3mic.dtsi b/arch/arm64/boot/dts/qcom/sc7280-herobrine-audio-rt5682-3mic.dtsi
index cf34334..1ca11a1 100644
--- a/arch/arm64/boot/dts/qcom/sc7280-herobrine-audio-rt5682-3mic.dtsi
+++ b/arch/arm64/boot/dts/qcom/sc7280-herobrine-audio-rt5682-3mic.dtsi
@@ -148,10 +148,6 @@ hp_i2c: &i2c2 {
bias-disable;
};
-&lpass_dmic01_clk_sleep {
- drive-strength = <2>;
-};
-
&lpass_dmic01_data {
bias-pull-down;
};
@@ -161,10 +157,6 @@ hp_i2c: &i2c2 {
bias-disable;
};
-&lpass_dmic23_clk_sleep {
- drive-strength = <2>;
-};
-
&lpass_dmic23_data {
bias-pull-down;
};
diff --git a/arch/arm64/boot/dts/qcom/sc7280-herobrine-audio-wcd9385.dtsi b/arch/arm64/boot/dts/qcom/sc7280-herobrine-audio-wcd9385.dtsi
index c72e53a..ae25520 100644
--- a/arch/arm64/boot/dts/qcom/sc7280-herobrine-audio-wcd9385.dtsi
+++ b/arch/arm64/boot/dts/qcom/sc7280-herobrine-audio-wcd9385.dtsi
@@ -167,10 +167,6 @@
bias-disable;
};
-&lpass_dmic01_clk_sleep {
- drive-strength = <2>;
-};
-
&lpass_dmic01_data {
bias-pull-down;
};
@@ -180,10 +176,6 @@
bias-disable;
};
-&lpass_dmic23_clk_sleep {
- drive-strength = <2>;
-};
-
&lpass_dmic23_data {
bias-pull-down;
};
@@ -194,30 +186,18 @@
bias-disable;
};
-&lpass_rx_swr_clk_sleep {
- bias-pull-down;
-};
-
&lpass_rx_swr_data {
drive-strength = <2>;
slew-rate = <1>;
bias-bus-hold;
};
-&lpass_rx_swr_data_sleep {
- bias-pull-down;
-};
-
&lpass_tx_swr_clk {
drive-strength = <2>;
slew-rate = <1>;
bias-disable;
};
-&lpass_tx_swr_clk_sleep {
- bias-pull-down;
-};
-
&lpass_tx_swr_data {
drive-strength = <2>;
slew-rate = <1>;
diff --git a/arch/arm64/boot/dts/qcom/sc7280-idp.dtsi b/arch/arm64/boot/dts/qcom/sc7280-idp.dtsi
index 8ca2281..f7efb99 100644
--- a/arch/arm64/boot/dts/qcom/sc7280-idp.dtsi
+++ b/arch/arm64/boot/dts/qcom/sc7280-idp.dtsi
@@ -576,10 +576,6 @@
bias-disable;
};
-&lpass_dmic01_clk_sleep {
- drive-strength = <2>;
-};
-
&lpass_dmic01_data {
bias-pull-down;
};
@@ -589,10 +585,6 @@
bias-disable;
};
-&lpass_dmic23_clk_sleep {
- drive-strength = <2>;
-};
-
&lpass_dmic23_data {
bias-pull-down;
};
@@ -603,30 +595,18 @@
bias-disable;
};
-&lpass_rx_swr_clk_sleep {
- bias-pull-down;
-};
-
&lpass_rx_swr_data {
drive-strength = <2>;
slew-rate = <1>;
bias-bus-hold;
};
-&lpass_rx_swr_data_sleep {
- bias-pull-down;
-};
-
&lpass_tx_swr_clk {
drive-strength = <2>;
slew-rate = <1>;
bias-disable;
};
-&lpass_tx_swr_clk_sleep {
- bias-pull-down;
-};
-
&lpass_tx_swr_data {
drive-strength = <2>;
slew-rate = <1>;
diff --git a/arch/arm64/boot/dts/qcom/sc7280.dtsi b/arch/arm64/boot/dts/qcom/sc7280.dtsi
index 448879d..4970864 100644
--- a/arch/arm64/boot/dts/qcom/sc7280.dtsi
+++ b/arch/arm64/boot/dts/qcom/sc7280.dtsi
@@ -2483,80 +2483,40 @@
function = "dmic1_clk";
};
- lpass_dmic01_clk_sleep: dmic01-clk-sleep-state {
- pins = "gpio6";
- function = "dmic1_clk";
- };
-
lpass_dmic01_data: dmic01-data-state {
pins = "gpio7";
function = "dmic1_data";
};
- lpass_dmic01_data_sleep: dmic01-data-sleep-state {
- pins = "gpio7";
- function = "dmic1_data";
- };
-
lpass_dmic23_clk: dmic23-clk-state {
pins = "gpio8";
function = "dmic2_clk";
};
- lpass_dmic23_clk_sleep: dmic23-clk-sleep-state {
- pins = "gpio8";
- function = "dmic2_clk";
- };
-
lpass_dmic23_data: dmic23-data-state {
pins = "gpio9";
function = "dmic2_data";
};
- lpass_dmic23_data_sleep: dmic23-data-sleep-state {
- pins = "gpio9";
- function = "dmic2_data";
- };
-
lpass_rx_swr_clk: rx-swr-clk-state {
pins = "gpio3";
function = "swr_rx_clk";
};
- lpass_rx_swr_clk_sleep: rx-swr-clk-sleep-state {
- pins = "gpio3";
- function = "swr_rx_clk";
- };
-
lpass_rx_swr_data: rx-swr-data-state {
pins = "gpio4", "gpio5";
function = "swr_rx_data";
};
- lpass_rx_swr_data_sleep: rx-swr-data-sleep-state {
- pins = "gpio4", "gpio5";
- function = "swr_rx_data";
- };
-
lpass_tx_swr_clk: tx-swr-clk-state {
pins = "gpio0";
function = "swr_tx_clk";
};
- lpass_tx_swr_clk_sleep: tx-swr-clk-sleep-state {
- pins = "gpio0";
- function = "swr_tx_clk";
- };
-
lpass_tx_swr_data: tx-swr-data-state {
pins = "gpio1", "gpio2", "gpio14";
function = "swr_tx_data";
};
-
- lpass_tx_swr_data_sleep: tx-swr-data-sleep-state {
- pins = "gpio1", "gpio2", "gpio14";
- function = "swr_tx_data";
- };
};
gpu: gpu@3d00000 {
--
2.7.4
^ permalink raw reply [flat|nested] 5+ messages in thread* Re: [PATCH v2] arm64: dts: qcom: sc7280: Remove unused sleep pin control nodes
2022-11-16 9:33 [PATCH v2] arm64: dts: qcom: sc7280: Remove unused sleep pin control nodes Srinivasa Rao Mandadapu
@ 2022-11-16 9:42 ` Krzysztof Kozlowski
2022-11-16 16:51 ` Doug Anderson
2022-12-06 18:19 ` Bjorn Andersson
2 siblings, 0 replies; 5+ messages in thread
From: Krzysztof Kozlowski @ 2022-11-16 9:42 UTC (permalink / raw)
To: Srinivasa Rao Mandadapu, vkoul, agross, andersson, robh+dt,
broonie, krzysztof.kozlowski+dt, linux-arm-msm, devicetree,
linux-kernel, quic_rohkumar, srinivas.kandagatla, dianders,
swboyd, judyhsiao, alsa-devel, quic_rjendra, konrad.dybcio, mka
On 16/11/2022 10:33, Srinivasa Rao Mandadapu wrote:
> Remove unused and redundant sleep pin control entries as they are
> not referenced anywhere in sc7280 based platform's device tree variants.
>
> Signed-off-by: Srinivasa Rao Mandadapu <quic_srivasam@quicinc.com>
> ---
> Chnges Since v1:
> -- Update subject prefixes and commit message.
Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH v2] arm64: dts: qcom: sc7280: Remove unused sleep pin control nodes
2022-11-16 9:33 [PATCH v2] arm64: dts: qcom: sc7280: Remove unused sleep pin control nodes Srinivasa Rao Mandadapu
2022-11-16 9:42 ` Krzysztof Kozlowski
@ 2022-11-16 16:51 ` Doug Anderson
2022-11-16 17:21 ` Krzysztof Kozlowski
2022-12-06 18:19 ` Bjorn Andersson
2 siblings, 1 reply; 5+ messages in thread
From: Doug Anderson @ 2022-11-16 16:51 UTC (permalink / raw)
To: Srinivasa Rao Mandadapu
Cc: vkoul, agross, andersson, robh+dt, broonie,
krzysztof.kozlowski+dt, linux-arm-msm, devicetree, linux-kernel,
quic_rohkumar, srinivas.kandagatla, swboyd, judyhsiao,
alsa-devel, quic_rjendra, konrad.dybcio, mka
Hi,
On Wed, Nov 16, 2022 at 1:33 AM Srinivasa Rao Mandadapu
<quic_srivasam@quicinc.com> wrote:
>
> Remove unused and redundant sleep pin control entries as they are
> not referenced anywhere in sc7280 based platform's device tree variants.
>
> Signed-off-by: Srinivasa Rao Mandadapu <quic_srivasam@quicinc.com>
> ---
> Chnges Since v1:
> -- Update subject prefixes and commit message.
>
> .../qcom/sc7280-herobrine-audio-rt5682-3mic.dtsi | 8 -----
> .../dts/qcom/sc7280-herobrine-audio-wcd9385.dtsi | 20 -----------
> arch/arm64/boot/dts/qcom/sc7280-idp.dtsi | 20 -----------
> arch/arm64/boot/dts/qcom/sc7280.dtsi | 40 ----------------------
> 4 files changed, 88 deletions(-)
Reported-by: Douglas Anderson <dianders@chromium.org>
Reviewed-by: Douglas Anderson <dianders@chromium.org>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH v2] arm64: dts: qcom: sc7280: Remove unused sleep pin control nodes
2022-11-16 16:51 ` Doug Anderson
@ 2022-11-16 17:21 ` Krzysztof Kozlowski
0 siblings, 0 replies; 5+ messages in thread
From: Krzysztof Kozlowski @ 2022-11-16 17:21 UTC (permalink / raw)
To: Doug Anderson, Srinivasa Rao Mandadapu
Cc: vkoul, agross, andersson, robh+dt, broonie,
krzysztof.kozlowski+dt, linux-arm-msm, devicetree, linux-kernel,
quic_rohkumar, srinivas.kandagatla, swboyd, judyhsiao,
alsa-devel, quic_rjendra, konrad.dybcio, mka
On 16/11/2022 17:51, Doug Anderson wrote:
> Hi,
>
> On Wed, Nov 16, 2022 at 1:33 AM Srinivasa Rao Mandadapu
> <quic_srivasam@quicinc.com> wrote:
>>
>> Remove unused and redundant sleep pin control entries as they are
>> not referenced anywhere in sc7280 based platform's device tree variants.
>>
>> Signed-off-by: Srinivasa Rao Mandadapu <quic_srivasam@quicinc.com>
>> ---
>> Chnges Since v1:
>> -- Update subject prefixes and commit message.
>>
>> .../qcom/sc7280-herobrine-audio-rt5682-3mic.dtsi | 8 -----
>> .../dts/qcom/sc7280-herobrine-audio-wcd9385.dtsi | 20 -----------
>> arch/arm64/boot/dts/qcom/sc7280-idp.dtsi | 20 -----------
>> arch/arm64/boot/dts/qcom/sc7280.dtsi | 40 ----------------------
>> 4 files changed, 88 deletions(-)
>
> Reported-by: Douglas Anderson <dianders@chromium.org>
I guess rather:
Suggested-by: Douglas Anderson <dianders@chromium.org>
It's not a bug, no Fixes tag.
> Reviewed-by: Douglas Anderson <dianders@chromium.org>
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH v2] arm64: dts: qcom: sc7280: Remove unused sleep pin control nodes
2022-11-16 9:33 [PATCH v2] arm64: dts: qcom: sc7280: Remove unused sleep pin control nodes Srinivasa Rao Mandadapu
2022-11-16 9:42 ` Krzysztof Kozlowski
2022-11-16 16:51 ` Doug Anderson
@ 2022-12-06 18:19 ` Bjorn Andersson
2 siblings, 0 replies; 5+ messages in thread
From: Bjorn Andersson @ 2022-12-06 18:19 UTC (permalink / raw)
To: robh+dt, konrad.dybcio, quic_srivasam, dianders,
krzysztof.kozlowski+dt, broonie, linux-kernel, linux-arm-msm,
swboyd, quic_rjendra, mka, devicetree, srinivas.kandagatla,
vkoul, judyhsiao, agross, quic_rohkumar, alsa-devel
On Wed, 16 Nov 2022 15:03:04 +0530, Srinivasa Rao Mandadapu wrote:
> Remove unused and redundant sleep pin control entries as they are
> not referenced anywhere in sc7280 based platform's device tree variants.
>
>
Applied, thanks!
[1/1] arm64: dts: qcom: sc7280: Remove unused sleep pin control nodes
commit: 7fa58dc94dd274c27cf1ab54b37d2dd54d7e18ac
Best regards,
--
Bjorn Andersson <andersson@kernel.org>
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2022-12-06 18:24 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-11-16 9:33 [PATCH v2] arm64: dts: qcom: sc7280: Remove unused sleep pin control nodes Srinivasa Rao Mandadapu
2022-11-16 9:42 ` Krzysztof Kozlowski
2022-11-16 16:51 ` Doug Anderson
2022-11-16 17:21 ` Krzysztof Kozlowski
2022-12-06 18:19 ` Bjorn Andersson
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®