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

From: Randy Dunlap
Date: Sun Jan 28 2024 - 21:59:38 EST


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@xxxxxxxxx>
> ---
> 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@xxxxxxxxxxxxx/

Thanks.
--
#Randy