On 8/24/23 13:29, Stefan Roese wrote:
On 8/21/23 01:31, Chris Packham wrote:
The CN9130-DB uses the SPI1 interface but had the pinctrl node labelled
as "cp0_spi0_pins". Use the label "cp0_spi1_pins" and update the node
name to "cp0-spi-pins-1" to avoid confusion with the pinctrl options for
SPI0.

Signed-off-by: Chris Packham <judge.pack...@gmail.com>

Reviewed-by: Stefan Roese <s...@denx.de>

Applied to u-boot-marvell/master

Thanks,
Stefan


Thanks,
Stefan

---

  arch/arm/dts/cn9130-db.dtsi | 2 +-
  arch/arm/dts/cn9130.dtsi    | 2 +-
  2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/arm/dts/cn9130-db.dtsi b/arch/arm/dts/cn9130-db.dtsi
index 1b28732ee752..4b21ff46d507 100644
--- a/arch/arm/dts/cn9130-db.dtsi
+++ b/arch/arm/dts/cn9130-db.dtsi
@@ -183,7 +183,7 @@
  /* U55 */
  &cp0_spi1 {
      pinctrl-names = "default";
-    pinctrl-0 = <&cp0_spi0_pins>;
+    pinctrl-0 = <&cp0_spi1_pins>;
      reg = <0x700680 0x50>,        /* control */
            <0x2000000 0x1000000>,    /* CS0 */
            <0 0xffffffff>,        /* CS1 */
diff --git a/arch/arm/dts/cn9130.dtsi b/arch/arm/dts/cn9130.dtsi
index 68b767a70639..efcb2e906b91 100644
--- a/arch/arm/dts/cn9130.dtsi
+++ b/arch/arm/dts/cn9130.dtsi
@@ -66,7 +66,7 @@
          marvell,pins = < 56 57 58 59 60 61 >;
          marvell,function = <14>;
      };
-    cp0_spi0_pins: cp0-spi-pins-0 {
+    cp0_spi1_pins: cp0-spi-pins-1 {
          marvell,pins = < 13 14 15 16 >;
          marvell,function = <3>;
      };

Viele Grüße,
Stefan Roese


Viele Grüße,
Stefan Roese

--
DENX Software Engineering GmbH,      Managing Director: Erika Unter
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-51 Fax: (+49)-8142-66989-80 Email: s...@denx.de

Reply via email to