On 02/08/2023 14:37, Konrad Dybcio wrote: > CPR3 and newer can be fed per-OPP voltage adjustment values for both > open- and closed-loop paths to make better decisions about settling > on the final voltage offset target. Document these properties. > > Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@xxxxxxxxxxxxx> > Tested-by: Jeffrey Hugo <quic_jhugo@xxxxxxxxxxx> > Signed-off-by: Konrad Dybcio <konrad.dybcio@xxxxxxxxxx> > --- > .../devicetree/bindings/opp/opp-v2-qcom-level.yaml | 14 ++++++++++++++ > 1 file changed, 14 insertions(+) > > diff --git a/Documentation/devicetree/bindings/opp/opp-v2-qcom-level.yaml b/Documentation/devicetree/bindings/opp/opp-v2-qcom-level.yaml > index a30ef93213c0..b203ea01b17a 100644 > --- a/Documentation/devicetree/bindings/opp/opp-v2-qcom-level.yaml > +++ b/Documentation/devicetree/bindings/opp/opp-v2-qcom-level.yaml > @@ -34,6 +34,20 @@ patternProperties: > minItems: 1 > maxItems: 2 > > + qcom,opp-cloop-vadj: > + description: | > + An array of per-thread values representing the closed-loop > + voltage adjustment value associated with this OPP node. > + $ref: /schemas/types.yaml#/definitions/int32-array I assume negative values are accepted on purpose. Adding these to example would be useful. Anyway: Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx> Best regards, Krzysztof