On 08/01/2014 03:40 PM, Simon Glass wrote:
Hi Stephen,

On 1 August 2014 00:09, Stephen Warren <swar...@wwwdotorg.org> wrote:
On 07/31/2014 04:13 PM, Simon Glass wrote:

Hi Stephen,

On 31 July 2014 21:20, Stephen Warren <swar...@wwwdotorg.org> wrote:

On 07/30/2014 03:49 AM, Simon Glass wrote:


If the sandbox device tree is provided to U-Boot (with the -d flag) then it
will use the device tree version in preference to the built-in device. The
only difference is the colour.

diff --git a/arch/sandbox/dts/sandbox.dts b/arch/sandbox/dts/sandbox.dts
+       uart0: serial {
+               compatible = "sandbox,serial";
+               u-boot,dm-pre-reloc;

...
+               text-colour = "cyan";



That's property should likely have a uboot, prefix, since it's
non-standard.

Can I not just declare a binding for 'sandbox,serial'?

Properties that are relevant only to a particular binding, rather than being
something quite generic an applicable to a whole class of devices, typically
have a vendor prefix.

A binding should/must exist for every node or compatible value. So, whether
you actually write the binding document or not makes no difference to the
names or vendor prefixes of the properties the binding uses.

So here you are saying it should be:

    sandbox,text-colour = "cyan";

Is that right?

I think that sounds right yes.
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to