diff mbox series

omap3: spi: Correct ti, pindir-d0-out-d1-in parsing

Message ID 20180322215502.3726-1-sjoerd.simons@collabora.co.uk
State Accepted
Commit 63018a3eddd5d9c64f7ee42615a63c6a67ab484d
Headers show
Series omap3: spi: Correct ti, pindir-d0-out-d1-in parsing | expand

Commit Message

Sjoerd Simons March 22, 2018, 9:55 p.m. UTC
The ti,pindir-d0-out-d1-in property is not expected to have a value
according to the device-tree binding, so treat it as a boolean not a
uint property.

Signed-off-by: Sjoerd Simons <sjoerd.simons@collabora.co.uk>
---

 drivers/spi/omap3_spi.c | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

Comments

Jagan Teki March 23, 2018, 5:32 p.m. UTC | #1
On Fri, Mar 23, 2018 at 3:25 AM, Sjoerd Simons
<sjoerd.simons@collabora.co.uk> wrote:
> The ti,pindir-d0-out-d1-in property is not expected to have a value
> according to the device-tree binding, so treat it as a boolean not a
> uint property.
>
> Signed-off-by: Sjoerd Simons <sjoerd.simons@collabora.co.uk>
> ---

Applied to u-boot-spi/master
diff mbox series

Patch

diff --git a/drivers/spi/omap3_spi.c b/drivers/spi/omap3_spi.c
index 053a67bbe0..1ac691a68e 100644
--- a/drivers/spi/omap3_spi.c
+++ b/drivers/spi/omap3_spi.c
@@ -630,8 +630,10 @@  static int omap3_spi_probe(struct udevice *dev)
 		(struct omap2_mcspi_platform_config*)dev_get_driver_data(dev);
 
 	priv->regs = (struct mcspi *)(devfdt_get_addr(dev) + data->regs_offset);
-	priv->pin_dir = fdtdec_get_uint(blob, node, "ti,pindir-d0-out-d1-in",
-					    MCSPI_PINDIR_D0_IN_D1_OUT);
+	if (fdtdec_get_bool(blob, node, "ti,pindir-d0-out-d1-in"))
+		priv->pin_dir = MCSPI_PINDIR_D0_OUT_D1_IN;
+	else
+		priv->pin_dir = MCSPI_PINDIR_D0_IN_D1_OUT;
 	priv->wordlen = SPI_DEFAULT_WORDLEN;
 	return 0;
 }