On Thu, 29 Sept 2022 at 10:09, Martyn Welch <martyn.we...@collabora.co.uk> wrote: > > Hi Simon, > > Is this approach acceptable to you (instead of adding legacy support > for this device)?
Reviewed-by: Simon Glass <s...@chromium.org> > > 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 >