This property is named differently than other SPI drivers with the same
property, as well as the property as used in Linux.

Signed-off-by: Sean Anderson <sean...@gmail.com>
---

Changes in v5:
- New

 arch/arc/dts/axs10x_mb.dtsi  |  3 ++-
 arch/arc/dts/hsdk.dts        |  3 ++-
 drivers/spi/designware_spi.c | 10 +++-------
 3 files changed, 7 insertions(+), 9 deletions(-)

diff --git a/arch/arc/dts/axs10x_mb.dtsi b/arch/arc/dts/axs10x_mb.dtsi
index 5b77642b8d..50e62788cf 100644
--- a/arch/arc/dts/axs10x_mb.dtsi
+++ b/arch/arc/dts/axs10x_mb.dtsi
@@ -97,7 +97,8 @@
                        spi-max-frequency = <4000000>;
                        clocks = <&apbclk>;
                        clock-names = "spi_clk";
-                       cs-gpio = <&cs_gpio 0>;
+                       num-cs = <1>;
+                       cs-gpios = <&cs_gpio 0>;
                        spi_flash@0 {
                                compatible = "jedec,spi-nor";
                                reg = <0>;
diff --git a/arch/arc/dts/hsdk.dts b/arch/arc/dts/hsdk.dts
index 34ef3a620a..719ffdbce5 100644
--- a/arch/arc/dts/hsdk.dts
+++ b/arch/arc/dts/hsdk.dts
@@ -120,7 +120,8 @@
                spi-max-frequency = <4000000>;
                clocks = <&cgu_clk CLK_SYS_SPI_REF>;
                clock-names = "spi_clk";
-               cs-gpio = <&cs_gpio 0>;
+               num-cs = <1>;
+               cs-gpios = <&cs_gpio 0>;
                spi_flash@0 {
                        compatible = "jedec,spi-nor";
                        reg = <0>;
diff --git a/drivers/spi/designware_spi.c b/drivers/spi/designware_spi.c
index 6e1c289297..765fa2f582 100644
--- a/drivers/spi/designware_spi.c
+++ b/drivers/spi/designware_spi.c
@@ -142,7 +142,8 @@ static int request_gpio_cs(struct udevice *bus)
        int ret;
 
        /* External chip select gpio line is optional */
-       ret = gpio_request_by_name(bus, "cs-gpio", 0, &priv->cs_gpio, 0);
+       ret = gpio_request_by_name(bus, "cs-gpios", 0, &priv->cs_gpio,
+                                  GPIOD_IS_OUT | GPIOD_IS_OUT_ACTIVE);
        if (ret == -ENOENT)
                return 0;
 
@@ -151,12 +152,7 @@ static int request_gpio_cs(struct udevice *bus)
                return ret;
        }
 
-       if (dm_gpio_is_valid(&priv->cs_gpio)) {
-               dm_gpio_set_dir_flags(&priv->cs_gpio,
-                                     GPIOD_IS_OUT | GPIOD_IS_OUT_ACTIVE);
-       }
-
-       debug("%s: used external gpio for CS management\n", __func__);
+       debug("%s: using external gpio for CS management\n", __func__);
 #endif
        return 0;
 }
-- 
2.25.0

Reply via email to