Hi Krzysztof - > Where did these two reviews happen? They were in-house reviews. Please feel free to remove them from the patch. > Where is any user of this? We are adding Kernel control of PoE LEDs and thought this might be useful to others, maybe those working on netdev, or anyone implementing PoE on their devices. Also, the Kernel >> Docs >> LEDs page states: "If required color or function is missing, please submit a patch to linux-leds@xxxxxxxxxxxxxxx", which is included here. Cheers, Tony O'Brien On Thu, 2025-02-13 at 10:31 +0100, Krzysztof Kozlowski wrote: > On Thu, Feb 13, 2025 at 01:58:41PM +1300, Tony O'Brien wrote: > > Add a define so network devices with a PoE PSE feature can provide > > status indications of connected PoE PDs. > > > > Reviewed-by: Ben Hoelker <ben.hoelker@xxxxxxxxxxxxxxxxxxx> > > Reviewed-by: Chris Packham <chris.packham@xxxxxxxxxxxxxxxxxxx> > > Where did these two reviews happen? > > > Signed-off-by: Tony O'Brien <tony.obrien@xxxxxxxxxxxxxxxxxxx> > > --- > > include/dt-bindings/leds/common.h | 1 + > > 1 file changed, 1 insertion(+) > > > > diff --git a/include/dt-bindings/leds/common.h b/include/dt- > > bindings/leds/common.h > > index 4f017bea0123..3a8958b81c87 100644 > > --- a/include/dt-bindings/leds/common.h > > +++ b/include/dt-bindings/leds/common.h > > @@ -94,6 +94,7 @@ > > #define LED_FUNCTION_MOBILE "mobile" > > #define LED_FUNCTION_MTD "mtd" > > #define LED_FUNCTION_PANIC "panic" > > +#define LED_FUNCTION_POE "poe" > > Where is any user of this? > > Best regards, > Krzysztof >
