mbox series

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

Message ID 20240902184222.24874-1-vassilisamir@gmail.com (mailing list archive)
Headers show
Series pressure: bmp280: Minor cleanup and interrupt support | expand

Message

Vasileios Amoiridis Sept. 2, 2024, 6:42 p.m. UTC
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

Comments

Andy Shevchenko Sept. 3, 2024, 2:36 p.m. UTC | #1
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.
Vasileios Amoiridis Sept. 4, 2024, 10:29 a.m. UTC | #2
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
Jonathan Cameron Sept. 7, 2024, 4:35 p.m. UTC | #3
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
Vasileios Amoiridis Sept. 10, 2024, 10:09 p.m. UTC | #4
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