[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <aN0oNgEp08BaGeTJ@wunner.de>
Date: Wed, 1 Oct 2025 15:10:14 +0200
From: Lukas Wunner <lukas@...ner.de>
To: Oleksij Rempel <o.rempel@...gutronix.de>
Cc: Andrew Lunn <andrew+netdev@...n.ch>,
"David S. Miller" <davem@...emloft.net>,
Eric Dumazet <edumazet@...gle.com>,
Jakub Kicinski <kuba@...nel.org>, Paolo Abeni <pabeni@...hat.com>,
Hubert Wi??niewski <hubert.wisniewski.25632@...il.com>,
Marek Szyprowski <m.szyprowski@...sung.com>, stable@...r.kernel.org,
kernel@...gutronix.de, linux-kernel@...r.kernel.org,
netdev@...r.kernel.org, Russell King <linux@...linux.org.uk>,
Xu Yang <xu.yang_2@....com>, linux-usb@...r.kernel.org
Subject: Re: [PATCH net v2 1/1] net: usb: asix: hold PM usage ref to avoid
PM/MDIO + RTNL deadlock
On Wed, Oct 01, 2025 at 03:04:32PM +0200, Oleksij Rempel wrote:
> @@ -1600,6 +1624,10 @@ static struct usb_driver asix_driver = {
> .resume = asix_resume,
> .reset_resume = asix_resume,
> .disconnect = usbnet_disconnect,
> + /* usbnet will force supports_autosuspend=1; we explicitly forbid RPM
> + * per-interface in bind to keep autosuspend disabled for this driver
> + * by using pm_runtime_forbid().
> + */
Looks like this code comment needs an update, now that you're no longer
using pm_runtime_forbid()?
Thanks,
Lukas
Powered by blists - more mailing lists