2024-01-28 19:57:57

by Kohshi Yamaguchi

[permalink] [raw]
Subject: [PATCH] doc: Fix malformed table in gadget-testing.rst

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 <[email protected]>
---
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.
--
2.34.1



2024-01-29 07:50:54

by Kohshi Yamaguchi

[permalink] [raw]
Subject: Re: [PATCH] doc: Fix malformed table in gadget-testing.rst

Hi Randy,

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

Best regards,
--
Kohshi Yamaguchi


2024-01-29 08:56:20

by Randy Dunlap

[permalink] [raw]
Subject: Re: [PATCH] doc: Fix malformed table in gadget-testing.rst

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 <[email protected]>
> ---
> 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/[email protected]/

Thanks.
--
#Randy