[src/trunk]: src/sys/arch/arm/imx Keep SCLK high in the inactive state. (SPI ...

[hkenken <hkenken%NetBSD.org@localhost>]

details:   https://anonhg.NetBSD.org/src/rev/f8edfaa2d051
branches:  trunk
changeset: 465280:f8edfaa2d051
user:      hkenken <hkenken%NetBSD.org@localhost>
date:      Fri Nov 15 01:10:42 2019 +0000

description:
Keep SCLK high in the inactive state. (SPI modes 2 and 3 only)

diffstat:

 sys/arch/arm/imx/imxspi.c |  9 ++++++---
 1 files changed, 6 insertions(+), 3 deletions(-)

diffs (49 lines):

diff -r 5455e5e94ba0 -r f8edfaa2d051 sys/arch/arm/imx/imxspi.c
--- a/sys/arch/arm/imx/imxspi.c Thu Nov 14 21:23:46 2019 +0000
+++ b/sys/arch/arm/imx/imxspi.c Fri Nov 15 01:10:42 2019 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: imxspi.c,v 1.6 2019/09/27 02:59:21 hkenken Exp $       */
+/*     $NetBSD: imxspi.c,v 1.7 2019/11/15 01:10:42 hkenken Exp $       */
 
 /*-
  * Copyright (c) 2014  Genetec Corporation.  All rights reserved.
@@ -32,7 +32,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: imxspi.c,v 1.6 2019/09/27 02:59:21 hkenken Exp $");
+__KERNEL_RCSID(0, "$NetBSD: imxspi.c,v 1.7 2019/11/15 01:10:42 hkenken Exp $");
 
 #include "opt_imxspi.h"
 #include "opt_fdt.h"
@@ -69,7 +69,7 @@
 
 #define        IMXCSPI_TYPE(type, x)                                                 \
        ((sc->sc_type == IMX31_CSPI) ? __CONCAT(CSPI_IMX31_, x) :             \
-           (sc->sc_type == IMX35_CSPI) ? __CONCAT(CSPI_IMX35_, x) : 0)
+        (sc->sc_type == IMX35_CSPI) ? __CONCAT(CSPI_IMX35_, x) : 0)
 #define        IMXCSPI(x)      __CONCAT(CSPI_, x)
 #define        IMXESPI(x)      __CONCAT(ECSPI_, x)
 #define        IMXSPI(x)       ((sc->sc_enhanced) ? IMXESPI(x) : IMXCSPI(x))
@@ -228,6 +228,7 @@
 
        config = bus_space_read_4(sc->sc_iot, sc->sc_ioh, ECSPI_CONFIGREG);
        config &= ~(__SHIFTIN(__BIT(slave), ECSPI_CONFIG_SCLK_POL) |
+           __SHIFTIN(__BIT(slave), ECSPI_CONFIG_SCLK_CTL) |
            __SHIFTIN(__BIT(slave), ECSPI_CONFIG_SCLK_PHA));
        switch (mode) {
        case SPI_MODE_0:
@@ -240,11 +241,13 @@
        case SPI_MODE_2:
                /* CPHA = 0, CPOL = 1 */
                config |= __SHIFTIN(__BIT(slave), ECSPI_CONFIG_SCLK_POL);
+               config |= __SHIFTIN(__BIT(slave), ECSPI_CONFIG_SCLK_CTL);
                break;
        case SPI_MODE_3:
                /* CPHA = 1, CPOL = 1 */
                config |= __SHIFTIN(__BIT(slave), ECSPI_CONFIG_SCLK_PHA);
                config |= __SHIFTIN(__BIT(slave), ECSPI_CONFIG_SCLK_POL);
+               config |= __SHIFTIN(__BIT(slave), ECSPI_CONFIG_SCLK_CTL);
                break;
        default:
                return EINVAL;