Correct timestamps read from the timestamp iio channel

These use the realtime clock, whereas Android expects the
system clock to be used as reference.

Change-Id: I90cb68042a7519570bb97995920a15c03f136fb4
Signed-off-by: Patrick Porlan <patrick.porlan@intel.com>

Be more careful with the amount of data we read from the iio dev fd

The iio drivers we have operate with a buffer length set at 2, and
can be set at larger sizes, however the current event integration
code path only handles a iio device report per loop.

For now, simply make sure we only read a single "scan", a.k.a. iio
device report, per loop. This could avoid dropping events if the
driver is producing them faster than we can consume them.

Change-Id: Ibfad7abbd445d6aae0345e2c828ca72c555c42ea
Signed-off-by: Patrick Porlan <patrick.porlan@intel.com>

Moping the floor: hamonize comments delimiters

// is fine, and I like it, but I like consistency even more

Change-Id: I906c5adc00f4ebe1cbe46e69248a3c80a0aa9403
Signed-off-by: Patrick Porlan <patrick.porlan@intel.com>

Moping the floor: remove superfluous f type specifiers

... as well as whitespaces, parentheses, etc.

Change-Id: I9f511420ae9dd783955e0f73d51e384712f98075
Signed-off-by: Patrick Porlan <patrick.porlan@intel.com>

Shift gyro drop count from common.h to filtering.h

Also, rename it with a more specific name.

Change-Id: Id87ad267b25949699cc29dcdd4cac28a5d572fbb
Signed-off-by: Patrick Porlan <patrick.porlan@intel.com>

Merge remote-tracking branch 'origin/abt/topic/gmin/l-dev/sensors/master' into gmin/l-dev/master

git log --pretty=format:"%h %ad | %s%d [%an]" m/abt/topic/gmin/l-dev/master..origin/abt/topic/gmin/l-dev/sensors/master:
  222bad1 Thu Nov 20 15:30:26 2014 +0200 | Move defines to appropriate c files (origin/abt/topic/gmin/l-dev/sensors/master) [Adriana Reus]
  ca985fc Thu Nov 20 14:52:50 2014 +0200 | Don't setup an upper limit for the magnetometer sanity_norm [Adriana Reus]
  1fdbe72 Wed Nov 19 14:28:11 2014 +0100 | Warning removal round [Patrick Porlan]

git diff --stat m/abt/topic/gmin/l-dev/master..HEAD:
  calibration.h         | 24 ++++--------------------
  compass-calibration.c | 51 +++++++++++++++++++++++++++++----------------------
  description.c         |  2 +-
  entry.c               | 25 ++++++++++++++++---------
  enumeration.c         |  4 ++--
  gyro-calibration.c    |  5 +++++
  transform.c           |  2 +-
  7 files changed, 58 insertions(+), 55 deletions(-)

Tracked-On: https://jira01.devtools.intel.com/browse/GMINL-4280
Signed-off-by: Adriana Reus <adriana.reus@intel.com>
Change-Id: Ifbca09ec6aa18eb8d45c8b41fb4fa1cd1cd36f2f

Use the driver provided timestamp channel if possible

There are several conditions though:
- a suitable iio channel is detected
- it can be enabled
- the scan report are of the expected size
- timestamps aren't nil

Change-Id: I7ee5aebd952225dacab170b9349640def4ddc892
Signed-off-by: Patrick Porlan <patrick.porlan@intel.com>

Magnetometer ts improvements

Attempt to better estimate timestamps for polling sensors.
Seems to slightly improve jitter on magnetometer, although
not in a very significant manner.

Tracked-On: https://jira01.devtools.intel.com/browse/GMINL-4216
Change-Id: I29bf0f4c100f3072b0cca9f1564518c37939ce20
Signed-off-by: Adriana Reus <adriana.reus@intel.com>
Reviewed-on: https://android.intel.com:443/302236

Populate quirks at enumeration

Currently quirks are populated when Android asks for activation.
This can be troublesome if we're enabling a virtual
sensor before the base sensor.

Change-Id: Ib988559a0e5e44767187380b928e7862adc40d4a
Signed-off-by: Adriana Reus <adriana.reus@intel.com>
Reviewed-on: https://android.intel.com:443/303492

Move defines to appropriate c files

Some clean-up

Change-Id: I5d04f329d0ee965ba6e9c943baf6682209922060
Signed-off-by: Adriana Reus <adriana.reus@intel.com>

Don't setup an upper limit for the magnetometer sanity_norm

CDD requires a minimum capability of 900 mT and this limits it.

Change-Id: I81596da46a944c4c99cd92498b72b2ebe914f19a
Signed-off-by: Adriana Reus <adriana.reus@intel.com>

Warning removal round

- mark unused parameters as such if we can't delete them
- delete truly unused parameters (ts in magneto calibration)

Change-Id: I9e1a17c962fb06f2ea08dc709ea4d295e4aaf70d
Signed-off-by: Patrick Porlan <patrick.porlan@intel.com>
Reviewed-on: https://android.intel.com:443/302995

Merge remote-tracking branch 'origin/abt/topic/gmin/l-dev/sensors/master' into gmin/l-dev/master

git log --pretty=format:"%h %ad | %s%d [%an]" m/abt/topic/gmin/l-dev/master..origin/abt/topic/gmin/l-dev/sensors/master:
  a60517d Mon Nov 17 12:41:39 2014 +0200 | Fix for broken indexes in compass calibration (origin/abt/topic/gmin/l-dev/sensors/master) [Adriana Reus]

git diff --stat m/abt/topic/gmin/l-dev/master..HEAD:
  compass-calibration.c | 10 +++++-----
  1 file changed, 5 insertions(+), 5 deletions(-)

Tracked-On: https://jira01.devtools.intel.com/browse/GMINL-4125
Change-Id: Iacd7781f98f6bb6c4944b1edb8b7ec7ef8ebdc6a
Signed-off-by: Adriana Reus <adriana.reus@intel.com>

Fix for broken indexes in compass calibration

My previous patch (Validate old cal_level) introduced an error
in the calibration level logic.

Tracked-On: https://jira01.devtools.intel.com/browse/GMINL-4108
Change-Id: I159cc1d5330147333eb25501525773db78b57573
Signed-off-by: Adriana Reus <adriana.reus@intel.com>

Merge remote-tracking branch 'origin/abt/topic/gmin/l-dev/sensors/master' into gmin/l-dev/master

git log --pretty=format:"%h %ad | %s%d [%an]" m/abt/topic/gmin/l-dev/master..origin/abt/topic/gmin/l-dev/sensors/master:
  bd8906e Fri Nov 14 16:31:49 2014 +0200 | Validate old cal_level (origin/abt/topic/gmin/l-dev/sensors/master) [Adriana Reus]
  41fef7b Fri Nov 14 16:33:42 2014 +0100 | GMINL-3234: Reduce magneto and gyro filter windows [Patrick Porlan]
  11fddd8 Fri Nov 14 15:43:44 2014 +0100 | GMINL-3234: Refactor a little bit our filtering code [Patrick Porlan]
  33fd15a Fri Nov 14 14:36:54 2014 +0100 | GMINL-3234: Reorganize filters a litle [Patrick Porlan]

git diff --stat m/abt/topic/gmin/l-dev/master..HEAD:
  compass-calibration.c |  16 +++--
  control.c             |  17 ++---
  enumeration.c         |   7 +-
  filtering.c           | 173 +++++++++++++++++++++++++++++++++++++++++++-------
  filtering.h           |  10 ++-
  transform.c           |  82 ++----------------------
  6 files changed, 176 insertions(+), 129 deletions(-)

