On 02-11-16, 11:03, Dave Gerlach wrote:
> >>+cpu0_opp_table: opp_table0 {
> >>+ compatible = "operating-points-v2-ti-am3352-cpu";
> >>+ ti,syscon-efuse = <&scm_conf 0x7fc 0x1fff 0>;
> >>+ ti,syscon-rev = <&scm_conf 0x600>;
> >>+
> >>+ /*
> >>+ * The three following nodes are marked with opp-suspend
> >>+ * because they can not be enabled simultaneously on a
> >>+ * single SoC.
> >>+ */
I missed reading this comment :(
>>+ opp50@300000000 {
> >>+ opp-hz = /bits/ 64 <300000000>;
> >>+ opp-microvolt = <950000 931000 969000>;
> >>+ opp-supported-hw = <0x06 0x0010>;
> >>+ opp-suspend;
> >>+ };
> >>+
> >>+ opp100@275000000 {
> >>+ opp-hz = /bits/ 64 <275000000>;
> >>+ opp-microvolt = <1100000 1078000 1122000>;
> >>+ opp-supported-hw = <0x01 0x00FF>;
> >>+ opp-suspend;
> >>+ };
> >>+
> >>+ opp100@300000000 {
> >>+ opp-hz = /bits/ 64 <300000000>;
> >>+ opp-microvolt = <1100000 1078000 1122000>;
> >>+ opp-supported-hw = <0x06 0x0020>;
> >>+ opp-suspend;
> >
> >Only one OPP in the table can be marked as suspend OPP.
> >
>
> Does that still apply when opp-supported-hw is involved? Based on the
> comment at the start of the table, those OPPs are all mutually exclusive and
> will not ever be enabled on the same piece of silicon, they represent the
> lowest OPP for each of three different supported-hw configurations.
You are right, its fine.
--
viresh
--
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
