On 01/02/2024 22:52, Daniel Golle wrote: > Add bindings for the MediaTek XFI T-PHY Ethernet SerDes PHY found in the > MediaTek MT7988 SoC which can operate at various interfaces modes: > > via USXGMII PCS: > * USXGMII > * 10GBase-R > * 5GBase-R > > via LynxI SGMII PCS: > * 2500Base-X > * 1000Base-X > * Cisco SGMII (MAC side) > > Signed-off-by: Daniel Golle <daniel@xxxxxxxxxxxxxx> > --- > .../bindings/phy/mediatek,xfi-tphy.yaml | 80 +++++++++++++++++++ > 1 file changed, 80 insertions(+) > create mode 100644 Documentation/devicetree/bindings/phy/mediatek,xfi-tphy.yaml > > diff --git a/Documentation/devicetree/bindings/phy/mediatek,xfi-tphy.yaml b/Documentation/devicetree/bindings/phy/mediatek,xfi-tphy.yaml > new file mode 100644 > index 0000000000000..e897118dcf7e6 > --- /dev/null > +++ b/Documentation/devicetree/bindings/phy/mediatek,xfi-tphy.yaml > @@ -0,0 +1,80 @@ > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/phy/mediatek,xfi-tphy.yaml# Please use compatible as filename. Your binding says only one is possible (const, not enum), so there is no reasoning for different filename. > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: MediaTek XFI T-PHY > + > +maintainers: > + - Daniel Golle <daniel@xxxxxxxxxxxxxx> > + > +description: > + The MediaTek XFI SerDes T-PHY provides the physical SerDes lanes > + used by the (10G/5G) USXGMII PCS and (1G/2.5G) LynxI PCS found in > + MediaTek's 10G-capabale SoCs. > + > +properties: > + $nodename: > + pattern: "^phy@[0-9a-f]+$" No need for nodename in individual bindings file. > + > + compatible: > + const: mediatek,mt7988-xfi-tphy > + > + reg: > + maxItems: 1 > + > + clocks: > + items: > + - description: XFI PHY clock > + - description: XFI register clock > + > + clock-names: > + items: > + - const: xfipll > + - const: topxtal > + > + resets: > + items: > + - description: PEXTP reset > + > + mediatek,usxgmii-performance-errata: > + $ref: /schemas/types.yaml#/definitions/flag > + description: > + One instance of the T-PHY on MT7988 suffers from a performance > + problem in 10GBase-R mode which needs a work-around in the driver. Can you explain what is this issue and errata about (except performance)? > + The work-around is enabled using this flag. > + > + "#phy-cells": > + const: 0 > + > +required: > + - compatible > + - reg > + - clocks > + - clock-names > + - resets > + - "#phy-cells" > + > +additionalProperties: false Best regards, Krzysztof