From: Mauro Carvalho Chehab <mchehab@kernel.org>
To: "Daniel W. S. Almeida" <dwlsalmeida@gmail.com>
Cc: linux-media@vger.kernel.org,
linux-kernel-mentees@lists.linuxfoundation.org,
p.zabel@pengutronix.de
Subject: Re: [Linux-kernel-mentees] [PATCH] Documentation: media: dtv-frontend.rst: fix a few minor typos
Date: Sun, 1 Dec 2019 08:57:41 +0100 [thread overview]
Message-ID: <20191201085741.67d9389c@kernel.org> (raw)
In-Reply-To: <20191126164051.39920-1-dwlsalmeida@gmail.com>
Em Tue, 26 Nov 2019 13:40:51 -0300
"Daniel W. S. Almeida" <dwlsalmeida@gmail.com> escreveu:
> From: "Daniel W. S. Almeida" <dwlsalmeida@gmail.com>
>
> Fix a few minor typos throughout the document without changing the
> meaning of the sentences.
>
> Signed-off-by: Daniel W. S. Almeida <dwlsalmeida@gmail.com>
Looks OK on my eyes.
> ---
> Documentation/media/kapi/dtv-frontend.rst | 16 ++++++++--------
> 1 file changed, 8 insertions(+), 8 deletions(-)
>
> diff --git a/Documentation/media/kapi/dtv-frontend.rst b/Documentation/media/kapi/dtv-frontend.rst
> index fbc5517c8d5a..b362109bb131 100644
> --- a/Documentation/media/kapi/dtv-frontend.rst
> +++ b/Documentation/media/kapi/dtv-frontend.rst
> @@ -15,8 +15,8 @@ The header file for this API is named ``dvb_frontend.h`` and located in
> Demodulator driver
> ^^^^^^^^^^^^^^^^^^
>
> -The demodulator driver is responsible to talk with the decoding part of the
> -hardware. Such driver should implement :c:type:`dvb_frontend_ops`, with
> +The demodulator driver is responsible for talking with the decoding part of the
> +hardware. Such driver should implement :c:type:`dvb_frontend_ops`, which
> tells what type of digital TV standards are supported, and points to a
> series of functions that allow the DVB core to command the hardware via
> the code under ``include/media/dvb_frontend.c``.
> @@ -120,7 +120,7 @@ Satellite TV reception is::
>
> .. |delta| unicode:: U+00394
>
> -The ``include/media/dvb_frontend.c`` has a kernel thread with is
> +The ``include/media/dvb_frontend.c`` has a kernel thread which is
> responsible for tuning the device. It supports multiple algorithms to
> detect a channel, as defined at enum :c:func:`dvbfe_algo`.
>
> @@ -220,11 +220,11 @@ Signal strength (:ref:`DTV-STAT-SIGNAL-STRENGTH`)
> - As the gain is visible through the set of registers that adjust the gain,
> typically, this statistics is always available [#f3]_.
>
> - - Drivers should try to make it available all the times, as this statistics
> + - Drivers should try to make it available all the times, as these statistics
> can be used when adjusting an antenna position and to check for troubles
> at the cabling.
>
> - .. [#f3] On a few devices, the gain keeps floating if no carrier.
> + .. [#f3] On a few devices, the gain keeps floating if there is no carrier.
> On such devices, strength report should check first if carrier is
> detected at the tuner (``FE_HAS_CARRIER``, see :c:type:`fe_status`),
> and otherwise return the lowest possible value.
> @@ -232,7 +232,7 @@ Signal strength (:ref:`DTV-STAT-SIGNAL-STRENGTH`)
> Carrier Signal to Noise ratio (:ref:`DTV-STAT-CNR`)
> - Signal to Noise ratio for the main carrier.
>
> - - Signal to Noise measurement depends on the device. On some hardware, is
> + - Signal to Noise measurement depends on the device. On some hardware, it is
> available when the main carrier is detected. On those hardware, CNR
> measurement usually comes from the tuner (e. g. after ``FE_HAS_CARRIER``,
> see :c:type:`fe_status`).
> @@ -323,8 +323,8 @@ A typical example of the logic that handle status and statistics is::
> .read_status = foo_get_status_and_stats,
> };
>
> -Statistics collect
> -^^^^^^^^^^^^^^^^^^
> +Statistics collection
> +^^^^^^^^^^^^^^^^^^^^^
>
> On almost all frontend hardware, the bit and byte counts are stored by
> the hardware after a certain amount of time or after the total bit/block
Cheers,
Mauro
_______________________________________________
Linux-kernel-mentees mailing list
Linux-kernel-mentees@lists.linuxfoundation.org
https://lists.linuxfoundation.org/mailman/listinfo/linux-kernel-mentees
WARNING: multiple messages have this Message-ID (diff)
From: Mauro Carvalho Chehab <mchehab@kernel.org>
To: "Daniel W. S. Almeida" <dwlsalmeida@gmail.com>
Cc: p.zabel@pengutronix.de, skhan@linuxfoundation.org,
linux-kernel-mentees@lists.linuxfoundation.org,
linux-media@vger.kernel.org
Subject: Re: [PATCH] Documentation: media: dtv-frontend.rst: fix a few minor typos
Date: Sun, 1 Dec 2019 08:57:41 +0100 [thread overview]
Message-ID: <20191201085741.67d9389c@kernel.org> (raw)
In-Reply-To: <20191126164051.39920-1-dwlsalmeida@gmail.com>
Em Tue, 26 Nov 2019 13:40:51 -0300
"Daniel W. S. Almeida" <dwlsalmeida@gmail.com> escreveu:
> From: "Daniel W. S. Almeida" <dwlsalmeida@gmail.com>
>
> Fix a few minor typos throughout the document without changing the
> meaning of the sentences.
>
> Signed-off-by: Daniel W. S. Almeida <dwlsalmeida@gmail.com>
Looks OK on my eyes.
> ---
> Documentation/media/kapi/dtv-frontend.rst | 16 ++++++++--------
> 1 file changed, 8 insertions(+), 8 deletions(-)
>
> diff --git a/Documentation/media/kapi/dtv-frontend.rst b/Documentation/media/kapi/dtv-frontend.rst
> index fbc5517c8d5a..b362109bb131 100644
> --- a/Documentation/media/kapi/dtv-frontend.rst
> +++ b/Documentation/media/kapi/dtv-frontend.rst
> @@ -15,8 +15,8 @@ The header file for this API is named ``dvb_frontend.h`` and located in
> Demodulator driver
> ^^^^^^^^^^^^^^^^^^
>
> -The demodulator driver is responsible to talk with the decoding part of the
> -hardware. Such driver should implement :c:type:`dvb_frontend_ops`, with
> +The demodulator driver is responsible for talking with the decoding part of the
> +hardware. Such driver should implement :c:type:`dvb_frontend_ops`, which
> tells what type of digital TV standards are supported, and points to a
> series of functions that allow the DVB core to command the hardware via
> the code under ``include/media/dvb_frontend.c``.
> @@ -120,7 +120,7 @@ Satellite TV reception is::
>
> .. |delta| unicode:: U+00394
>
> -The ``include/media/dvb_frontend.c`` has a kernel thread with is
> +The ``include/media/dvb_frontend.c`` has a kernel thread which is
> responsible for tuning the device. It supports multiple algorithms to
> detect a channel, as defined at enum :c:func:`dvbfe_algo`.
>
> @@ -220,11 +220,11 @@ Signal strength (:ref:`DTV-STAT-SIGNAL-STRENGTH`)
> - As the gain is visible through the set of registers that adjust the gain,
> typically, this statistics is always available [#f3]_.
>
> - - Drivers should try to make it available all the times, as this statistics
> + - Drivers should try to make it available all the times, as these statistics
> can be used when adjusting an antenna position and to check for troubles
> at the cabling.
>
> - .. [#f3] On a few devices, the gain keeps floating if no carrier.
> + .. [#f3] On a few devices, the gain keeps floating if there is no carrier.
> On such devices, strength report should check first if carrier is
> detected at the tuner (``FE_HAS_CARRIER``, see :c:type:`fe_status`),
> and otherwise return the lowest possible value.
> @@ -232,7 +232,7 @@ Signal strength (:ref:`DTV-STAT-SIGNAL-STRENGTH`)
> Carrier Signal to Noise ratio (:ref:`DTV-STAT-CNR`)
> - Signal to Noise ratio for the main carrier.
>
> - - Signal to Noise measurement depends on the device. On some hardware, is
> + - Signal to Noise measurement depends on the device. On some hardware, it is
> available when the main carrier is detected. On those hardware, CNR
> measurement usually comes from the tuner (e. g. after ``FE_HAS_CARRIER``,
> see :c:type:`fe_status`).
> @@ -323,8 +323,8 @@ A typical example of the logic that handle status and statistics is::
> .read_status = foo_get_status_and_stats,
> };
>
> -Statistics collect
> -^^^^^^^^^^^^^^^^^^
> +Statistics collection
> +^^^^^^^^^^^^^^^^^^^^^
>
> On almost all frontend hardware, the bit and byte counts are stored by
> the hardware after a certain amount of time or after the total bit/block
Cheers,
Mauro
next prev parent reply other threads:[~2019-12-01 7:57 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-11-26 16:40 [Linux-kernel-mentees] [PATCH] Documentation: media: dtv-frontend.rst: fix a few minor typos Daniel W. S. Almeida
2019-11-26 16:40 ` Daniel W. S. Almeida
2019-12-01 7:57 ` Mauro Carvalho Chehab [this message]
2019-12-01 7:57 ` Mauro Carvalho Chehab
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=20191201085741.67d9389c@kernel.org \
--to=mchehab@kernel.org \
--cc=dwlsalmeida@gmail.com \
--cc=linux-kernel-mentees@lists.linuxfoundation.org \
--cc=linux-media@vger.kernel.org \
--cc=p.zabel@pengutronix.de \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.