From e95ce7682eeacc543c3731d4af1bd726702555e7 Mon Sep 17 00:00:00 2001 From: li feng Date: Tue, 13 Dec 2016 17:17:51 -0800 Subject: bmp280: handle divided by 0 case In bmp280 driver bmp280_set_data_rate(), parameter rate may be 0 On reef ISH enabled board, this caused unhandled exception. BUG=None BRANCH=None TEST=On reef ISH enabled board, verified barometer bmp280 sensor initialized succefully. Change-Id: I7cdae8efbc618fca1551fd9fe809cb69ba0cee34 Signed-off-by: li feng Reviewed-on: https://chromium-review.googlesource.com/419744 Commit-Ready: Li1 Feng Tested-by: Li1 Feng Reviewed-by: Aaron Durbin --- driver/baro_bmp280.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) (limited to 'driver/baro_bmp280.c') diff --git a/driver/baro_bmp280.c b/driver/baro_bmp280.c index 3a3f3e3b6d..d755b278fd 100644 --- a/driver/baro_bmp280.c +++ b/driver/baro_bmp280.c @@ -352,13 +352,14 @@ static int bmp280_set_data_rate(const struct motion_sensor_t *s, int rate, { struct bmp280_drv_data_t *data = BMP280_GET_DATA(s); int durn, i, ret; - int period = 1000000 / rate; /* Period in ms */ + int period; /* Period in ms */ if (rate == 0) { /* Set to sleep mode */ data->rate = 0; return bmp280_set_power_mode(s, BMP280_SLEEP_MODE); - } + } else + period = 1000000 / rate; /* reset power mode, waking from sleep */ if (!data->rate) { -- cgit v1.2.1