Hi Simon, Is this approach acceptable to you (instead of adding legacy support for this device)?
Martyn On Fri, 2022-09-16 at 11:24 +0100, Martyn Welch wrote: > The support added later in this series tweaks the PMIC voltages in > the > SPL. Enable support for the rn5t567 in SPL builds to allow this to be > done > cleanly. > > Signed-off-by: Martyn Welch <martyn.we...@collabora.co.uk> > --- > > Changes in v3: > - New patch (replaces addition of legacy support for rn5t567) > > drivers/power/pmic/Kconfig | 8 ++++++++ > drivers/power/pmic/Makefile | 2 +- > 2 files changed, 9 insertions(+), 1 deletion(-) > > diff --git a/drivers/power/pmic/Kconfig b/drivers/power/pmic/Kconfig > index 66b16b06e0..8e10ecae2e 100644 > --- a/drivers/power/pmic/Kconfig > +++ b/drivers/power/pmic/Kconfig > @@ -300,6 +300,14 @@ config PMIC_RN5T567 > regulators Real-Time Clock and 4 GPIOs. This driver provides > register access only. > > +config SPL_PMIC_RN5T567 > + bool "Enable driver for Ricoh RN5T567 PMIC in SPL" > + depends on SPL_DM_PMIC > + ---help--- > + The RN5T567 is a PMIC with 4 step-down DC/DC converters, 5 > LDO > + regulators Real-Time Clock and 4 GPIOs. This driver provides > + register access only. > + > config PMIC_TPS65090 > bool "Enable driver for Texas Instruments TPS65090 PMIC" > ---help--- > diff --git a/drivers/power/pmic/Makefile > b/drivers/power/pmic/Makefile > index f73b326255..ae34e6b33e 100644 > --- a/drivers/power/pmic/Makefile > +++ b/drivers/power/pmic/Makefile > @@ -22,7 +22,7 @@ obj-$(CONFIG_$(SPL_)PMIC_AXP) += axp.o > obj-$(CONFIG_PMIC_MAX8997) += max8997.o > obj-$(CONFIG_PMIC_PM8916) += pm8916.o > obj-$(CONFIG_$(SPL_TPL_)PMIC_RK8XX) += rk8xx.o > -obj-$(CONFIG_PMIC_RN5T567) += rn5t567.o > +obj-$(CONFIG_$(SPL_)PMIC_RN5T567) += rn5t567.o > obj-$(CONFIG_PMIC_TPS65090) += tps65090.o > obj-$(CONFIG_PMIC_S5M8767) += s5m8767.o > obj-$(CONFIG_DM_PMIC_TPS65910) += pmic_tps65910_dm.o
