Re: [PATCH] iio: dps310: Add pressure sensing capability

From: Eddie James
Date: Mon May 06 2019 - 17:04:53 EST



On 5/5/19 7:09 AM, Jonathan Cameron wrote:
On Fri, 3 May 2019 02:34:30 +0000
Joel Stanley <joel@xxxxxxxxx> wrote:

Hi Eddie,

On Wed, 1 May 2019 at 14:43, Eddie James <eajames@xxxxxxxxxxxxx> wrote:
The DPS310 supports measurement of pressure, so support that in the
driver. Use background measurement like the temperature sensing and
default to lowest precision and lowest measurement rate.
Upstream didn't accept my patch as they wanted it to support pressure
in addition to temperature first. I didn't ever get around to doing
that.

I suggest you send my original patch and device tree binding along
with this one as a series.


The dps310 is already in the upstream witherspoon dts somehow :)



Cheers,

Joel
Exactly as Joel says. Note there were some other minor bits and pieces
in the last review I can find of Joel's driver that will also need
tidying up..

Great to see this driver making progress.


Thanks Joel and Jonathan, my mistake as I didn't base the patch on linux-next. I have submitted the series with a few cleanup changes for Joel's patch.


Thanks!

Eddie


Thanks,

Jonathan