Tracked-On: https://jira01.devtools.intel.com/browse/GMINL-4053
Change-Id: Ic521c3932544e8e1b853570c9b6f226618f73172

Validate old cal_level

In case we play with the cal level setting for a platform
it's best to check that the old one is not in fact outdated.
For example we lowered it for the ecs so the check is to make
sure this takes effect in case of an OTA update for example.

Change-Id: I78cba3722edd15ca23dc5f8c75bd8f0eef74d50e
Signed-off-by: Adriana Reus <adriana.reus@intel.com>

GMINL-3234: Reduce magneto and gyro filter windows

Use 20 samples for the magnetometer moving average, rather than
the previous 30. I believe that reduces latency that the camera
is sensitive to.

Also reduce the median filter set to 5 samples, rather than 7,
in the goal of increasing the "freshness" of gyro readings, with
an increase of noise level though.

At some point in the future we should make these per-sensor params.

Change-Id: I772c1fb720b5a1ebd9ed3d376bb038a1d0fed615
Tracked-On: https://jira01.devtools.intel.com/browse/GMINL-3234
Signed-off-by: Patrick Porlan <patrick.porlan@intel.com>

GMINL-3234: Refactor a little bit our filtering code

filtering.h has a simpler interface now, and filtering structures
are now uniformly allocated/cleaned on enable/disable, for sensors
marked as noisy.

This paves the way for per-sensor filter type and parameters.

Change-Id: I1e867570e79c3a56bac0734307a023c1101bb826
Tracked-On: https://jira01.devtools.intel.com/browse/GMINL-3234
Signed-off-by: Patrick Porlan <patrick.porlan@intel.com>

GMINL-3234: Reorganize filters a litle

The moving average filter has a new home: was transform.c, now
filtering.c, with a new name also.

Change-Id: I9670dc687f883bba205ee2525ff364d628d54828
Tracked-On: https://jira01.devtools.intel.com/browse/GMINL-3234
Signed-off-by: Patrick Porlan <patrick.porlan@intel.com>

Merge remote-tracking branch 'origin/abt/topic/gmin/l-dev/sensors/master' into gmin/l-dev/master

git log --pretty=format:"%h %ad | %s%d [%an]" m/abt/topic/gmin/l-dev/master..origin/abt/topic/gmin/l-dev/sensors/master:
  55a8356 Wed Nov 12 17:53:07 2014 +0100 | GMINL-3234: Temporarily remove uncalibrated gyroscope (origin/abt/topic/gmin/l-dev/sensors/master) [Patrick Porlan]
  89dc581 Wed Nov 12 15:02:07 2014 +0100 | GMINL-3234: Allow for a minimal sensor sampling rate [Patrick Porlan]
  e66ecb2 Tue Nov 11 18:21:14 2014 +0200 | Compass calibration minor fixes [Adriana Reus]
  f2f754d Wed Nov 12 14:56:28 2014 +0100 | GMINL-3234: Tweak magneto filter window size [Patrick Porlan]
  0813206 Tue Nov 11 12:37:29 2014 +0200 | Configurable calibration steps [Adriana Reus]
  fdb0f76 Mon Nov 10 19:56:36 2014 +0200 | Don't let the timestamp get too far behind [Adriana Reus]

git diff --stat m/abt/topic/gmin/l-dev/master..HEAD:
  common.h              |  3 +++
  compass-calibration.c | 16 ++++++++++++++--
  control.c             |  4 ++++
  description.c         | 36 +++++++++++++++++++++++++++++++++++-
  description.h         |  1 +
  enumeration.c         | 10 ++++++++--
  transform.c           |  2 +-
  7 files changed, 66 insertions(+), 6 deletions(-)

Tracked-On: https://jira01.devtools.intel.com/browse/GMINL-4014
Change-Id: Ib0eb72773199038832c11be6024e7c0c1e3ee09c
Signed-off-by: Adriana Reus <adriana.reus@intel.com>

GMINL-3234: Temporarily remove uncalibrated gyroscope

The camera app seem to be opening both gyros during panorama
creation, and I suspect that this is causing problems, even
though I can't confirm it, and I don't understand why. As
this sensor type is simply a nice-to-have, I prefer disabling
it pending further investigation.

Change-Id: I01b6324fb8c0e3d01c1dbc71096bee17e766c333
Tracked-On: https://jira01.devtools.intel.com/browse/GMINL-3234
Signed-off-by: Patrick Porlan <patrick.porlan@intel.com>

GMINL-3234: Allow for a minimal sensor sampling rate

The magnetometer is used at 5 Hz by default, which makes calibration
and filtering operate less than optimally. Allow for a minimum sampling
rate specifier, though the min_freq optional parameter.

Change-Id: I7fbb894c72f72cca17d2609454e70f5f8c7874e7
Signed-off-by: Patrick Porlan <patrick.porlan@intel.com>

Compass calibration minor fixes

Change-Id: Iccc77d87a160f6a48eb9c74c18ccfd0d1a2b2368
Signed-off-by: Adriana Reus <adriana.reus@intel.com>

GMINL-3234: Tweak magneto filter window size

100 events is way too much, and besides, this value is being reduced
to one second worth of data samples anyway. That should reduce CPU
load and latency a little bit on fast magnetometers. On the ECS
Cruise 7 it shouldn't change much as the BMC150 operates at 30 Hz max.

Change-Id: I078d071dd7c5fb0b06e0fd01739679cff0ac8bef
Tracked-On: https://jira01.devtools.intel.com/browse/GMINL-3234
Signed-off-by: Patrick Porlan <patrick.porlan@intel.com>

Configurable calibration steps

Tracked-On: https://jira01.devtools.intel.com/browse/GMINL-3783
Change-Id: I90bb09187c5d6fd8002f65c92612c381b4ea4bf5
Signed-off-by: Adriana Reus <adriana.reus@intel.com>

Don't let the timestamp get too far behind

Tracked-On: https://jira01.devtools.intel.com/browse/GMINL-3896
Change-Id: I151d9cc964524998fb4fa5dc388395e5e1857eb7
Signed-off-by: Adriana Reus <adriana.reus@intel.com>

Merge remote-tracking branch 'origin/abt/topic/gmin/l-dev/sensors/master' into gmin/l-dev/master

git log --pretty=format:"%h %ad | %s%d [%an]" m/abt/topic/gmin/l-dev/master..origin/abt/topic/gmin/l-dev/sensors/master:
  467a854 Fri Nov 7 14:30:05 2014 +0200 | Change enable_count with flag (origin/abt/topic/gmin/l-dev/sensors/master) [Adriana Reus]
  40a6350 Mon Nov 3 16:04:30 2014 +0200 | Magneto calibration tweaks [Adriana Reus]
  8e4a72f Thu Oct 30 16:27:00 2014 +0100 | GMINL-3234: Clamp very low gyro reading to zero [Patrick Porlan]

git diff --stat m/abt/topic/gmin/l-dev/master..HEAD:
  calibration.h         |  1 -
  common.h              |  2 +-
  compass-calibration.c | 10 ++++----
  control.c             | 37 ++++++++++++++---------------
  gyro-calibration.c    |  6 ++---
  transform.c           | 65 ++++++++++++++++++++++++++++++++++++++++++++++++++-
  6 files changed, 91 insertions(+), 30 deletions(-)

