[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <20221112154440.5fd1ef6e@jic23-huawei>
Date: Sat, 12 Nov 2022 15:44:40 +0000
From: Jonathan Cameron <jic23@...nel.org>
To: Cosmin Tanislav <demonsingur@...il.com>
Cc: Lars-Peter Clausen <lars@...afoo.de>,
Michael Hennerich <Michael.Hennerich@...log.com>,
Nuno Sá <nuno.sa@...log.com>,
Rob Herring <robh+dt@...nel.org>,
Krzysztof Kozlowski <krzysztof.kozlowski+dt@...aro.org>,
linux-iio@...r.kernel.org, devicetree@...r.kernel.org,
linux-kernel@...r.kernel.org
Subject: Re: [PATCH v4 00/13] Support more parts in LTC2983
On Sun, 6 Nov 2022 17:14:58 +0000
Jonathan Cameron <jic23@...nel.org> wrote:
> On Thu, 3 Nov 2022 15:00:28 +0200
> Cosmin Tanislav <demonsingur@...il.com> wrote:
>
> > Add support for the following parts:
> > * LTC2984
> > * LTC2986
> > * LTM2985
> >
> > The LTC2984 is a variant of the LTC2983 with EEPROM.
> > The LTC2986 is a variant of the LTC2983 with only 10 channels,
> > EEPROM and support for active analog temperature sensors.
> > The LTM2985 is software-compatible with the LTC2986.
> >
> > Also, make bulk write buffer DMA-safe.
> > Besides that, refine the binding.
> Replying at least partly so I can see I already looked at this...
>
> Looks good to me. Will pick up once DT bindings maintainers are
> happy.
Series applied to the togreg branch of iio.git and initially pushed out
as testing. I tweaked patch 11 as requested whilst applying.
Thanks,
Jonathan
>
> Thanks,
>
> Jonathan
>
> >
> > V1 -> V2:
> > * add Fixes tag
> > * add patch that fixes the regmap_bulk_write() call with stack allocated
> > buffer
> > * add patch that refines the binding in preperation for adding new
> > parts support to it
> > * do not use stack allocated buffer for writing the EEPROM key
> >
> > V2 -> V3:
> > * drop minItems if equal to maxItems
> > * drop adi,single-ended conditions because of recent dtschema
> > restrictions (even though they're valid in jsonschema and the last
> > dtschema tag)
> > * drop "allocate iio channels once" patch since it's been picked as a fix
> >
> > V3 -> V4:
> > * add pipe for 'adi,mux-delay-config-us' description
> > * add pipe for 'adi,rtd-curve' description
> > * keep newlines after descriptions
> > * move 'adi,current-rotate' to conditional
> > * put 'adi,custom-steinhart' in the original place
> > * keep 'minItems' for 'adi,custom-steinhart'
> > * override 'enum' to forbid auto-range in 'adi,excitation-current-nanoamp'
> > when using custom thermistors
> > * split refine patch into multiple patches
> > - use generic node name in example
> > - describe broken mux delay property
> > - refine descriptions
> > - change default excitation for custom thermistors
> > - require 4 wire rtd for current rotate
> > - require custom sensor tables
> > - describe matrix items
> > - remove qutations from phandle ref
> > - use hex for sensor address
> > - add default values
> >
> > Cosmin Tanislav (13):
> > iio: temperature: ltc2983: make bulk write buffer DMA-safe
> > dt-bindings: iio: temperature: ltc2983: add default values
> > dt-bindings: iio: temperature: ltc2983: use hex for sensor address
> > dt-bindings: iio: temperature: ltc2983: remove qutations from phandle
> > ref
> > dt-bindings: iio: temperature: ltc2983: describe matrix items
> > dt-bindings: iio: temperature: ltc2983: require custom sensor tables
> > dt-bindings: iio: temperature: ltc2983: require 4 wire rtd for current
> > rotate
> > dt-bindings: iio: temperature: ltc2983: change default excitation for
> > custom thermistors
> > dt-bindings: iio: temperature: ltc2983: refine descriptions
> > dt-bindings: iio: temperature: ltc2983: describe broken mux delay
> > property
> > dt-bindings: iio: temperature: ltc2983: use generic node name in
> > example
> > dt-bindings: iio: temperature: ltc2983: support more parts
> > iio: temperature: ltc2983: support more parts
> >
> > .../bindings/iio/temperature/adi,ltc2983.yaml | 335 +++++++++++-------
> > drivers/iio/temperature/ltc2983.c | 193 +++++++++-
> > 2 files changed, 389 insertions(+), 139 deletions(-)
> >
>
Powered by blists - more mailing lists