Re: [PATCH v4 2/4] Documentation: dt: add bindings for ti-cpufreq

[Dave Gerlach <d-gerlach@ti.com>]

On 01/20/2017 12:43 PM, Tony Lindgren wrote:
> * Dave Gerlach <d-gerlach@ti.com> [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
>