can: m_can: m_can_config_endisable(): remove double clearing of clock stop request bit
authorSean Nyekjaer <sean@geanix.com>
Wed, 11 Dec 2019 06:32:27 +0000 (07:32 +0100)
committerMarc Kleine-Budde <mkl@pengutronix.de>
Thu, 10 Dec 2020 09:10:43 +0000 (10:10 +0100)
The CSR bit is already cleared when arriving here so remove this section of
duplicate code.

The registers set in m_can_config_endisable() is set to same exact values as
before this patch.

Signed-off-by: Sean Nyekjaer <sean@geanix.com>
Acked-by: Sriram Dash <sriram.dash@samsung.com>
Acked-by: Dan Murphy <dmurphy@ti.com>
Link: https://lore.kernel.org/r/20191211063227.84259-1-sean@geanix.com
Fixes: f524f829b75a ("can: m_can: Create a m_can platform framework")
Signed-off-by: Marc Kleine-Budde <mkl@pengutronix.de>
drivers/net/can/m_can/m_can.c

index 05c978d1c53d9354e545ead661f0f02ae36620d4..8f389df26afc08dcfaffa2747d6d5078033530f1 100644 (file)
@@ -380,10 +380,6 @@ void m_can_config_endisable(struct m_can_classdev *cdev, bool enable)
                cccr &= ~CCCR_CSR;
 
        if (enable) {
-               /* Clear the Clock stop request if it was set */
-               if (cccr & CCCR_CSR)
-                       cccr &= ~CCCR_CSR;
-
                /* enable m_can configuration */
                m_can_write(cdev, M_CAN_CCCR, cccr | CCCR_INIT);
                udelay(5);