On Tue, Sep 17, 2024 at 07:52:59AM +0000, Daniel Machon wrote:
> > >    reg:
> > >      items:
> > > @@ -85,6 +95,12 @@ allOf:
> > >            contains:
> > >              enum:
> > >                - microchip,lan966x-pinctrl
> > > +              - microchip,lan9698-pinctrl
> > > +              - microchip,lan9696-pinctrl
> > > +              - microchip,lan9694-pinctrl
> > > +              - microchip,lan9693-pinctrl
> > > +              - microchip,lan9692-pinctrl
> > 
> > > +              - microchip,lan9691-pinctrl
> > 
> > This should work on its own, since the other devices here have it as a
> > fallback.
> 
> Just to be clear - we only need the "microchip,lan9691-pinctrl" here?

Ye, in the enum you should only need that compatible (in addition to the
exsting one) because lan9691 is the fallback for all the others and
therefore the compatible must contain it.