Tracked-On: https://jira01.devtools.intel.com/browse/GMINL-3832
Change-Id: I0b48e2d201d1cb27992cbf5897468abae6d9065d
Signed-off-by: Adriana Reus <adriana.reus@intel.com>

Change enable_count with flag

Tracked-On: https://jira01.devtools.intel.com/browse/GMINL-3725
Change-Id: I50eae55b9128ce4e3c647dbed633fea933b522dd
Signed-off-by: Adriana Reus <adriana.reus@intel.com>

Magneto calibration tweaks

Tracked-On: https://jira01.devtools.intel.com/browse/GMINL-3514
Change-Id: I7c1f61310218551ce84fd0a497f57b850490d4a9
Signed-off-by: Adriana Reus <adriana.reus@intel.com>

GMINL-3234: Clamp very low gyro reading to zero

This avoids accumulating error over long periods of time.
Incidently, be more cautious with the gyro status field,
as it's not valid for the uncal gyro.

Tracked-On: https://jira01.devtools.intel.com/browse/GMINL-3234
Change-Id: Ic4fa964bb31c3e223eabf23382be27f51e3df529
Signed-off-by: Patrick Porlan <patrick.porlan@intel.com>

Merge remote-tracking branch 'origin/abt/topic/gmin/l-dev/sensors/master' into gmin/l-dev/master

git log --pretty=format:"%h %ad | %s%d [%an]" m/abt/topic/gmin/l-dev/master..origin/abt/topic/gmin/l-dev/sensors/master:
  f883d43 Mon Oct 27 14:36:06 2014 +0200 | Provision the preferred scale (origin/abt/topic/gmin/l-dev/sensors/master) [Viorel Suman]
  9b29687 Thu Oct 30 12:56:39 2014 +0200 | Handle max_freq parameter [Adriana Reus]
  64c5e96 Tue Oct 28 12:48:34 2014 +0200 | Reset gyro filtering for each enabling. [Adriana Reus]
  537fd28 Mon Oct 27 19:04:40 2014 +0200 | Fix initialization points for event_count and meta_data_pending [Adriana Reus]
  94c3ac8 Tue Oct 28 12:22:01 2014 +0200 | Fix bug in gyro filtering call. [Adriana Reus]

git diff --stat m/abt/topic/gmin/l-dev/master..HEAD:
  common.h      |  1 -
  control.c     |  2 ++
  description.c | 11 ++++++++++-
  enumeration.c | 24 ++++++++++++++++++++----
  filtering.c   | 14 ++++++++++++++
  filtering.h   |  4 ++--
  6 files changed, 48 insertions(+), 8 deletions(-)

Tracked-On: https://jira01.devtools.intel.com/browse/GMINL-3440
Change-Id: I944e1a752ca2a93f6ce3728246aaf317729bac02
Signed-off-by: Adriana Reus <adriana.reus@intel.com>

Provision the preferred scale

The patch enables the provisioning of the preferred
scale for a specific sensor.

Tracked-On: https://jira01.devtools.intel.com/browse/GMINL-3156
Change-Id: I6884ea03a98621e61c39ce956e9b19624e8682f0
Signed-off-by: Viorel Suman <viorel.suman@intel.com>
Reviewed-on: https://android.intel.com:443/293561

Handle max_freq parameter

Tracked-On: https://jira01.devtools.intel.com/browse/GMINL-1535
Change-Id: I5aea4c040e2da587aff7bed5b08df26ea0950bda
Signed-off-by: Adriana Reus <adriana.reus@intel.com>
Reviewed-on: https://android.intel.com:443/294928

Reset gyro filtering for each enabling.

Issue: GMINL-2659
Change-Id: I1d7703cb384c8ffb40539955a91fd1d3c17a074e
Signed-off-by: Adriana Reus <adriana.reus@intel.com>

Fix initialization points for event_count and meta_data_pending

Issue: GMINL-2659
Change-Id: Iec77dacc2eec496ebe5f63950840de3c3ce46aca
Signed-off-by: Adriana Reus <adriana.reus@intel.com>

Fix bug in gyro filtering call.

Issue: GMINL-2659
Change-Id: I49b96138fa2de50ef1f324205b354769e66a18e6
Signed-off-by: Adriana Reus <adriana.reus@intel.com>

Merge remote-tracking branch 'origin/abt/topic/gmin/l-dev/sensors/master' into gmin/l-dev/master

git log --pretty=format:"%h %ad | %s%d [%an]" m/abt/topic/gmin/l-dev/master..origin/abt/topic/gmin/l-dev/sensors/master:
  f4cb7dc Fri Oct 24 15:17:45 2014 +0200 | GMINL-2659: Keep recent events history for fusion-like processing (origin/abt/topic/gmin/l-dev/sensors/master) [Patrick Porlan]
  7c24cdc Mon Oct 27 13:11:25 2014 +0200 | Scale magnetic events within a sanity limit [Adriana Reus]
  ff21ce0 Thu Oct 23 16:27:42 2014 +0300 | Keep gyro cal between reboots [Adriana Reus]
  910e5cc Wed Oct 22 15:13:51 2014 +0300 | Collect a minimum number of events for filtering for noisy sensors [Adriana Reus]
  fc476f1 Fri Oct 17 15:34:20 2014 +0200 | IRDA-2056: Add type field to sensor_info structure [Patrick Porlan]

git diff --stat m/abt/topic/gmin/l-dev/master..HEAD:
  calibration.h         |  2 ++
  common.h              | 22 ++++++++++++++++++++-
  compass-calibration.c | 45 +++++++++++++++++++++++++------------------
  control.c             | 48 +++++++++++++++++++++-------------------------
  description.c         | 32 ++++++-------------------------
  enumeration.c         | 12 ++++++------
  filtering.c           | 53 +++++++++++++++++++++++++++++++++++++++++++++++++++
  filtering.h           |  2 ++
  gyro-calibration.c    | 41 +++++++++++++++++++++++++++++++++++++--
  transform.c           | 38 ++++++++++++++++++++++--------------
  10 files changed, 201 insertions(+), 94 deletions(-)

Tracked-On: https://jira01.devtools.intel.com/browse/GMINL-3169
Change-Id: I6e0ec40cfd34c7cf3eef2428ee6b33de7f7173c4
Signed-off-by: Adriana Reus <adriana.reus@intel.com>

Merge remote-tracking branch 'origin/abt/topic/gmin/l-dev/sensors/master' into gmin/l-dev/master

git log --pretty=format:"%h %ad | %s%d [%an]" m/abt/topic/gmin/l-dev/master..origin/abt/topic/gmin/l-dev/sensors/master:
  f4cb7dc Fri Oct 24 15:17:45 2014 +0200 | GMINL-2659: Keep recent events history for fusion-like processing (origin/abt/topic/gmin/l-dev/sensors/master) [Patrick Porlan]
  7c24cdc Mon Oct 27 13:11:25 2014 +0200 | Scale magnetic events within a sanity limit [Adriana Reus]
  ff21ce0 Thu Oct 23 16:27:42 2014 +0300 | Keep gyro cal between reboots [Adriana Reus]
  910e5cc Wed Oct 22 15:13:51 2014 +0300 | Collect a minimum number of events for filtering for noisy sensors [Adriana Reus]
  fc476f1 Fri Oct 17 15:34:20 2014 +0200 | IRDA-2056: Add type field to sensor_info structure [Patrick Porlan]

