linux-kernel - Re: [PATCH RESEND v4 0/5] iio: accel: mc3230: add mount matrix, of match and mc3510c support

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] [day] [month] [year] [list]

Message-ID: <20250118162010.08ac38f4@jic23-huawei>
Date: Sat, 18 Jan 2025 16:20:10 +0000
From: Jonathan Cameron <jic23@...nel.org>
To: Jonathan Cameron <Jonathan.Cameron@...wei.com>
Cc: Vasiliy Doylov via B4 Relay
 <devnull+nekodevelopper.gmail.com@...nel.org>, <nekodevelopper@...il.com>,
 Lars-Peter Clausen <lars@...afoo.de>, Rob Herring <robh@...nel.org>,
 Krzysztof Kozlowski <krzk+dt@...nel.org>, Conor Dooley
 <conor+dt@...nel.org>, <linux-iio@...r.kernel.org>,
 <linux-kernel@...r.kernel.org>, <devicetree@...r.kernel.org>, "Krzysztof
 Kozlowski" <krzysztof.kozlowski@...aro.org>
Subject: Re: [PATCH RESEND v4 0/5] iio: accel: mc3230: add mount matrix, of
 match and mc3510c support

On Thu, 16 Jan 2025 16:42:11 +0000
Jonathan Cameron <Jonathan.Cameron@...wei.com> wrote:

> On Thu, 16 Jan 2025 16:52:41 +0300
> Vasiliy Doylov via B4 Relay <devnull+nekodevelopper.gmail.com@...nel.org> wrote:
> 
> Always say why you are resending.

Other than that, looks good and you seem to have addressed all
comments on v3, so applied to the testing branch of iio.git.
Note I'll be rebasing that on rc1 once available and only then pushing
it out as togreg.  In the meantime it will get some build bot coverage.

Thanks,

Jonathan

> 
> Jonathan
> 
> > Changes includes:
> > - Add mount matrix handling
> > - Add match table to work with DT
> > - Add MC3510C support
> > 
> > MC3510C use same registors as MC3230, but different value scale.
> > 
> > Signed-off-by: Vasiliy Doylov <nekodevelopper@...il.com>
> > ---
> > Changes in v4:
> > - Fixed commit messages
> > - Added const modificator to chip_infos
> > - Return name->chip_id->product_code->scale order
> > - `{}` replaced by `{ }` (with space)
> > - Link to v3: https://lore.kernel.org/r/20250112-mainlining-mc3510c-v3-0-9ee6520ab69d@gmail.com
> > 
> > Changes in v2:
> > - Ordered commits
> > - Fixed comment style
> > - Ordered struct members
> > - Fixed device table
> > - MC5310C commit splitted
> > - Link to v1: https://lore.kernel.org/r/20250111-mainlining-mc3510c-v1-0-57be503addf8@gmail.com
> > 
> > Changes in v3:
> > - Fixed patch (poped stash)
> > - Link to v2: https://lore.kernel.org/all/20250112-mainlining-mc3510c-v2-0-322804a545cf@gmail.com
> > 
> > ---
> > Vasiliy Doylov (5):
> >       dt-bindings: iio: accel: mc3230: document mc3510c
> >       iio: accel: mc3230: add mount matrix support
> >       iio: accel: mc3230: add OF match table
> >       iio: accel: mc3230: add multiple devices support
> >       iio: accel: mc3230: add mc3510c support
> > 
> >  .../devicetree/bindings/trivial-devices.yaml       |  2 +
> >  drivers/iio/accel/mc3230.c                         | 95 ++++++++++++++++++----
> >  2 files changed, 80 insertions(+), 17 deletions(-)
> > ---
> > base-commit: 2b88851f583d3c4e40bcd40cfe1965241ec229dd
> > change-id: 20250111-mainlining-mc3510c-564702fba487
> > 
> > Best regards,  
> 
>