Драйвер акселерометра kxcjk1013 использует идентификационную матрицу, несмотря на то, что правила hwdb и udev определяют ⇐ Linux

[Anonymous]

Я использую ноутбук Teclast F6 Pro 2-в-1 с двумя акселерометрами kxcjk1013, который используется для ориентации экрана и переключения режима планшета.
Хотя датчик кажется работоспособным ( например, я могу прочитать значения ускорения, используя cat /sys/bus/iio/devices/iio:device*/in_accel_*_raw, ix X11 ориентация экрана меняется при повороте монитора), ориентация, сообщаемая драйвером, неверна. Это предотвращает активацию режима планшета, поскольку два датчика никогда не отображаются в правильной ориентации относительно друг друга.

В частности, я вижу следующее сообщение в dmesg:

Код: Выделить всё

kxcjk1013 i2c-KIOX020A:00: mounting matrix not found: using identity...
kxcjk1013 i2c-KIOX010A:00: mounting matrix not found: using identity...

Из исходного кода драйвера kxcjk1013 я понимаю, что это происходит, когда драйверу не удается найти матрицу монтирования в ACPI, sysfs или свойствах устройства.
Драйвер сначала попытается получить свойства из данных платформы, если это не удастся, затем из ACPI, если это не удастся, то из IIO, а если это не удастся, то он прибегнет к идентификационной матрице, что и происходит. здесь.
Фрагменты кода

Код: Выделить всё

kxcjk1013_get_mount_matrix

Код: Выделить всё

static const struct iio_mount_matrix *
kxcjk1013_get_mount_matrix(const struct iio_dev *indio_dev,
const struct iio_chan_spec *chan)
{
struct kxcjk1013_data *data = iio_priv(indio_dev);

return &data->orientation;
}

Код: Выделить всё

kxcjk1013_probe

Код: Выделить всё

    pdata = dev_get_platdata(&client->dev);
if (pdata) {
data->active_high_intr = pdata->active_high_intr;
data->orientation = pdata->orientation;
} else {
data->active_high_intr = true; /* default polarity */

if (!iio_read_acpi_mount_matrix(&client->dev, &data->orientation, "ROTM")) {
ret = iio_read_mount_matrix(&client->dev, &data->orientation);
if (ret)
return ret;
}

}

Метод ACPI (при сбое dev_get_platdata)

Код: Выделить всё

/**
* iio_read_acpi_mount_matrix() - Read accelerometer mount matrix info from ACPI
* @dev:        Device structure
* @orientation:    iio_mount_matrix struct to fill
* @acpi_method:    ACPI method name to read the matrix from, usually "ROTM"
*
* Try to read the mount-matrix by calling the specified method on the device's
* ACPI firmware-node. If the device has no ACPI firmware-node; or the method
* does not exist then this will fail silently.  This expects the method to
* return data in the ACPI "ROTM" format defined by Microsoft:
* https://learn.microsoft.com/en-us/windows-hardware/drivers/sensors/sensors-acpi-entries
* This is a Microsoft extension and not part of the official ACPI spec.
* The method name is configurable because some dual-accel setups define 2 mount
* matrices in a single ACPI device using separate "ROMK" and "ROMS" methods.
*
* Returns: true if the matrix was successfully, false otherwise.
*/
bool iio_read_acpi_mount_matrix(struct device *dev,
struct iio_mount_matrix *orientation,
char *acpi_method)
{
struct acpi_buffer buffer = { ACPI_ALLOCATE_BUFFER, NULL };
char *str;
union acpi_object *obj, *elements;
acpi_handle handle;
acpi_status status;
int i, j, val[3];
bool ret = false;

handle = ACPI_HANDLE(dev);
if (!handle)
return false;

if (!acpi_has_method(handle, acpi_method))
return false;

status = acpi_evaluate_object(handle, acpi_method, NULL, &buffer);
if (ACPI_FAILURE(status)) {
dev_err(dev, "Failed to get ACPI mount matrix: %d\n", status);
return false;
}

obj = buffer.pointer;
if (obj->type != ACPI_TYPE_PACKAGE || obj->package.count != 3) {
dev_err(dev, "Unknown ACPI mount matrix package format\n");
goto out_free_buffer;
}

elements = obj->package.elements;
for (i = 0; i < 3; i++) {
if (elements[i].type != ACPI_TYPE_STRING) {
dev_err(dev, "Unknown ACPI mount matrix element format\n");
goto out_free_buffer;
}

str = elements[i].string.pointer;
if (sscanf(str, "%d %d %d", &val[0], &val[1], &val[2]) != 3) {
dev_err(dev, "Incorrect ACPI mount matrix string format\n");
goto out_free_buffer;
}

for (j = 0; j < 3; j++) {
switch (val[j]) {
case -1: str = "-1"; break;
case 0:  str = "0";  break;
case 1:  str = "1";  break;
default:
dev_err(dev, "Invalid value in ACPI mount matrix: %d\n", val[j]);
goto out_free_buffer;
}
orientation->rotation[i * 3 + j] = str;
}
}

ret = true;

out_free_buffer:
kfree(buffer.pointer);
return ret;
}

