Re: [PATCH 2/2] net: dsa: rzn1_a5psw: use devm to enable clocks

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

To: Rosen Penev <rosenp@xxxxxxxxx>
Subject: Re: [PATCH 2/2] net: dsa: rzn1_a5psw: use devm to enable clocks
From: Andrew Lunn <andrew@xxxxxxx>
Date: Mon, 7 Jul 2025 07:56:28 +0200
Cc: netdev@xxxxxxxxxxxxxxx, Vladimir Oltean <olteanv@xxxxxxxxx>, "David S. Miller" <davem@xxxxxxxxxxxxx>, Eric Dumazet <edumazet@xxxxxxxxxx>, Jakub Kicinski <kuba@xxxxxxxxxx>, Paolo Abeni <pabeni@xxxxxxxxxx>, Clément Léger <clement.leger@xxxxxxxxxxx>, open list <linux-kernel@xxxxxxxxxxxxxxx>, "open list:RENESAS RZ/N1 A5PSW SWITCH DRIVER" <linux-renesas-soc@xxxxxxxxxxxxxxx>
In-reply-to: <20250707003918.21607-3-rosenp@gmail.com>
References: <20250707003918.21607-1-rosenp@gmail.com> <20250707003918.21607-3-rosenp@gmail.com>

On Sun, Jul 06, 2025 at 05:39:18PM -0700, Rosen Penev wrote:
> The remove function has these in the wrong order. The switch should be
> unregistered last. Simpler to use devm so that the right thing is done.
> 
> Signed-off-by: Rosen Penev <rosenp@xxxxxxxxx>

Reviewed-by: Andrew Lunn <andrew@xxxxxxx>

    Andrew

References:
- [PATCH 0/2] net: dsa: rzn1_a5psw: add COMPILE_TEST
  - From: Rosen Penev
- [PATCH 2/2] net: dsa: rzn1_a5psw: use devm to enable clocks
  - From: Rosen Penev

Prev by Date: Re: [PATCH 1/2] net: dsa: rzn1_a5psw: add COMPILE_TEST
Next by Date: Re: [PATCH 1/2] net: dsa: rzn1_a5psw: add COMPILE_TEST
Previous by thread: [PATCH 2/2] net: dsa: rzn1_a5psw: use devm to enable clocks
Index(es):
- Date
- Thread

[Index of Archives] [Linux Samsung SOC] [Linux Wireless] [Linux Kernel] [ATH6KL] [Linux Bluetooth] [Linux Netdev] [Kernel Newbies] [IDE] [Security] [Git] [Netfilter] [Bugtraq] [Yosemite News] [MIPS Linux] [ARM Linux] [Linux Security] [Linux RAID] [Linux ATA RAID] [Samba] [Device Mapper]

Powered by Linux