Hi,
On Thu, Dec 13, 2018 at 8:41 PM Viresh Kumar <viresh.kumar@xxxxxxxxxx> wrote:
>
> On 12-12-18, 14:18, Jordan Crouse wrote:
> > + gpu_opp_table: opp-table {
> > + compatible = "operating-points-v2-qcom-level";
>
> I think you need to mention "operating-points-v2" as well here.
Are you saying the above should be:
compatible = "operating-points-v2-qcom-level", "operating-points-v2";
If so I'm not sure I agree. It's _not_ really compatible with the
"operating-points-v2" binding. If you had a driver that had never
heard of "operating-points-v2-qcom-level" and had only heard of
"operating-points-v2" and it took a look at this node it would have no
idea what to do with it.
I'll also note that other instances posted to the list don't list both:
https://patchwork.kernel.org/patch/10725801/ - RPM / RPMH PD bindings
https://patchwork.kernel.org/patch/10725793/ - RPMH PD device tree for sdm845
The bindings patch also makes no mention of needing
"operating-points-v2". I think the common case when a fallback is
required it is explicitly called out in the bindings:
https://patchwork.kernel.org/patch/10725803/ - qcom-opp bindings
-Doug
