On Tue, 07 Jul 2026 15:04:37 +0100 Rodrigo Alencar via B4 Relay <[email protected]> wrote:
> From: Rodrigo Alencar <[email protected]> > > Add custom ABI documentation files for the DDS AD9910 with sysfs and > debugfs entries. The sysfs ones control parameters from the Digital Ramp > Generator and OSK engine. The debugfs attributes are read-only and > evalutate into source channel labels for each DDS core parameter. > > Signed-off-by: Rodrigo Alencar <[email protected]> Sashiko called out on earlier patch adding IIO_FREQUENCY that it might need ABI docs in the main ABI file. Now we only add that when a driver first uses, it but I do see some standard stuff being added in patch 11 for offset and scale, so I'd expect to see a patch to sysfs-bus-iio Key being to define the units. That may mean adding a _raw entry even though you don't actually use it in that patch. > --- > Documentation/ABI/testing/debugfs-iio-ad9910 | 60 > ++++++++++++++++++++++ > .../ABI/testing/sysfs-bus-iio-frequency-ad9910 | 31 +++++++++++ > MAINTAINERS | 2 + > 3 files changed, 93 insertions(+) > > diff --git a/Documentation/ABI/testing/debugfs-iio-ad9910 > b/Documentation/ABI/testing/debugfs-iio-ad9910 > new file mode 100644 > index 000000000000..b4a125bc1007 > --- /dev/null > +++ b/Documentation/ABI/testing/debugfs-iio-ad9910 > @@ -0,0 +1,60 @@ > +What: /sys/kernel/debug/iio/iio:deviceX/frequency_source > +KernelVersion: 7.3 > +Contact: [email protected] > +Description: > + Read-only. Reports the channel label that corresponds to which > of the Maybe reference label via iio:deviceX/*_label or something like that to make the point it is in the related per channel label sysfs file. > + on-chip signal generators is currently driving the output > frequency. > + > + Resolution order (first match wins): > + I wonder if this is correct place to provide this info. Sooner or later I'd expect the attribute to be used in another driver and annoyingly we can only document them in one place or the docs build goes wrong. So maybe just he bit above should be here and this priority stuff belongs in the per driver main doc? (I'm wondering if an earlier me told you to put it here - sorry if so!) > + - ``ram`` — RAM playback is enabled and its destination is > + frequency, phase (polar), amplitude, or polar. > + - ``drg_frequency`` — the Digital Ramp Generator is enabled > + and its destination is frequency. > + - ``parallel_frequency`` — the parallel data port is enabled > + and the frequency scan element is active. > + - ``profileN`` (N = 0..7) — none of the above modulation > + sources are driving frequency, so the currently selected > + single-tone profile register is in effect. > +
