Re: [PATCH 0/4] iio: ad3552r-hs: add support for internal ramp generator

[Jonathan Cameron <jic23@kernel.org>]

On Fri, 21 Mar 2025 21:28:47 +0100
Angelo Dureghello <adureghello@baylibre.com> wrote:

> Add support to enable the HDL IP core internal ramp generator,
> actually managed by the adi-axi-dac backend. 

What is it for?  Circuit testing or something else?
We have in the past had pattern generators in IIO (currently under
frequency drivers, though I'm not sure what we have in the way of
waveforms in the stuff outside staging) so I'd like to be sure
this is about debug rather than a pattern that is actually expected
to be useful.

Jonathan

> 
> It works this way:
> 
> /sys/bus/iio/devices/iio:device0# echo 1 > buffer0/out_voltage0_en 
> /sys/bus/iio/devices/iio:device0# echo 1 > buffer0/out_voltage1_en                                           
> /sys/bus/iio/devices/iio:device0# echo 1 > buffer0/enable 
> 
> Activating ramp generator:
> 
> /sys/kernel/debug/iio/iio:device0# echo -n backend-ramp-generator > data_source
> 
> Deactivating:
> 
> /sys/kernel/debug/iio/iio:device0# echo -n iio-buffer > data_source
> 
> Signed-off-by: Angelo Dureghello <adureghello@baylibre.com>
> ---
> Angelo Dureghello (4):
>       docs: iio: add documentation for ad3552r driver
>       iio: backend: add support for data source get
>       iio: dac: adi-axi-dac: add data source get
>       iio: dac: ad3552r-hs: add support for internal ramp
> 
>  Documentation/iio/ad3552r.rst      |  65 +++++++++++++++++++++++
>  Documentation/iio/index.rst        |   1 +
>  MAINTAINERS                        |   1 +
>  drivers/iio/dac/ad3552r-hs.c       | 106 ++++++++++++++++++++++++++++++++++---
>  drivers/iio/dac/adi-axi-dac.c      |  27 ++++++++++
>  drivers/iio/industrialio-backend.c |  28 ++++++++++
>  include/linux/iio/backend.h        |   5 ++
>  7 files changed, 227 insertions(+), 6 deletions(-)
> ---
> base-commit: eb870a5af7db1e5ca59330875125230b28e630f9
> change-id: 20250321-wip-bl-ad3552r-fixes-4a386944c170
> 
> Best regards,