On 09/02/2013 01:28 AM, Alexandre Courbot wrote:

Hi Tomasz!

On Fri, Aug 30, 2013 at 5:20 PM, Tomasz Figa [off-list ref] wrote:

quoted

quoted

+Required properties:
+- compatible : "tl,trusted-foundations"
+- version-major : major version number of Trusted Foundations firmware
+- version-minor: minor version number of Trusted Foundations firmware

Hmm, maybe you could simply define a single version property that could
have multiple cells? Like:

        firmware {
                compatible = "tl,trusted-foundations";
                version = <2 8>;
        };

I'm fine this way too, but do we have other bindings that use the same
scheme? What is the general convention for version number bindings?

I don't know if there are enough cases of this for there to be a
convention. A 2-cell property seems fine to me.

quoted

quoted

+       This option allows the kernel to invoke the secure monitor whenever
+       required on devices using Trusted Foundations.
+
+       Devices using Trusted Foundations should pass a device tree
containing +    a node compatible with "tl,trusted-foundations" to
signal the presence +   of the secure monitor.

What about pointing to the documentation file instead?

Yes, that would make more sense.

Possibly. What about when the binding document is no longer part of the
kernel though? Perhaps we could reference the documentation in some way
other than by the pathname within the kernel source tree though, e.g.
'see the device tree binding documentation for
compatible="tl,trusted-foundations"'?