[PATCH net-next v6 01/12] dt-bindings: net: airoha: Add EN7581 ethernet-ports properties
Lorenzo Bianconi
lorenzo at kernel.org
Fri May 15 02:12:00 PDT 2026
> On Mon, May 11, 2026 at 12:49:27PM +0200, Lorenzo Bianconi wrote:
> > EN7581 and AN7583 SoCs support connecting multiple external SerDes to GDM3
> > or GDM4 ports via a hw arbiter that manages the traffic in a TDM manner.
> > As a result multiple net_devices can connect to the same GDM{3,4} port
> > and there is a theoretical "1:n" relation between GDM ports and
> > net_devices.
> > Introduce the ethernet-port property in order to model a given net_device
> > that is connected via the external arbiter to the GDM{3,4} port (that
> > is represented by the ethernet property. Please note GDM1 or GDM2 does not
> > support the connection with the external arbiter and are represented
> > by ethernet property.
> >
> > Signed-off-by: Lorenzo Bianconi <lorenzo at kernel.org>
> > ---
> > .../devicetree/bindings/net/airoha,en7581-eth.yaml | 64 +++++++++++++++++++++-
> > 1 file changed, 63 insertions(+), 1 deletion(-)
> >
> > diff --git a/Documentation/devicetree/bindings/net/airoha,en7581-eth.yaml b/Documentation/devicetree/bindings/net/airoha,en7581-eth.yaml
> > index fbe2ddcdd909..642f300c0945 100644
> > --- a/Documentation/devicetree/bindings/net/airoha,en7581-eth.yaml
> > +++ b/Documentation/devicetree/bindings/net/airoha,en7581-eth.yaml
> > @@ -130,6 +130,50 @@ patternProperties:
> > maximum: 4
> > description: GMAC port identifier
> >
> > + '#address-cells':
> > + const: 1
>
> blank line
ack, I will fix it in v7.
>
> > + '#size-cells':
> > + const: 0
> > +
> > + allOf:
> > + - if:
> > + properties:
> > + reg:
> > + contains:
> > + items:
> > + - enum:
> > + - 3
> > + - 4
> > + then:
> > + properties:
> > + '#address-cells':
> > + const: 1
> > + '#size-cells':
> > + const: 0
>
> Why do you have these twice? Drop this one.
ack, I will fix it in v7.
>
> > +
> > + patternProperties:
> > + "^ethernet-port@[0-5]$":
> > + type: object
> > + unevaluatedProperties: false
> > + $ref: ethernet-controller.yaml#
> > + description: External ethernet port ID available on the GDM port
> > +
> > + properties:
> > + compatible:
> > + const: airoha,eth-port
> > +
> > + reg:
> > + maxItems: 1
>
> Instead, 'maximum: 5'.
ack, I will fix it in v7.
>
> > + description: External ethernet port identifier
> > +
> > + required:
> > + - reg
> > + - compatible
> > +
> > + required:
> > + - "#address-cells"
> > + - "#size-cells"
> > +
> > required:
> > - reg
> > - compatible
> > @@ -191,9 +235,27 @@ examples:
> > #address-cells = <1>;
> > #size-cells = <0>;
> >
> > - mac: ethernet at 1 {
> > + mac1: ethernet at 1 {
>
> Just drop unused labels.
ack, I will fix it in v7.
Regards,
Lorenzo
>
> > compatible = "airoha,eth-mac";
> > reg = <1>;
> > };
> > +
> > + mac4: ethernet at 4 {
> > + compatible = "airoha,eth-mac";
> > + reg = <4>;
> > +
> > + #address-cells = <1>;
> > + #size-cells = <0>;
> > +
> > + ethernet-port at 0 {
> > + compatible = "airoha,eth-port";
> > + reg = <0>;
> > + };
> > +
> > + ethernet-port at 1 {
> > + compatible = "airoha,eth-port";
> > + reg = <1>;
> > + };
> > + };
> > };
> > };
> >
> > --
> > 2.54.0
> >
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 228 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-mediatek/attachments/20260515/dbdb42b3/attachment.sig>
More information about the Linux-mediatek
mailing list