From: "Markus Stockhausen" <markus.stockhausen@gmx.de>
To: "'Jakub Kicinski'" <kuba@kernel.org>
Cc: <andrew@lunn.ch>, <hkallweit1@gmail.com>, <linux@armlinux.org.uk>,
<davem@davemloft.net>, <edumazet@google.com>, <pabeni@redhat.com>,
<netdev@vger.kernel.org>, <chris.packham@alliedtelesis.co.nz>,
<daniel@makrotopia.org>, <robh@kernel.org>, <krzk+dt@kernel.org>,
<conor+dt@kernel.org>, <devicetree@vger.kernel.org>
Subject: AW: [PATCH net-next v4 3/8] net: phy: add (*notify_phy_attach/detach)() hooks to struct mii_bus
Date: Tue, 21 Jul 2026 21:59:54 +0200 [thread overview]
Message-ID: <009001dd194b$835acaf0$8a1060d0$@gmx.de> (raw)
In-Reply-To: <20260721181549.855625-1-kuba@kernel.org>
HI Jakub,
>Von: Jakub Kicinski <kuba@kernel.org>
>Gesendet: Dienstag, 21. Juli 2026 20:16
>An: markus.stockhausen@gmx.de
>Betreff: Re: [PATCH net-next v4 3/8] net: phy: add
(*notify_phy_attach/detach)() hooks to struct mii_bus
>
> This is an AI-generated review of your patch. The human sending this
> email has considered the AI review valid, or at least plausible.
> ---
> net: phy: add (*notify_phy_attach/detach)() hooks to struct mii_bus
> ...
> [Medium]
> Is the pairing between notify_phy_attach and notify_phy_detach
> symmetric on all failure paths?
thanks for the check. I already asked for guidance on this [1]
but got no feedback yet. Maybe someone can assist what is
the best way to make these calls symmetrical.
Markus
[1] https://lore.kernel.org/netdev/008601dd0d6d$f3b143e0$db13cba0$@gmx.de/
next prev parent reply other threads:[~2026-07-21 20:00 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-07-09 6:41 [PATCH net-next v4 0/8] net: mdio: realtek-rtl9300: Add RTL83xx support Markus Stockhausen
2026-07-09 6:41 ` [PATCH net-next v4 1/8] dt-bindings: net: realtek,rtl9301-mdio: Add RTL83xx series Markus Stockhausen
2026-07-09 6:41 ` [PATCH net-next v4 2/8] net: mdio: realtek-rtl9300: Add polling documentation Markus Stockhausen
2026-07-09 6:41 ` [PATCH net-next v4 3/8] net: phy: add (*notify_phy_attach/detach)() hooks to struct mii_bus Markus Stockhausen
2026-07-21 18:15 ` Jakub Kicinski
2026-07-21 19:59 ` Markus Stockhausen [this message]
2026-07-21 20:45 ` Jakub Kicinski
2026-07-09 6:41 ` [PATCH net-next v4 4/8] net: mdio: realtek-rtl9300: Configure hardware polling during probing Markus Stockhausen
2026-07-09 6:41 ` [PATCH net-next v4 5/8] net: mdio: realtek-rtl9300: Add page tracking Markus Stockhausen
2026-07-09 6:41 ` [PATCH net-next v4 6/8] net: mdio: realtek-rtl9300: Increase MDIO timeout Markus Stockhausen
2026-07-09 6:41 ` [PATCH net-next v4 7/8] net: mdio: realtek-rtl9300: Add support for RTL838x Markus Stockhausen
2026-07-09 6:41 ` [PATCH net-next v4 8/8] net: mdio: realtek-rtl9300: Add support for RTL839x Markus Stockhausen
2026-07-21 15:17 ` AW: [PATCH net-next v4 0/8] net: mdio: realtek-rtl9300: Add RTL83xx support Markus Stockhausen
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to='009001dd194b$835acaf0$8a1060d0$@gmx.de' \
--to=markus.stockhausen@gmx.de \
--cc=andrew@lunn.ch \
--cc=chris.packham@alliedtelesis.co.nz \
--cc=conor+dt@kernel.org \
--cc=daniel@makrotopia.org \
--cc=davem@davemloft.net \
--cc=devicetree@vger.kernel.org \
--cc=edumazet@google.com \
--cc=hkallweit1@gmail.com \
--cc=krzk+dt@kernel.org \
--cc=kuba@kernel.org \
--cc=linux@armlinux.org.uk \
--cc=netdev@vger.kernel.org \
--cc=pabeni@redhat.com \
--cc=robh@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox