| Message ID | 20181004193410.7265-2-oskari@lemmela.net (mailing list archive) |
|---|---|
| State | Superseded, archived |
| Headers | show |
| Series | AXP8x3 AC and battery power supply support | expand |
On Thu, 4 Oct 2018 22:34:07 +0300 Oskari Lemmela <oskari@lemmela.net> wrote: > AXP803 PMIC is register compatible with AXP813. > > AXP803/AXP813 are capable to limit input current and supply voltage. > > Signed-off-by: Oskari Lemmela <oskari@lemmela.net> Hi Oskari, It would be better to split this up as different aspects are going to ideally be applied through different kernel trees. I know that already got commented on for the later patch but just wanted to point out it applies here as well. One comment inline about avoiding wildcards given the propensity of hardware manufacturers to confuse us on their naming schemes. Jonathan > --- > Documentation/devicetree/bindings/gpio/gpio-axp209.txt | 3 ++- > Documentation/devicetree/bindings/iio/adc/axp20x_adc.txt | 3 ++- > .../devicetree/bindings/power/supply/axp20x_ac_power.txt | 4 ++++ > .../devicetree/bindings/power/supply/axp20x_battery.txt | 1 + > 4 files changed, 9 insertions(+), 2 deletions(-) > > diff --git a/Documentation/devicetree/bindings/gpio/gpio-axp209.txt b/Documentation/devicetree/bindings/gpio/gpio-axp209.txt > index fc42b2caa06d..da3c0efb5e76 100644 > --- a/Documentation/devicetree/bindings/gpio/gpio-axp209.txt > +++ b/Documentation/devicetree/bindings/gpio/gpio-axp209.txt > @@ -11,6 +11,7 @@ This driver employs the per-pin muxing pattern. > Required properties: > - compatible: Should be one of: > - "x-powers,axp209-gpio" > + - "x-powers,axp803-gpio" > - "x-powers,axp813-gpio" > - #gpio-cells: Should be two. The first cell is the pin number and the > second is the GPIO flags. > @@ -67,7 +68,7 @@ GPIO0 | gpio_in, gpio_out, ldo, adc > GPIO1 | gpio_in, gpio_out, ldo, adc > GPIO2 | gpio_in, gpio_out > > -axp813 > +axp8x3 Avoid wild cards like this. It if far too common to have a hardware manufacturer not keep to what seems like an established naming convention and introduce something entirely different in the middle of an existing series of names. > ------ > GPIO | Functions > ------------------------ > diff --git a/Documentation/devicetree/bindings/iio/adc/axp20x_adc.txt b/Documentation/devicetree/bindings/iio/adc/axp20x_adc.txt > index 7a6313913923..247616099171 100644 > --- a/Documentation/devicetree/bindings/iio/adc/axp20x_adc.txt > +++ b/Documentation/devicetree/bindings/iio/adc/axp20x_adc.txt > @@ -4,6 +4,7 @@ Required properties: > - compatible: should be one of: > - "x-powers,axp209-adc", > - "x-powers,axp221-adc", > + - "x-powers,axp803-adc", > - "x-powers,axp813-adc", > - #io-channel-cells: should be 1, > > @@ -39,7 +40,7 @@ AXP22x > 2 | batt_chrg_i > 3 | batt_dischrg_i > > -AXP813 > +AXP8x3 > ------ > 0 | pmic_temp > 1 | gpio0_v > diff --git a/Documentation/devicetree/bindings/power/supply/axp20x_ac_power.txt b/Documentation/devicetree/bindings/power/supply/axp20x_ac_power.txt > index 826e8a879121..97276a71e961 100644 > --- a/Documentation/devicetree/bindings/power/supply/axp20x_ac_power.txt > +++ b/Documentation/devicetree/bindings/power/supply/axp20x_ac_power.txt > @@ -4,6 +4,8 @@ Required Properties: > - compatible: One of: > "x-powers,axp202-ac-power-supply" > "x-powers,axp221-ac-power-supply" > + "x-powers,axp803-ac-power-supply" > + "x-powers,axp813-ac-power-supply" > > This node is a subnode of the axp20x PMIC. > > @@ -13,6 +15,8 @@ reading ADC channels from the AXP20X ADC. > The AXP22X is only able to tell if an AC power supply is present and > usable. > > +The AXP8X3 is able to limit current and supply voltage > + > Example: > > &axp209 { > diff --git a/Documentation/devicetree/bindings/power/supply/axp20x_battery.txt b/Documentation/devicetree/bindings/power/supply/axp20x_battery.txt > index 41916f69902c..780ebd7e3b84 100644 > --- a/Documentation/devicetree/bindings/power/supply/axp20x_battery.txt > +++ b/Documentation/devicetree/bindings/power/supply/axp20x_battery.txt > @@ -4,6 +4,7 @@ Required Properties: > - compatible, one of: > "x-powers,axp209-battery-power-supply" > "x-powers,axp221-battery-power-supply" > + "x-powers,axp803-battery-power-supply" > "x-powers,axp813-battery-power-supply" > > This node is a subnode of its respective PMIC DT node.
diff --git a/Documentation/devicetree/bindings/gpio/gpio-axp209.txt b/Documentation/devicetree/bindings/gpio/gpio-axp209.txt index fc42b2caa06d..da3c0efb5e76 100644 --- a/Documentation/devicetree/bindings/gpio/gpio-axp209.txt +++ b/Documentation/devicetree/bindings/gpio/gpio-axp209.txt @@ -11,6 +11,7 @@ This driver employs the per-pin muxing pattern. Required properties: - compatible: Should be one of: - "x-powers,axp209-gpio" + - "x-powers,axp803-gpio" - "x-powers,axp813-gpio" - #gpio-cells: Should be two. The first cell is the pin number and the second is the GPIO flags. @@ -67,7 +68,7 @@ GPIO0 | gpio_in, gpio_out, ldo, adc GPIO1 | gpio_in, gpio_out, ldo, adc GPIO2 | gpio_in, gpio_out -axp813 +axp8x3 ------ GPIO | Functions ------------------------ diff --git a/Documentation/devicetree/bindings/iio/adc/axp20x_adc.txt b/Documentation/devicetree/bindings/iio/adc/axp20x_adc.txt index 7a6313913923..247616099171 100644 --- a/Documentation/devicetree/bindings/iio/adc/axp20x_adc.txt +++ b/Documentation/devicetree/bindings/iio/adc/axp20x_adc.txt @@ -4,6 +4,7 @@ Required properties: - compatible: should be one of: - "x-powers,axp209-adc", - "x-powers,axp221-adc", + - "x-powers,axp803-adc", - "x-powers,axp813-adc", - #io-channel-cells: should be 1, @@ -39,7 +40,7 @@ AXP22x 2 | batt_chrg_i 3 | batt_dischrg_i -AXP813 +AXP8x3 ------ 0 | pmic_temp 1 | gpio0_v diff --git a/Documentation/devicetree/bindings/power/supply/axp20x_ac_power.txt b/Documentation/devicetree/bindings/power/supply/axp20x_ac_power.txt index 826e8a879121..97276a71e961 100644 --- a/Documentation/devicetree/bindings/power/supply/axp20x_ac_power.txt +++ b/Documentation/devicetree/bindings/power/supply/axp20x_ac_power.txt @@ -4,6 +4,8 @@ Required Properties: - compatible: One of: "x-powers,axp202-ac-power-supply" "x-powers,axp221-ac-power-supply" + "x-powers,axp803-ac-power-supply" + "x-powers,axp813-ac-power-supply" This node is a subnode of the axp20x PMIC. @@ -13,6 +15,8 @@ reading ADC channels from the AXP20X ADC. The AXP22X is only able to tell if an AC power supply is present and usable. +The AXP8X3 is able to limit current and supply voltage + Example: &axp209 { diff --git a/Documentation/devicetree/bindings/power/supply/axp20x_battery.txt b/Documentation/devicetree/bindings/power/supply/axp20x_battery.txt index 41916f69902c..780ebd7e3b84 100644 --- a/Documentation/devicetree/bindings/power/supply/axp20x_battery.txt +++ b/Documentation/devicetree/bindings/power/supply/axp20x_battery.txt @@ -4,6 +4,7 @@ Required Properties: - compatible, one of: "x-powers,axp209-battery-power-supply" "x-powers,axp221-battery-power-supply" + "x-powers,axp803-battery-power-supply" "x-powers,axp813-battery-power-supply" This node is a subnode of its respective PMIC DT node.
AXP803 PMIC is register compatible with AXP813. AXP803/AXP813 are capable to limit input current and supply voltage. Signed-off-by: Oskari Lemmela <oskari@lemmela.net> --- Documentation/devicetree/bindings/gpio/gpio-axp209.txt | 3 ++- Documentation/devicetree/bindings/iio/adc/axp20x_adc.txt | 3 ++- .../devicetree/bindings/power/supply/axp20x_ac_power.txt | 4 ++++ .../devicetree/bindings/power/supply/axp20x_battery.txt | 1 + 4 files changed, 9 insertions(+), 2 deletions(-)