On 25/05/2022 09:16, Viresh Kumar wrote:
> On 13-05-22, 08:13, Krzysztof Kozlowski wrote:
>> + ufs_opp_table: opp-table {
>> + compatible = "operating-points-v2";
>> +
>> + opp-50000000 {
>> + opp-hz = /bits/ 64 <50000000
>> + 0
>> + 0
>> + 37500000
>> + 0
>> + 0
>> + 0
>> + 0
>> + // FIXME: value 0 copied from freq-table-hz
>> + 0>;
>
> One general comment, I think this should follow how we specify
> multiple voltages or other fields and so each frequency should be part
> of a different < > braces. Like: opp-hz = /bits/ 64 <5000000>, <0>, ....
>
> Whatever is there between < > seems to be connected, like
> min/max/target for voltage.
>
> The code will process both in a similar way though eventually.
OK, I can change to such format.
Best regards,
Krzysztof
