commit 1cd700015318727d6d42236ab6274f1949fb08ba
author Mathias Agopian <mathias@google.com> Wed Jul 21 15:59:50 2010 -0700
committer Mathias Agopian <mathias@google.com> Wed Jun 27 17:07:53 2012 -0700
tree f209f8348eb9e6b7cbd5f6f03c2f8e21a622b548
parent 50df2959e58fc7408f98d11d77c8428397dca445

propagate sensor event rate properly

Change-Id: I32e67d30e4295285a6827956cc8161b2025d70bc

services/sensorservice/SensorService.cpp

diff --git a/services/sensorservice/SensorService.cpp b/services/sensorservice/SensorService.cpp
index fec9153..a4f6549 100644
--- a/services/sensorservice/SensorService.cpp
+++ b/services/sensorservice/SensorService.cpp
@@ -77,6 +77,9 @@
 
 // ---------------------------------------------------------------------------
 
+// 100 events/s max
+static const nsecs_t MINIMUM_EVENT_PERIOD = ms2ns(10);
+
 SensorService::SensorService()
     : Thread(false),
       mSensorDevice(0),
@@ -284,7 +287,6 @@
     status_t err = NO_ERROR;
     Mutex::Autolock _l(mLock);
     SensorRecord* rec = mActiveSensors.valueFor(handle);
-    LOGW("sensor (handle=%d) is not enabled", handle);
     if (rec) {
         // see if this connection becomes inactive
         connection->removeSensor(handle);
@@ -310,6 +312,12 @@
     if (mInitCheck != NO_ERROR)
         return mInitCheck;
 
+    if (ns < 0)
+        return BAD_VALUE;
+
+    if (ns < MINIMUM_EVENT_PERIOD)
+        ns = MINIMUM_EVENT_PERIOD;
+
     status_t err = NO_ERROR;
     Mutex::Autolock _l(mLock);