linux-kernel - Re: [RESEND PATCH v8 0/6] Add support for ADIS16550

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20250217143354.0d1c4a2d@jic23-huawei>
Date: Mon, 17 Feb 2025 14:33:54 +0000
From: Jonathan Cameron <jic23@...nel.org>
To: Robert Budai <robert.budai@...log.com>
Cc: Lars-Peter Clausen <lars@...afoo.de>, Michael Hennerich
 <Michael.Hennerich@...log.com>, Nuno Sa <nuno.sa@...log.com>, "Ramona
 Gradinariu" <ramona.gradinariu@...log.com>, Antoniu Miclaus
 <antoniu.miclaus@...log.com>, Rob Herring <robh@...nel.org>, "Krzysztof
 Kozlowski" <krzk+dt@...nel.org>, Conor Dooley <conor+dt@...nel.org>,
 "Jonathan Corbet" <corbet@....net>, <linux-iio@...r.kernel.org>,
 <devicetree@...r.kernel.org>, <linux-kernel@...r.kernel.org>,
 <linux-doc@...r.kernel.org>
Subject: Re: [RESEND PATCH v8 0/6] Add support for ADIS16550

On Mon, 17 Feb 2025 12:57:44 +0200
Robert Budai <robert.budai@...log.com> wrote:

> The ADIS16550 is a complete inertial system that includes a triaxis gyroscope
> and a triaxis accelerometer. Each inertial sensor in the ADIS16550 combines
> industry leading MEMS only technology with signal conditioning that optimizes
> dynamic performance. The factory calibration characterizes each sensor for
> sensitivity, bias, and alignment. As a result, each sensor has its own dynamic
> compensation formulas that provide accurate sensor measurements.
> 
Hi Robert,

The cover letter of any RESEND should always start with why you are doing so.
If this was for the tiny fixup you mentioned it should have been v9 and
not have been sent for a few days at least.  If everything else is fine
I don't mind making that sort of fixup whilst applying anyway!

Jonathan


> Robert Budai (6):
>   iio: imu: adis: Add custom ops struct
>   iio: imu: adis: Add reset to custom ops
>   iio: imu: adis: Add DIAG_STAT register
>   dt-bindings: iio: Add adis16550 bindings
>   iio: imu: adis16550: add adis16550 support
>   docs: iio: add documentation for adis16550 driver
> 
>  .../bindings/iio/imu/adi,adis16550.yaml       |   74 ++
>  Documentation/iio/adis16550.rst               |  376 ++++++
>  Documentation/iio/index.rst                   |    1 +
>  MAINTAINERS                                   |   10 +
>  drivers/iio/imu/Kconfig                       |   13 +
>  drivers/iio/imu/Makefile                      |    1 +
>  drivers/iio/imu/adis.c                        |   35 +-
>  drivers/iio/imu/adis16550.c                   | 1149 +++++++++++++++++
>  include/linux/iio/imu/adis.h                  |   34 +-
>  9 files changed, 1680 insertions(+), 13 deletions(-)
>  create mode 100644 Documentation/devicetree/bindings/iio/imu/adi,adis16550.yaml
>  create mode 100644 Documentation/iio/adis16550.rst
>  create mode 100644 drivers/iio/imu/adis16550.c
>