New issue
Advanced search Search tips

Issue 799607 link

Starred by 1 user

Issue metadata

Status: Started
Owner:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug



Sign in to add a comment

kernel: check sensor support (chrome/ARC++)

Project Member Reported by gwendal@chromium.org, Jan 5 2018

Issue description

Loading 4.14 on eve, kernel crashes when ARC++ starts.

I lost the console-log, but it is in cros_ec_sensors_ring.o, accessing a offset from a NULL pointer.

 
Status: Started (was: Untriaged)
Project Member

Comment 2 by bugdroid1@chromium.org, Jan 10 2018

Labels: merge-merged-chromeos-4.14
The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/kernel/+/37a68af97ca63b98e529a60a2d955e71a5ab4d57

commit 37a68af97ca63b98e529a60a2d955e71a5ab4d57
Author: Gwendal Grignou <gwendal@chromium.org>
Date: Wed Jan 10 08:10:46 2018

CHROMIUM: iio: cros_ec: Fix a kernel panic when processing first ring message

indio_dev was not initialized. It is only used in the
cros_ec_sensors_ring driver, which is not upstream.

BUG=chromium:799607
TEST=On eve, check ARC++ sensor app is working with AIDA64

Change-Id: I605979b781ae2d38f328b8e00b7e66148260c552
Signed-off-by: Gwendal Grignou <gwendal@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/851415
Commit-Ready: ChromeOS CL Exonerator Bot <chromiumos-cl-exonerator@appspot.gserviceaccount.com>
Reviewed-by: Guenter Roeck <groeck@chromium.org>

[modify] https://crrev.com/37a68af97ca63b98e529a60a2d955e71a5ab4d57/drivers/iio/common/cros_ec_sensors/cros_ec_sensors_core.c

Project Member

Comment 3 by bugdroid1@chromium.org, Jan 10 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/kernel/+/12fc95c7ee8119c4621aaa6d46fd26c19cd267c1

commit 12fc95c7ee8119c4621aaa6d46fd26c19cd267c1
Author: Gwendal Grignou <gwendal@chromium.org>
Date: Wed Jan 10 08:10:48 2018

CHROMIUM: iio: cros_ec: Readd 'id' sysfs entry

It is not part of upstream as this entry is only used by Android Sensor
HAL to interpret ring buffer information.

BUG=chromium:799607
TEST=On eve, check ARC++ sensor app is working with AIDA64

Change-Id: I402c4ad17c7dfbd797a50dd4ef6a72b7931e4885
Signed-off-by: Gwendal Grignou <gwendal@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/853094
Commit-Ready: ChromeOS CL Exonerator Bot <chromiumos-cl-exonerator@appspot.gserviceaccount.com>
Reviewed-by: Guenter Roeck <groeck@chromium.org>

[modify] https://crrev.com/12fc95c7ee8119c4621aaa6d46fd26c19cd267c1/drivers/iio/common/cros_ec_sensors/cros_ec_sensors_core.c

Sign in to add a comment