<br><br><div class="gmail_quote">On Sat, Nov 17, 2012 at 9:21 AM, Arnd Bergmann <span dir="ltr"><<a href="mailto:arnd@arndb.de" target="_blank">arnd@arndb.de</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im">On Saturday 17 November 2012, Christian Daudt wrote:<br>
> >> diff --git a/Documentation/devicetree/bindings/arm/bcm/bcm281xx.txt b/Documentation/devicetree/bindings/arm/bcm/bcm281xx.txt<br>
> ><br>
> >> +Required root node property:<br>
> >> +<br>
> >> +compatible = "bcm,bcm281xx";<br>
> ><br>
> > Hmm. I still tend to think we should list the specific SoC rather than<br>
> > the family here, but I guess if you're sure there won't be any SW<br>
> > compatibility between the different SoCs in the series, this is fine.<br>
> ><br>
><br>
</div>> right now I expect to be able to use the family thoughout, so I'll stick to it.<br>
><br>
<br>
But that's not how the compatible property is supposed to work: Each compatible<br>
value should refer to a specific property, and not include wildcards.<br>
If you have one driver (or platform file in this case) that can deal with<br>
a whole family of devices, the normal solution is to list the name of the oldest<br>
part in the compatible string as the most generic part, and then list the<br>
name of the new part as the specific one.<br><span class="HOEnZb"><font color="#888888"><br></font></span></blockquote><div>At that point doesn't the 'oldest part' become the wildcard  ? I'm not very attached to names, so I'm ok changing bcm281xx above to bcm11351, which happens to be the first chip I'm submitting a board for. But then bcm11351 will just become 'the chip name used to represent the family' right ? I had followed the fact that omap does use omap5 in omap5.dtsi - and afaik tegra2 and tegra3 are family names, not chip models, and are used in dtsi. But then again a bunch of chip models are used to represent the families too...</div>
<div> Let me know if you want me to submit a modified patchset. Shouldn't take me more than 5 minutes anyways :)</div><div><br></div><div> Thanks,</div><div>   csd</div><div><br></div><div> </div></div><br>