Thread (12 messages) 12 messages, 4 authors, 2014-10-26
STALE4293d

[PATCH] arm: dts: add initial support for TBS2910 Matrix ARM mini PC

From: sebastian.hesselbarth@gmail.com (Sebastian Hesselbarth)
Date: 2014-10-21 12:03:51

On 21.10.2014 13:57, S?ren Moch wrote:
quoted
quoted
+/ {
+    model = "TBS2910 Matrix ARM mini PC";
+    compatible = "tbs,imx6q-tbs2910", "fsl,imx6q";
[...]
Also, board compatible should only be "tbs,tbs2910".
Similar boards use e.g.
     compatible = "fsl,imx6q-sabresd", "fsl,imx6q";
     compatible = "udoo,imx6q-udoo", "fsl,imx6q";
     compatible = "wand,imx6q-wandboard", "fsl,imx6q";

So is it really better to use only "tbs,tbs2910" here?
Hmm, having the SoC name again in the board compatible seems
odd to me, e.g. we have "google,chromecast" without the SoC
name in it.

Anyway, it is just a compatible and if it is common for imx-
based boards or Shawn/Sascha like to have it this way you
should stick with that, of course.

Sebastian
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help