summaryrefslogtreecommitdiffstats
path: root/libsensors/lps331ap.c
diff options
context:
space:
mode:
authorDheeraj CVR <cvr.dheeraj@gmail.com>2015-06-11 16:44:25 +0400
committerforkbomb <keepcalm444@gmail.com>2015-11-25 08:34:55 +1100
commitfc862f3a4c433c67605f12ad2d9addc6fc1bc831 (patch)
treea753c448934c1ef459b302d5d60d46a07f694d6b /libsensors/lps331ap.c
parentea6c7d9c8609022e19200d888f449139139d9173 (diff)
downloaddevice_samsung_i9300-fc862f3a4c433c67605f12ad2d9addc6fc1bc831.zip
device_samsung_i9300-fc862f3a4c433c67605f12ad2d9addc6fc1bc831.tar.gz
device_samsung_i9300-fc862f3a4c433c67605f12ad2d9addc6fc1bc831.tar.bz2
libsensors: switch to portable typedefs to match callbacks
Using a strict basetype only coding style is not a good idea especially when we are dealing with callbacks which use portable typedef like int64_t. This kind of coding style would most likely end up in overflows at various places especially when dealing with timestamps and sensor delays which are caused when typecasting datatypes of different size. Switch to portable typedefs and get rid of "long". Change-Id: I75b9cace7602345dba9095f046292e6d4db07df4
Diffstat (limited to 'libsensors/lps331ap.c')
-rw-r--r--libsensors/lps331ap.c11
1 files changed, 5 insertions, 6 deletions
diff --git a/libsensors/lps331ap.c b/libsensors/lps331ap.c
index 5c3007b..a8ef55a 100644
--- a/libsensors/lps331ap.c
+++ b/libsensors/lps331ap.c
@@ -150,13 +150,12 @@ int lps331ap_deactivate(struct smdk4x12_sensors_handlers *handlers)
return 0;
}
-int lps331ap_set_delay(struct smdk4x12_sensors_handlers *handlers, long int delay)
+int lps331ap_set_delay(struct smdk4x12_sensors_handlers *handlers, int64_t delay)
{
struct lps331ap_data *data;
- int d;
int rc;
- ALOGD("%s(%p, %ld)", __func__, handlers, delay);
+ ALOGD("%s(%p, %" PRId64 ")", __func__, handlers, delay);
if (handlers == NULL || handlers->data == NULL)
return -EINVAL;
@@ -164,11 +163,11 @@ int lps331ap_set_delay(struct smdk4x12_sensors_handlers *handlers, long int dela
data = (struct lps331ap_data *) handlers->data;
if (delay < 10000000)
- d = 10;
+ delay = 10;
else
- d = delay / 1000000;
+ delay = delay / 1000000;
- rc = sysfs_value_write(data->path_delay, d);
+ rc = sysfs_value_write(data->path_delay, delay);
if (rc < 0) {
ALOGE("%s: Unable to write sysfs value", __func__);
return -1;