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

Issue 2198543002: Accelerometer: find trigger name (Closed)

Created:
4 years, 4 months ago by gwendal1
Modified:
4 years, 4 months ago
Reviewers:
jonross, oshima
CC:
chromium-reviews, oshima+watch_chromium.org
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Accelerometer: find trigger name Do not assume the sysfs trigger to use is trigger0. Instead find the trigger which name is "sysfstrig0". This is the name used by accelerometer-init.sh udev script. BUG=chromium:632486 TEST=With proper accelerometer-init.sh, check that chrome rotate screen even when cros-ec-ring driver is loaded. R=jonross@chromium.org Signed-off-by: Gwendal Grignou <gwendal@chromium.org>; Committed: https://crrev.com/1ade62df9003dc43cc47037d8207a2a6b89c3014 Cr-Commit-Position: refs/heads/master@{#410407}

Patch Set 1 #

Patch Set 2 : Accelerometer: find trigger name #

Unified diffs Side-by-side diffs Delta from patch set Stats (+43 lines, -7 lines) Patch
M chromeos/accelerometer/accelerometer_reader.cc View 4 chunks +43 lines, -7 lines 0 comments Download

Messages

Total messages: 10 (5 generated)
jonross
On 2016/08/08 17:29:07, gwendal wrote: > mailto:gwendal@chromium.org changed reviewers: > + mailto:oshima@chromium.org lgtm
4 years, 4 months ago (2016-08-08 17:31:43 UTC) #3
oshima
rs lgtm
4 years, 4 months ago (2016-08-08 17:48:48 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2198543002/20001
4 years, 4 months ago (2016-08-08 17:56:58 UTC) #6
commit-bot: I haz the power
Committed patchset #2 (id:20001)
4 years, 4 months ago (2016-08-08 20:01:20 UTC) #8
commit-bot: I haz the power
4 years, 4 months ago (2016-08-08 20:04:18 UTC) #10
Message was sent while issue was closed.
Patchset 2 (id:??) landed as
https://crrev.com/1ade62df9003dc43cc47037d8207a2a6b89c3014
Cr-Commit-Position: refs/heads/master@{#410407}

Powered by Google App Engine
This is Rietveld 408576698