[PATCH 1/4] power: regulator: Trigger probe of regulators which are always-on or boot-on

Marek Vasut marex at denx.de
Sun Jul 28 20:35:27 CEST 2024


On 7/28/24 7:55 PM, Svyatoslav Ryhel wrote:
> нд, 28 лип. 2024 р. о 19:38 Marek Vasut <marex at denx.de> пише:
>>
>> On 6/27/24 1:55 AM, 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.
>>
>> Is there anything blocking this series from being applied ?
> 
> This patchset causes PMIC regulators probe too early which results in
> i2c line setup failure. These patches MUST NOT be applied in this form
> since they will break at least 15 Tegra 3 devices which use DM PMIC,
> maybe more.

Thank you for testing. I do not have any tegra 3 devices, but this 
patchset does not do anything with pinmuxing. If a regulator is probed, 
all of its dependencies (i2c bus, pinmux configuration, etc.) should be 
probed as well. Can you have a look at what the problem with pinmuxing 
is on tegra 3? It seems it might be unrelated to this patchset and would 
eventually show up elsewhere?


More information about the U-Boot mailing list