On Fri, Jan 5, 2018 at 1:04 AM, Enric Balletbo i Serra <enric.balletbo@xxxxxxxxxxxxx> wrote: > Hi Gwendal, > > On 05/01/18 01:32, Gwendal Grignou wrote: >> Add driver for light and proximity sensors exposed by EC. >> >> Reviewed-on: https://chromium-review.googlesource.com/291360 >> Reviewed-by: Dmitry Torokhov <dtor@xxxxxxxxxxxx> >> [rebase44(groeck): Adjust for API changes] >> [rebase414(gwendal): Adjust for API changes] >> Signed-off-by: Guenter Roeck <groeck@xxxxxxxxxxxx> >> (cherry picked from commit 17e781c5f2747236ea1f9de6906e04aa92d383cb) >> Signed-off-by: Gwendal Grignou <gwendal@xxxxxxxxxxxx> >> --- >> drivers/iio/common/cros_ec_sensors/Kconfig | 9 + >> drivers/iio/common/cros_ec_sensors/Makefile | 1 + >> .../common/cros_ec_sensors/cros_ec_light_prox.c | 260 +++++++++++++++++++++ > > Isn't this the driver that is already available in mainline at > drivers/iio/light/cros_ec_light_prox.c ? Yes it is, my patch is wrong, like the pressure sensor, 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