From fcfd32f04a58e1baf54b39d97c7b37eabb0770db Mon Sep 17 00:00:00 2001 From: Gwendal Grignou Date: Fri, 16 Oct 2015 11:06:35 -0700 Subject: [PATCH] motion: fix manage_activity interface Declare optional parameters are const structure. These parameters, when used, are just read by the sensor driver. BRANCH=smaug BUG=None TEST=compile Change-Id: I8f2a9291e1908922831fb5e2a524bb6edd0e0f65 Signed-off-by: Gwendal Grignou Reviewed-on: https://chromium-review.googlesource.com/306696 Reviewed-by: Alec Berg --- common/motion_sense.c | 2 +- driver/accelgyro_bmi160.c | 2 +- include/accelgyro.h | 2 +- 3 files changed, 3 insertions(+), 3 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/driver/accelgyro_bmi160.c b/driver/accelgyro_bmi160.c index 00c5e76090..03024db3f8 100644 --- a/driver/accelgyro_bmi160.c +++ b/driver/accelgyro_bmi160.c @@ -661,7 +661,7 @@ void normalize(const struct motion_sensor_t *s, vector_3_t v, uint8_t *data) int manage_activity(const struct motion_sensor_t *s, enum motionsensor_activity activity, int enable, - struct ec_motion_sense_activity *param) + const struct ec_motion_sense_activity *param) { int ret; struct bmi160_drv_data_t *data = BMI160_GET_DATA(s); diff --git a/include/accelgyro.h b/include/accelgyro.h index 201fd1d07d..699fcda6df 100644 --- a/include/accelgyro.h +++ b/include/accelgyro.h @@ -133,7 +133,7 @@ struct accelgyro_drv { int (*manage_activity)(const 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.