On Sun, May 14, 2023 at 07:39:28PM +0100, Conor Dooley wrote: > On Mon, May 15, 2023 at 12:56:47AM +0800, Jisheng Zhang wrote: > > > +title: Bouffalo Lab Technology SoC-based boards > > I know you're only propagating an existing pattern, but the "SoC-based" > looks rather odd! > > > +properties: > > + $nodename: > > + const: '/' > > + compatible: > > + oneOf: > > + - description: Carrier boards for the Sipeed M1s SoM > > + items: > > + - enum: > > + - sipeed,m1s-dock > > BTW, do you know of any other m1s compatible docks? > I couldn't find any other ones via Google, so maybe it is just worth FWICT, there's no other m1s compatible docks, but in theory the SoM can be used to build different docks, I just keep it as is since you are fine with it too ;) > swapping the enum here for another const. > Either is fine by me though. > Reviewed-by: Conor Dooley <conor.dooley@xxxxxxxxxxxxx> > > Thanks, > Conor.