netdev - Re: [net 11/15] can: tcan4x5x: tcan4x5x_can_remove(): fix order of deregistration

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <67debcb1-5fb6-76bd-b1b1-24d97389aa6b@ti.com>
Date:   Tue, 17 Nov 2020 08:00:45 -0600
From:   Dan Murphy <dmurphy@...com>
To:     Marc Kleine-Budde <mkl@...gutronix.de>, <netdev@...r.kernel.org>
CC:     <davem@...emloft.net>, <kuba@...nel.org>,
        <linux-can@...r.kernel.org>, <kernel@...gutronix.de>
Subject: Re: [net 11/15] can: tcan4x5x: tcan4x5x_can_remove(): fix order of
 deregistration

Marc

On 11/15/20 11:41 AM, Marc Kleine-Budde wrote:
> Change the order in tcan4x5x_can_remove() to be the exact inverse of
> tcan4x5x_can_probe(). First m_can_class_unregister(), then power down the
> device.
>
> Fixes: 5443c226ba91 ("can: tcan4x5x: Add tcan4x5x driver to the kernel")
> Cc: Dan Murphy <dmurphy@...com>
> Link: http://lore.kernel.org/r/20201019154233.1262589-10-mkl@pengutronix.de
> Signed-off-by: Marc Kleine-Budde <mkl@...gutronix.de>
> ---
>   drivers/net/can/m_can/tcan4x5x.c | 4 ++--
>   1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/net/can/m_can/tcan4x5x.c b/drivers/net/can/m_can/tcan4x5x.c
> index f058bd9104e9..4fdb7121403a 100644
> --- a/drivers/net/can/m_can/tcan4x5x.c
> +++ b/drivers/net/can/m_can/tcan4x5x.c
> @@ -527,10 +527,10 @@ static int tcan4x5x_can_remove(struct spi_device *spi)
>   {
>   	struct tcan4x5x_priv *priv = spi_get_drvdata(spi);
>   
> -	tcan4x5x_power_enable(priv->power, 0);
> -
>   	m_can_class_unregister(priv->mcan_dev);
>   
> +	tcan4x5x_power_enable(priv->power, 0);
> +
>   	return 0;
>   }
>   

Reviewed-by: Dan Murphy<dmurphy@...com>