Linux Kernel Mentees Archive on lore.kernel.org
 help / color / Atom feed
* [Linux-kernel-mentees] [PATCH] Documentation: media: dtv-frontend.rst: fix a few minor typos
@ 2019-11-26 16:40 Daniel W. S. Almeida
  2019-12-01  7:57 ` Mauro Carvalho Chehab
  0 siblings, 1 reply; 2+ messages in thread
From: Daniel W. S. Almeida @ 2019-11-26 16:40 UTC (permalink / raw)
  To: mchehab, p.zabel; +Cc: linux-media, linux-kernel-mentees, Daniel W. S. Almeida

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>
---
 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
-- 
2.24.0

_______________________________________________
Linux-kernel-mentees mailing list
Linux-kernel-mentees@lists.linuxfoundation.org
https://lists.linuxfoundation.org/mailman/listinfo/linux-kernel-mentees

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [Linux-kernel-mentees] [PATCH] Documentation: media: dtv-frontend.rst: fix a few minor typos
  2019-11-26 16:40 [Linux-kernel-mentees] [PATCH] Documentation: media: dtv-frontend.rst: fix a few minor typos Daniel W. S. Almeida
@ 2019-12-01  7:57 ` Mauro Carvalho Chehab
  0 siblings, 0 replies; 2+ messages in thread
From: Mauro Carvalho Chehab @ 2019-12-01  7:57 UTC (permalink / raw)
  To: Daniel W. S. Almeida; +Cc: linux-media, linux-kernel-mentees, p.zabel

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

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, back to index

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-11-26 16:40 [Linux-kernel-mentees] [PATCH] Documentation: media: dtv-frontend.rst: fix a few minor typos Daniel W. S. Almeida
2019-12-01  7:57 ` Mauro Carvalho Chehab

Linux Kernel Mentees Archive on lore.kernel.org

Archives are clonable:
	git clone --mirror https://lore.kernel.org/linux-kernel-mentees/0 linux-kernel-mentees/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 linux-kernel-mentees linux-kernel-mentees/ https://lore.kernel.org/linux-kernel-mentees \
		linux-kernel-mentees@lists.linuxfoundation.org
	public-inbox-index linux-kernel-mentees

Example config snippet for mirrors

Newsgroup available over NNTP:
	nntp://nntp.lore.kernel.org/org.linuxfoundation.lists.linux-kernel-mentees


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git