| Message ID | 20250301-exynos7870-pmic-regulators-v3-0-808d0b47a564@disroot.org |
|---|---|
| Headers | show |
| Series | Introduce support for Exynos7870's S2MPU05 PMIC and its regulators | expand |
On Sat, Mar 01, 2025 at 01:07:12AM +0530, Kaustabh Chakraborty wrote: > S2MPU05 is a PMIC found in Exynos7870 devices, which controls voltage > regulators (21 LDOs and 5 BUCKs). Provide documentation for devicetree > definitions, regulator naming patterns, etc. > > Signed-off-by: Kaustabh Chakraborty <kauschluss@disroot.org> > --- > .../bindings/regulator/samsung,s2mpu05.yaml | 47 ++++++++++++++++++++++ > 1 file changed, 47 insertions(+) Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> Best regards, Krzysztof
On Sat, Mar 01, 2025 at 01:07:14AM +0530, Kaustabh Chakraborty wrote: > S2MPU05 is a PMIC by manufactured by Samsung, particularly used in > Exynos7870 based devices. Add driver support for controlling its 21 LDO > and 5 BUCK regulators. > > Signed-off-by: Kaustabh Chakraborty <kauschluss@disroot.org> > Acked-by: Mark Brown <broonie@kernel.org> > --- > drivers/regulator/Kconfig | 4 +- > drivers/regulator/s2mps11.c | 92 ++++++++++++++++++++++++++++++++++++++++++++- > 2 files changed, 93 insertions(+), 3 deletions(-) Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> Best regards, Krzysztof
On Sat, 01 Mar 2025, Kaustabh Chakraborty wrote: > Exynos7870 devices use Samsung S2MPU05 as its primary PMIC. Add support > for it in the existing PMIC driver. Additionally, also add support for > voltage regulators which can be accessed and controlled from the PMIC > itself. > > Patches from mfd and regulator subsystems have been placed together in > this series. Here, both patches from both subsystems depend on the other: > 1. The regulator driver patch includes a header file which describes the > PMIC registers. This header is introduced in a PMIC patch. > 2. The PMIC dt-binding patch references the regulator documentation. > > Note that 1. is a build dependency, but 2. is not. > > This patch series is a part of Exynos7870 upstreaming. > > Signed-off-by: Kaustabh Chakraborty <kauschluss@disroot.org> > --- > Changes in v3: > - Lowercase "ldo" and "buck" in regulator dt-bindings and driver patches. > - Add back missing Reviewed-by: tag in [PATCH v2 2/3]. > - Link to v2: https://lore.kernel.org/r/20250219-exynos7870-pmic-regulators-v2-0-1ea86fb332f7@disroot.org > > Changes in v2: > - Drop applied [PATCH 2/4]. > - Added myself as maintainer in s2mpu05-pmic DT docs. > - Edited LDO description to explain missing LDOs. > - Added all missing regulator control registers intended for CP. > - Modify regulator_desc_s2mpu05_ldo* macros to allow choosing register > suffixes (CTRL, CTRL1, etc). Subsequently, drop Reviewed-by: tag in > [PATCH 4/4]. > - Take over ownership of patches by the co-author, upon their request. > - Link to v1: https://lore.kernel.org/r/20250204-exynos7870-pmic-regulators-v1-0-05adad38102c@disroot.org > > --- > Kaustabh Chakraborty (3): > regulator: dt-bindings: add documentation for s2mpu05-pmic regulators > mfd: sec: add support for S2MPU05 PMIC > regulator: s2mps11: Add support for S2MPU05 regulators > > .../bindings/regulator/samsung,s2mpu05.yaml | 47 ++++++ > drivers/mfd/sec-core.c | 12 ++ > drivers/mfd/sec-irq.c | 34 ++++ > drivers/regulator/Kconfig | 4 +- > drivers/regulator/s2mps11.c | 92 ++++++++++- > include/linux/mfd/samsung/core.h | 1 + > include/linux/mfd/samsung/irq.h | 44 +++++ > include/linux/mfd/samsung/s2mpu05.h | 183 +++++++++++++++++++++ > 8 files changed, 414 insertions(+), 3 deletions(-) > --- > base-commit: be5c7bbb3a64baf884481a1ba0c2f8fb2f93f7c3 > change-id: 20250203-exynos7870-pmic-regulators-26512b79a29b > > Best regards, > -- > Kaustabh Chakraborty <kauschluss@disroot.org> Applied the set. I'll send out a pull-request if all goes well with build testing. Note to self: ib-mfd-regulator-6.15
On Sat, 01 Mar 2025 01:07:11 +0530, Kaustabh Chakraborty wrote: > Exynos7870 devices use Samsung S2MPU05 as its primary PMIC. Add support > for it in the existing PMIC driver. Additionally, also add support for > voltage regulators which can be accessed and controlled from the PMIC > itself. > > Patches from mfd and regulator subsystems have been placed together in > this series. Here, both patches from both subsystems depend on the other: > 1. The regulator driver patch includes a header file which describes the > PMIC registers. This header is introduced in a PMIC patch. > 2. The PMIC dt-binding patch references the regulator documentation. > > [...] Applied, thanks! [1/3] regulator: dt-bindings: add documentation for s2mpu05-pmic regulators commit: 07ef6dc942741b918dd0dcbb951e0ae3dd6b53b9 [2/3] mfd: sec: add support for S2MPU05 PMIC commit: ed33479b7beb2b2dc9649a4e7474b47253d554f9 [3/3] regulator: s2mps11: Add support for S2MPU05 regulators commit: 169cd52fd9445b30379ea6deafa28a260d489699 -- Lee Jones [李琼斯]
Exynos7870 devices use Samsung S2MPU05 as its primary PMIC. Add support for it in the existing PMIC driver. Additionally, also add support for voltage regulators which can be accessed and controlled from the PMIC itself. Patches from mfd and regulator subsystems have been placed together in this series. Here, both patches from both subsystems depend on the other: 1. The regulator driver patch includes a header file which describes the PMIC registers. This header is introduced in a PMIC patch. 2. The PMIC dt-binding patch references the regulator documentation. Note that 1. is a build dependency, but 2. is not. This patch series is a part of Exynos7870 upstreaming. Signed-off-by: Kaustabh Chakraborty <kauschluss@disroot.org> --- Changes in v3: - Lowercase "ldo" and "buck" in regulator dt-bindings and driver patches. - Add back missing Reviewed-by: tag in [PATCH v2 2/3]. - Link to v2: https://lore.kernel.org/r/20250219-exynos7870-pmic-regulators-v2-0-1ea86fb332f7@disroot.org Changes in v2: - Drop applied [PATCH 2/4]. - Added myself as maintainer in s2mpu05-pmic DT docs. - Edited LDO description to explain missing LDOs. - Added all missing regulator control registers intended for CP. - Modify regulator_desc_s2mpu05_ldo* macros to allow choosing register suffixes (CTRL, CTRL1, etc). Subsequently, drop Reviewed-by: tag in [PATCH 4/4]. - Take over ownership of patches by the co-author, upon their request. - Link to v1: https://lore.kernel.org/r/20250204-exynos7870-pmic-regulators-v1-0-05adad38102c@disroot.org --- Kaustabh Chakraborty (3): regulator: dt-bindings: add documentation for s2mpu05-pmic regulators mfd: sec: add support for S2MPU05 PMIC regulator: s2mps11: Add support for S2MPU05 regulators .../bindings/regulator/samsung,s2mpu05.yaml | 47 ++++++ drivers/mfd/sec-core.c | 12 ++ drivers/mfd/sec-irq.c | 34 ++++ drivers/regulator/Kconfig | 4 +- drivers/regulator/s2mps11.c | 92 ++++++++++- include/linux/mfd/samsung/core.h | 1 + include/linux/mfd/samsung/irq.h | 44 +++++ include/linux/mfd/samsung/s2mpu05.h | 183 +++++++++++++++++++++ 8 files changed, 414 insertions(+), 3 deletions(-) --- base-commit: be5c7bbb3a64baf884481a1ba0c2f8fb2f93f7c3 change-id: 20250203-exynos7870-pmic-regulators-26512b79a29b Best regards,