All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] arm64: dts: qcom: sc7280: Move the SD CD GPIO pin out of the dtsi file
@ 2021-08-30 15:06 Douglas Anderson
  2021-08-30 15:51 ` Matthias Kaehlcke
                   ` (2 more replies)
  0 siblings, 3 replies; 4+ messages in thread
From: Douglas Anderson @ 2021-08-30 15:06 UTC (permalink / raw)
  To: bjorn.andersson
  Cc: sbhanu, swboyd, mka, Douglas Anderson, Andy Gross, Rob Herring,
	devicetree, linux-arm-msm, linux-kernel

There's nothing magical about GPIO91 and boards could use different
GPIOs for card detect. Move the pin out of the dtsi file and to the
only existing board file.

Signed-off-by: Douglas Anderson <dianders@chromium.org>
---

 arch/arm64/boot/dts/qcom/sc7280-idp.dtsi | 1 +
 arch/arm64/boot/dts/qcom/sc7280.dtsi     | 4 ----
 2 files changed, 1 insertion(+), 4 deletions(-)

diff --git a/arch/arm64/boot/dts/qcom/sc7280-idp.dtsi b/arch/arm64/boot/dts/qcom/sc7280-idp.dtsi
index 371a2a9dcf7a..ed9f9283b71d 100644
--- a/arch/arm64/boot/dts/qcom/sc7280-idp.dtsi
+++ b/arch/arm64/boot/dts/qcom/sc7280-idp.dtsi
@@ -336,6 +336,7 @@ data {
 	};
 
 	sd-cd {
+		pins = "gpio91";
 		bias-pull-up;
 	};
 };
diff --git a/arch/arm64/boot/dts/qcom/sc7280.dtsi b/arch/arm64/boot/dts/qcom/sc7280.dtsi
index 53a21d086178..99e946a33fbb 100644
--- a/arch/arm64/boot/dts/qcom/sc7280.dtsi
+++ b/arch/arm64/boot/dts/qcom/sc7280.dtsi
@@ -1573,10 +1573,6 @@ cmd {
 				data {
 					pins = "sdc2_data";
 				};
-
-				sd-cd {
-					pins = "gpio91";
-				};
 			};
 
 			sdc2_off: sdc2-off {
-- 
2.33.0.259.gc128427fd7-goog


^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH] arm64: dts: qcom: sc7280: Move the SD CD GPIO pin out of the dtsi file
  2021-08-30 15:06 [PATCH] arm64: dts: qcom: sc7280: Move the SD CD GPIO pin out of the dtsi file Douglas Anderson
@ 2021-08-30 15:51 ` Matthias Kaehlcke
  2021-08-30 22:59 ` Stephen Boyd
  2021-09-16 15:27 ` Doug Anderson
  2 siblings, 0 replies; 4+ messages in thread
From: Matthias Kaehlcke @ 2021-08-30 15:51 UTC (permalink / raw)
  To: Douglas Anderson
  Cc: bjorn.andersson, sbhanu, swboyd, Andy Gross, Rob Herring,
	devicetree, linux-arm-msm, linux-kernel

On Mon, Aug 30, 2021 at 08:06:37AM -0700, Douglas Anderson wrote:
> There's nothing magical about GPIO91 and boards could use different
> GPIOs for card detect. Move the pin out of the dtsi file and to the
> only existing board file.
> 
> Signed-off-by: Douglas Anderson <dianders@chromium.org>

Reviewed-by: Matthias Kaehlcke <mka@chromium.org>

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] arm64: dts: qcom: sc7280: Move the SD CD GPIO pin out of the dtsi file
  2021-08-30 15:06 [PATCH] arm64: dts: qcom: sc7280: Move the SD CD GPIO pin out of the dtsi file Douglas Anderson
  2021-08-30 15:51 ` Matthias Kaehlcke
@ 2021-08-30 22:59 ` Stephen Boyd
  2021-09-16 15:27 ` Doug Anderson
  2 siblings, 0 replies; 4+ messages in thread
From: Stephen Boyd @ 2021-08-30 22:59 UTC (permalink / raw)
  To: Douglas Anderson, bjorn.andersson
  Cc: sbhanu, mka, Andy Gross, Rob Herring, devicetree, linux-arm-msm,
	linux-kernel

Quoting Douglas Anderson (2021-08-30 08:06:37)
> There's nothing magical about GPIO91 and boards could use different
> GPIOs for card detect. Move the pin out of the dtsi file and to the
> only existing board file.
>
> Signed-off-by: Douglas Anderson <dianders@chromium.org>
> ---

Reviewed-by: Stephen Boyd <swboyd@chromium.org>

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] arm64: dts: qcom: sc7280: Move the SD CD GPIO pin out of the dtsi file
  2021-08-30 15:06 [PATCH] arm64: dts: qcom: sc7280: Move the SD CD GPIO pin out of the dtsi file Douglas Anderson
  2021-08-30 15:51 ` Matthias Kaehlcke
  2021-08-30 22:59 ` Stephen Boyd
@ 2021-09-16 15:27 ` Doug Anderson
  2 siblings, 0 replies; 4+ messages in thread
From: Doug Anderson @ 2021-09-16 15:27 UTC (permalink / raw)
  To: Bjorn Andersson
  Cc: Shaik Sajida Bhanu, Stephen Boyd, Matthias Kaehlcke, Andy Gross,
	Rob Herring,
	open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS,
	linux-arm-msm, LKML

Hi,

On Mon, Aug 30, 2021 at 8:07 AM Douglas Anderson <dianders@chromium.org> wrote:
>
> There's nothing magical about GPIO91 and boards could use different
> GPIOs for card detect. Move the pin out of the dtsi file and to the
> only existing board file.
>
> Signed-off-by: Douglas Anderson <dianders@chromium.org>
> ---
>
>  arch/arm64/boot/dts/qcom/sc7280-idp.dtsi | 1 +
>  arch/arm64/boot/dts/qcom/sc7280.dtsi     | 4 ----
>  2 files changed, 1 insertion(+), 4 deletions(-)

If there are no concerns with this patch, I think it's ready to land
now. Thanks!

-Doug

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2021-09-16 15:28 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-08-30 15:06 [PATCH] arm64: dts: qcom: sc7280: Move the SD CD GPIO pin out of the dtsi file Douglas Anderson
2021-08-30 15:51 ` Matthias Kaehlcke
2021-08-30 22:59 ` Stephen Boyd
2021-09-16 15:27 ` Doug Anderson

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.