From: Francesco Dolcini Date: Thu, 4 Jan 2024 23:57:23 +0000 (+0100) Subject: can: m_can: remove redundant check for pm_clock_support X-Git-Tag: v6.9-rc1~159^2~219^2~4 X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=e517293fd72d9044902efae59f05203203a60736;p=thirdparty%2Flinux.git can: m_can: remove redundant check for pm_clock_support m_can_clk_start() already skip starting the clock when clock support is disabled, remove the redundant check in m_can_class_register(). This also solves the imbalance with m_can_clk_stop() that is called afterward in the same function before the return. Signed-off-by: Francesco Dolcini Reviewed-by: Markus Schneider-Pargmann Link: https://lore.kernel.org/all/20240104235723.46931-1-francesco@dolcini.it [mkl: rebased to net-next/main] Signed-off-by: Marc Kleine-Budde --- diff --git a/drivers/net/can/m_can/m_can.c b/drivers/net/can/m_can/m_can.c index b7dbce4c342a7..290880ce5329c 100644 --- a/drivers/net/can/m_can/m_can.c +++ b/drivers/net/can/m_can/m_can.c @@ -2312,11 +2312,9 @@ int m_can_class_register(struct m_can_classdev *cdev) } } - if (cdev->pm_clock_support) { - ret = m_can_clk_start(cdev); - if (ret) - return ret; - } + ret = m_can_clk_start(cdev); + if (ret) + return ret; if (cdev->is_peripheral) { ret = can_rx_offload_add_manual(cdev->net, &cdev->offload,