[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <20241031213803.18091910@jic23-huawei>
Date: Thu, 31 Oct 2024 21:38:03 +0000
From: Jonathan Cameron <jic23@...nel.org>
To: ahaslam@...libre.com
Cc: lars@...afoo.de, Michael.Hennerich@...log.com, robh@...nel.org,
krzk+dt@...nel.org, conor+dt@...nel.org, nuno.sa@...log.com,
dlechner@...libre.com, linux-iio@...r.kernel.org,
devicetree@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH v3 0/6] Improvements and Enhancements for AD5791 DAC
Driver
On Thu, 31 Oct 2024 08:17:40 +0100
ahaslam@...libre.com wrote:
> From: Axel Haslam <ahaslam@...libre.com>
>
> These patches aim to improve on the ad5791 driver:
> - make use of chip_info / match tables, and drop device enum id.
> - Add reset, clr and ldac gpios that have to be set to the correct level in case they
> are not hardwired on the setup/PCB.
> - simplify probe by using the devm_* functions to automatically free resources.
Applied. Thanks,
> ---
> Changes in v3:
> - v2 is missing the version prefix. Im sending v3 just with the added review-by tag.
> - Add review-by tag from David Lechner
> - Link to V2: https://lore.kernel.org/all/94a03835-bdd1-4243-88c7-0ad85784fe36@baylibre.com/
>
> Changes in v2:
> - Fix probe error print using uninitialized ret.
> - Add documentation for new struct parameters
> - Add review-by tags to device tree bindings
> - Link to V1: https://lore.kernel.org/all/CAKXjFTPwN2TYW6sq1kj3miZ0f5OqKX0aTk8eGf1sj9TBk1_e=A@mail.gmail.com/T/
>
> Axel Haslam (6):
> dt-bindings: iio: dac: ad5791: Add optional reset, clr and ldac gpios
> dt-bindings: iio: dac: ad5791: Add required voltage supplies
> iio: dac: ad5791: Include chip_info in device match tables
> iio: dac: ad5791: Add reset, clr and ldac gpios
> iio: dac: ad5791: Use devm_regulator_get_enable_read_voltage
> iio: dac: ad5791: Use devm_iio_device_register
>
> .../bindings/iio/dac/adi,ad5791.yaml | 39 ++++
> drivers/iio/dac/ad5791.c | 203 ++++++++----------
> 2 files changed, 131 insertions(+), 111 deletions(-)
>
Powered by blists - more mailing lists