pressure: bmp280: Improve pushing of data to buffer

[PATCH v3 0/2] pressure: bmp280: Improve pushing of data to buffer

Posted by Vasileios Amoiridis 1 month, 4 weeks ago

Changes in v3:

[PATCH v3 1/2]:
	- Remove fixes tag

[PATCH v3 2/2]:
	- Use internal s32 *chans variable to better visualize what is
	  taking place in the data->sensor_data.
	- Use proper size/alignment to the data->sensor_data.

P.S. After this patchseries is applied, I will rebase this [1] and resend it.

[1]: https://lore.kernel.org/linux-iio/20240914002900.45158-1-vassilisamir@gmail.com/

---
v2: https://lore.kernel.org/linux-iio/20240929112511.100292-1-vassilisamir@gmail.com/
v1: https://lore.kernel.org/linux-iio/20240823172017.9028-1-vassilisamir@gmail.com/

Vasileios Amoiridis (2):
  iio: pressure: bmp280: Use unsigned type for raw values
  iio: pressure: bmp280: Use char instead of s32 for data buffer

 drivers/iio/pressure/bmp280-core.c | 69 +++++++++++++++++++-----------
 drivers/iio/pressure/bmp280.h      |  4 +-
 2 files changed, 46 insertions(+), 27 deletions(-)

-- 
2.25.1

Re: [PATCH v3 0/2] pressure: bmp280: Improve pushing of data to buffer

Posted by Jonathan Cameron 1 month, 3 weeks ago

On Mon, 30 Sep 2024 22:23:51 +0200
Vasileios Amoiridis <vassilisamir@gmail.com> wrote:

> Changes in v3:
> 
> [PATCH v3 1/2]:
> 	- Remove fixes tag
> 
> [PATCH v3 2/2]:
> 	- Use internal s32 *chans variable to better visualize what is
> 	  taking place in the data->sensor_data.
> 	- Use proper size/alignment to the data->sensor_data.
> 
> P.S. After this patchseries is applied, I will rebase this [1] and resend it.
> 
> [1]: https://lore.kernel.org/linux-iio/20240914002900.45158-1-vassilisamir@gmail.com/
Applied to the togreg branch of iio.git and pushed out as testing for all the normal
boring reasons.

Thanks,
Jonathan

> 
> ---
> v2: https://lore.kernel.org/linux-iio/20240929112511.100292-1-vassilisamir@gmail.com/
> v1: https://lore.kernel.org/linux-iio/20240823172017.9028-1-vassilisamir@gmail.com/
> 
> Vasileios Amoiridis (2):
>   iio: pressure: bmp280: Use unsigned type for raw values
>   iio: pressure: bmp280: Use char instead of s32 for data buffer
> 
>  drivers/iio/pressure/bmp280-core.c | 69 +++++++++++++++++++-----------
>  drivers/iio/pressure/bmp280.h      |  4 +-
>  2 files changed, 46 insertions(+), 27 deletions(-)
>

Re: [PATCH v3 0/2] pressure: bmp280: Improve pushing of data to buffer

Posted by Vasileios Amoiridis 1 month, 3 weeks ago

On Sun, Oct 06, 2024 at 03:05:17PM +0100, Jonathan Cameron wrote:
> On Mon, 30 Sep 2024 22:23:51 +0200
> Vasileios Amoiridis <vassilisamir@gmail.com> wrote:
> 
> > Changes in v3:
> > 
> > [PATCH v3 1/2]:
> > 	- Remove fixes tag
> > 
> > [PATCH v3 2/2]:
> > 	- Use internal s32 *chans variable to better visualize what is
> > 	  taking place in the data->sensor_data.
> > 	- Use proper size/alignment to the data->sensor_data.
> > 
> > P.S. After this patchseries is applied, I will rebase this [1] and resend it.
> > 
> > [1]: https://lore.kernel.org/linux-iio/20240914002900.45158-1-vassilisamir@gmail.com/
> Applied to the togreg branch of iio.git and pushed out as testing for all the normal
> boring reasons.
> 
> Thanks,
> Jonathan
> 

Hi Jonathan, thanks a lot!

Cheers,
Vasilis

> > 
> > ---
> > v2: https://lore.kernel.org/linux-iio/20240929112511.100292-1-vassilisamir@gmail.com/
> > v1: https://lore.kernel.org/linux-iio/20240823172017.9028-1-vassilisamir@gmail.com/
> > 
> > Vasileios Amoiridis (2):
> >   iio: pressure: bmp280: Use unsigned type for raw values
> >   iio: pressure: bmp280: Use char instead of s32 for data buffer
> > 
> >  drivers/iio/pressure/bmp280-core.c | 69 +++++++++++++++++++-----------
> >  drivers/iio/pressure/bmp280.h      |  4 +-
> >  2 files changed, 46 insertions(+), 27 deletions(-)
> > 
>