Hi Rob, On 03/09/24 8:13 pm, Rob Herring wrote: > EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe > > On Tue, Sep 03, 2024 at 04:17:05PM +0530, Parthiban Veerasooran wrote: >> The LAN8650/1 combines a Media Access Controller (MAC) and an Ethernet >> PHY to enable 10BASE-T1S networks. The Ethernet Media Access Controller >> (MAC) module implements a 10 Mbps half duplex Ethernet MAC, compatible >> with the IEEE 802.3 standard and a 10BASE-T1S physical layer transceiver >> integrated into the LAN8650/1. The communication between the Host and the >> MAC-PHY is specified in the OPEN Alliance 10BASE-T1x MACPHY Serial >> Interface (TC6). >> >> Reviewed-by: Conor Dooley <conor.dooley@xxxxxxxxxxxxx> >> Reviewed-by: Andrew Lunn <andrew@xxxxxxx> >> Signed-off-by: Parthiban Veerasooran <Parthiban.Veerasooran@xxxxxxxxxxxxx> >> --- >> .../bindings/net/microchip,lan8650.yaml | 80 +++++++++++++++++++ >> MAINTAINERS | 1 + >> 2 files changed, 81 insertions(+) >> create mode 100644 Documentation/devicetree/bindings/net/microchip,lan8650.yaml >> >> diff --git a/Documentation/devicetree/bindings/net/microchip,lan8650.yaml b/Documentation/devicetree/bindings/net/microchip,lan8650.yaml >> new file mode 100644 >> index 000000000000..b7b755b27b78 >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/net/microchip,lan8650.yaml >> @@ -0,0 +1,80 @@ >> +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) >> +%YAML 1.2 >> +--- >> +$id: http://devicetree.org/schemas/net/microchip,lan8650.yaml# >> +$schema: http://devicetree.org/meta-schemas/core.yaml# >> + >> +title: Microchip LAN8650/1 10BASE-T1S MACPHY Ethernet Controllers >> + >> +maintainers: >> + - Parthiban Veerasooran <parthiban.veerasooran@xxxxxxxxxxxxx> >> + >> +description: >> + The LAN8650/1 combines a Media Access Controller (MAC) and an Ethernet >> + PHY to enable 10BASE‑T1S networks. The Ethernet Media Access Controller >> + (MAC) module implements a 10 Mbps half duplex Ethernet MAC, compatible >> + with the IEEE 802.3 standard and a 10BASE-T1S physical layer transceiver >> + integrated into the LAN8650/1. The communication between the Host and >> + the MAC-PHY is specified in the OPEN Alliance 10BASE-T1x MACPHY Serial >> + Interface (TC6). >> + >> +allOf: >> + - $ref: /schemas/net/ethernet-controller.yaml# >> + - $ref: /schemas/spi/spi-peripheral-props.yaml# >> + >> +properties: >> + compatible: >> + oneOf: >> + - const: microchip,lan8650 >> + - items: >> + - const: microchip,lan8651 >> + - const: microchip,lan8650 >> + >> + reg: >> + maxItems: 1 >> + >> + interrupts: >> + description: >> + Interrupt from MAC-PHY asserted in the event of Receive Chunks >> + Available, Transmit Chunk Credits Available and Extended Status >> + Event. >> + maxItems: 1 >> + >> + spi-max-frequency: >> + minimum: 15000000 >> + maximum: 25000000 >> + > >> + "#address-cells": >> + const: 1 >> + >> + "#size-cells": >> + const: 0 > > What are these for? They apply to child nodes, yet you have no child > nodes defined. Ah yes, thanks for pointing it out. By mistake they are added by referring spi node. I will remove them in the next version. Best regards, Parthiban V > > Rob
