Re: [PATCH v2 1/1] media: i2c: ov02c10: Keep power on and use reset for power management

[Bryan O'Donoghue <bod@kernel.org>]

On 26/01/2026 11:58, Saikiran B wrote:
> "I don't think we've established the regulator is at fault. That's the
> feedback I'm giving you here. ... vreg_cam_1p8: regulator-cam-1p8 {
> compatible = "regulator-fixed";"
> 
> Just to clarify on the regulators: on the Slim 7x, the camera supplies
> (avdd, dvdd, dovdd) are all RPMh-controlled LDOs (pm8010 and pm8550),
> not generic fixed regulators.

Slim7x - not the Dell right ;)

> As I've confirmed that the qcom-rpmh-regulator driver doesn't natively
> support active discharge or parsing off-on-delay-us (generic
> property), which explains why the physical discharge constraint wasn't
> being respected.

No, the RPMh firmware should know how to do that. Not the Linux side, 
this is the part of your brown-out story that doesn't make sense.

BTW, did you try my given sequence - particularly the XSHUTDOWN in 
power_on(); ?

If the XSHUTDOWN pin is for example floating or not in the correct 
logical state when you power-on, the chip may not initialise correctly.

Which could lead you to conclude - you are having a regulator problem, 
when in fact you are having a sensor state-machine init problem.

?

---
bod