mbox series

[v3,0/2] power: supply: gpio-charger: add support for default charge current limit

Message ID 20241218-default-charge-current-limit-v3-0-b26118cf06b5@liebherr.com (mailing list archive)
Headers show
Series power: supply: gpio-charger: add support for default charge current limit | expand

Message

Dimitri Fedrau via B4 Relay Dec. 18, 2024, 8:34 a.m. UTC
With DT properties charge-current-limit-gpios and
charge-current-limit-mapping one can define charge current limits in uA
using up to 32 GPIOs. At the moment the driver defaults to smallest charge
current limitation for safety reasons. When disabling charging is
supported, which should be common, the driver defaults to non charging on
probe. By having a default, charging can be enabled on probe for such
devices.

Signed-off-by: Dimitri Fedrau <dimitri.fedrau@liebherr.com>
---
Changes in v3:
- Fix commit msg for "dt-bindings: power: supply: gpio-charger: add support for default charge current limit"
  Add use case
- Link to v2: https://lore.kernel.org/r/20241213-default-charge-current-limit-v2-0-45886fce905c@liebherr.com

Changes in v2:
- renamed DT property charge-current-limit-default to
  charge-current-limit-default-microamp
- Added dependency on charge-current-limit-mapping
- Added intention and use case to commit descriptions
- Added charge-current-limit-default-microamp in bindings example
- Link to v1: https://lore.kernel.org/r/20241211-default-charge-current-limit-v1-0-7819ba06ee2a@liebherr.com

---
Dimitri Fedrau (2):
      dt-bindings: power: supply: gpio-charger: add support for default charge current limit
      power: supply: gpio-charger: add support for default charge current limit

 .../devicetree/bindings/power/supply/gpio-charger.yaml      |  6 ++++++
 drivers/power/supply/gpio-charger.c                         | 13 +++++++++++++
 2 files changed, 19 insertions(+)
---
base-commit: 57cb041f61d4abcf8dfa41259df27d081ab4cb6a
change-id: 20241209-default-charge-current-limit-69ae7945061a

Best regards,

Comments

Sebastian Reichel Dec. 21, 2024, 3:54 p.m. UTC | #1
On Wed, 18 Dec 2024 09:34:59 +0100, Dimitri Fedrau wrote:
> With DT properties charge-current-limit-gpios and
> charge-current-limit-mapping one can define charge current limits in uA
> using up to 32 GPIOs. At the moment the driver defaults to smallest charge
> current limitation for safety reasons. When disabling charging is
> supported, which should be common, the driver defaults to non charging on
> probe. By having a default, charging can be enabled on probe for such
> devices.
> 
> [...]

Applied, thanks!

[1/2] dt-bindings: power: supply: gpio-charger: add support for default charge current limit
      commit: 0a219d6fd7ab89177a6fa49539def43694154d50
[2/2] power: supply: gpio-charger: add support for default charge current limit
      commit: de68987480cb92702eab14cf62ec2aa1bae5c867

Best regards,