Метод iio (если ACPI не работает)

Код: Выделить всё

/**
* iio_read_mount_matrix() - retrieve iio device mounting matrix from
*                           device "mount-matrix" property
* @dev:    device the mounting matrix property is assigned to
* @matrix: where to store retrieved matrix
*
* If device is assigned no mounting matrix property, a default 3x3 identity
* matrix will be filled in.
*
* Returns: 0 if success, or a negative error code on failure.
*/
int iio_read_mount_matrix(struct device *dev, struct iio_mount_matrix *matrix)
{
size_t len = ARRAY_SIZE(iio_mount_idmatrix.rotation);
int err;

err = device_property_read_string_array(dev, "mount-matrix", matrix->rotation, len);
if (err == len)
return 0;

if (err >= 0)
/* Invalid number of matrix entries. */
return -EINVAL;

if (err != -EINVAL)
/* Invalid matrix declaration format. */
return err;

/* Matrix was not declared at all: fallback to identity.  */
return iio_setup_mount_idmatrix(dev, matrix);
}
EXPORT_SYMBOL(iio_read_mount_matrix);

Резервная матрица идентификации (в случае сбоя iio)

Код: Выделить всё

static int iio_setup_mount_idmatrix(const struct device *dev,
struct iio_mount_matrix *matrix)
{
*matrix = iio_mount_idmatrix;
dev_info(dev, "mounting matrix not found: using identity...\n");
return 0;
}

Чтение свойств устройства

Код: Выделить всё

int device_property_read_string_array(const struct device *dev, const char *propname,
const char **val, size_t nval)
{
return fwnode_property_read_string_array(dev_fwnode(dev), propname, val, nval);
}
EXPORT_SYMBOL_GPL(device_property_read_string_array);

Похоже, что получить матрицу монтирования как из ACPI, так и из Sysfs не удается.

Код: Выделить всё

iio_read_acpi_mount_matrix

Код: Выделить всё

err = device_property_read_string_array(dev, "mount-matrix", matrix->rotation, len);

не работает должным образом с учетом полученных входных данных, поскольку он использует резервную матрицу идентификации в iio_read_mount_matrix. В качестве имени свойства он использует матрицу-mount, а свойство в устройстве называется in_accel_mount_matrix или ACCEL_MOUNT_MATRIX.

Вот ключевые моменты моей настройки и наблюдений:

• Поведение водителя:
  • Похоже, что драйвер ожидает свойство mount-matrix либо через ACPI, либо через sysfs.
  • Однако в я вижу только cat /sys/bus/iio/ устройства/iio:device0/, атрибут существует как in_accel_mount_matrix, а не mount-matrix.
  • Это несоответствие имен? Переводит ли драйвер между in_accel_mount_matrix и mount-matrix?
