public inbox for linux-usb@vger.kernel.org
 help / color / mirror / Atom feed
* [PATCH] docs: usb: fix literal block marker in usbmon verification example
@ 2022-06-04 15:54 Justin Swartz
  2022-06-05  8:39 ` Bagas Sanjaya
  2022-06-09 15:58 ` Jonathan Corbet
  0 siblings, 2 replies; 5+ messages in thread
From: Justin Swartz @ 2022-06-04 15:54 UTC (permalink / raw)
  Cc: Justin Swartz, Greg Kroah-Hartman, Jonathan Corbet, linux-usb,
	linux-doc, linux-kernel

The "Verify that bus sockets are present" example was not properly
formatted due to a typo in the literal block marker.

Signed-off-by: Justin Swartz <justin.swartz@risingedge.co.za>
---
 Documentation/usb/usbmon.rst | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Documentation/usb/usbmon.rst b/Documentation/usb/usbmon.rst
index b0bd51080..6d5ec1e62 100644
--- a/Documentation/usb/usbmon.rst
+++ b/Documentation/usb/usbmon.rst
@@ -42,7 +42,7 @@ if usbmon is built into the kernel::
 	# modprobe usbmon
 	#
 
-Verify that bus sockets are present:
+Verify that bus sockets are present::
 
 	# ls /sys/kernel/debug/usb/usbmon
 	0s  0u  1s  1t  1u  2s  2t  2u  3s  3t  3u  4s  4t  4u
-- 
2.30.2


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

* Re: [PATCH] docs: usb: fix literal block marker in usbmon verification example
  2022-06-04 15:54 [PATCH] docs: usb: fix literal block marker in usbmon verification example Justin Swartz
@ 2022-06-05  8:39 ` Bagas Sanjaya
  2022-06-05  8:58   ` Justin Swartz
  2022-06-09 15:58 ` Jonathan Corbet
  1 sibling, 1 reply; 5+ messages in thread
From: Bagas Sanjaya @ 2022-06-05  8:39 UTC (permalink / raw)
  To: Justin Swartz
  Cc: Greg Kroah-Hartman, Jonathan Corbet, linux-usb, linux-doc,
	linux-kernel

On 6/4/22 22:54, Justin Swartz wrote:
> The "Verify that bus sockets are present" example was not properly
> formatted due to a typo in the literal block marker.
> 

Missing second colon?
iff --git a/Documentation/usb/usbmon.rst b/Documentation/usb/usbmon.rst
> index b0bd51080..6d5ec1e62 100644
> --- a/Documentation/usb/usbmon.rst
> +++ b/Documentation/usb/usbmon.rst
> @@ -42,7 +42,7 @@ if usbmon is built into the kernel::
>  	# modprobe usbmon
>  	#
>  
> -Verify that bus sockets are present:
> +Verify that bus sockets are present::
>  
>  	# ls /sys/kernel/debug/usb/usbmon
>  	0s  0u  1s  1t  1u  2s  2t  2u  3s  3t  3u  4s  4t  4u

Otherwise, the literal block rendered correctly.

Reviewed-by: Bagas Sanjaya <bagasdotme@gmail.com>

-- 
An old man doll... just what I always wanted! - Clara

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

* Re: [PATCH] docs: usb: fix literal block marker in usbmon verification example
  2022-06-05  8:39 ` Bagas Sanjaya
@ 2022-06-05  8:58   ` Justin Swartz
  2022-06-05  9:40     ` Bagas Sanjaya
  0 siblings, 1 reply; 5+ messages in thread
From: Justin Swartz @ 2022-06-05  8:58 UTC (permalink / raw)
  To: Bagas Sanjaya
  Cc: Greg Kroah-Hartman, Jonathan Corbet, linux-usb, linux-doc,
	linux-kernel

On 2022-06-05 10:39, Bagas Sanjaya wrote:
> On 6/4/22 22:54, Justin Swartz wrote:
>> The "Verify that bus sockets are present" example was not properly
>> formatted due to a typo in the literal block marker.
>> 
> 
> Missing second colon?

