> > > > 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. OK, got it. Thanks Conor. /Daniel