Hi Simon,

On Tue, Feb 27, 2018 at 9:22 AM, Simon Horman [off-list ref] wrote:

On Mon, Feb 26, 2018 at 07:03:44PM +0100, Geert Uytterhoeven wrote:

quoted

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.

quoted

--- 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.

quoted

     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.

quoted

+    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))

quoted

     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 ;-)

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert at linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds