Re: [PATCH v2 9/9] dt-bindings: Add documentation for rt5033 mfd, regulator and charger

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Krzysztof,

On 19.04.23 10:42, Krzysztof Kozlowski wrote:
On 18/04/2023 23:37, Jakob Hauser wrote:

+properties:
+  compatible:
+    const: richtek,rt5033-charger
+
+  richtek,pre-microamp:
+    description:
+      Current of pre-charge mode. The pre-charge current levels are 350 mA to
+      650 mA programmed by I2C per 100 mA.

minimum:
maximum:
multipleOf: 100

Same for other cases.

The "multipleOf: 100" doesn't seen appropriate to me when the choice is
350, 450, 550, 650. Those are not multiples of 100. It's more of a step
size. I didn't find a general property for step size. Listing them as
"enum" would be another possibility, I guess, but not an elegant one.
Especially for property "richtek,const-microvolt" there are 30
possibilities.

Ahh, right. You can use enum here and min/max for other cases, where
multipleOf cannot be used.

+  richtek,eoc-microamp:
+    description:
+      This property is end of charge current. Its level ranges from 150 mA to
+      600 mA. Between 150 mA and 300 mA in 50 mA steps, between 300 mA and 600 mA
+      in 100 mA steps.
+    maxItems: 1

Here are two different step sizes. The first few are 50 mA steps (150,
200, 250, 300 mA) and then it changes to 100 mA steps (300, 400, 500,
600 mA). How to deal with that? Again I guess "enum" would be a
possibility, but again not a nice one.

enum

Thanks for the reply. Looking through the properties, I get the following result.

  richtek,pre-microamp:
    description:
      Current of pre-charge mode. The pre-charge current levels are
      350 mA to 650 mA programmed by I2C per 100 mA.
    maxItems: 1
    enum: [350000, 450000, 550000, 650000]

  richtek,fast-microamp:
    description:
      Current of fast-charge mode. The fast-charge current levels are
      700 mA to 2000 mA programmed by I2C per 100 mA.
    maxItems: 1
    minimum: 700000
    maximum: 2000000
    multipleOf: 100000

  richtek,eoc-microamp:
    description:
      This property is end of charge current. Its level ranges from
      150 mA to 600 mA. Between 150 mA and 300 mA in 50 mA steps,
      between 300 mA and 600 mA in 100 mA steps.
    maxItems: 1
    enum: [150000, 200000, 250000, 300000, 400000, 500000, 600000]

  richtek,pre-threshold-microvolt:
    description:
      Voltage of pre-charge mode. If the battery voltage is below the
      pre-charge threshold voltage, the charger is in pre-charge mode
      with pre-charge current. Its levels are 2.3 V to 3.8 V programmed
      by I2C per 0.1 V.
    maxItems: 1
    minimum: 2300000
    maximum: 3800000
    multipleOf: 100000

  richtek,const-microvolt:
    description:
      Battery regulation voltage of constant voltage mode. This voltage
      levels from 3.65 V to 4.4 V by I2C per 0.025 V.
    maxItems: 1
    minimum: 3650000
    maximum: 4400000
    multipleOf: 25000

Kind regards,
Jakob



[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux