[v1] iio: pressure: bmp280: Minor cleanup

[PATCH v1 0/3] iio: pressure: bmp280: Minor cleanup

Posted by Vasileios Amoiridis 1 year, 2 months ago

This series adds the SPI interface description on the device-tree file
of the sensor, adds proper self-described sized variables and performs
a minor optimization in time variable names.

Vasileios Amoiridis (3):
  dt-bindings: iio: pressure: bmp085: Add SPI interface
  iio: pressure: bmp280: Use sizeof() for denominator
  iio: pressure: bmp280: Make time vars intuitive and move to fsleep

 .../bindings/iio/pressure/bmp085.yaml         | 35 +++++++++++++++++
 drivers/iio/pressure/bmp280-core.c            | 39 ++++++++++---------
 drivers/iio/pressure/bmp280.h                 |  8 ++--
 3 files changed, 59 insertions(+), 23 deletions(-)


base-commit: a61ff7eac77e86de828fe28c4e42b8ae9ec2b195
-- 
2.43.0

Re: [PATCH v1 0/3] iio: pressure: bmp280: Minor cleanup

Posted by Jonathan Cameron 1 year, 2 months ago

On Fri, 29 Nov 2024 00:24:47 +0100
Vasileios Amoiridis <vassilisamir@gmail.com> wrote:

> This series adds the SPI interface description on the device-tree file
> of the sensor, adds proper self-described sized variables and performs
> a minor optimization in time variable names.
> 
> Vasileios Amoiridis (3):
>   dt-bindings: iio: pressure: bmp085: Add SPI interface
>   iio: pressure: bmp280: Use sizeof() for denominator
>   iio: pressure: bmp280: Make time vars intuitive and move to fsleep
Hi Vasileios,

I took a quick look and nothing to add to the other reviews.

Thanks,

Jonathan

> 
>  .../bindings/iio/pressure/bmp085.yaml         | 35 +++++++++++++++++
>  drivers/iio/pressure/bmp280-core.c            | 39 ++++++++++---------
>  drivers/iio/pressure/bmp280.h                 |  8 ++--
>  3 files changed, 59 insertions(+), 23 deletions(-)
> 
> 
> base-commit: a61ff7eac77e86de828fe28c4e42b8ae9ec2b195

Re: [PATCH v1 0/3] iio: pressure: bmp280: Minor cleanup

Posted by Vasileios Amoiridis 1 year, 2 months ago

On Sat, Nov 30, 2024 at 02:30:35PM +0000, Jonathan Cameron wrote:
> On Fri, 29 Nov 2024 00:24:47 +0100
> Vasileios Amoiridis <vassilisamir@gmail.com> wrote:
> 
> > This series adds the SPI interface description on the device-tree file
> > of the sensor, adds proper self-described sized variables and performs
> > a minor optimization in time variable names.
> > 
> > Vasileios Amoiridis (3):
> >   dt-bindings: iio: pressure: bmp085: Add SPI interface
> >   iio: pressure: bmp280: Use sizeof() for denominator
> >   iio: pressure: bmp280: Make time vars intuitive and move to fsleep
> Hi Vasileios,
> 
> I took a quick look and nothing to add to the other reviews.
> 
> Thanks,
> 
> Jonathan
> 

Hi Jonathan,

Thanks, I will go on with a v2.

Cheers,
Vasilis

> > 
> >  .../bindings/iio/pressure/bmp085.yaml         | 35 +++++++++++++++++
> >  drivers/iio/pressure/bmp280-core.c            | 39 ++++++++++---------
> >  drivers/iio/pressure/bmp280.h                 |  8 ++--
> >  3 files changed, 59 insertions(+), 23 deletions(-)
> > 
> > 
> > base-commit: a61ff7eac77e86de828fe28c4e42b8ae9ec2b195
>