Hi Heiko On Wed, Jun 15, 2022 at 8:23 AM Heiko Thiery <heiko.thi...@gmail.com> wrote: > > Hi Marek, > > [SNIP] > > > > diff --git a/board/freescale/imx8mn_evk/spl.c > > > b/board/freescale/imx8mn_evk/spl.c > > > index 14cb51368f..0d9909a662 100644 > > > --- a/board/freescale/imx8mn_evk/spl.c > > > +++ b/board/freescale/imx8mn_evk/spl.c > > > @@ -83,6 +83,15 @@ int power_init_board(void) > > > #ifdef CONFIG_IMX8MN_LOW_DRIVE_MODE > > > /* Set VDD_SOC/VDD_DRAM to 0.8v for low drive mode */ > > > pmic_reg_write(dev, PCA9450_BUCK1OUT_DVS0, 0x10); > > > +#elif defined(CONFIG_TARGET_IMX8MN_DDR3L_EVK) > > > + /* Set VDD_SOC to 0.85v for DDR3L at 1600MTS */ > > > + pmic_reg_write(dev, PCA9450_BUCK1OUT_DVS0, 0x14); > > > + > > > + /* Disable the BUCK2 */ > > > + pmic_reg_write(dev, PCA9450_BUCK2CTRL, 0x48); > > > + > > > + /* Set NVCC_DRAM to 1.35v */ > > > + pmic_reg_write(dev, PCA9450_BUCK6OUT, 0x1E); > > > #else > > > > All this part is not done by the spl pmic driver? > > I saw that you added the PCA9450 driver. Do you know if this > initialization can be done by the driver when CONFIG_SPL_DM_REGULATOR > is enabled? If I see this correctly, it can't be done. Is that > correct?
+&i2c1 { + u-boot,dm-spl; +}; + +&{/soc@0/bus@30800000/i2c@30a20000/pmic@4b} { + u-boot,dm-spl; +}; + +&{/soc@0/bus@30800000/i2c@30a20000/pmic@4b/regulators} { + u-boot,dm-spl; +}; + +&pinctrl_i2c1 { + u-boot,dm-spl; +}; + +&pinctrl_pmic { + u-boot,dm-spl; +}; + Maybe something like this should work. Now question is about should be done in pre-reloc or not Michael > > -- > Heiko -- Michael Nazzareno Trimarchi Co-Founder & Chief Executive Officer M. +39 347 913 2170 mich...@amarulasolutions.com __________________________________ Amarula Solutions BV Joop Geesinkweg 125, 1114 AB, Amsterdam, NL T. +31 (0)85 111 9172 i...@amarulasolutions.com www.amarulasolutions.com