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

[Marek Vasut]

On 9/25/24 3:04 PM, Svyatoslav Ryhel wrote:
> ср, 25 вер. 2024 р. о 15:48 Marek Vasut <marex at denx.de> пише:
>>
>> On 9/25/24 12:18 PM, Svyatoslav Ryhel wrote:
>>
>> [...]
>>
>>> Hello there!
>>> I was digging this when I had some free time and found that with
>>> patches from Marek the only difference is that function
>>> i2c_get_chip_for_busnum is not called for PMIC's main i2c address
>>
>> Is it possible this is called earlier, before UART output is
>> initialized, so it does not show up in the log below ?
>>
> It is highly unlikely, this function is called when uart is already
> working and pinmux is set.

It could be called earlier now, try to enable CONFIG_BOOTSTAGE and add 
BOOTSTAGE_MARKER() to points of interest (like the I2C functions). You 
should be able to see whether that function was called in 'bootstage' 
command output, even if UART wasn't active at that point yet.

>> Could it be that it is called before your pinmux is properly
>> initialized, hence no UART, and that is why the I2C communication fails?
>>
> But if we assume that pinmux is not set, then how we resolve situation
> when pinmux and regulator enable are both set by probe after bind and
> regulators are probed before pinmux. Regulators will fail, they will
> not deffer till pinmux probes. Or you propose return to pre-DM version
> of pinmux? Inconsistency.

I am not going to guess what the problem is, currently it is unclear. 
Once we know what the problem is, we can determine the best solution.

>> So far, I only found Toradex Tegra T20 board here, no T30.
> What do you mean? Specify please

I have Toradex Tegra T20 SoM and board here, probably not useful.