Add RT5033 charger device driver
Status from this message: Applied, thanks!
Get series with b4
b4 am [email protected]Versions newest first · linked by author and title
- v6 Latest posting 2/210 patches 16 repliesApplied
- v6 posting 1/210 patches 0 repliesSuperseded
- v510 patches 9 repliesSuperseded
Earlier versions · 4
| Version | Posted | Patches | Replies | Status |
|---|---|---|---|---|
| v4 | 8 patches | 12 replies | Superseded | |
| v3 | 8 patches | 11 replies | Superseded | |
| v2 | 9 patches | 17 replies | Superseded | |
| v1 | 10 patches | 19 replies | Applied |
Changes in v6 from the cover letter
- Patch 5: In function rt5033_charger_probe() after
calling rt5033_charger_dt_init() replaced the return value from "-ENODEV" to "PTR_ERR(charger->chg)", as suggested by Christophe.
- Patch 9: Changed the patch from adding "power-supplies: true" to replacing "additionalProperties: false" by "unevaluatedProperties: false", as suggested by Krzysztof.
Patches in v6 posting 2/2 Applied
| # | Subject and files | Size | Replies | Applied |
|---|---|---|---|---|
| 1/10 | mfd: rt5033: Drop rt5033-battery sub-devicedrivers/ | +0 | 0 replies | |
| 2/10 | mfd: rt5033: Fix chip revision readoutdrivers/ include/ | +7 | 0 replies | |
| 3/10 | mfd: rt5033: Fix STAT_MASK, HZ_MASK and AICR definesinclude/ | +5 | 0 replies | |
| 4/10 | mfd: rt5033: Apply preparatory changes before adding rt5033-charger driverinclude/ include/ | +36 | 2 replies | |
| 5/10 | power: supply: rt5033_charger: Add RT5033 charger device driverdrivers/ drivers/ drivers/ +1 more | +481 | 0 replies | |
| 6/10 | power: supply: rt5033_charger: Add cable detection and USB OTG supplydrivers/ | +274 | 6 replies | |
| 7/10 | power: supply: rt5033_battery: Move struct rt5033_battery to battery driverdrivers/ include/ | +8 | 0 replies | |
| 8/10 | power: supply: rt5033_battery: Adopt status property from chargerdrivers/ | +23 | 0 replies | |
| 9/10 | dt-bindings: power: supply: rt5033-battery: Apply unevaluatedPropertiesDocumentation/ | +1 | 2 replies | |
| 10/10 | dt-bindings: Add rt5033 mfd, regulator and chargerDocumentation/ Documentation/ | +203 | 0 replies |
Reviews trailers per patch
7 of 10 patches have a Reviewed-by or Acked-by.
| Patch | Lee Jones | Linus Walleij | Raymond Hackley | Sebastian Reichel | Krzysztof Kozlowski | Conor Dooley |
|---|---|---|---|---|---|---|
| 1/10 | A-b | R-b | ||||
| 2/10 | ||||||
| 3/10 | Acked-for-MFD-by | |||||
| 4/10 | ||||||
| 5/10 | R-b | T-b | A-b | |||
| 6/10 | T-b | A-b | ||||
| 7/10 | A-b | |||||
| 8/10 | A-b | |||||
| 9/10 | A-b | A-b | ||||
| 10/10 | R-b | R-b |
Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes. Scroll the table sideways for all 6 reviewers.
Cover letter [PATCH v6 00/10 RESEND] Add RT5033 charger device driver
This patchset adds the charger driver "rt5033-charger". It is part of the multifunction device rt5033. The patchset is based on an older version by Beomho Seo of March 2015. For more information on the history and setup of the patchset see the cover sheet of version v1, there is a link further down below the changelog.
RESEND: Sorry for spamming. The first try of sending v6 got interrupted and was split into two threads on the lore list. Therefore sending it again.
Changes in v6:
- Patch 5: In function rt5033_charger_probe() after
calling rt5033_charger_dt_init() replaced the return value from "-ENODEV" to "PTR_ERR(charger->chg)", as suggested by Christophe.
- Patch 9: Changed the patch from adding "power-supplies: true" to replacing "additionalProperties: false" by "unevaluatedProperties: false", as suggested by Krzysztof.
v1: https://lore.kernel.org/linux-pm/[email protected]/T/#t
v2: https://lore.kernel.org/linux-pm/[email protected]/T/#t
v3: https://lore.kernel.org/linux-pm/[email protected]/T/#t
v4: https://lore.kernel.org/linux-pm/[email protected]/T/#t
v5: https://lore.kernel.org/linux-pm/[email protected]/T/#t
The result of the patchset v6 can be seen at:
https://github.com/Jakko3/linux/blob/rt5033-charger_v6/drivers/power/supply/rt5033_charger.c
Jakob Hauser (9):
mfd: rt5033: Fix chip revision readout
mfd: rt5033: Fix STAT_MASK, HZ_MASK and AICR defines
mfd: rt5033: Apply preparatory changes before adding rt5033-charger
driver
power: supply: rt5033_charger: Add RT5033 charger device driver
power: supply: rt5033_charger: Add cable detection and USB OTG supply
power: supply: rt5033_battery: Move struct rt5033_battery to battery
driver
power: supply: rt5033_battery: Adopt status property from charger
dt-bindings: power: supply: rt5033-battery: Apply
unevaluatedProperties
dt-bindings: Add rt5033 mfd, regulator and charger
Stephan Gerhold (1):
mfd: rt5033: Drop rt5033-battery sub-device
.../bindings/mfd/richtek,rt5033.yaml | 138 ++++ .../power/supply/richtek,rt5033-battery.yaml | 2 +- .../power/supply/richtek,rt5033-charger.yaml | 65 ++ drivers/mfd/rt5033.c | 8 +- drivers/power/supply/Kconfig | 8 + drivers/power/supply/Makefile | 1 + drivers/power/supply/rt5033_battery.c | 38 +- drivers/power/supply/rt5033_charger.c | 744 ++++++++++++++++++ include/linux/mfd/rt5033-private.h | 64 +- include/linux/mfd/rt5033.h | 24 - 10 files changed, 1034 insertions(+), 58 deletions(-) create mode 100644 Documentation/devicetree/bindings/mfd/richtek,rt5033.yaml create mode 100644 Documentation/devicetree/bindings/power/supply/richtek,rt5033-charger.yaml create mode 100644 drivers/power/supply/rt5033_charger.c
Signature · 1 line
2.39.2