On Fri, Oct 13, 2023 at 03:04:10PM +0200, Linus Walleij wrote: > Hi Andrew, > > thanks for reviewing! > > On Fri, Oct 13, 2023 at 2:43 PM Andrew Lunn <andrew@xxxxxxx> wrote: > > > > +properties: > > > + compatible: > > > + oneOf: > > > + - enum: > > > + - marvell,mv88e6060 > > > > The 6060 is a separate driver. Its not part of mv88e6xxx. So it should > > have a binding document of its own. > > It really doesn't matter to the DT bindings. > It is not the job of DT to reflect the state of Linux. > > In another operating system they might all be the same driver. > Or all four variants have their own driver. > > If the hardware is distinctly different so a lot of the properties > are unique then it may be warranted with a separate DT > binding, for the sake of keeping bindings simpler and > coherent. What i want to avoid is giving the impression that the mv88e6060 implements something when in fact it does not. That device does not have the interrupt controller, so all the interrupt properties are invalid for it. The hardware does not allow access to the EEPROM, so the EEPROM length property is not valid, etc. If you do want to make it part of the binding, it would be good to add constraints based on the compatible to indicate which properties are actually valid for the hardware. The 88e6060 actually has an open datasheet. https://www.insidegadgets.com/wp-content/uploads/2014/07/88E6060.pdf Andrew