Re: [PATCH v3 0/2] regulator: qcom-rpmh: Add off-on-delay support

[Mark Brown <broonie@kernel.org>]

[-- Attachment #1: Type: text/plain, Size: 1400 bytes --]

On Wed, Sep 09, 2026 at 02:34:21AM +0530, Kamal Wadhwa wrote:

Please fix your mail client to word wrap within paragraphs at something
substantially less than 80 columns.  Doing this makes your messages much
easier to read and reply to.

> However recently on Qualcomm's QCS8300 RB4 boards we started seeing "NOP OUT" issue
> for UFS driver, where this patch seems to be helping.

> Also, Rob had earlier raised a few points to Saikiran on this patch series
> (https://lore.kernel.org/all/20260129174829.GA1324020-robh@kernel.org/):

> 3. If we can avoid turning the regulator off itself, so we may not need this delay

> point 3 - We evaluated this for UFS - but the problem is that the issue happens in
> the defer probe path, so if we avoid turning off the regulator, it leads to long
> error prints appearing for the regulator being left ON. So we couldn't take that
> path.

I think Rob might be thinking here of something like the runtime PM
autosuspend delay where once the final reference is dropped we wait some
time before we actually implement the power off in hardware, thus a
quick power cycle would never actually get implemented.  That does sound
like a good idea if we've got issues with quick cycling, obviously the
actual delay would still be needed but if you've got quick bounces
happening routinely then a delay like that would allow you to avoid both
power off and ramp delays.

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]