git diff --stat m/abt/topic/gmin/l-dev/master..HEAD:
  calibration.h         |  2 ++
  common.h              | 22 ++++++++++++++++++++-
  compass-calibration.c | 45 +++++++++++++++++++++++++------------------
  control.c             | 48 +++++++++++++++++++++-------------------------
  description.c         | 32 ++++++-------------------------
  enumeration.c         | 12 ++++++------
  filtering.c           | 53 +++++++++++++++++++++++++++++++++++++++++++++++++++
  filtering.h           |  2 ++
  gyro-calibration.c    | 41 +++++++++++++++++++++++++++++++++++++--
  transform.c           | 38 ++++++++++++++++++++++--------------
  10 files changed, 201 insertions(+), 94 deletions(-)

Change-Id: I6e0ec40cfd34c7cf3eef2428ee6b33de7f7173c4

GMINL-2659: Keep recent events history for fusion-like processing

We may wish to correlate recent input from several active sensors.
In preparation to that, keep the last 100 events we got for the
entire sensors collection. This is intended as a lightweight record,
built and used from the main sensor sample retrieval thread exclusively
so we don't have to deal with performance-degrading synchronization
primitives. This data will enable multi-sensors filtering decisions.

Change-Id: Ied6559c7dd91e94bebbeb1bc60182d1db3103d8c
Signed-off-by: Patrick Porlan <patrick.porlan@intel.com>

Scale magnetic events within a sanity limit

Scale magnetic events even if we don't have a calibration.
If the values are considered "wrong" by Android it does not compute
values for the virtual sensors until we have a calibration and
sane looking values in place

Change-Id: I94347d98f541c85d9734ce347a070abca351de2c
Signed-off-by: Adriana Reus <adriana.reus@intel.com>

Keep gyro cal between reboots

Issue: GMINL-2743
Change-Id: I232e979d62d12f31168c0bfdcfcd3b224d5237b3
Signed-off-by: Adriana Reus <adriana.reus@intel.com>
Reviewed-on: https://android.intel.com:443/292457

Collect a minimum number of events for filtering for noisy sensors

This is to make sure we have at least a couple of events for
the filtering queue and to make sure we are not sensing events
that can affect the overall mean or standard deviation.

Change-Id: If359915e7281a553758eb79ec10ea0559f9490f8
Signed-off-by: Adriana Reus <adriana.reus@intel.com>
Reviewed-on: https://android.intel.com:443/291953

IRDA-2056: Add type field to sensor_info structure

We were getting this information through the associated catalog
entry but it's considerably more readable this way.

Change-Id: I7b4489738e755a2831930c0a1a335a2fb9e731cc
Signed-off-by: Patrick Porlan <patrick.porlan@intel.com>

Merge remote-tracking branch 'origin/abt/topic/gmin/l-dev/sensors/master' into gmin/l-dev/master

git log --pretty=format:"%h %ad | %s%d [%an]" m/abt/topic/gmin/l-dev/master..origin/abt/topic/gmin/l-dev/sensors/master:
  5b9cb96 Thu Oct 16 10:33:57 2014 +0200 | IRDA-2056: Header file cleanup (origin/abt/topic/gmin/l-dev/sensors/master) [Patrick Porlan]
  af41894 Wed Oct 15 14:30:40 2014 +0200 | IRDA-2056: Rename continuous quirk flag and fix the location where it's used [Patrick Porlan]
  7755b1c Wed Oct 15 14:29:25 2014 +0200 | IRDA-2056: Cosmetic whitespace shuffling [Patrick Porlan]
  138a7ad Wed Oct 15 14:27:10 2014 +0200 | IRDA-2056: Link the terse flag to the variable we're now using to decide whether or not to operate in "driver only sending events on motion" mode. [Patrick Porlan]
  4b21c9f Wed Oct 15 14:22:02 2014 +0200 | IRDA-2056: Remove a superfluous check [Patrick Porlan]

git diff --stat m/abt/topic/gmin/l-dev/master..HEAD:
  control.c     |  4 ++--
  description.c |  7 +++----
  description.h | 35 +++++++++++++++--------------------
  enumeration.c | 59 ++++++++++++++++++++++-------------------------------------
  4 files changed, 42 insertions(+), 63 deletions(-)

Tracked-On:https://jira01.devtools.intel.com/browse/GMINL-2821
Change-Id: I36bcd5eee3af84edc2c311a72f032bb14a3546b7
Signed-off-by: Adriana Reus <adriana.reus@intel.com>

IRDA-2056: Header file cleanup

Order and symetry reduce the reader's cognitive load...

Change-Id: Ifa47d3a109898a16565eac7315bc35711bda4b9c
Signed-off-by: Patrick Porlan <patrick.porlan@intel.com>
Reviewed-on: https://android.intel.com:443/289221

IRDA-2056: Rename continuous quirk flag and fix the location where it's used

This flag is used to force use of the continuous trigger, typically
in cases where the motion trigger would otherwise be autoselected.

Also, there are cases, such as with the Invensense driver that we
have on the T100, where a single trigger will control several
sensors because they're associated to the same iio device. In that
case having a single sensor forced to continous mode will imply that
no sensor co-located on the same iio device can be in "only send
events whenever there's motion" mode.

Change-Id: I4d01d2d964b70dead3e67c54bdd92d7d938cbf23
Signed-off-by: Patrick Porlan <patrick.porlan@intel.com>
Reviewed-on: https://android.intel.com:443/289220

IRDA-2056: Cosmetic whitespace shuffling

Change-Id: Ie6aecf3f7f595e4c8bd4ca9cd15b1562b81dfe92
Signed-off-by: Patrick Porlan <patrick.porlan@intel.com>
Reviewed-on: https://android.intel.com:443/289219

IRDA-2056: Link the terse flag to the variable we're now using to decide
whether or not to operate in "driver only sending events on
motion" mode.

Change-Id: I9600e4d8611715b2a55a523613af1dec368a8893
Signed-off-by: Patrick Porlan <patrick.porlan@intel.com>
Reviewed-on: https://android.intel.com:443/289218

IRDA-2056: Remove a superfluous check

We don't really have to check whether or not a sensor type is
defined as continuous by Android when we encounter a motion
trigger for the associated sensor. Simply record the name, and
we'll decide later if, and when, it has to be selected.

Change-Id: Ibc77470db1d66cc67f5e799f0db454e3157f3945
Signed-off-by: Patrick Porlan <patrick.porlan@intel.com>
Reviewed-on: https://android.intel.com:443/289217

Merge remote-tracking branch 'origin/abt/topic/gmin/l-dev/sensors/master' into gmin/l-dev/master

git log --pretty=format:"%h %ad | %s%d [%an]" m/abt/topic/gmin/l-dev/master..origin/abt/topic/gmin/l-dev/sensors/master:
  8f3c496 Mon Oct 20 20:12:36 2014 +0300 | Synchronize our timestamps with the SystemClock.getRealtimeNanos (origin/abt/topic/gmin/l-dev/sensors/master) [Adriana Reus]
  60c1864 Wed Oct 15 14:13:13 2014 +0200 | IRDA-2056: Minor rework to the trigger switching code [Patrick Porlan]
  58f7faf Wed Oct 15 14:11:32 2014 +0200 | IRDA-2056: Minor cosmetic tweaks [Patrick Porlan]
  dd4d8ca Wed Oct 15 14:03:24 2014 +0200 | IRDA-2056: Refine the test we use to decide to apply gyro calibration [Patrick Porlan]
  db7e7fb Wed Oct 15 11:32:36 2014 +0200 | IRDA-2056: Fix accuracy indicator for gyroscope readings. [Patrick Porlan]
  5caa616 Fri Oct 17 10:01:29 2014 +0300 | Include acceptable threshold on TS [Viorel Suman]

