From: Tom Parkin <tparkin@katalix.com>
To: Siddhant Gupta <siddhantgupta416@gmail.com>
Cc: davem@davemloft.net, kuba@kernel.org, corbet@lwn.net,
netdev@vger.kernel.org, linux-doc@vger.kernel.org,
linux-kernel@vger.kernel.org, mamtashukla555@gmail.com,
himadrispandya@gmail.com
Subject: Re: [PATCH] Documentation: networking: Fix Column span alignment warnings in l2tp.rst
Date: Wed, 18 Nov 2020 10:23:07 +0000 [thread overview]
Message-ID: <20201118102307.GA4903@katalix.com> (raw)
In-Reply-To: <20201117095207.GA16407@Sleakybeast>
[-- Attachment #1: Type: text/plain, Size: 4104 bytes --]
On Tue, Nov 17, 2020 at 15:22:07 +0530, Siddhant Gupta wrote:
> Fix Column span alignment problem warnings in the file
>
Thanks for the patch, Siddhant.
Could you provide some information on how these warnings were
triggered? Using Sphinx 2.4.4 I can't reproduce any warnings for
l2tp.rst using the "make htmldocs" target.
> Signed-off-by: Siddhant Gupta <siddhantgupta416@gmail.com>
> ---
> Documentation/networking/l2tp.rst | 26 +++++++++++++-------------
> 1 file changed, 13 insertions(+), 13 deletions(-)
>
> diff --git a/Documentation/networking/l2tp.rst b/Documentation/networking/l2tp.rst
> index 498b382d25a0..0c0ac4e70586 100644
> --- a/Documentation/networking/l2tp.rst
> +++ b/Documentation/networking/l2tp.rst
> @@ -171,7 +171,8 @@ DEBUG N Debug flags.
> ================== ======== ===
> Attribute Required Use
> ================== ======== ===
> -CONN_ID N Identifies the tunnel id to be queried.
> +CONN_ID N Identifies the tunnel id
> + to be queried.
> Ignored in DUMP requests.
> ================== ======== ===
>
> @@ -208,8 +209,8 @@ onto the new session. This is covered in "PPPoL2TP Sockets" later.
> ================== ======== ===
> Attribute Required Use
> ================== ======== ===
> -CONN_ID Y Identifies the parent tunnel id of the session
> - to be destroyed.
> +CONN_ID Y Identifies the parent tunnel id
> + of the session to be destroyed.
> SESSION_ID Y Identifies the session id to be destroyed.
> IFNAME N Identifies the session by interface name. If
> set, this overrides any CONN_ID and SESSION_ID
> @@ -222,13 +223,12 @@ IFNAME N Identifies the session by interface name. If
> ================== ======== ===
> Attribute Required Use
> ================== ======== ===
> -CONN_ID Y Identifies the parent tunnel id of the session
> - to be modified.
> +CONN_ID Y Identifies the parent tunnel
> + id of the session to be modified.
> SESSION_ID Y Identifies the session id to be modified.
> -IFNAME N Identifies the session by interface name. If
> - set, this overrides any CONN_ID and SESSION_ID
> - attributes. Currently supported for L2TPv3
> - Ethernet sessions only.
> +IFNAME N Identifies the session by interface name. If set,
> + this overrides any CONN_ID and SESSION_ID
> + attributes. Currently supported for L2TPv3 Ethernet sessions only.
> DEBUG N Debug flags.
> RECV_SEQ N Enable rx data sequence numbers.
> SEND_SEQ N Enable tx data sequence numbers.
> @@ -243,10 +243,10 @@ RECV_TIMEOUT N Timeout to wait when reordering received
> ================== ======== ===
> Attribute Required Use
> ================== ======== ===
> -CONN_ID N Identifies the tunnel id to be queried.
> - Ignored for DUMP requests.
> -SESSION_ID N Identifies the session id to be queried.
> - Ignored for DUMP requests.
> +CONN_ID N Identifies the tunnel id
> + to be queried. Ignored for DUMP requests.
> +SESSION_ID N Identifies the session id
> + to be queried. Ignored for DUMP requests.
> IFNAME N Identifies the session by interface name.
> If set, this overrides any CONN_ID and
> SESSION_ID attributes. Ignored for DUMP
> --
> 2.25.1
>
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
next prev parent reply other threads:[~2020-11-18 10:33 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-11-17 9:52 [PATCH] Documentation: networking: Fix Column span alignment warnings in l2tp.rst Siddhant Gupta
2020-11-18 10:23 ` Tom Parkin [this message]
2020-11-18 11:14 ` siddhant gupta
2020-11-18 20:09 ` Jakub Kicinski
2020-11-19 9:45 ` Tom Parkin
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20201118102307.GA4903@katalix.com \
--to=tparkin@katalix.com \
--cc=corbet@lwn.net \
--cc=davem@davemloft.net \
--cc=himadrispandya@gmail.com \
--cc=kuba@kernel.org \
--cc=linux-doc@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=mamtashukla555@gmail.com \
--cc=netdev@vger.kernel.org \
--cc=siddhantgupta416@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.