diff mbox series

doc: Fix malformed table in gadget-testing.rst

Message ID 20240128195731.40003-1-kohshi54.yam@gmail.com (mailing list archive)
State New
Headers show
Series doc: Fix malformed table in gadget-testing.rst | expand

Commit Message

Kohshi Yamaguchi Jan. 28, 2024, 7:57 p.m. UTC
The table describing the NCM function attributes in gadget-testing.rst
was malformed, causing Sphinx build warnings. This patch fixes the table
format to align with the reStructuredText specifications.

The border lines of the table were not properly aligned with the column
headers, which was corrected by adjusting the border line lengths to
match the headers.

Signed-off-by: Kohshi Yamaguchi <kohshi54.yam@gmail.com>
---
 Documentation/usb/gadget-testing.rst | 22 +++++++++++-----------
 1 file changed, 11 insertions(+), 11 deletions(-)

Comments

Randy Dunlap Jan. 29, 2024, 2:59 a.m. UTC | #1
Hi,

On 1/28/24 11:57, Kohshi Yamaguchi wrote:
> The table describing the NCM function attributes in gadget-testing.rst
> was malformed, causing Sphinx build warnings. This patch fixes the table
> format to align with the reStructuredText specifications.
> 
> The border lines of the table were not properly aligned with the column
> headers, which was corrected by adjusting the border line lengths to
> match the headers.
> 
> Signed-off-by: Kohshi Yamaguchi <kohshi54.yam@gmail.com>
> ---
>  Documentation/usb/gadget-testing.rst | 22 +++++++++++-----------
>  1 file changed, 11 insertions(+), 11 deletions(-)
> 
> diff --git a/Documentation/usb/gadget-testing.rst b/Documentation/usb/gadget-testing.rst
> index 8cd62c466d20..1d1737f246df 100644
> --- a/Documentation/usb/gadget-testing.rst
> +++ b/Documentation/usb/gadget-testing.rst
> @@ -448,17 +448,17 @@ Function-specific configfs interface
>  The function name to use when creating the function directory is "ncm".
>  The NCM function provides these attributes in its function directory:
>  
> -	===============   ==================================================
> -	ifname		  network device interface name associated with this
> -			  function instance
> -	qmult		  queue length multiplier for high and super speed
> -	host_addr	  MAC address of host's end of this
> -			  Ethernet over USB link
> -	dev_addr	  MAC address of device's end of this
> -			  Ethernet over USB link
> -	max_segment_size  Segment size required for P2P connections. This
> -			  will set MTU to (max_segment_size - 14 bytes)
> -	===============   ==================================================
> +	================   ==================================================
> +	ifname		   network device interface name associated with this
> +			   function instance
> +	qmult		   queue length multiplier for high and super speed
> +	host_addr	   MAC address of host's end of this
> +			   Ethernet over USB link
> +	dev_addr	   MAC address of device's end of this
> +			   Ethernet over USB link
> +	max_segment_size   Segment size required for P2P connections. This
> +			   will set MTU to (max_segment_size - 14 bytes)
> +	================   ==================================================
>  
>  and after creating the functions/ncm.<instance name> they contain default
>  values: qmult is 5, dev_addr and host_addr are randomly selected.

or like this:

https://lore.kernel.org/lkml/20240128045347.25909-1-rdunlap@infradead.org/

Thanks.
Kohshi Yamaguchi Jan. 29, 2024, 7:50 a.m. UTC | #2
Hi Randy,

Thank you for reviewing my patch and pointing out the similar change you posted.
I appreciate your feedback.

Best regards,
--
Kohshi Yamaguchi
diff mbox series

Patch

diff --git a/Documentation/usb/gadget-testing.rst b/Documentation/usb/gadget-testing.rst
index 8cd62c466d20..1d1737f246df 100644
--- a/Documentation/usb/gadget-testing.rst
+++ b/Documentation/usb/gadget-testing.rst
@@ -448,17 +448,17 @@  Function-specific configfs interface
 The function name to use when creating the function directory is "ncm".
 The NCM function provides these attributes in its function directory:
 
-	===============   ==================================================
-	ifname		  network device interface name associated with this
-			  function instance
-	qmult		  queue length multiplier for high and super speed
-	host_addr	  MAC address of host's end of this
-			  Ethernet over USB link
-	dev_addr	  MAC address of device's end of this
-			  Ethernet over USB link
-	max_segment_size  Segment size required for P2P connections. This
-			  will set MTU to (max_segment_size - 14 bytes)
-	===============   ==================================================
+	================   ==================================================
+	ifname		   network device interface name associated with this
+			   function instance
+	qmult		   queue length multiplier for high and super speed
+	host_addr	   MAC address of host's end of this
+			   Ethernet over USB link
+	dev_addr	   MAC address of device's end of this
+			   Ethernet over USB link
+	max_segment_size   Segment size required for P2P connections. This
+			   will set MTU to (max_segment_size - 14 bytes)
+	================   ==================================================
 
 and after creating the functions/ncm.<instance name> they contain default
 values: qmult is 5, dev_addr and host_addr are randomly selected.