commit 5c49ec30b57a18ac022601473c9524f40554c9cf
author Mathias Agopian <mathias@google.com> Wed Jul 21 18:13:39 2010 -0700
committer Android Git Automerger <android-git-automerger@android.com> Wed Jul 21 18:13:39 2010 -0700
tree 9ac6fd8c1c7315ca8d785d009ce36345eb38c429
parent 409c484b6d235b52b69e87a44b1f51a1ac5ca1f9
parent 1f2ec4070aa9c999d41dc50a892a896e894c5710

am 1f2ec407: am 23e8de26: propagate sensor event rate properly

Merge commit '1f2ec4070aa9c999d41dc50a892a896e894c5710'

* commit '1f2ec4070aa9c999d41dc50a892a896e894c5710':
  propagate sensor event rate properly

core/jni/android_hardware_SensorManager.cpp

diff --git a/core/jni/android_hardware_SensorManager.cpp b/core/jni/android_hardware_SensorManager.cpp
index 232c8af..7b23418 100644
--- a/core/jni/android_hardware_SensorManager.cpp
+++ b/core/jni/android_hardware_SensorManager.cpp
@@ -100,13 +100,13 @@
 
 static jboolean
 sensors_enable_sensor(JNIEnv *env, jclass clazz,
-        jint nativeQueue, jstring name, jint sensor, jint enable)
+        jint nativeQueue, jstring name, jint sensor, jint delay)
 {
     sp<SensorEventQueue> queue(reinterpret_cast<SensorEventQueue *>(nativeQueue));
     if (queue == 0) return JNI_FALSE;
     status_t res;
-    if (enable) {
-        res = queue->enableSensor(sensor);
+    if (delay >= 0) {
+        res = queue->enableSensor(sensor, delay);
     } else {
         res = queue->disableSensor(sensor);
     }

include/gui/SensorEventQueue.h

diff --git a/include/gui/SensorEventQueue.h b/include/gui/SensorEventQueue.h
index bb03c12..ad36dac 100644
--- a/include/gui/SensorEventQueue.h
+++ b/include/gui/SensorEventQueue.h
@@ -62,10 +62,12 @@
 
     status_t enableSensor(Sensor const* sensor) const;
     status_t disableSensor(Sensor const* sensor) const;
-    status_t enableSensor(int32_t handle) const;
-    status_t disableSensor(int32_t handle) const;
     status_t setEventRate(Sensor const* sensor, nsecs_t ns) const;
 
+    // these are here only to support SensorManager.java
+    status_t enableSensor(int32_t handle, int32_t ms) const;
+    status_t disableSensor(int32_t handle) const;
+
 private:
     sp<PollLoop> getPollLoop() const;
     sp<ISensorEventConnection> mSensorEventConnection;

libs/gui/SensorEventQueue.cpp

diff --git a/libs/gui/SensorEventQueue.cpp b/libs/gui/SensorEventQueue.cpp
index cc98656..4b46842 100644
--- a/libs/gui/SensorEventQueue.cpp
+++ b/libs/gui/SensorEventQueue.cpp
@@ -114,8 +114,12 @@
     return mSensorEventConnection->enableDisable(sensor->getHandle(), false);
 }
 
-status_t SensorEventQueue::enableSensor(int32_t handle) const {
-    return mSensorEventConnection->enableDisable(handle, true);
+status_t SensorEventQueue::enableSensor(int32_t handle, int32_t ms) const {
+    status_t err = mSensorEventConnection->enableDisable(handle, true);
+    if (err == NO_ERROR) {
+        mSensorEventConnection->setEventRate(handle, ms2ns(ms));
+    }
+    return err;
 }
 
 status_t SensorEventQueue::disableSensor(int32_t handle) const {

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);