From: Krzysztof Kozlowski <krzk@kernel.org>
To: Sander Vanheule <sander@svanheule.net>
Cc: Jonas Jelonek <jelonek.jonas@gmail.com>,
Oleksij Rempel <o.rempel@pengutronix.de>,
Kory Maincent <kory.maincent@bootlin.com>,
Andrew Lunn <andrew+netdev@lunn.ch>,
"David S. Miller" <davem@davemloft.net>,
Eric Dumazet <edumazet@kernel.org>,
Jakub Kicinski <kuba@kernel.org>,
Paolo Abeni <pabeni@redhat.com>, Rob Herring <robh@kernel.org>,
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 1/2] dt-bindings: net: pse-pd: realtek,pse: add gen1 I2C transport
Date: Tue, 20 Oct 2026 09:27:27 +0200 [thread overview]
Message-ID: <20261020-noisy-amorphous-jackal-1df08a@quoll> (raw)
In-Reply-To: <20261018161248.376205-2-sander@svanheule.net>
On Sun, Oct 18, 2026 at 06:12:47PM +0200, Sander Vanheule wrote:
> Like the gen2 protocol, the gen1 protocol can also be used over both a
> raw I2C transport and an SMBUs transport. Add a compatible for the
> former and the compatible for the Ubiquiti Unifi Switch Lite 16 PoE
> device that uses it.
>
> Signed-off-by: Sander Vanheule <sander@svanheule.net>
> ---
> .../bindings/net/pse-pd/realtek,pse-mcu-gen1.yaml | 7 +++++++
> 1 file changed, 7 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/net/pse-pd/realtek,pse-mcu-gen1.yaml b/Documentation/devicetree/bindings/net/pse-pd/realtek,pse-mcu-gen1.yaml
> index 3bb32349c28c..526f8525bb4d 100644
> --- a/Documentation/devicetree/bindings/net/pse-pd/realtek,pse-mcu-gen1.yaml
> +++ b/Documentation/devicetree/bindings/net/pse-pd/realtek,pse-mcu-gen1.yaml
> @@ -54,6 +54,12 @@ properties:
> - zyxel,gs1920-24hp-v2-pse
> - const: realtek,pse-mcu-gen1-smbus
>
> + # I2C, raw framing
> + - items:
> + - enum:
> + - ubnt,usw-lite-16-poe-pse
> + - const: realtek,pse-mcu-gen1-i2c
> +
> # UART
> - items:
> - enum:
> @@ -100,6 +106,7 @@ allOf:
> contains:
> enum:
> - realtek,pse-mcu-gen1-smbus
> + - realtek,pse-mcu-gen1-i2c
That's the same device as smbus, no? Same device, same compatible.
You can have property deciding about protocol, if this cannot be deduced
from the clients.
Best regards,
Krzysztof
next prev parent reply other threads:[~2026-10-20 7:27 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-10-18 16:12 [PATCH net-next 0/2] net: pse-pd: Plain I2C transport for Realtek Gen1 Sander Vanheule
2026-10-18 16:12 ` [PATCH net-next 1/2] dt-bindings: net: pse-pd: realtek,pse: add gen1 I2C transport Sander Vanheule
2026-10-20 7:27 ` Krzysztof Kozlowski [this message]
2026-10-18 16:12 ` [PATCH net-next 2/2] net: pse-pd: realtek-pse-mcu: " Sander Vanheule
2026-10-19 5:51 ` Oleksij Rempel
2026-10-19 6:56 ` Jonas Jelonek
2026-10-19 9:16 ` [PATCH net-next 0/2] net: pse-pd: Plain I2C transport for Realtek Gen1 Kory Maincent
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=20261020-noisy-amorphous-jackal-1df08a@quoll \
--to=krzk@kernel.org \
--cc=andrew+netdev@lunn.ch \
--cc=conor+dt@kernel.org \
--cc=davem@davemloft.net \
--cc=devicetree@vger.kernel.org \
--cc=edumazet@kernel.org \
--cc=jelonek.jonas@gmail.com \
--cc=kory.maincent@bootlin.com \
--cc=krzk+dt@kernel.org \
--cc=kuba@kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=netdev@vger.kernel.org \
--cc=o.rempel@pengutronix.de \
--cc=pabeni@redhat.com \
--cc=robh@kernel.org \
--cc=sander@svanheule.net \
/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
all inboxes | Powered by JetHome®