On Wed, Apr 5, 2017 at 1:48 AM, Quentin Schulz <quentin.schulz@xxxxxxxxxxxxxxxxxx> wrote: > Hi Liam, > > On 05/04/2017 10:38, Liam Breck wrote: >> [reduced CC list] >> >> Hi Quentin, >> >> On Wed, Apr 5, 2017 at 1:10 AM, Quentin Schulz >> <quentin.schulz@xxxxxxxxxxxxxxxxxx> wrote: >>> The X-Powers AXP20X and AXP22X PMICs can have a battery as power supply. >>> >>> This patch adds the DT binding documentation for the battery power >>> supply which gets various data from the PMIC, such as the battery status >>> (charging, discharging, full, dead), current max limit, current current, >>> battery capacity (in percentage), voltage max and min limits, current >>> voltage and battery capacity (in Ah). >>> >>> Signed-off-by: Quentin Schulz <quentin.schulz@xxxxxxxxxxxxxxxxxx> >>> Acked-by: Chen-Yu Tsai <wens@xxxxxxxx> >>> Acked-by: Maxime Ripard <maxime.ripard@xxxxxxxxxxxxxxxxxx> >>> --- >>> >>> v5: >>> - removed DT property example from monitored-battery, >>> >>> v4: >>> - added monitored-battery optional property, >>> - added example with battery, >>> >>> v3: >>> - removed constant charge current property, now should use the WIP >>> battery framework, >>> >>> v2: >>> - changed DT node name from ac_power_supply to ac-power-supply, >>> - removed io-channels and io-channel-names from DT (the IIO mapping is >>> done in the IIO ADC driver now), >>> - added x-powers,constant-charge-current property to set the maximal >>> default constant current charge of the battery, >>> >>> .../bindings/power/supply/axp20x_battery.txt | 28 ++++++++++++++++++++++ >>> 1 file changed, 28 insertions(+) >>> create mode 100644 Documentation/devicetree/bindings/power/supply/axp20x_battery.txt >>> >>> diff --git a/Documentation/devicetree/bindings/power/supply/axp20x_battery.txt b/Documentation/devicetree/bindings/power/supply/axp20x_battery.txt >>> new file mode 100644 >>> index 0000000..63826fd >>> --- /dev/null >>> +++ b/Documentation/devicetree/bindings/power/supply/axp20x_battery.txt >>> @@ -0,0 +1,28 @@ >>> +AXP20x and AXP22x battery power supply >>> + >>> +Required Properties: >>> + - compatible, one of: >>> + "x-powers,axp209-battery-power-supply" >>> + "x-powers,axp221-battery-power-supply" >>> + >>> +Optional properties: >>> + - monitored-battery, phandle to a fixed battery >> >> Say here what properties of the battery your driver considers, e.g. >> https://patchwork.kernel.org/patch/9660987/ >> > > Sorry but I'm completely lost in your patch series, this isn't your > patch series for the battery framework. How do I know from which I > should take the name of the DT property then? That's a link from a subset, with just bq27xxx fuel gauge stuff; it demonstrates documenting the properties a driver considers from battery node. Here's a link into previous full series: https://patchwork.kernel.org/patch/9633605/ >>> + >>> +This node is a subnode of the axp20x/axp22x PMIC. >>> + >>> +The AXP20X and AXP22X can read the battery voltage, charge and discharge >>> +currents of the battery by reading ADC channels from the AXP20X/AXP22X >>> +ADC. >>> + >>> +Example: >>> + >>> +battery: battery { >>> + compatible = "fixed-battery"; >> >> This is now "simple-battery". Also needs properties. >> > > I've removed all the code in the driver to take care of the monitored > battery info until your framework is merged. Since there are still > discussion around the name to give to the different DT properties and > since it's not required in this driver, I'll add them later to the DT > binding. The DT maintainers want complete-file patches, so maybe drop this patch until later? > Quentin > > -- > Quentin Schulz, Free Electrons > Embedded Linux and Kernel engineering > http://free-electrons.com -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html