Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1935)

Unified Diff: device/generic_sensor/platform_sensor_iio.cc

Issue 2370343002: [sensors] Ambient light sensor implementation for ChromeOS and Linux. (Closed)
Patch Set: change SetTaskRunner to SetFileTaskRunner as per offline discussion with Mikhail Created 4 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: device/generic_sensor/platform_sensor_iio.cc
diff --git a/device/generic_sensor/platform_sensor_iio.cc b/device/generic_sensor/platform_sensor_iio.cc
new file mode 100644
index 0000000000000000000000000000000000000000..c5fa590ebd39ea8db93a320c8023954b93fd7490
--- /dev/null
+++ b/device/generic_sensor/platform_sensor_iio.cc
@@ -0,0 +1,114 @@
+// Copyright 2016 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "device/generic_sensor/platform_sensor_iio.h"
+
+#include "base/threading/thread.h"
+#include "base/timer/timer.h"
+#include "device/generic_sensor/iio/platform_sensor_utils_iio.h"
+#include "device/generic_sensor/iio/sensor_data_iio.h"
+
+namespace device {
+
+namespace {
+
+// Checks if at least one value has been changed.
+bool HaveValuesChanged(const SensorReading& lhs, const SensorReading& rhs) {
+ return lhs.values[0] != rhs.values[0] || lhs.values[1] != rhs.values[1] ||
+ lhs.values[2] != rhs.values[2];
+}
+
+} // namespace
+
+PlatformSensorIio::PlatformSensorIio(
+ mojom::SensorType type,
+ mojo::ScopedSharedBufferMapping mapping,
+ PlatformSensorProvider* provider,
+ const SensorDataIio& data,
+ std::unique_ptr<SensorReader> sensor_reader,
+ scoped_refptr<base::SingleThreadTaskRunner> polling_thread_task_runner_)
+ : PlatformSensor(type, std::move(mapping), provider),
+ timer_(new base::RepeatingTimer()),
+ default_configuration_(data.default_configuration),
+ reporting_mode_(data.reporting_mode),
+ sensor_reader_(std::move(sensor_reader)),
+ polling_thread_task_runner_(polling_thread_task_runner_),
+ weak_factory_(this) {}
+
+PlatformSensorIio::~PlatformSensorIio() {
+ polling_thread_task_runner_->DeleteSoon(FROM_HERE, timer_);
+}
+
+mojom::ReportingMode PlatformSensorIio::GetReportingMode() {
+ return reporting_mode_;
+}
+
+bool PlatformSensorIio::StartSensor(
+ const PlatformSensorConfiguration& configuration) {
+ DCHECK(task_runner_->BelongsToCurrentThread());
+ return polling_thread_task_runner_->PostTask(
+ FROM_HERE, base::Bind(&PlatformSensorIio::BeginPoll,
+ weak_factory_.GetWeakPtr(), configuration));
+}
+
+void PlatformSensorIio::StopSensor() {
+ DCHECK(task_runner_->BelongsToCurrentThread());
+ polling_thread_task_runner_->PostTask(
+ FROM_HERE, base::Bind(&PlatformSensorIio::StopPoll, this));
+}
+
+bool PlatformSensorIio::CheckSensorConfiguration(
+ const PlatformSensorConfiguration& configuration) {
+ DCHECK(task_runner_->BelongsToCurrentThread());
+ // TODO(maksims): make this sensor dependent.
+ // For example, in case of accelerometer, check current polling frequency
+ // exposed by iio driver.
+ return configuration.frequency() > 0 &&
+ configuration.frequency() <=
+ mojom::SensorConfiguration::kMaxAllowedFrequency;
+}
+
+PlatformSensorConfiguration PlatformSensorIio::GetDefaultConfiguration() {
+ DCHECK(task_runner_->BelongsToCurrentThread());
+ return default_configuration_;
+}
+
+void PlatformSensorIio::BeginPoll(
+ const PlatformSensorConfiguration& configuration) {
+ DCHECK(polling_thread_task_runner_->BelongsToCurrentThread());
+ timer_->Start(FROM_HERE, base::TimeDelta::FromMicroseconds(
+ base::Time::kMicrosecondsPerSecond /
+ configuration.frequency()),
+ this, &PlatformSensorIio::PollForReadingData);
+}
+
+void PlatformSensorIio::StopPoll() {
+ DCHECK(polling_thread_task_runner_->BelongsToCurrentThread());
+ timer_->Stop();
+}
+
+void PlatformSensorIio::PollForReadingData() {
+ DCHECK(polling_thread_task_runner_->BelongsToCurrentThread());
+
+ SensorReading reading;
+ if (!sensor_reader_->ReadSensorReading(&reading)) {
+ task_runner_->PostTask(
+ FROM_HERE, base::Bind(&PlatformSensorIio::NotifySensorError, this));
+ StopPoll();
+ return;
+ }
+
+ bool notifyNeeded = false;
+ if (GetReportingMode() == mojom::ReportingMode::ON_CHANGE) {
+ if (!HaveValuesChanged(reading, old_values_))
+ return;
+ notifyNeeded = true;
+ }
+
+ old_values_ = reading;
+ reading.timestamp = (base::TimeTicks::Now() - base::TimeTicks()).InSecondsF();
+ UpdateSensorReading(reading, notifyNeeded);
+}
+
+} // namespace device

Powered by Google App Engine
This is Rietveld 408576698