[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20240623113400.5dcc1d0d@jic23-huawei>
Date: Sun, 23 Jun 2024 11:34:00 +0100
From: Jonathan Cameron <jic23@...nel.org>
To: Antoniu Miclaus <antoniu.miclaus@...log.com>
Cc: Ramona Gradinariu <ramona.gradinariu@...log.com>, Lars-Peter Clausen
<lars@...afoo.de>, "Michael Hennerich" <Michael.Hennerich@...log.com>, Rob
Herring <robh@...nel.org>, Krzysztof Kozlowski <krzk+dt@...nel.org>, Conor
Dooley <conor+dt@...nel.org>, Jonathan Corbet <corbet@....net>, Jun Yan
<jerrysteve1101@...il.com>, Matti Vaittinen <mazziesaccount@...il.com>,
Mario Limonciello <mario.limonciello@....com>, Mehdi Djait
<mehdi.djait.k@...il.com>, <linux-iio@...r.kernel.org>,
<devicetree@...r.kernel.org>, <linux-kernel@...r.kernel.org>,
<linux-doc@...r.kernel.org>
Subject: Re: [PATCH v2 3/3] docs: iio: add documentation for adxl380 driver
On Fri, 21 Jun 2024 13:17:05 +0300
Antoniu Miclaus <antoniu.miclaus@...log.com> wrote:
> Add documentation for adxl380 driver which describes the driver
> device files and shows how the user may use the ABI for various
> scenarios (configuration, measurement, etc.).
>
> Signed-off-by: Ramona Gradinariu <ramona.gradinariu@...log.com>
> Signed-off-by: Antoniu Miclaus <antoniu.miclaus@...log.com>
Looks good. Only comment is to make sure you run a docs build test.
We had one similar hexdump case blow up a few weeks back.
If you already have, just stick a note under the --- in the next version.
Everyone hates building the docs because it's slow so I know I sometimes
forget to do so and suspect others do as well!
> +Obtain buffered data:
> +
> +.. code-block:: bash
We recently got bitten by a processing issue for one of these hex dumps and changed it
to unformatted text. Make sure you do an HTML docs build for this and check there
are no similar warnings (honestly I never really understood what was going wrong with the
parser).
If you do need unformatted text
Obtain buffered data::
plus the indent works.
> +
> + root:/sys/bus/iio/devices/iio:device0> hexdump -C /dev/iio\:device0
> + ...
> + 002bc300 f7 e7 00 a8 fb c5 24 80 f7 e7 01 04 fb d6 24 80 |......$.......$.|
> + 002bc310 f7 f9 00 ab fb dc 24 80 f7 c3 00 b8 fb e2 24 80 |......$.......$.|
> + 002bc320 f7 fb 00 bb fb d1 24 80 f7 b1 00 5f fb d1 24 80 |......$...._..$.|
> + 002bc330 f7 c4 00 c6 fb a6 24 80 f7 a6 00 68 fb f1 24 80 |......$....h..$.|
> + 002bc340 f7 b8 00 a3 fb e7 24 80 f7 9a 00 b1 fb af 24 80 |......$.......$.|
> + 002bc350 f7 b1 00 67 fb ee 24 80 f7 96 00 be fb 92 24 80 |...g..$.......$.|
> + 002bc360 f7 ab 00 7a fc 1b 24 80 f7 b6 00 ae fb 76 24 80 |...z..$......v$.|
> + 002bc370 f7 ce 00 a3 fc 02 24 80 f7 c0 00 be fb 8b 24 80 |......$.......$.|
> + 002bc380 f7 c3 00 93 fb d0 24 80 f7 ce 00 d8 fb c8 24 80 |......$.......$.|
> + 002bc390 f7 bd 00 c0 fb 82 24 80 f8 00 00 e8 fb db 24 80 |......$.......$.|
> + 002bc3a0 f7 d8 00 d3 fb b4 24 80 f8 0b 00 e5 fb c3 24 80 |......$.......$.|
> + 002bc3b0 f7 eb 00 c8 fb 92 24 80 f7 e7 00 ea fb cb 24 80 |......$.......$.|
> + 002bc3c0 f7 fd 00 cb fb 94 24 80 f7 e3 00 f2 fb b8 24 80 |......$.......$.|
> + ...
Powered by blists - more mailing lists