On 01/20/2017 12:43 PM, Tony Lindgren wrote:
* Dave Gerlach <d-gerlach@xxxxxx> [170119 10:32]:
On 01/19/2017 12:07 PM, Rob Herring wrote:
+ /*
+ * The three following nodes are marked with opp-suspend
+ * because they can not be enabled simultaneously on a
+ * single SoC.
+ */
+ opp50@300000000 {
What's the 50, 100, 120 in the names?
Those are the names of the OPPs given in Table 5-7 of the AM335x data
manual, seen here http://www.ti.com/lit/ds/symlink/am3359.pdf . I typically
reference the table and document in the commit message of the actual DT
patches but didn't here for the binding.
If you don't need the names, you could maybe use:
opp100: opp@275000000 {
...
};
opp200: opp@300000000 {
...
};
Thanks for the suggestion, but we have duplicate frequencies that have
different voltages defined for them, so the scheme opp@<freq> won't work
because then we end up with duplicate nodes. OPPs are a defined by a
number of parameters, not just frequency, so always naming the nodes
opp@<freq> won't cover all scenarios.
Regards,
Dave
...
Regards,
Tony
--
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
