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)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Wed, 30 Dec 2020 10:53:47 +0000 (11:53 +0100)
[ Upstream commit c9f4cad6cdfe350ce2637e57f7f2aa7ff326bcc6 ]

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>
Signed-off-by: Sasha Levin <sashal@kernel.org>
drivers/net/can/m_can/m_can.c

index 61a93b19203799a71c07cf73a78309f12118ef4b..7fc4ac1582afc46e85c37246d0630feab3b41b32 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);