That is correct.


>> -Verify that bus sockets are present:
>> +Verify that bus sockets are present::
>> 
>>  	# ls /sys/kernel/debug/usb/usbmon
>>  	0s  0u  1s  1t  1u  2s  2t  2u  3s  3t  3u  4s  4t  4u
> 
> Otherwise, the literal block rendered correctly.

The block is not rendered correctly without the second colon.
See: https://docs.kernel.org/usb/usbmon.html

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

* Re: [PATCH] docs: usb: fix literal block marker in usbmon verification example
  2022-06-05  8:58   ` Justin Swartz
@ 2022-06-05  9:40     ` Bagas Sanjaya
  0 siblings, 0 replies; 5+ messages in thread
From: Bagas Sanjaya @ 2022-06-05  9:40 UTC (permalink / raw)
  To: Justin Swartz
  Cc: Greg Kroah-Hartman, Jonathan Corbet, linux-usb, linux-doc,
	linux-kernel

On 6/5/22 15:58, Justin Swartz wrote:
> On 2022-06-05 10:39, Bagas Sanjaya wrote:
>> On 6/4/22 22:54, Justin Swartz wrote:
>>> The "Verify that bus sockets are present" example was not properly
>>> formatted due to a typo in the literal block marker.
>>>
>>
>> Missing second colon?
> 
> That is correct.
> 
> 
>>> -Verify that bus sockets are present:
>>> +Verify that bus sockets are present::
>>>
>>>      # ls /sys/kernel/debug/usb/usbmon
>>>      0s  0u  1s  1t  1u  2s  2t  2u  3s  3t  3u  4s  4t  4u
>>
>> Otherwise, the literal block rendered correctly.
> 
> The block is not rendered correctly without the second colon.
> See: https://docs.kernel.org/usb/usbmon.html

Thanks for the explanation.

I mean that for the literal block part, it renders correctly with
the second colon (which this patch is about), so the documentation
error is fixed (although not reported by Sphinx when performing
htmldocs build).

So for this patch, I gave
Reviewed-by: Bagas Sanjaya <bagasdotme@gmail.com>

-- 
An old man doll... just what I always wanted! - Clara

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

* Re: [PATCH] docs: usb: fix literal block marker in usbmon verification example
  2022-06-04 15:54 [PATCH] docs: usb: fix literal block marker in usbmon verification example Justin Swartz
  2022-06-05  8:39 ` Bagas Sanjaya
@ 2022-06-09 15:58 ` Jonathan Corbet
  1 sibling, 0 replies; 5+ messages in thread
From: Jonathan Corbet @ 2022-06-09 15:58 UTC (permalink / raw)
  To: Justin Swartz
  Cc: Justin Swartz, Greg Kroah-Hartman, linux-usb, linux-doc,
	linux-kernel

Justin Swartz <justin.swartz@risingedge.co.za> writes:

> The "Verify that bus sockets are present" example was not properly
> formatted due to a typo in the literal block marker.
>
> Signed-off-by: Justin Swartz <justin.swartz@risingedge.co.za>
> ---
>  Documentation/usb/usbmon.rst | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/Documentation/usb/usbmon.rst b/Documentation/usb/usbmon.rst
> index b0bd51080..6d5ec1e62 100644
> --- a/Documentation/usb/usbmon.rst
> +++ b/Documentation/usb/usbmon.rst
> @@ -42,7 +42,7 @@ if usbmon is built into the kernel::
>  	# modprobe usbmon
>  	#
>  
> -Verify that bus sockets are present:
> +Verify that bus sockets are present::
>  

Applied, thanks.

jon

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

end of thread, other threads:[~2022-06-09 15:58 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2022-06-04 15:54 [PATCH] docs: usb: fix literal block marker in usbmon verification example Justin Swartz
2022-06-05  8:39 ` Bagas Sanjaya
2022-06-05  8:58   ` Justin Swartz
2022-06-05  9:40     ` Bagas Sanjaya
2022-06-09 15:58 ` Jonathan Corbet

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox