Hi Rick, On 12 December 2016 at 09:57, Rick Bronson <[email protected]> wrote: > Hi Simon, > >> >> On 9 December 2016 at 18:12, Rick Bronson <[email protected]> wrote: >> > Hi All, >> > >> > How do I enable a particular regulator upon boot? I have two >> > identically set LDO entries: >> > >> > vccio_en: LDO_REG1 { >> > regulator-always-on; >> > regulator-boot-on; >> > regulator-min-microvolt = <3300000>; >> > regulator-max-microvolt = <3300000>; >> > regulator-name = "vccio_en"; >> > regulator-state-mem { >> > regulator-on-in-suspend; >> > regulator-suspend-microvolt = > <3300000>; >> > }; >> > }; >> > >> > vcc33_mic: LDO_REG2 { >> > regulator-always-on; >> > regulator-boot-on; >> > regulator-min-microvolt = <3300000>; >> > regulator-max-microvolt = <3300000>; >> > regulator-name = "vcc33_mic"; >> > regulator-state-mem { >> > regulator-on-in-suspend; >> > regulator-suspend-microvolt = > <3300000>; >> > }; >> > }; >> > >> > Yet one is enabled, the other disabled, any idea why?: >> > >> > => regulator status >> > Name Enabled uV mA Mode >> > ... >> > vccio_en enabled 3300000 - - >> > vcc33_mic disabled 3300000 - - >> > >> > And oddly, the uV values actually don't come from the DT but from >> > the rk808_ldo table in drivers/power/regulator/rk808.c >> >> Do you think this is happening by PMIC settings (in the device) rather >> than through U-Boot? > > Think I found the reason for this, it's this way because of the way > BOOT0, BOOT1 are strapped on the RK808. > >> >> > >> > Any ideas? >> > >> > Thanks for any help. >> > >> >> There is a function called regulators_enable_boot_on() which enables >> all boot-on regulators that have a fixed voltage, but I don't think >> that is called with rockchip. >> >> Now that I look at it, I cannot see why I put the voltage values in >> the driver. They should come form DT. > > Do you think the right way to solve this is (from common/board_r.c): > > __weak int power_init_board(void) > { > regulators_enable_boot_on(false); > return 0; > } > > Tried this but it seems to introduce a race condition because often, it > hangs.
This should only enable regulators - never disable them. I suggest trying to call regulator_autoset() for each regulator in turn in your function so that you know what causes the problem. I can't really imagine why enabling a regulator could cause a problem though. You might consider putting the code in board_init(). > > Cheers, > > Rick > > Regards, Simon > _______________________________________________ U-Boot mailing list [email protected] http://lists.denx.de/mailman/listinfo/u-boot
