On 02/06/16 12:14, Rob Herring wrote:
> On Sun, May 29, 2016 at 01:12:02PM +0200, Carlo Caione wrote:
[...]
> > +Example of the node using the secure monitor:
> > +
> > + #include <dt-bindings/firmware/meson-sm.h>
> > +
> > + ...
> > +
> > + efuse: efuse {
> > + compatible = "amlogic,meson-gxbb-efuse";
> > + secure-monitor = <&sm>;
>
> Why do you need this? Given there can only be one node, just use
> of_find_compatible_node();
of_find_compatible_node() in the driver works fine if we have only one
possible compatible property for the secure-monitor node. In this case
we have a different compatible property for each SoC (GXBB in this case)
so I think having the phandle in the DT is still the best thing to do.
Any thought on this Rob?
--
Carlo Caione
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
