Skip to content

Commit

Permalink
scsi: ufs: dt-bindings: common: Add OPP table
Browse files Browse the repository at this point in the history
Except scaling UFS and bus clocks, it's necessary to scale also the
voltages of regulators or power domain performance state levels.  Adding
Operating Performance Points table allows to adjust power domain
performance state, depending on the UFS clock speed.

OPPv2 deprecates previous property limited to clock scaling:
freq-table-hz.

Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Reviewed-by: Rob Herring <robh@kernel.org>
Signed-off-by: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
Link: https://lore.kernel.org/r/20231012172129.65172-2-manivannan.sadhasivam@linaro.org
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
  • Loading branch information
Krzysztof Kozlowski authored and Martin K. Petersen committed Oct 17, 2023
1 parent 058676b commit e820de1
Showing 1 changed file with 32 additions and 3 deletions.
35 changes: 32 additions & 3 deletions Documentation/devicetree/bindings/ufs/ufs-common.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -20,11 +20,25 @@ properties:
items:
- description: Minimum frequency for given clock in Hz
- description: Maximum frequency for given clock in Hz
deprecated: true
description: |
Preferred is operating-points-v2.
Array of <min max> operating frequencies in Hz stored in the same order
as the clocks property. If this property is not defined or a value in the
array is "0" then it is assumed that the frequency is set by the parent
clock or a fixed rate clock source.
as the clocks property. If either this property or operating-points-v2 is
not defined or a value in the array is "0" then it is assumed that the
frequency is set by the parent clock or a fixed rate clock source.
operating-points-v2:
description:
Preferred over freq-table-hz.
If present, each OPP must contain array of frequencies stored in the same
order for each clock. If clock frequency in the array is "0" then it is
assumed that the frequency is set by the parent clock or a fixed rate
clock source.

opp-table:
type: object

interrupts:
maxItems: 1
Expand Down Expand Up @@ -75,8 +89,23 @@ properties:

dependencies:
freq-table-hz: [ clocks ]
operating-points-v2: [ clocks, clock-names ]

required:
- interrupts

allOf:
- if:
required:
- freq-table-hz
then:
properties:
operating-points-v2: false
- if:
required:
- operating-points-v2
then:
properties:
freq-table-hz: false

additionalProperties: true

0 comments on commit e820de1

Please sign in to comment.