Linux Documentation
 help / color / mirror / Atom feed
From: Jonathan Cameron <jic23@kernel.org>
To: Bagas Sanjaya <bagasdotme@gmail.com>
Cc: linux-iio@vger.kernel.org, linux-kernel@vger.kernel.org,
	linux-doc@vger.kernel.org, Lars-Peter Clausen <lars@metafoo.de>,
	Cosmin Tanislav <cosmin.tanislav@analog.com>,
	Andy Shevchenko <andy.shevchenko@gmail.com>
Subject: Re: [PATCH] Documentation: ad4130: format list of in_voltage-voltage_filter_mode_available modes
Date: Sun, 6 Nov 2022 17:28:48 +0000	[thread overview]
Message-ID: <20221106172848.374831a5@jic23-huawei> (raw)
In-Reply-To: <20221106140233.74112-1-bagasdotme@gmail.com>

On Sun,  6 Nov 2022 21:02:33 +0700
Bagas Sanjaya <bagasdotme@gmail.com> wrote:

> Sphinx reports two warnings on sysfs documentation for AD4130 driver:
> 
> Documentation/ABI/testing/sysfs-bus-iio-adc-ad4130:2: WARNING: Unexpected indentation.
> Documentation/ABI/testing/sysfs-bus-iio-adc-ad4130:2: WARNING: Block quote ends without a blank line; unexpected unindent.
> 
> These are due to misformatting of sinc* modes list. Format it with bullet
> list. Since each entry spans multiple lines, separate each with a blank
> line.
> 
> Fixes: 5bdef39c5c6e1a ("iio: adc: ad4130: add AD4130 driver")
> Signed-off-by: Bagas Sanjaya <bagasdotme@gmail.com>

Applied to the togreg branch of iio.git and pushed out initially as testing for
the autobuilders to look at it (or at least look at the other stuff that I've
applied on that branch today).

Thanks!

Jonathan

> ---
>  .../ABI/testing/sysfs-bus-iio-adc-ad4130      | 48 +++++++++++--------
>  1 file changed, 29 insertions(+), 19 deletions(-)
> 
> diff --git a/Documentation/ABI/testing/sysfs-bus-iio-adc-ad4130 b/Documentation/ABI/testing/sysfs-bus-iio-adc-ad4130
> index d9555751d21ce2..f24ed6687e900e 100644
> --- a/Documentation/ABI/testing/sysfs-bus-iio-adc-ad4130
> +++ b/Documentation/ABI/testing/sysfs-bus-iio-adc-ad4130
> @@ -3,25 +3,35 @@ KernelVersion:  6.2
>  Contact:	linux-iio@vger.kernel.org
>  Description:
>  		Reading returns a list with the possible filter modes.
> -		"sinc4"       - Sinc 4. Excellent noise performance. Long 1st
> -				conversion time. No natural 50/60Hz rejection.
> -		"sinc4+sinc1" - Sinc4 + averaging by 8. Low 1st conversion time.
> -		"sinc3"	      - Sinc3. Moderate 1st conversion time. Good noise
> -				performance.
> -		"sinc3+rej60" - Sinc3 + 60Hz rejection. At a sampling frequency
> -				of 50Hz, achieves simultaneous 50Hz and 60Hz
> -				rejection.
> -		"sinc3+sinc1" - Sinc3 + averaging by 8. Low 1st conversion time.
> -				Best used with a sampling frequency of at least
> -				216.19Hz.
> -		"sinc3+pf1"   - Sinc3 + Post Filter 1.
> -				53dB rejection @ 50Hz, 58dB rejection @ 60Hz.
> -		"sinc3+pf2"   - Sinc3 + Post Filter 2.
> -				70dB rejection @ 50Hz, 70dB rejection @ 60Hz.
> -		"sinc3+pf3"   - Sinc3 + Post Filter 3.
> -				99dB rejection @ 50Hz, 103dB rejection @ 60Hz.
> -		"sinc3+pf4"   - Sinc3 + Post Filter 4.
> -				103dB rejection @ 50Hz, 109dB rejection @ 60Hz.
> +
> +		  * "sinc4"       - Sinc 4. Excellent noise performance. Long
> +                    1st conversion time. No natural 50/60Hz rejection.
> +
> +		  * "sinc4+sinc1" - Sinc4 + averaging by 8. Low 1st conversion
> +		    time.
> +
> +		  * "sinc3"	      - Sinc3. Moderate 1st conversion time.
> +		    Good noise performance.
> +
> +		  * "sinc3+rej60" - Sinc3 + 60Hz rejection. At a sampling
> +		    frequency of 50Hz, achieves simultaneous 50Hz and 60Hz
> +		    rejection.
> +
> +		  * "sinc3+sinc1" - Sinc3 + averaging by 8. Low 1st conversion
> +		    time. Best used with a sampling frequency of at least
> +		    216.19Hz.
> +
> +		  * "sinc3+pf1"   - Sinc3 + Post Filter 1. 53dB rejection @
> +		    50Hz, 58dB rejection @ 60Hz.
> +
> +		  * "sinc3+pf2"   - Sinc3 + Post Filter 2. 70dB rejection @
> +		    50Hz, 70dB rejection @ 60Hz.
> +
> +		  * "sinc3+pf3"   - Sinc3 + Post Filter 3. 99dB rejection @
> +		    50Hz, 103dB rejection @ 60Hz.
> +
> +		  * "sinc3+pf4"   - Sinc3 + Post Filter 4. 103dB rejection @
> +		    50Hz, 109dB rejection @ 60Hz.
>  
>  What:		/sys/bus/iio/devices/iio:deviceX/in_voltageY-voltageZ_filter_mode
>  KernelVersion:  6.2
> 
> base-commit: b8bee0f98634cc1ce8cc8bf0e1db025734ee0c2b


      reply	other threads:[~2022-11-06 17:29 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-11-06 14:02 [PATCH] Documentation: ad4130: format list of in_voltage-voltage_filter_mode_available modes Bagas Sanjaya
2022-11-06 17:28 ` Jonathan Cameron [this message]

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=20221106172848.374831a5@jic23-huawei \
    --to=jic23@kernel.org \
    --cc=andy.shevchenko@gmail.com \
    --cc=bagasdotme@gmail.com \
    --cc=cosmin.tanislav@analog.com \
    --cc=lars@metafoo.de \
    --cc=linux-doc@vger.kernel.org \
    --cc=linux-iio@vger.kernel.org \
    --cc=linux-kernel@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