linux-kernel - Re: [PATCH v4] ARM: dts: r9a06g032: add r9a06g032-rzn1d400-eb board device-tree

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <CAMuHMdXH6Fcm5xzS5UWpqBm4gFZ4bJBL_paDmpqCYvCO=uyWYg@mail.gmail.com>
Date: Fri, 11 Apr 2025 12:07:43 +0200
From: Geert Uytterhoeven <geert@...ux-m68k.org>
To: Wolfram Sang <wsa+renesas@...g-engineering.com>, 
	Geert Uytterhoeven <geert@...ux-m68k.org>, Thomas Bonnefille <thomas.bonnefille@...tlin.com>, 
	Magnus Damm <magnus.damm@...il.com>, Rob Herring <robh@...nel.org>, 
	Krzysztof Kozlowski <krzk+dt@...nel.org>, Conor Dooley <conor+dt@...nel.org>, 
	Thomas Petazzoni <thomas.petazzoni@...tlin.com>, Miquèl Raynal <miquel.raynal@...tlin.com>, 
	linux-renesas-soc@...r.kernel.org, devicetree@...r.kernel.org, 
	linux-kernel@...r.kernel.org, Clément Léger <clement.leger@...tlin.com>, 
	Niklas Söderlund <niklas.soderlund+renesas@...natech.se>
Subject: Re: [PATCH v4] ARM: dts: r9a06g032: add r9a06g032-rzn1d400-eb board device-tree

Hi Wolfram,

On Fri, 11 Apr 2025 at 11:53, Wolfram Sang
<wsa+renesas@...g-engineering.com> wrote:
> > > +       pinctrl-0 = <&pins_eth1>, <&pins_eth2>, <&pins_eth3>, <&pins_eth4>,
> > > +                   <&pins_mdio1>;
> > > +
> > > +       mdio {
> > > +               /* CN15 and CN16 switches must be configured in MDIO2 mode */
> > > +               switch0phy1: ethernet-phy@1 {
> > > +                       reg = <1>;
> > > +                       leds {
> > > +                               #address-cells = <1>;
> > > +                               #size-cells = <0>;
> > > +
> > > +                               led@0 {
> > > +                                       reg = <0>;
> >
> > color = <LED_COLOR_ID_GREEN>;
> >
> > > +                               };
> > > +                               led@1 {
> > > +                                       reg = <1>;
> >
> > color = <LED_COLOR_ID_ORANGE>;
> >
> > > +                               };
> >
> > The above should also have one of:
> >
> >     function = LED_FUNCTION_LAN;
> >     function = LED_FUNCTION_SPEED_LAN;
> >
> > I don't know the LED function mapping.
>
> I have an incremental fix for the LEDs to this patch. Thomas cannot
> really do it because he doesn't have the board. I was waiting with my
> patch until this patch is upstream, but I better send it out now, so you
> can squash it into this one?

Yes please.This patch is becoming too much of a dependency.
Thanks!

Gr{oetje,eeting}s,

                        Geert

-- 
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@...ux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds