From: Rob Herring <robh@kernel.org>
To: Daniel Golle <daniel@makrotopia.org>
Cc: Hauke Mehrtens <hauke@hauke-m.de>, Andrew Lunn <andrew@lunn.ch>,
Vladimir Oltean <olteanv@gmail.com>,
"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>,
netdev@vger.kernel.org, devicetree@vger.kernel.org,
linux-kernel@vger.kernel.org
Subject: Re: [PATCH net-next v3 1/3] dt-bindings: net: dsa: lantiq,gswip: reference common PHY properties
Date: Mon, 9 Feb 2026 17:54:13 -0600 [thread overview]
Message-ID: <20260209235413.GA2137170-robh@kernel.org> (raw)
In-Reply-To: <f556ef8be75e37a2f864b9d905a78962bbe76d18.1769916962.git.daniel@makrotopia.org>
On Sun, Feb 01, 2026 at 03:41:53AM +0000, Daniel Golle wrote:
> Reference the common PHY properties so RX and TX SerDes lane polarity
> of the SGMII/1000Base-X/2500Base-X port can be configured.
>
> Signed-off-by: Daniel Golle <daniel@makrotopia.org>
> ---
> v3: commit message: clarify that the intention is to configure polarity
> at port level, as opposed to the internal polarity of the PCS component
> v2: use allOf to include PHY common properties, add example use
> ---
> Documentation/devicetree/bindings/net/dsa/lantiq,gswip.yaml | 4 ++++
> 1 file changed, 4 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/net/dsa/lantiq,gswip.yaml b/Documentation/devicetree/bindings/net/dsa/lantiq,gswip.yaml
> index f601e5f9fa6a..b4a31cde4322 100644
> --- a/Documentation/devicetree/bindings/net/dsa/lantiq,gswip.yaml
> +++ b/Documentation/devicetree/bindings/net/dsa/lantiq,gswip.yaml
> @@ -105,6 +105,8 @@ patternProperties:
> patternProperties:
> "^(ethernet-)?port@[0-6]$":
> $ref: dsa-port.yaml#
> + allOf:
> + - $ref: /schemas/phy/phy-common-props.yaml#
If you respin, put both $ref's under the allOf.
Acked-by: Rob Herring (Arm) <robh@kernel.org>
> unevaluatedProperties: false
>
> properties:
> @@ -288,6 +290,7 @@ examples:
>
> - |
> #include <dt-bindings/leds/common.h>
> + #include <dt-bindings/phy/phy.h>
>
> mdio {
> #address-cells = <1>;
> @@ -320,6 +323,7 @@ examples:
> label = "wan";
> phy-mode = "1000base-x";
> managed = "in-band-status";
> + tx-polarity = <PHY_POL_INVERT>;
> };
>
> port@5 {
> --
> 2.52.0
next prev parent reply other threads:[~2026-02-09 23:54 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-02-01 3:41 [PATCH net-next v3 0/3] net: dsa: mxl-gsw1xx: setup polarities and validate chip Daniel Golle
2026-02-01 3:41 ` [PATCH net-next v3 1/3] dt-bindings: net: dsa: lantiq,gswip: reference common PHY properties Daniel Golle
2026-02-09 23:54 ` Rob Herring [this message]
2026-02-01 3:42 ` [PATCH net-next v3 2/3] net: dsa: mxl-gsw1xx: configure SerDes port polarities Daniel Golle
2026-02-01 3:42 ` [PATCH net-next v3 3/3] net: dsa: mxl-gsw1xx: validate chip ID Daniel Golle
2026-02-01 8:31 ` [PATCH net-next v3 0/3] net: dsa: mxl-gsw1xx: setup polarities and validate chip Vladimir Oltean
2026-02-10 8:20 ` patchwork-bot+netdevbpf
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=20260209235413.GA2137170-robh@kernel.org \
--to=robh@kernel.org \
--cc=andrew@lunn.ch \
--cc=conor+dt@kernel.org \
--cc=daniel@makrotopia.org \
--cc=davem@davemloft.net \
--cc=devicetree@vger.kernel.org \
--cc=edumazet@google.com \
--cc=hauke@hauke-m.de \
--cc=krzk+dt@kernel.org \
--cc=kuba@kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=netdev@vger.kernel.org \
--cc=olteanv@gmail.com \
--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.