| From: |
| André Draszik <andre.draszik-AT-linaro.org> |
| To: |
| Tudor Ambarus <tudor.ambarus-AT-linaro.org>, Rob Herring <robh-AT-kernel.org>, Conor Dooley <conor+dt-AT-kernel.org>, Krzysztof Kozlowski <krzk-AT-kernel.org>, Liam Girdwood <lgirdwood-AT-gmail.com>, Mark Brown <broonie-AT-kernel.org>, Lee Jones <lee-AT-kernel.org>, Linus Walleij <linus.walleij-AT-linaro.org>, Bartosz Golaszewski <brgl-AT-bgdev.pl>, Krzysztof Kozlowski <krzk+dt-AT-kernel.org> |
| Subject: |
| [PATCH v3 00/20] Samsung S2MPG10 regulator and S2MPG11 PMIC drivers |
| Date: |
| Mon, 03 Nov 2025 19:14:39 +0000 |
| Message-ID: |
| <20251103-s2mpg1x-regulators-v3-0-b8b96b79e058@linaro.org> |
| Cc: |
| Peter Griffin <peter.griffin-AT-linaro.org>, Will McVicker <willmcvicker-AT-google.com>, kernel-team-AT-android.com, linux-kernel-AT-vger.kernel.org, linux-samsung-soc-AT-vger.kernel.org, devicetree-AT-vger.kernel.org, linux-gpio-AT-vger.kernel.org, André Draszik <andre.draszik-AT-linaro.org> |
| Archive-link: |
| Article |
This series extends the existing S2MPG10 PMIC driver to add support for
the regulators, and adds new S2MPG11 core and regulator drivers.
This series must be applied in-order, due to the regulator drivers
depending on headers & definitions added by the bindings and core
drivers. I would expect them all to go via the MFD tree.
The patches are kept together in one series, due to S2MPG11 and its
regulators being very similar to S2MPG10.
The Samsung S2MPG11 PMIC is a Power Management IC for mobile
applications with buck converters, various LDOs, power meters, and
additional GPIO interfaces. It typically complements an S2MPG10 PMIC in
a main/sub configuration as the sub-PMIC and both are used on the
Google Pixel 6 and 6 Pro (oriole / raven).
A DT update for Oriole / Raven to enable these is required which I will
send out separately.
Cheers,
Andre'
Signed-off-by: André Draszik <andre.draszik@linaro.org>
---
Changes in v3:
- Krzysztof:
- split s2mpg10 core binding into separate file
- drop PCTRLSEL values that can be described using standard
properties, and update remaining macro names (self)
- drop maxItems:1 where not needed (Krzysztof)
- samsung,ext-control-gpios -> enable-gpios
- fix LDO20M_EN pin name -> VLDO20M_EN
- move all binding patternProperties to top-level
- one (driver) instance per actual rail, not per rail type (LDO or
buck)
- new macro REGULATOR_LINEAR_VRANGE() (patch 10)
- address some (pre-existing) checkpatch warnings)
- various updates to regulator driver (patch 16 & 19).See footer there
- more descriptive commit messages
- Link to v2: https://lore.kernel.org/r/20250606-s2mpg1x-regulators-v2-...
Changes in v2:
- s2mpg11 also exposes additional GPIOs, update all relevant commit
messages (nevertheless, GPIOs are out of scope in this series)
- fix some commit message typos: s2mp1 -> s2mpg1
- patch2: drop | (literal style mark) from samsung,ext-control-gpios
- patch5: add | to vinb*-supply description for better formatting
- patch13: update ::of_parse_cb assignment
- patch15: drop duplicated ::of_parse_cb assignment
- Link to v1: https://lore.kernel.org/r/20250604-s2mpg1x-regulators-v1-...
---
André Draszik (20):
dt-bindings: firmware: google,gs101-acpm-ipc: convert regulators to lowercase
dt-bindings: mfd: samsung,s2mps11: split s2mpg10 into separate file
regulator: dt-bindings: add s2mpg10-pmic regulators
regulator: dt-bindings: add s2mpg11-pmic regulators
dt-bindings: mfd: samsung,s2mpg10: Link s2mpg10-pmic to its regulators
dt-bindings: mfd: samsung,s2mpg10: Add s2mpg11-pmic
dt-bindings: firmware: google,gs101-acpm-ipc: update PMIC examples
mfd: sec-common: Instantiate s2mpg10 bucks and ldos separately
mfd: sec: Add support for S2MPG11 PMIC via ACPM
regulator: add REGULATOR_LINEAR_VRANGE macro
regulator: s2mps11: drop two needless variable initialisations
regulator: s2mps11: use dev_err_probe() where appropriate
regulator: s2mps11: place constants on right side of comparison tests
regulator: s2mps11: update node parsing (allow -supply properties)
regulator: s2mps11: refactor handling of external rail control
regulator: s2mps11: add S2MPG10 regulator
regulator: s2mps11: refactor S2MPG10 ::set_voltage_time() for S2MPG11 reuse
regulator: s2mps11: refactor S2MPG10 regulator macros for S2MPG11 reuse
regulator: s2mps11: add S2MPG11 regulator
regulator: s2mps11: more descriptive gpio consumer name
.../bindings/firmware/google,gs101-acpm-ipc.yaml | 44 +-
.../devicetree/bindings/mfd/samsung,s2mpg10.yaml | 124 ++
.../devicetree/bindings/mfd/samsung,s2mps11.yaml | 29 +-
.../regulator/samsung,s2mpg10-regulator.yaml | 158 +++
.../regulator/samsung,s2mpg11-regulator.yaml | 136 +++
MAINTAINERS | 1 +
drivers/mfd/sec-acpm.c | 213 +++-
drivers/mfd/sec-common.c | 88 +-
drivers/mfd/sec-irq.c | 67 +-
drivers/regulator/s2mps11.c | 1193 ++++++++++++++++++--
.../regulator/samsung,s2mpg10-regulator.h | 53 +
include/linux/mfd/samsung/core.h | 1 +
include/linux/mfd/samsung/irq.h | 99 ++
include/linux/mfd/samsung/s2mpg10.h | 24 +
include/linux/mfd/samsung/s2mpg11.h | 434 +++++++
include/linux/regulator/driver.h | 5 +
16 files changed, 2510 insertions(+), 159 deletions(-)
---
base-commit: 131f3d9446a6075192cdd91f197989d98302faa6
change-id: 20250603-s2mpg1x-regulators-7a41c8399abf
Best regards,
--
André Draszik <andre.draszik@linaro.org>