On Fri, Jan 5, 2018 at 1:06 AM, Enric Balletbo i Serra <enric.balletbo@xxxxxxxxxxxxx> wrote: > Hi Gwendal, > > On 05/01/18 01:32, Gwendal Grignou wrote: >> Add a driver when EC presents a barometer. >> >> BUG=chrome-os-partner:57111 >> TEST=Check the values returned by the EC >> cat scale >> 0.040000000 >> cat in_pressure_raw >> 24097 >> scale * in_pressure_raw --> 963.88 hPa >> >> Reviewed-on: https://chromium-review.googlesource.com/382832 >> Reviewed-by: Dmitry Torokhov <dtor@xxxxxxxxxxxx> >> (cherry picked from commit 686988372cdc ("CHROMIUM: iio: cros_ec: >> add cros_ec barometer driver")) >> Signed-off-by: Gwendal Grignou <gwendal@xxxxxxxxxxxx> >> --- >> drivers/iio/common/cros_ec_sensors/Kconfig | 7 + >> drivers/iio/common/cros_ec_sensors/Makefile | 1 + >> drivers/iio/common/cros_ec_sensors/cros_ec_baro.c | 213 ++++++++++++++++++++++ > > Isn't this the driver that is already available in mainline at > drivers/iio/pressure/cros_ec_baro.c ? Yes it is, my patch is wrong, the code is already in. Thanks, Gwendal. > > iirc the maintainer told me to move the driver to his iio sensor type directory > when I posted this driver. I did a quick look at there are some few differences > but essentially is the same driver so maybe what we need to do is sync the one > is in mainline with the latest version in chromeos kernel. > > Best regards, > Enric > -- To unsubscribe from this list: send the line "unsubscribe linux-iio" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
