[PATCH v5 0/7] pressure: bmp280: Minor cleanup and interrupt support

Vasileios Amoiridis posted 7 patches 1 year, 3 months ago
There is a newer version of this series
.../bindings/iio/pressure/bmp085.yaml         |  22 +-
drivers/iio/pressure/bmp280-core.c            | 682 +++++++++++++++---
drivers/iio/pressure/bmp280-i2c.c             |   4 +-
drivers/iio/pressure/bmp280-spi.c             |   4 +-
drivers/iio/pressure/bmp280.h                 |  52 ++
5 files changed, 675 insertions(+), 89 deletions(-)
[PATCH v5 0/7] pressure: bmp280: Minor cleanup and interrupt support
Posted by Vasileios Amoiridis 1 year, 3 months ago
Depends on this: https://lore.kernel.org/linux-iio/20240823172017.9028-1-vassilisamir@gmail.com

Changes in v5:

[PATCH v5 1/7]:
	- Use local variable instead of direct assignment to improve
	  readability

[PATCH v5 2/7]:
	- Use local variable instead of direct assignment to improve
	  readability

[PATCH v5 4/7]:
	- Improve commit message
	- Use unit suffix in time variables
	- Use local variable instead of direct assignment to improve
	  readability
	- Use 8 elements per line in the const int array

[PATCH v5 5/7]:
	- Use not in the allOf:int: to reduce if cases
	- Use type:boolean

[PATCH v5 6/7]:
	- Use local variable instead of direct assignment to improve
	  readability
	- Use irq_get_trigger_type()
	- Make return paths consistent

[PATCH v5 7/7]:
	- Use local variable instead of direct assignment to improve
	  readbility

---
v4: https://lore.kernel.org/linux-iio/20240828205128.92145-1-vassilisamir@gmail.com
v3: https://lore.kernel.org/linux-iio/20240823181714.64545-1-vassilisamir@gmail.com
v2: https://lore.kernel.org/linux-iio/20240725231039.614536-1-vassilisamir@gmail.com
v1: https://lore.kernel.org/linux-iio/20240711211558.106327-1-vassilisamir@gmail.com

Vasileios Amoiridis (7):
  iio: pressure: bmp280: Use bulk read for humidity calibration data
  iio: pressure: bmp280: Add support for bmp280 soft reset
  iio: pressure: bmp280: Remove config error check for IIR filter
    updates
  iio: pressure: bmp280: Use sleep and forced mode for oneshot captures
  dt-bindings: iio: pressure: bmp085: Add interrupts for BMP3xx and
    BMP5xx devices
  iio: pressure: bmp280: Add data ready trigger support
  iio: pressure: bmp280: Move bmp085 interrupt to new configuration

 .../bindings/iio/pressure/bmp085.yaml         |  22 +-
 drivers/iio/pressure/bmp280-core.c            | 682 +++++++++++++++---
 drivers/iio/pressure/bmp280-i2c.c             |   4 +-
 drivers/iio/pressure/bmp280-spi.c             |   4 +-
 drivers/iio/pressure/bmp280.h                 |  52 ++
 5 files changed, 675 insertions(+), 89 deletions(-)


base-commit: 0f718e10da81446df0909c9939dff2b77e3b4e95
prerequisite-patch-id: e4f81f31f4fbb2aa872c0c74ed4511893eee0c9a
-- 
2.25.1
Re: [PATCH v5 0/7] pressure: bmp280: Minor cleanup and interrupt support
Posted by Andy Shevchenko 1 year, 3 months ago
On Mon, Sep 02, 2024 at 08:42:15PM +0200, Vasileios Amoiridis wrote:
> Depends on this: https://lore.kernel.org/linux-iio/20240823172017.9028-1-vassilisamir@gmail.com

Reviewed-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>

for the patches 1,2, and 3.

-- 
With Best Regards,
Andy Shevchenko
Re: [PATCH v5 0/7] pressure: bmp280: Minor cleanup and interrupt support
Posted by Vasileios Amoiridis 1 year, 3 months ago
On Tue, Sep 03, 2024 at 05:36:57PM +0300, Andy Shevchenko wrote:
> On Mon, Sep 02, 2024 at 08:42:15PM +0200, Vasileios Amoiridis wrote:
> > Depends on this: https://lore.kernel.org/linux-iio/20240823172017.9028-1-vassilisamir@gmail.com
> 
> Reviewed-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
> 
> for the patches 1,2, and 3.
> 
> -- 
> With Best Regards,
> Andy Shevchenko
> 
> 

Thank you very much for the reviews Andy.

Cheers,
Vasilis
Re: [PATCH v5 0/7] pressure: bmp280: Minor cleanup and interrupt support
Posted by Jonathan Cameron 1 year, 3 months ago
On Wed, 4 Sep 2024 12:29:55 +0200
Vasileios Amoiridis <vassilisamir@gmail.com> wrote:

> On Tue, Sep 03, 2024 at 05:36:57PM +0300, Andy Shevchenko wrote:
> > On Mon, Sep 02, 2024 at 08:42:15PM +0200, Vasileios Amoiridis wrote:  
> > > Depends on this: https://lore.kernel.org/linux-iio/20240823172017.9028-1-vassilisamir@gmail.com  
> > 
> > Reviewed-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
> > 
> > for the patches 1,2, and 3.
> > 
> > -- 
> > With Best Regards,
> > Andy Shevchenko
> > 
> >   
> 
> Thank you very much for the reviews Andy.

Too many patches floating around at the moment (not yours!) so I'm going
to reduce that number where I can just to make it easier to keep track
of what needs more focus.

So picked up patches 1-3 on the togreg branch of iio.git and pushed
out as testing. Note these are probably 6.13 material now.

Jonathan

> 
> Cheers,
> Vasilis
Re: [PATCH v5 0/7] pressure: bmp280: Minor cleanup and interrupt support
Posted by Vasileios Amoiridis 1 year, 3 months ago
On Sat, Sep 07, 2024 at 05:35:37PM +0100, Jonathan Cameron wrote:
> On Wed, 4 Sep 2024 12:29:55 +0200
> Vasileios Amoiridis <vassilisamir@gmail.com> wrote:
> 
> > On Tue, Sep 03, 2024 at 05:36:57PM +0300, Andy Shevchenko wrote:
> > > On Mon, Sep 02, 2024 at 08:42:15PM +0200, Vasileios Amoiridis wrote:  
> > > > Depends on this: https://lore.kernel.org/linux-iio/20240823172017.9028-1-vassilisamir@gmail.com  
> > > 
> > > Reviewed-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
> > > 
> > > for the patches 1,2, and 3.
> > > 
> > > -- 
> > > With Best Regards,
> > > Andy Shevchenko
> > > 
> > >   
> > 
> > Thank you very much for the reviews Andy.
> 
> Too many patches floating around at the moment (not yours!) so I'm going
> to reduce that number where I can just to make it easier to keep track
> of what needs more focus.
> 
> So picked up patches 1-3 on the togreg branch of iio.git and pushed
> out as testing. Note these are probably 6.13 material now.
> 
> Jonathan
> 
> > 
> > Cheers,
> > Vasilis
> 

Hi Jonathan,

Thank you very much for the continuous effort!

Cheers,
Vasilis