• < strong>Подробности ACPI:
  • Устройство обнаруживается через ACPI, но здесь не определена матрица монтирования (

    Код: Выделить всё

    /sys/bus/acpi/devices/KIOX010A:00/

    ).
  • Атрибут DRIVER для устройства ACPI пуст. Могу ли я применить правила udev для устройств ACPI для решения этой проблемы? Использует ли ACPI отдельный драйвер или мне следует ожидать, что здесь будет ссылка на драйвер, указанный выше?
  • Я декомпилировал таблицы ACPI (DSDT) и нашел устройства здесь, но матрицы монтирования нет. определено здесь. Могу ли я определить это задним числом?
  • Присутствует устройство ACPI для акселерометра (

    Код: Выделить всё

    /sys/bus/acpi/devices/KIOX010A:00/

    ), но с ним не связан ни один драйвер (

    Код: Выделить всё

    DRIVER==""

    ).
  • Может ли отсутствие драйвера привести к игнорированию матрицы монтирования из ACPI?
  < /li>
• Правила udev и hwdb:
  • Матрица монтирования правильно определено через 60-sensor.hwdb:

    Код: Выделить всё

    ## Teclast F6 Pro (2 sensors)
    sensor:modalias:acpi:KIOX010A*:dmi:*:svnTECLAST:pnF6Pro:*
    ACCEL_MOUNT_MATRIX=-1, 0, 0; 0, -1, 0; 0, 0, -1
    ACCEL_LOCATION=display
    
    sensor:modalias:acpi:KIOX020A*:dmi:*:svnTECLAST:pnF6Pro:*
    ACCEL_MOUNT_MATRIX=0, 1, 0; 1, 0, 0; 0, 0, 1
    ACCEL_LOCATION=base
    

  • Я определил это дополнительно с помощью правил udev в /etc/udev/rules.d/99- i2c-sensor.rules, следующим образом:

    Код: Выделить всё

    ACTION=="add|change", KERNEL=="iio:device0", ATTR{in_accel_mount_matrix}="0, 1, 0; 1, 0, 0; 0, 0, 1"
    ACTION=="add|change", KERNEL=="iio:device1", ATTR{in_accel_mount_matrix}="-1, 0, 0; 0, -1, 0; 0, 0, -1"
    

  • Однако на поведение водителя это не влияет. Кажется, не выбрана правильная матрица.

Вопрос:

Драйвер:

• Несоответствие имен между mount-matrix ( в драйвере ядра) и in_accel_mount_matrix (в свойствах устройства) вероятная причина проблемы? Если да, то как я могу решить эту проблему?
• Соответствует ли формат матрицы монтирования (разделенные запятой и точкой с запятой) драйвером?
• Как я могу дополнительно отладить поведение ядра и почему оно не может прочитать матрицу монтирования? Либо путем репликации поведения ядра в отдельном скрипте, либо путем изменения и компиляции самого ядра.

Свойства и конфигурация устройства

• Почему правила hwdb и udev не работают? Это проблема из-за того, что драйвер не смотрит сюда, или из-за того, что данные, предоставленные драйверу, не соответствуют ожиданиям?
• Могу ли я вручную отредактировать данные платформы, чтобы их можно было прочитать здесь ?
• Могу ли я добавить правила udev для ACPI или I2C, аналогично тому, как я это сделал для IIO?
• Если матрица определена в hwdb для ACPI, почему он не отображается как атрибут для устройство?
• Есть ли какие-либо дополнительные шаги или альтернативы отладки, которые я могу попробовать (например, создание собственного наложения дерева устройств, исправлений ядра или других методов)?

Будем очень признательны за любые советы или предложения о том, как заставить драйвер kxcjk1013 подобрать правильную матрицу крепления!

Дайте мне знать, если вам нужны дальнейшие уточнения на вопрос.

Подробнее здесь: https://stackoverflow.com/questions/792 ... udev-rules