All the proposals were implemented, and 2 extra patches were added (Patch 2 and patch 4) in order to have better logical split between patches. Changes in v2: Patch 1: Sorted and removed headers as per request. Patch 2: *NEW* Patch, adds coefficients for IIO units in the chip_info structure as per request, remove them from the read_* functions. Patch 3: Patch 2 of v1. Added RAW values as well, as per request. Patch 4: *NEW* Remove the temperature reading from inside read_* functions so the addition of a buffer for userspace is facilitated + make the code much more intuitive. Patch 5: Patch 3 of v1. No logical change, only minor typos as per request. Patch 6: Patch 4 of v1. Previous commits allowed for much cleaner approach as per request. Dropped filling of extra buffer in the read_* functions. Patch 4 allows to put extra buffer in the union of the chip_info and fill the buffer in the buffer_handler function. [1] https://lore.kernel.org/linux-iio/20240303165300.468011-1-vassilisamir@xxxxxxxxx Vasileios Amoiridis (6): iio: pressure: BMP280 core driver headers sorting iio: pressure: Simplify read_* functions iio: pressure: add SCALE and RAW values for channels iio: pressure: Simplify and make more clear temperature readings iio: pressure: Add timestamp and scan_masks for BM280 driver iio: pressure: Add triggered buffer support for BMP280 driver drivers/iio/pressure/Kconfig | 2 + drivers/iio/pressure/bmp280-core.c | 431 +++++++++++++++++++++-------- drivers/iio/pressure/bmp280.h | 18 +- 3 files changed, 326 insertions(+), 125 deletions(-) -- 2.25.1
