summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorGwendal Grignou <gwendal@chromium.org>2015-10-16 11:06:35 -0700
committerChromeOS bot <3su6n15k.default@developer.gserviceaccount.com>2015-10-29 06:19:54 +0000
commit1a54995c6fe24bc14d3f5ff20222643e034d3004 (patch)
treecb1fc778c09106d43147eef519056b741d7bec01
parentf4e9a84cb0b0e2b29dea32f8c1990db54b77faf0 (diff)
downloadchrome-ec-1a54995c6fe24bc14d3f5ff20222643e034d3004.tar.gz
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 <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/309590
-rw-r--r--common/motion_sense.c2
-rw-r--r--driver/accelgyro_bmi160.c2
-rw-r--r--include/accelgyro.h2
3 files changed, 3 insertions, 3 deletions
diff --git a/common/motion_sense.c b/common/motion_sense.c
index cb2bab524e..96f525ab29 100644
--- a/common/motion_sense.c
+++ b/common/motion_sense.c
@@ -1136,7 +1136,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 1e645333a6..ce7b371840 100644
--- a/driver/accelgyro_bmi160.c
+++ b/driver/accelgyro_bmi160.c
@@ -652,7 +652,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.