Re: [PATCH 2/3] iio: pressure: dps310: reset the chip at probe

From: Jonathan Cameron

Date: Sun Nov 01 2026 - 12:31:02 EST


On Sun, 1 Nov 2026 12:58:55 +0200
Rupesh Majhi <zoone.rupert@xxxxxxxxx> wrote:

> On Sun, 01 Nov 2026 01:04:44 +0000
> Jonathan Cameron <jic23@xxxxxxxxxx> wrote:
>
> > > Reset the chip first, as remove already does.
> > Why do we do it in remove? Does it put it in a power saving mode
> > or similar? It's fairly unusual to bother resetting in that path
> > given as you've seen here, we pretty much have to reset in probe.
>
> Yes, in effect. Driver leaves the chip measuring in background mode,
> and soft reset returns it to standby (datasheet 4.1). It came with the
> original driver without a stated reason.
>
> Writing 0 to MEAS_CTRL would also stop it, if you'd rather not reset
> there.
I would prefer that as it would avoid the need for documentation
to say that the reset is there to put it into a low power mode.
If there hadn't been any other way to do it (unlike writing that 0)
then I'd have asked for a comment.

Thanks,

Jonathan

>
> Rupesh