[PATCH v2 1/5] power: regulator: Trigger probe of regulators which are always-on or boot-on
Marek Vasut
marex at denx.de
Fri Sep 27 00:16:51 CEST 2024
On 9/26/24 5:05 PM, Jonas Karlman wrote:
> Hi Marek,
Hi,
> On 2024-09-25 04:21, Marek Vasut wrote:
>> In case a regulator DT node contains regulator-always-on or regulator-boot-on
>> property, make sure the regulator gets correctly configured by U-Boot on start
>> up. Unconditionally probe such regulator drivers. This is a preparatory patch
>> for introduction of .regulator_post_probe() which would trigger the regulator
>> configuration.
>>
>> Parsing of regulator-always-on and regulator-boot-on DT property has been
>> moved to regulator_post_bind() as the information is required early, the
>> rest of the DT parsing has been kept in regulator_pre_probe() to avoid
>> slowing down the boot process.
>>
>> Signed-off-by: Marek Vasut <marex at denx.de>
>
> [snip]
>
>>
>> - debug("'%s' of dev: '%s', has nonunique value: '%s\n",
>> - property, dev->name, uc_pdata->name);
>> + /*
>> + * In case the regulator has regulator-always-on or
>> + * regulator-boot-on DT property, trigger probe() to
>> + * configure its default state during startup.
>> + */
>> + if (uc_pdata->always_on && uc_pdata->boot_on)
>
> This check for always_on _and_ boot_on does not fully match the commit
> message, comment or the old behavior of regulators_enable_boot_on()
> where any always_on _or_ boot_on would trigger autoset().
This should be ORR, thanks for spotting this, fixed in V3.
More information about the U-Boot
mailing list