linux-kernel - Re: [PATCH 0/3] Some sx9310 iio driver updates

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]

Date:   Sun, 26 Jul 2020 12:56:36 +0100
From:   Jonathan Cameron <jic23@...nel.org>
To:     Stephen Boyd <swboyd@...omium.org>
Cc:     linux-kernel@...r.kernel.org, linux-iio@...r.kernel.org,
        Hartmut Knaack <knaack.h@....de>,
        Lars-Peter Clausen <lars@...afoo.de>,
        Peter Meerwald-Stadler <pmeerw@...erw.net>,
        Rob Herring <robh+dt@...nel.org>,
        Douglas Anderson <dianders@...omium.org>,
        Daniel Campello <campello@...omium.org>
Subject: Re: [PATCH 0/3] Some sx9310 iio driver updates

On Fri, 24 Jul 2020 14:33:24 -0700
Stephen Boyd <swboyd@...omium.org> wrote:

> These patches are only related because I'm looking at this driver. The
> first one resends the DT binding for the driver that was merged in
> v5.8-rc1 with a small change to update for proper regulators. The second
> patch fixes a few printks that are missing newlines and should be
> totally non-trivial to apply. The third patch changes whoami to unsigned
> to avoid confusing. The fourth patch drops channel_users because it's
> unused. The final patch adds support to enable the svdd and vdd supplies
> so that this driver can work on a board I have where the svdd supply
> isn't enabled at boot and needs to be turned on before this driver
> starts to communicate with the chip.
Hi Stephen,

I clearly made a mess of picking this driver up in the first place.

Anyhow, we now have two series in flight for the driver that, whilst
mostly independent (I think) will result in at least some fuzz.
If possible could you work with Daniel to send me one single series
with all the changes?

Thanks,

Jonathan

> 
> Changes from v1:
>  * Two new patches for whoami and channel_uesrs
>  * Use bulk regulator APIs for supplies patch
>  * Support both regulators
> 
> Daniel Campello (1):
>   dt-bindings: iio: Add bindings for sx9310 sensor
> 
> Stephen Boyd (4):
>   iio: sx9310: Add newlines to printks
>   iio: sx9310: whoami is unsigned
>   iio: sx9310: Drop channel_users[]
>   iio: sx9310: Enable vdd and svdd regulators at probe
> 
>  .../iio/proximity/semtech,sx9310.yaml         | 60 +++++++++++++++++++
>  drivers/iio/proximity/sx9310.c                | 46 +++++++++++---
>  2 files changed, 97 insertions(+), 9 deletions(-)
>  create mode 100644 Documentation/devicetree/bindings/iio/proximity/semtech,sx9310.yaml
> 
> Cc: Hartmut Knaack <knaack.h@....de>
> Cc: Lars-Peter Clausen <lars@...afoo.de>
> Cc: Peter Meerwald-Stadler <pmeerw@...erw.net>
> Cc: Rob Herring <robh+dt@...nel.org>
> Cc: Douglas Anderson <dianders@...omium.org>
> Cc: Daniel Campello <campello@...omium.org>
> 
> base-commit: b3a9e3b9622ae10064826dccb4f7a52bd88c7407