On 15.08.2021 00:26, David Bauer wrote: > Hi Heiner, > > On 8/14/21 8:33 PM, Heiner Kallweit wrote: >> On 14.08.2021 20:11, David Bauer wrote: >>> Add binding documentation for the Realtek RTL8152 / RTL8153 USB ethernet >>> adapters. >>> >>> Signed-off-by: David Bauer <mail@xxxxxxxxxxxxxxx> >>> --- >>> .../bindings/net/realtek,rtl8152.yaml | 43 +++++++++++++++++++ >>> 1 file changed, 43 insertions(+) >>> create mode 100644 Documentation/devicetree/bindings/net/realtek,rtl8152.yaml >>> >>> diff --git a/Documentation/devicetree/bindings/net/realtek,rtl8152.yaml b/Documentation/devicetree/bindings/net/realtek,rtl8152.yaml >>> new file mode 100644 >>> index 000000000000..ab760000b3a6 >>> --- /dev/null >>> +++ b/Documentation/devicetree/bindings/net/realtek,rtl8152.yaml >>> @@ -0,0 +1,43 @@ >>> +# SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause >>> +%YAML 1.2 >>> +--- >>> +$id: http://devicetree.org/schemas/net/realtek,rtl8152.yaml# >>> +$schema: http://devicetree.org/meta-schemas/core.yaml# >>> + >>> +title: Realtek RTL8152/RTL8153 series USB ethernet >>> + >>> +maintainers: >>> + - David Bauer <mail@xxxxxxxxxxxxxxx> >>> + >>> +properties: >>> + compatible: >>> + oneOf: >>> + - items: >>> + - enum: >>> + - realtek,rtl8152 >>> + - realtek,rtl8153 >>> + >>> + reg: >>> + description: The device number on the USB bus >>> + >>> + realtek,led-data: >>> + $ref: /schemas/types.yaml#/definitions/uint32 >>> + description: Value to be written to the LED configuration register. >>> + >> >> +Pavel as LED subsystem maintainer >> >> There's an ongoing discussion (with certain decisions taken already) about >> how to configure network device LEDs. > > Thanks, I didn't knew about this. > > Is there any place where i can read up specifics about > this topic? > A recent mail thread about network device LEDs is here: https://lore.kernel.org/netdev/20210716212427.821834-6-anthony.l.nguyen@xxxxxxxxx/ To cut a long story short: LED subsystem maintainer has ideas how a unified solution could like, and he has some work-in-progress patches. And some statements exist how not to do it and what to avoid. But there's still some open issues, therefore no solution is available yet. It's not really clear how to go on with network device LED support in the meantime. > Best > David > Heiner >> >>> +required: >>> + - compatible >>> + - reg >>> + >>> +examples: >>> + - | >>> + usb@100 { >>> + reg = <0x100 0x100>; >>> + #address-cells = <1>; >>> + #size-cells = <0>; >>> + >>> + usb-eth@2 { >>> + compatible = "realtek,rtl8153"; >>> + reg = <0x2>; >>> + realtek,led-data = <0x87>; >>> + }; >>> + }; >>> >>