git diff --stat m/abt/topic/gmin/l-dev/master..HEAD:
  common.h           |  3 +++
  control.c          | 61 ++++++++++++++++++++++++++++++++++++++----------------
  gyro-calibration.c |  2 --
  transform.c        | 19 +++++++++++++++--
  utils.c            | 42 ++++++++++++++++++++++++++++++++-----
  utils.h            |  7 ++++---
  6 files changed, 104 insertions(+), 30 deletions(-)
~

Tracked-On:https://jira01.devtools.intel.com/browse/GMINL-2673
Change-Id: Ice355c5abc14344b224b5a933cb1a9519ec305a0
Signed-off-by: Adriana Reus <adriana.reus@intel.com>

Synchronize our timestamps with the SystemClock.getRealtimeNanos

It seems we get unsynced after a suspend/resume cycle.
Sync the clocks once per sensor enabling.
CDD required us to be in sync anyway.

Tracked-On: https://jira01.devtools.intel.com/browse/GMINL-1535
Change-Id: I2422551837832575492ee2d0c1a7adb1163b3167
Signed-off-by: Adriana Reus <adriana.reus@intel.com>

IRDA-2056: Minor rework to the trigger switching code

Isolate the test that we use to switch from the motion trigger to
the continous one (for accelerometer based games that are sensitive
to small motion thresholds) into a function. Also, specifically
deactivate this behaviour for drivers marked as terse: for those it
may make sense to initialize the motion trigger name field with the
same string as the initial trigger name field, so let's check for
the terse flag.

Change-Id: Ia666caed8f04345aca7c09d6a686107590ffc590
Signed-off-by: Patrick Porlan <patrick.porlan@intel.com>

IRDA-2056: Minor cosmetic tweaks

Change-Id: I6032b66272dd6056b192ba52a6d2f573eca4d779
Signed-off-by: Patrick Porlan <patrick.porlan@intel.com>

IRDA-2056: Refine the test we use to decide to apply gyro calibration

The terse flag should only be used with drivers that a) don't
expose a recognizable motion trigger (we currently look for the
any-motion substring), b) are not for sensor types that Android
defines as continuous, and c) are nonetheless on-change. As of
today the only remaining example of this that I can think of
are the XPS 12 ISH drivers, so I prefer not to use this flag
to decide whether or not to apply calibration. Checking whether
we're using the initial or motion trigger seems better, if not
fully satisfactory.

Change-Id: I24c6632497f758e1301da7927ec94bf17f6fe84d
Signed-off-by: Patrick Porlan <patrick.porlan@intel.com>

IRDA-2056: Fix accuracy indicator for gyroscope readings.

Report medium accuracy rather than the default (unreliable) one
if we're not using calibration.

Change-Id: I648075681eed0a82bc081babe8a0825096bbe4a4

Include acceptable threshold on TS

The patch takes in consideration the acceptable threshold when adjusting
the event timestamp. The accelerometer threshold is enforced to "1" in
order to make happy virtual sensors test cases - virtual sensors events
are clones of accel events (so are the timestamps), see "process" method
in GravitySensor.cpp, OrientationSensor.cpp, RotationVectorSensor.cpp
from frameworks/native/services/sensorservice.

Tracked-On: https://jira01.devtools.intel.com/browse/GMINL-264
Change-Id: Ie9e05949154ed2f4e3bac4f1d405bb41c6a7ddcb
Signed-off-by: Viorel Suman <viorel.suman@intel.com>

Merge remote-tracking branch 'origin/abt/topic/gmin/l-dev/sensors/master' into gmin/l-dev/master

git log --pretty=format:"%h %ad | %s%d [%an]" m/abt/topic/gmin/l-dev/master..origin/abt/topic/gmin/l-dev/sensors/master:
  c5d1ab6 Wed Oct 15 19:42:45 2014 +0300 | Limit max frequency to 400Hz (origin/abt/topic/gmin/l-dev/sensors/master) [Adriana Reus]
  c1b848a Mon Oct 13 09:32:10 2014 +0300 | Fix using s as index to array before check [Viorel Suman]
  d52af6c Mon Aug 11 14:46:19 2014 +0300 | sens: update properties name and value max size [Octavian Purdila]
  e338f33 Tue Jul 29 17:20:00 2014 +0300 | sens: HAL library path autodetect [Octavian Purdila]
  a2b5a4f Tue Jul 29 16:08:57 2014 +0300 | sens: use Android.mk to get the HAL sources for the Linux build [Octavian Purdila]

git diff --stat m/abt/topic/gmin/l-dev/master..HEAD:
  Android.mk                |  4 +---
  Makefile                  | 11 ++++++-----
  common.h                  |  2 +-
  control.c                 |  6 +++---
  linux/cutils/properties.h |  4 ++--
  sens.c                    | 31 +++++++++++++++++++++++++++----
  6 files changed, 40 insertions(+), 18 deletions(-)

Tracked-On: https://jira01.devtools.intel.com/browse/GMINL-2463
Change-Id: I54b6ee99aa16c5cc96b33ab9c5878ce634f3f845
Signed-off-by: Adriana Reus <adriana.reus@intel.com>

Merge remote-tracking branch 'origin/abt/topic/gmin/l-dev/sensors/master' into gmin/l-dev/master

git log --pretty=format:"%h %ad | %s%d [%an]" m/abt/topic/gmin/l-dev/master..origin/abt/topic/gmin/l-dev/sensors/master:
  c5d1ab6 Wed Oct 15 19:42:45 2014 +0300 | Limit max frequency to 400Hz (origin/abt/topic/gmin/l-dev/sensors/master) [Adriana Reus]
  c1b848a Mon Oct 13 09:32:10 2014 +0300 | Fix using s as index to array before check [Viorel Suman]
  d52af6c Mon Aug 11 14:46:19 2014 +0300 | sens: update properties name and value max size [Octavian Purdila]
  e338f33 Tue Jul 29 17:20:00 2014 +0300 | sens: HAL library path autodetect [Octavian Purdila]
  a2b5a4f Tue Jul 29 16:08:57 2014 +0300 | sens: use Android.mk to get the HAL sources for the Linux build [Octavian Purdila]

git diff --stat m/abt/topic/gmin/l-dev/master..HEAD:
  Android.mk                |  4 +---
  Makefile                  | 11 ++++++-----
  common.h                  |  2 +-
  control.c                 |  6 +++---
  linux/cutils/properties.h |  4 ++--
  sens.c                    | 31 +++++++++++++++++++++++++++----
  6 files changed, 40 insertions(+), 18 deletions(-)

Change-Id: I54b6ee99aa16c5cc96b33ab9c5878ce634f3f845

Limit max frequency to 400Hz

It seems that having the accel and the gyro work at very high
frequencies (> 500) at the same time does not go very well with the
overall cpu loading and can affect other threads, for example the magnetometer
events don't come as fast as cts would like it given these conditions.

Tracked-On: https://jira01.devtools.intel.com/browse/GMINL-2227
Change-Id: I69480b393794afb072654451a468c30bb821466a
Signed-off-by: Adriana Reus <adriana.reus@intel.com>

Fix using s as index to array before check

The patch fixes the using of s as index to array
sensor_info before check.

Issue: GMINL-2312
Change-Id: I17f474526c9e48bc598e067106738d50b87398a5
Signed-off-by: Viorel Suman <viorel.suman@intel.com>

