| Message ID | 20200724213329.899216-1-swboyd@chromium.org (mailing list archive) |
|---|---|
| Headers | show |
| Series | Some sx9310 iio driver updates | expand |
On Fri, 24 Jul 2020 14:33:24 -0700 Stephen Boyd <swboyd@chromium.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@gmx.de> > Cc: Lars-Peter Clausen <lars@metafoo.de> > Cc: Peter Meerwald-Stadler <pmeerw@pmeerw.net> > Cc: Rob Herring <robh+dt@kernel.org> > Cc: Douglas Anderson <dianders@chromium.org> > Cc: Daniel Campello <campello@chromium.org> > > base-commit: b3a9e3b9622ae10064826dccb4f7a52bd88c7407
Quoting Jonathan Cameron (2020-07-26 04:56:36) > On Fri, 24 Jul 2020 14:33:24 -0700 > Stephen Boyd <swboyd@chromium.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? > Sure. No problem for me to work with Daniel. Thanks.