On Tue, Feb 27, 2018 at 10:25:42PM +0100, Geert Uytterhoeven wrote: > Hi Simon, > > On Tue, Feb 27, 2018 at 9:22 AM, Simon Horman <horms@xxxxxxxxxxxx> wrote: > > On Mon, Feb 26, 2018 at 07:03:44PM +0100, Geert Uytterhoeven wrote: > >> The Renesas Salvator-X development board can be equipped with an R-Car > >> H3, M3-W, or M3-N SiP, which are pin-compatible. > >> > >> Document board part number and compatible values for the version with > >> R-Car M3-N. > >> > >> The board part number was extracted from a big patch by Takeshi Kihara > >> in the BSP. > > >> --- a/Documentation/devicetree/bindings/arm/shmobile.txt > >> +++ b/Documentation/devicetree/bindings/arm/shmobile.txt > >> @@ -108,6 +108,8 @@ Boards: > >> compatible = "renesas,salvator-x", "renesas,r8a7795" > >> - Salvator-X (RTP0RC7796SIPB0011S) > > > > Here we have: Name (Part No) > > That's the standard (highest number of occurrences) way. > > >> compatible = "renesas,salvator-x", "renesas,r8a7796" > >> + - Salvator-X (RTP0RC7796SIPB0011S (M3N)) > > > > Here we have: Name (Part No (SoC)) > > This one is special: it's basically an M3-N prototype board made by taking > a Salvator-X/M3-W board, and replacing the M3-W SiP by an M3-N SiP. > Hence the board part number is the same. > > >> + compatible = "renesas,salvator-x", "renesas,r8a77965" > >> - Salvator-XS (Salvator-X 2nd version, RTP0RC7795SIPB0012S) > > > > And here we have: Name (Version, Part No) > > AFAIU, "Salvator-XS" really means "Salvator-X 2nd version". > So it's not "Version", but "Alternative Board Name". > There's a similar thing with the ULCB boards: > > - H3ULCB (R-Car Starter Kit Premier, RTP0RC7795SKBX0010SA00 (H3 ES1.1)) > H3ULCB (R-Car Starter Kit Premier, RTP0RC77951SKBX010SA00 (H3 ES2.0)) > - M3ULCB (R-Car Starter Kit Pro, RTP0RC7796SKBX0010SA09 (M3 ES1.0)) > > >> compatible = "renesas,salvator-xs", "renesas,r8a7795" > >> - Salvator-XS (Salvator-X 2nd version, RTP0RC7796SIPB0012S) > > > > Can we think about making this more consistent? > > I'm afraid not. Naming seems to become only more complex over time... > Unless you have great hands and great ideas ;-) Not so many good ideas this morning. I've applied this as is. We can revisit if the need arises. -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