sens: update properties name and value max size

Update the Linux properties name and value max size to match those in
Android.

Change-Id: I9ac24242b1056a5e65f921a31d9ec583d166bae2
Signed-off-by: Octavian Purdila <octavian.purdila@intel.com>

sens: HAL library path autodetect

Change-Id: Ic32e932ad5f179f7cba710c21e07e6cb0c1d3623
Signed-off-by: Octavian Purdila <octavian.purdila@intel.com>

sens: use Android.mk to get the HAL sources for the Linux build

Also, use the same name for the Linux build HAL (sensors.gmin.so).

Change-Id: I3290c6347a3746634d721367eb2b6b42d92e759e
Signed-off-by: Octavian Purdila <octavian.purdila@intel.com>

Merge remote-tracking branch 'origin/abt/topic/gmin/l-dev/sensors/master' into gmin/l-dev/master

git log --pretty=format:"%h %ad | %s%d [%an]" m/abt/topic/gmin/l-dev/master..origin/abt/topic/gmin/l-dev/sensors/master:
  8fb9851 Tue Oct 7 16:01:48 2014 +0300 | Retry when sysfs_write_str fails (origin/abt/topic/gmin/l-dev/sensors/master) [Viorel Suman]
  d679bc6 Tue Sep 30 17:07:53 2014 +0300 | Acquisition routine review [Viorel Suman]
  b1efe3a Fri Oct 3 15:12:44 2014 +0300 | Some tweaks to the median filter [Adriana Reus]
  d1b25b5 Mon Oct 6 16:17:16 2014 +0300 | Fixing some compiler warnings [Adriana Reus]
  344447a Fri Sep 26 12:15:15 2014 +0300 | Upgrade HAL version to 1_3 [Adriana Reus]
  2e4e1a2 Thu Sep 25 14:42:49 2014 +0300 | Implement batch and flush functions [Adriana Reus]
  b24d658 Fri Oct 3 10:24:36 2014 +0300 | Min/Max delay for Light and Temperature [Viorel Suman]

git diff --stat m/abt/topic/gmin/l-dev/master..HEAD:
  common.h      |   6 +++
  control.c     | 124 ++++++++++++++++++++++++++++++++++++----------------------
  control.h     |   3 +-
  description.c | 113 +++++++++++++++++++++++++++++++++++-----------------
  description.h |   2 +
  entry.c       |  19 +++++++--
  enumeration.c |  28 ++++---------
  filtering.c   |  69 +++++++++++++++++++-------------
  filtering.h   |  26 +++---------
  transform.c   |   2 +-
  utils.c       |   2 +-
  utils.h       |   1 +
  12 files changed, 239 insertions(+), 156 deletions(-)

Change-Id: Ic4fb49db812d123bc903e9106f5a2b7c220ef490
Signed-off-by: Adriana Reus <adriana.reus@intel.com>

Retry when sysfs_write_str fails

The sysfs_write_str might fail if called
often on the same file. We retry a few times
in that case.

Change-Id: I42cdc2484728ab837a35b54d01ed87abd6bdcb0c
Signed-off-by: Viorel Suman <viorel.suman@intel.com>

Acquisition routine review

The OVERHEAD_THRESHOLD is a quite subjective
platform specific constant, so the current patch
tries to adjust in a more elegant way the target_time
passed to pthread_cond_timedwait.

Issue: GMINL-1535
Change-Id: Ic4156b1cf5e712eee9ca7e1cc597b0b3fef3432e
Signed-off-by: Viorel Suman <viorel.suman@intel.com>

Some tweaks to the median filter

Made it field number agnostic, also window size agnostic.

Change-Id: I3ac5afaeec4cb8e7be6e0bccbb6dd34324316592
Signed-off-by: Adriana Reus <adriana.reus@intel.com>

Fixing some compiler warnings

vendor/intel/hardware/iio-sensors/./entry.c:29:2: warning: implicit declaration of function 'sensor_get_quirks' [-Wimplicit-function-declaration]

vendor/intel/hardware/iio-sensors/./enumeration.c:338:2: warning: format '%ld' expects argument of type 'long int', but argument 7 has type 'int32_t' [-Wformat=]

vendor/intel/hardware/iio-sensors/./description.c:443:2: warning: implicit declaration of function 'sysfs_read_str' [-Wimplicit-function-declaration]

vendor/intel/hardware/iio-sensors/./description.c:460:4: warning: implicit declaration of function 'isspace' [-Wimplicit-function-declaration]

Change-Id: I6baa6ead295e24d18bf8af5c8db3293d565738d8
Signed-off-by: Adriana Reus <adriana.reus@intel.com>

Upgrade HAL version to 1_3

Signed-off-by: Adriana Reus <adriana.reus@intel.com>
Change-Id: Iecf94131c3e6cd62b6eb3d4027c08ec16a210632

Implement batch and flush functions

We need these so that we can upgrade the HAL.
Even if we don't have batch yet, the 1_3 API is requiring them.

Change-Id: I47da935b38bf1b051e3b5a18951d8f4724a82e31
Signed-off-by: Adriana Reus <adriana.reus@intel.com>

Min/Max delay for Light and Temperature

The patch reviews the get_min_delay and
get_max_delay methods, and sets the min delay
for polling light and temperature sensors.

Change-Id: I75303c672c7ccc5ec5fe4d7f879ed934cda1f55a
Signed-off-by: Viorel Suman <viorel.suman@intel.com>

Merge remote-tracking branch 'origin/abt/topic/gmin/l-dev/sensors/master' into gmin/l-dev/master

git log --pretty=format:"%h %ad | %s%d [%an]" m/abt/topic/gmin/l-dev/master..origin/abt/topic/gmin/l-dev/sensors/master:
  66e23b6 Tue Sep 30 17:03:23 2014 +0300 | load/set_timestamp functions implementation (origin/abt/topic/gmin/l-dev/sensors/master) [Viorel Suman]
  5426187 Mon Sep 29 17:59:12 2014 +0200 | IRDA-1184: Fix event duplication condition [Patrick Porlan]
  91169af Mon Sep 29 17:26:00 2014 +0200 | IRDA-1184: Fix poll time calculation routine [Patrick Porlan]
  2afe869 Tue Sep 23 16:23:10 2014 +0300 | Moving average optimization [Viorel Suman]
  6cd353a Wed Oct 1 12:08:58 2014 +0300 | Minor clean-up [Adriana Reus]
  c8bc431 Wed Oct 1 11:42:12 2014 +0300 | Don't setup the timestamp artificially for non-continous sensors [Adriana Reus]
  58dfca4 Thu Sep 25 15:37:21 2014 +0300 | Clamp frequency to smallest supported rate if requested lower [Adriana Reus]
  77fd2f6 Wed Sep 24 16:43:08 2014 +0200 | IRDA-1184: Alternate between triggers depending on sampling rate [Patrick Porlan]
  c230e47 Thu Sep 18 23:22:31 2014 +0200 | control: Retry when buffer enabling fails [Samuel Ortiz]
  8977477 Thu Sep 25 12:12:44 2014 +0300 | Initialize fifoReservedEventCount and fifoMaxEventCount [Adriana Reus]

