From: Jonathan Cameron <jic23@kernel.org>
To: Wang Zihan <jiyu03@qq.com>
Cc: linux-iio@vger.kernel.org, linux-doc@vger.kernel.org
Subject: Re: [PATCH v2] iio: adxl313: fix typos in documentation
Date: Mon, 4 May 2026 17:43:26 +0100 [thread overview]
Message-ID: <20260504174326.353b6f02@jic23-huawei> (raw)
In-Reply-To: <tencent_EC262467CD5E46CA97CF2B2FDCBCA0565C08@qq.com>
On Sat, 2 May 2026 14:07:00 +0800
Wang Zihan <jiyu03@qq.com> wrote:
> Add missing space in "ADXL313is" and correct "a single types"
> to "a single type".
>
> Signed-off-by: Wang Zihan <jiyu03@qq.com>
> ---
> Documentation/iio/adxl313.rst | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/Documentation/iio/adxl313.rst b/Documentation/iio/adxl313.rst
> index 966e72c01..3662153a6 100644
> --- a/Documentation/iio/adxl313.rst
> +++ b/Documentation/iio/adxl313.rst
> @@ -11,7 +11,7 @@ This driver supports Analog Device's ADXL313 on SPI/I2C bus.
>
> * `ADXL313 <https://www.analog.com/ADXL313>`_
>
> -The ADXL313is a low noise density, low power, 3-axis accelerometer with
> +The ADXL313 is a low noise density, low power, 3-axis accelerometer with
> selectable measurement ranges. The ADXL313 supports the ±0.5 g, ±1 g, ±2 g and
> ±4 g ranges.
>
> @@ -112,7 +112,7 @@ apply the following formula:
> Where _offset and _scale are device attributes. If no _offset attribute is
> present, simply assume its value is 0.
>
> -The ADXL313 driver offers data for a single types of channels, the table below
> +The ADXL313 driver offers data for a single type of channels, the table below
ooh. That's fun grammar. Intent is that there are multiple channels of one type
but the form used in your corrected version is at best unwieldy given plural
singular mix. "a single type of channel" is fine for instance but this isn't really.
I'd rewrite as
The ADXL313 driver offers data for multiple channels of a single type. The table
below...
(so split he thing up into two sentences as well).
> shows the measurement units for the processed value, which are defined by the
> IIO framework:
>
next prev parent reply other threads:[~2026-05-04 16:43 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-05-02 6:07 [PATCH v2] iio: adxl313: fix typos in documentation Wang Zihan
2026-05-02 12:45 ` Joshua Crofts
2026-05-04 16:43 ` Jonathan Cameron [this message]
2026-05-04 17:21 ` Joshua Crofts
-- strict thread matches above, loose matches on Subject: below --
2026-05-02 12:53 Wang Zihan
2026-05-04 16:38 ` Jonathan Cameron
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20260504174326.353b6f02@jic23-huawei \
--to=jic23@kernel.org \
--cc=jiyu03@qq.com \
--cc=linux-doc@vger.kernel.org \
--cc=linux-iio@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox