| Message ID | 20210119174421.226541-5-angelogioacchino.delregno@somainline.org |
|---|---|
| State | Changes Requested, archived |
| Headers | show |
| Series | Really implement Qualcomm LAB/IBB regulators | expand |
| Context | Check | Description |
|---|---|---|
| robh/checkpatch | success | |
| robh/dt-meta-schema | success | |
| robh/dtbs-check | success |
On Tue, Jan 19, 2021 at 06:44:18PM +0100, AngeloGioacchino Del Regno wrote: > Document properties to configure soft start and discharge resistor > for LAB and IBB respectively. > > Signed-off-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@somainline.org> > Reviewed-by: Bjorn Andersson <bjorn.andersson@linaro.org> > --- > .../bindings/regulator/qcom-labibb-regulator.yaml | 10 ++++++++++ > 1 file changed, 10 insertions(+) > > diff --git a/Documentation/devicetree/bindings/regulator/qcom-labibb-regulator.yaml b/Documentation/devicetree/bindings/regulator/qcom-labibb-regulator.yaml > index 53853ec20fe2..7a507692f1ba 100644 > --- a/Documentation/devicetree/bindings/regulator/qcom-labibb-regulator.yaml > +++ b/Documentation/devicetree/bindings/regulator/qcom-labibb-regulator.yaml > @@ -22,6 +22,11 @@ properties: > type: object > > properties: > + qcom,soft-start-us: > + $ref: /schemas/types.yaml#/definitions/uint32 Don't need a type when you have standard units. > + description: Regulator soft start time in microseconds. > + enum: [200, 400, 600, 800] > + default: 200 > > interrupts: > maxItems: 1 > @@ -35,6 +40,11 @@ properties: > type: object > > properties: > + qcom,discharge-resistor-kohms: -kohms is not standard units as documented in property-units.txt (and .yaml in dtschema). Please use '-ohms'. Given this is already applied, send an incremental patch. > + $ref: /schemas/types.yaml#/definitions/uint32 > + description: Discharge resistor value in KiloOhms. > + enum: [300, 64, 32, 16] > + default: 300 > > interrupts: > maxItems: 1 > -- > 2.30.0 >
diff --git a/Documentation/devicetree/bindings/regulator/qcom-labibb-regulator.yaml b/Documentation/devicetree/bindings/regulator/qcom-labibb-regulator.yaml index 53853ec20fe2..7a507692f1ba 100644 --- a/Documentation/devicetree/bindings/regulator/qcom-labibb-regulator.yaml +++ b/Documentation/devicetree/bindings/regulator/qcom-labibb-regulator.yaml @@ -22,6 +22,11 @@ properties: type: object properties: + qcom,soft-start-us: + $ref: /schemas/types.yaml#/definitions/uint32 + description: Regulator soft start time in microseconds. + enum: [200, 400, 600, 800] + default: 200 interrupts: maxItems: 1 @@ -35,6 +40,11 @@ properties: type: object properties: + qcom,discharge-resistor-kohms: + $ref: /schemas/types.yaml#/definitions/uint32 + description: Discharge resistor value in KiloOhms. + enum: [300, 64, 32, 16] + default: 300 interrupts: maxItems: 1