git diff --stat m/abt/topic/gmin/l-dev/master..HEAD:
  calibration.h         |    8 -----
  common.h              |    1 +
  compass-calibration.c |    6 ++++
  control.c             |   93 +++++++++++++++++++++++++++++++++++++++++--------
  description.h         |    3 ++
  enumeration.c         |    6 ++++
  transform.c           |   41 ++++++++++++++--------
  utils.c               |   15 ++++++--
  utils.h               |    2 ++
  9 files changed, 135 insertions(+), 40 deletions(-)

Change-Id: Idc427a94cdc7645185dba5f57e5d8359d7a46815
Signed-off-by: Adriana Reus <adriana.reus@intel.com>

load/set_timestamp functions implementation

The patch implements the load_timestamp and
set_timestamp functions aimed to provide a
common way to manage the timestamps.

Change-Id: If5d5d86ed9f83298385e7736345f7eee2884418c
Signed-off-by: Viorel Suman <viorel.suman@intel.com>

IRDA-1184: Fix event duplication condition

Now that we can dynamically switch between continous and motion
triggers, checking for the terse flag is not necessary, and is
confusing.

Issue: IRDA-1184

Change-Id: Iaed039550a2d4d30b1cb1aa1c03785d4bcbe6b70
Signed-off-by: Patrick Porlan <patrick.porlan@intel.com>

IRDA-1184: Fix poll time calculation routine

The poll delay should be maximum if we're operating in continuous mode
even if the sensor supports a motion trigger. This should very slightly
lower CPU consumption, avoiding a few unrequired exits from epoll.

Issue: IRDA-1184

Change-Id: I026be3de5fed49b77f7d9ac3718ab7c71e527d9d
Signed-off-by: Patrick Porlan <patrick.porlan@intel.com>

Moving average optimization

The patch optimizes the moving average calculation
by introducing the moving history sum calculated
while the new data is recorded.

Change-Id: I9a8a505c983ecd48c1e160495ea45030dabbf535
Signed-off-by: Viorel Suman <viorel.suman@intel.com>

Minor clean-up

Change-Id: Ib2b81f09c6a640f858b2564cb8232622693f3c4c
Signed-off-by: Adriana Reus <adriana.reus@intel.com>

Don't setup the timestamp artificially for non-continous sensors

Change-Id: I52c2e8777f28ea8a958f3078486ec0dcce4e505d
Signed-off-by: Adriana Reus <adriana.reus@intel.com>

Clamp frequency to smallest supported rate if requested lower

As per HAL spec requirements.
Signed-off-by: Adriana Reus <adriana.reus@intel.com>
Change-Id: I31df5dfef5bf5ba463aeaa295369fea1f19cf96d

IRDA-1184: Alternate between triggers depending on sampling rate

The BMC150 accel driver any-motion trigger is too jerky for certain
games such as the "sniper shooter" one, or "doodle jump". Switch back
to continuous event generation mode if a sampling rate above 25 is
requested, which should keep both the launcher (15 events per second)
and games (60 events per second) happy.

Issue: IRDA-1184

Change-Id: I16000d57e17506aa7e7c0a4f83ced7739e01dd01
Signed-off-by: Patrick Porlan <patrick.porlan@intel.com>

control: Retry when buffer enabling fails

On suspend/resume cycles, enabling the data buffer might fail and thus
would keep the driver in sleep mode.
We retry a few times in that case, giving the driver a few ms to come
back from suspend.

Issue: IRDA-603
Change-Id: Icd781d6574052d1c87a9f0b29b68ea72aa4dbbb2
Signed-off-by: Samuel Ortiz <sameo@linux.intel.com>

Initialize fifoReservedEventCount and fifoMaxEventCount

Set them up to 0 as we are not implementing batch yet,
will be set up appropriately when we will have batch mode.

Change-Id: Ia6d3fc46d12ce453deb150e312afb6ae5043a973
Signed-off-by: Adriana Reus <adriana.reus@intel.com>

Merge remote-tracking branch 'origin/abt/topic/gmin/l-dev/sensors/master' into gmin/l-dev/master

git log --pretty=format:"%h %ad | %s%d [%an]" origin/abt/topic/gmin/l-dev/master..origin/abt/topic/gmin/l-dev/sensors/master:
  ccf4852 Wed Sep 24 14:53:45 2014 +0300 | Adapt polling rate to get a real frequency closer to the advertised one (origin/abt/topic/gmin/l-dev/sensors/master) [Adriana Reus]
  83d771c Wed Sep 24 12:33:11 2014 +0300 | Median filter implementation [Adriana Reus]
  38059a0 Thu Sep 18 16:08:42 2014 +0300 | The event timestamp as function of frequency [Viorel Suman]
  eb6f309 Thu Sep 18 14:36:04 2014 +0300 | Revert "[Workaround] Fix event timestamp recurrency" [Viorel Suman]

git diff --stat origin/abt/topic/gmin/l-dev/master..HEAD:
  Android.mk    |   1 +
  common.h      |   5 ++-
  control.c     |  36 +++++++++++++++-----
  description.c |   6 +++-
  description.h |   1 +
  enumeration.c |  28 +++++++++++++++
  filtering.c   | 108 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
  filtering.h   |  25 ++++++++++++++
  transform.c   |   3 +-
  9 files changed, 202 insertions(+), 11 deletions(-)

Change-Id: I74cefcc04c195180362a9c34f4d57cda9e5151f1
Signed-off-by: Viorel Suman <viorel.suman@intel.com>

Adapt polling rate to get a real frequency closer to the advertised one

When polling try to compensate for the iio overhead in
order to try to get a frequency closer to the advertised one.
It helps for a bunch of cts tests.

Signed-off-by: Adriana Reus <adriana.reus@intel.com>
Change-Id: I9fd6352d9515faff368167f8f2be66869cf75eda

Median filter implementation

Use a median filter for gyroscope. It seems better
suited for the type of noise we are observing and it
also solves cts standard deviation failures

Signed-off-by: Adriana Reus <adriana.reus@intel.com>
Change-Id: Ib53846cfb92912ae66d16e6993e6f97d4a3e3328

The event timestamp as function of frequency

The patch guards the sensor event timestamp
as function of frequency in order to fix the
"EventGapVerification" and "FrequencyVerification"
from the new L-Dessert SingleSensorTests CTS test
case. It also introduces the CONTINUOUS_DRIVER quirk
used to enforce the continuous driver mode.

Issue: GMINL-544
Change-Id: I5b6accd1c7fa84e3eaeeb48c77fe86bc824781b3
Signed-off-by: Viorel Suman <viorel.suman@intel.com>

Revert "[Workaround] Fix event timestamp recurrency"

This reverts commit b1f370d00598b68a52c754495e6455a8af5c2b35.
Change-Id: If9aa055b01ad2326f5c567a7c25f207188b206eb

Merge remote-tracking branch 'origin/abt/topic/gmin/l-dev/sensors/master' into gmin/l-dev/master

git log --pretty=format:"%h %ad | %s%d [%an]" m/abt/topic/gmin/l-dev/master..origin/abt/topic/gmin/l-dev/sensors/master:
  ca5db17 Wed Sep 17 17:26:14 2014 +0300 | Setup some sanity limitations for magnetometer calibration (origin/abt/topic/gmin/l-dev/sensors/master) [Adriana Reus]
  642a2d2 Wed Sep 17 15:38:04 2014 +0300 | Change the uncalibrated vs calibrated order. [Adriana Reus]
  3dc768c Tue Sep 16 14:08:10 2014 +0300 | Don't calibrate gyro if terse driver [Adriana Reus]

