From: Rob Herring <robh@kernel.org>
To: Buday Csaba <buday.csaba@prolan.hu>
Cc: Andrew Lunn <andrew@lunn.ch>,
Heiner Kallweit <hkallweit1@gmail.com>,
Russell King <linux@armlinux.org.uk>,
"David S. Miller" <davem@davemloft.net>,
Eric Dumazet <edumazet@google.com>,
Jakub Kicinski <kuba@kernel.org>, Paolo Abeni <pabeni@redhat.com>,
Krzysztof Kozlowski <krzk+dt@kernel.org>,
Conor Dooley <conor+dt@kernel.org>,
Philipp Zabel <p.zabel@pengutronix.de>,
Florian Fainelli <f.fainelli@gmail.com>,
netdev@vger.kernel.org, devicetree@vger.kernel.org,
linux-kernel@vger.kernel.org
Subject: Re: [PATCH net-next v3 3/4] dt-bindings: net: mdio: add phy-id-read-needs-reset property
Date: Sun, 26 Oct 2025 16:20:26 -0500 [thread overview]
Message-ID: <20251026212026.GA2959311-robh@kernel.org> (raw)
In-Reply-To: <3a93b87f2bf7ea87c1251d2360e11e710772dd92.1760620093.git.buday.csaba@prolan.hu>
On Fri, Oct 17, 2025 at 06:10:10PM +0200, Buday Csaba wrote:
> Some Ethernet PHYs require a hard reset before accessing their MDIO
> registers. When the ID is not provided by a compatible string,
> reading the PHY ID may fail on such devices.
>
> This patch introduces a new device tree property called
> `phy-id-read-needs-reset`, which can be used to hard reset the
> PHY before attempting to read its ID via MDIO.
If your phy needs special handling, provide a compatible and use that.
>
> Signed-off-by: Buday Csaba <buday.csaba@prolan.hu>
> ---
> V2 -> V3: unchanged
> V1 -> V2: added this DT binding
> ---
> Documentation/devicetree/bindings/net/ethernet-phy.yaml | 8 ++++++++
> 1 file changed, 8 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/net/ethernet-phy.yaml b/Documentation/devicetree/bindings/net/ethernet-phy.yaml
> index 2ec2d9fda..b570f8038 100644
> --- a/Documentation/devicetree/bindings/net/ethernet-phy.yaml
> +++ b/Documentation/devicetree/bindings/net/ethernet-phy.yaml
> @@ -215,6 +215,14 @@ properties:
> Delay after the reset was deasserted in microseconds. If
> this property is missing the delay will be skipped.
>
> + phy-id-read-needs-reset:
> + $ref: /schemas/types.yaml#/definitions/flag
> + description:
> + Some PHYs require a hard reset before accessing MDIO registers.
> + This workaround allows auto-detection of the PHY ID in such cases.
> + When the PHY ID is provided with the 'compatible' string, setting
> + this property has no effect.
> +
> sfp:
> $ref: /schemas/types.yaml#/definitions/phandle
> description:
> --
> 2.39.5
>
>
next prev parent reply other threads:[~2025-10-26 21:20 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-10-17 16:10 [PATCH net-next v3 0/4] net: mdio: implement optional PHY reset before MDIO access Buday Csaba
2025-10-17 16:10 ` [PATCH net-next v3 1/4] net: mdio: common handling of phy reset properties Buday Csaba
2025-10-17 16:10 ` [PATCH net-next v3 2/4] net: mdio: change property read from fwnode_property_read_u32() to device_property_read_u32() Buday Csaba
2025-10-17 16:10 ` [PATCH net-next v3 3/4] dt-bindings: net: mdio: add phy-id-read-needs-reset property Buday Csaba
2025-10-26 21:20 ` Rob Herring [this message]
2025-10-17 16:10 ` [PATCH net-next v3 4/4] net: mdio: reset PHY before attempting to access registers in fwnode_mdiobus_register_phy Buday Csaba
2025-10-20 9:16 ` Philipp Zabel
2025-10-21 7:24 ` Buday Csaba
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=20251026212026.GA2959311-robh@kernel.org \
--to=robh@kernel.org \
--cc=andrew@lunn.ch \
--cc=buday.csaba@prolan.hu \
--cc=conor+dt@kernel.org \
--cc=davem@davemloft.net \
--cc=devicetree@vger.kernel.org \
--cc=edumazet@google.com \
--cc=f.fainelli@gmail.com \
--cc=hkallweit1@gmail.com \
--cc=krzk+dt@kernel.org \
--cc=kuba@kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux@armlinux.org.uk \
--cc=netdev@vger.kernel.org \
--cc=p.zabel@pengutronix.de \
--cc=pabeni@redhat.com \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.