summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorGwendal Grignou <gwendal@chromium.org>2015-10-16 11:06:35 -0700
committerChromeOS Commit Bot <chromeos-commit-bot@chromium.org>2016-08-01 16:41:07 +0000
commit8a9a10da54e6d2201511de5e638b4064bf849a01 (patch)
tree161772d0adc8c027f7eabadc682b17b18e62d977
parent3239b84b22019c7c9b7d78e0e3d06e2f0369b547 (diff)
downloadchrome-ec-8a9a10da54e6d2201511de5e638b4064bf849a01.tar.gz
BACKPORT: motion: fix manage_activity interface
Declare optional parameters are const structure. These parameters, when used, are just read by the sensor driver. BRANCH=smaug, cyan BUG=b:27849483 TEST=compile Change-Id: I8f2a9291e1908922831fb5e2a524bb6edd0e0f65 Signed-off-by: Gwendal Grignou <gwendal@chromium.org> Reviewed-on: https://chromium-review.googlesource.com/306696 Reviewed-by: Alec Berg <alecaberg@chromium.org> (cherry picked from commit fcfd32f04a58e1baf54b39d97c7b37eabb0770db) Signed-off-by: Gwendal Grignou <gwendal@chromium.org> Reviewed-on: https://chromium-review.googlesource.com/359389
-rw-r--r--common/motion_sense.c2
-rw-r--r--include/accelgyro.h2
2 files changed, 2 insertions, 2 deletions
diff --git a/common/motion_sense.c b/common/motion_sense.c
index 2095882a6f..d6b94d0128 100644
--- a/common/motion_sense.c
+++ b/common/motion_sense.c
@@ -1119,7 +1119,7 @@ static int host_cmd_motion_sense(struct host_cmd_handler_args *args)
ret = sensor->drv->manage_activity(sensor,
in->set_activity.activity,
in->set_activity.enable,
- NULL);
+ &in->set_activity);
}
if (ret != EC_RES_SUCCESS)
return ret;
diff --git a/include/accelgyro.h b/include/accelgyro.h
index a941c2fcb9..dd7d50a6d9 100644
--- a/include/accelgyro.h
+++ b/include/accelgyro.h
@@ -133,7 +133,7 @@ struct accelgyro_drv {
int (*manage_activity)(struct motion_sensor_t *s,
enum motionsensor_activity activity,
int enable,
- struct ec_motion_sense_activity *data);
+ const struct ec_motion_sense_activity *data);
/**
* List activities managed by the sensors.
* @s Pointer to sensor data.