git diff --stat m/abt/topic/gmin/l-dev/master..HEAD:
  calibration.h         |    3 +++
  compass-calibration.c |   18 ++++++++++++++++++
  control.c             |   19 +++++++++++--------
  transform.c           |    3 ++-
  4 files changed, 34 insertions(+), 9 deletions(-)

Change-Id: I08d7310a03dcacfc2285496d199873840191d3a2

Setup some sanity limitations for magnetometer calibration

We never considered a certain magnetic field strength
because it varies depending on the location. Added some
sanity checking though because sometimes the calibration
gets a big bias and reduces the data values.

Change-Id: I7020b68f0bbd6ed0ba9bded1dbd3936034d92aa5
Signed-off-by: Adriana Reus <adriana.reus@intel.com>

Change the uncalibrated vs calibrated order.

Send the calibrated event before the corresponding uncalibrated
one for gyro. It pleases Cts Verifier.

Signed-off-by: Adriana Reus <adriana.reus@intel.com>
Change-Id: I0e1be4d38b47ba3d1fcb6ee33ba43c0b99afe31f

Don't calibrate gyro if terse driver

If we get events on a on-change basis it's better not to
attempt a calibration since we may receive the same event
duplicated and may end up with a wrong bias.

Issue: GMINL-1234
Signed-off-by: Adriana Reus <adriana.reus@intel.com>
Change-Id: Ieb99ee794299f3112f3149757a5a2ae0cdfc256e

Merge remote-tracking branch 'origin/abt/topic/gmin/l-dev/sensors/master' into gmin/l-dev/master

git log --pretty=format:"%h %ad | %s%d [%an]" m/abt/topic/gmin/l-dev/master..origin/abt/topic/gmin/l-dev/sensors/master:
  298c194 Fri Aug 29 13:57:07 2014 +0200 | Refine sensor property get function to filter by iio dev name (origin/abt/topic/gmin/l-dev/sensors/master) [Patrick Porlan]
  b1f370d Mon Sep 15 14:06:56 2014 +0300 | [Workaround] Fix event timestamp recurrency [Adriana Reus]
  8b0553e Mon Sep 15 14:05:29 2014 +0300 | Limit max frequency to 1000 Hz [Adriana Reus]
  05135a5 Mon Sep 15 14:03:27 2014 +0300 | Revert "STPK-1429: Always wait for at least 1 ms before repeating samples" [Adriana Reus]

git diff --stat m/abt/topic/gmin/l-dev/master..HEAD:
  common.h      |    1 +
  control.c     |   13 ++++---------
  description.c |   25 ++++++++++++++++++++++++-
  3 files changed, 29 insertions(+), 10 deletions(-)

Change-Id: Ic8e63f31165e0e63e41c32d745cde37dae17da9b

Refine sensor property get function to filter by iio dev name

This enables having multiple sensors of the same type.

ro.iio.anglvel.bmg160.name is picked up instead of
ro.iio.anglvel.name if the iio device name fort he sensor we
are querying is bmg160.

Change-Id: I62995ea293b765585778a2190bff31c6a75c3ef5
Signed-off-by: Patrick Porlan <patrick.porlan@intel.com>

[Workaround] Fix event timestamp recurrency

A lot of new cts test for sensors expect to have
very regular intervals for events or else they will
report event gaps. Getting the timestamp right before
reporting seems to level them.

Issue: GMINL-544
Change-Id: Ie9554ba3ff95a5fa2c5bb29083181bf23776765e
Signed-off-by: Adriana Reus <adriana.reus@intel.com>

Limit max frequency to 1000 Hz

Android sets up 1000 hz for DELAY_FASTEST. This conflicts
with some cts tests for sensors that report higher frequencies
(example bmg160 gyro). Since none of the current sensors actually
report these kind of high values to userspace it also
doesn't make much sense to duplicate events in a higher rate.

Issue: GMINL-544
Change-Id: I2ded62ca7345b0f49c4b8b20ce3022055bacbdd1
Signed-off-by: Adriana Reus <adriana.reus@intel.com>

Revert "STPK-1429: Always wait for at least 1 ms before repeating samples"

This adds an artificial limitation for frequency which makes
some cts tests fail. They expect to receive higher frequencies
(example 1000 hz) but this limits the frequency at aroung 850 hz.
I wasn't able to see any side effect from removing this, so revert
at least for the moment.

This reverts commit 4c189692e67bc68d520d7adf462348ea4fd54900.

Issue: GMINL-544

Merge remote-tracking branch 'origin/abt/topic/gmin/l-dev/sensors/master' into gmin/l-dev/master

git log --pretty=format:"%h %ad | %s%d [%an]" m/abt/topic/gmin/l-dev/master..origin/abt/topic/gmin/l-dev/sensors/master:
  28c665e Mon Aug 18 11:28:26 2014 +0300 | Fix build for x86_64 (origin/abt/topic/gmin/l-dev/sensors/master) [Irina Tirdea]
  4acf533 Thu Sep 4 13:54:01 2014 +0300 | Populate minDelay field. [Adriana Reus]
  61a5b42 Thu Aug 28 19:20:01 2014 +0200 | GMIN-2992: Prefix uncalibrated gyro name with Uncalibrated [Patrick Porlan]
  7d7c4b0 Wed Aug 27 18:12:10 2014 +0200 | GMIN-3015: Initialize accuracy flag on accel and gyro samples [Patrick Porlan]
  25e58ad Wed Aug 27 18:36:43 2014 +0200 | GMIN-3015: Refine a little bit the magnetometer accuracy indicator [Patrick Porlan]

git diff --stat m/abt/topic/gmin/l-dev/master..HEAD:
  compass-calibration.c |   31 +++++++++++++++++++++----
  control.c             |   15 +++++++-----
  description.c         |   62 +++++++++++++++++++++++++++++++++++++++++++++++++
  enumeration.c         |    8 +++++++
  gyro-calibration.c    |    5 ++++
  transform.c           |    2 ++
  6 files changed, 112 insertions(+), 11 deletions(-)

Change-Id: I2c9474899c85908c2c69eab675bf90b751cdca3a

Fix build for x86_64

target  C: sensors.gmin <= vendor/intel/hardware/iio-sensors/./control.c
vendor/intel/hardware/iio-sensors/./control.c: In function 'acquisition_routine':
vendor/intel/hardware/iio-sensors/./control.c:337:10: error: cast from pointer
to integer of different size [-Werror=pointer-to-int-cast]
  int s = (int) param;
          ^
vendor/intel/hardware/iio-sensors/./control.c: In function
'start_acquisition_thread':
vendor/intel/hardware/iio-sensors/./control.c:453:5: error: cast to pointer
from integer of different size [-Werror=int-to-pointer-cast]
     (void*) s);
     ^

Change-Id: Id994ceab05378e53b38bfafa4ead9121eb4163e8
Signed-off-by: Irina Tirdea <irina.tirdea@intel.com>

Populate minDelay field.

Populate minDelay field with what data we find in
'sampling_frequencies_available'.
For sensors in poll mode where we don't have a way to
request a frequency, artificially set up a limit per
CDD requirements.

Signed-off-by: Adriana Reus <adriana.reus@intel.com>
Change-Id: I962e6e352a07f42b4833a83908a4b984a3f17235

GMIN-2992: Prefix uncalibrated gyro name with Uncalibrated

Otherwise it's not clear in many applications which of the two
gyroscopes is the uncalibrated one.

Issue: GMIN-2992

Change-Id: I5610e16290e0e8914efb8bf5aa3a8250bc141509
Signed-off-by: Patrick Porlan <patrick.porlan@intel.com>