Re: [PATCH 1/4] dt-bindings: net: add bitfield defines for Ethernet speeds

From: Andrew Lunn
Date: Thu May 05 2022 - 08:17:48 EST


On Thu, May 05, 2022 at 07:19:41AM +0200, Rafał Miłecki wrote:
> On 3.05.2022 17:36, Rafał Miłecki wrote:
> > From: Rafał Miłecki <rafal@xxxxxxxxxx>
> >
> > This allows specifying multiple Ethernet speeds in a single DT uint32
> > value.
> >
> > Signed-off-by: Rafał Miłecki <rafal@xxxxxxxxxx>
>
> Ansuel please check if my patchset conflicts in any way with your work.
>
> Andrew suggested to combine both but right now I don't see it as
> necessary.
>
> I'd still appreciate your review of my work. Such binding may be
> required for some hardware controlled LEDs setup too I guess.

Please look at the LED binding. It is an LED you are trying to
control, so that is the binding you should be using. How do you
describe this functionality using that binding. Ansuel code will give
you the framework to actually do the implementation within.

Andrew