]> git.proxmox.com Git - mirror_ubuntu-bionic-kernel.git/commitdiff
can: m_can: add missing delay after setting CCCR_INIT bit
authorDong Aisheng <b29396@freescale.com>
Wed, 29 Oct 2014 10:45:24 +0000 (18:45 +0800)
committerMarc Kleine-Budde <mkl@pengutronix.de>
Tue, 18 Nov 2014 20:35:05 +0000 (21:35 +0100)
The spec mentions there may be a delay until the value written to INIT can be
read back due to the synchronization mechanism between the two clock domains.
But it does not indicate the exact clock cycles needed. The 5us delay is a
test value and seems ok.

Without the delay, CCCR.CCE bit may fail to be set and then the initialization
fail sometimes when do repeatly up and down.

Signed-off-by: Dong Aisheng <b29396@freescale.com>
Signed-off-by: Marc Kleine-Budde <mkl@pengutronix.de>
drivers/net/can/m_can/m_can.c

index 98f7e0ea7f6a7eafb2009f123d863f85ae3dce81..3ad7d88720b7302644c59c314a835882aeb3bde4 100644 (file)
@@ -296,6 +296,7 @@ static inline void m_can_config_endisable(const struct m_can_priv *priv,
        if (enable) {
                /* enable m_can configuration */
                m_can_write(priv, M_CAN_CCCR, cccr | CCCR_INIT);
+               udelay(5);
                /* CCCR.CCE can only be set/reset while CCCR.INIT = '1' */
                m_can_write(priv, M_CAN_CCCR, cccr | CCCR_INIT | CCCR_CCE);
        } else {