mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
* [PATCH] Documentation: net: dsa: Fix bracket
@ 2026-09-16 12:34 Manuel Ebner
  2026-09-16 12:34 ` [PATCH] docs: ethtool: " Manuel Ebner
                   ` (3 more replies)
  0 siblings, 4 replies; 8+ messages in thread
From: Manuel Ebner @ 2026-09-16 12:34 UTC (permalink / raw)
  To: David S. Miller, Eric Dumazet, Jakub Kicinski, Paolo Abeni,
	Simon Horman, Jonathan Corbet, Shuah Khan, Randy Dunlap
  Cc: Manuel Ebner, netdev, linux-doc, linux-kernel

Add missing ')'.

Signed-off-by: Manuel Ebner <manuelebnerli@mailbox.org>
---
 Documentation/networking/dsa/dsa.rst | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Documentation/networking/dsa/dsa.rst b/Documentation/networking/dsa/dsa.rst
index 42a99f5df..7edfdd555 100644
--- a/Documentation/networking/dsa/dsa.rst
+++ b/Documentation/networking/dsa/dsa.rst
@@ -864,7 +864,7 @@ The drivers which act upon the ``dsa_db`` argument in ``port_fdb_add``,
 DSA associates each offloaded bridge and each offloaded LAG with a one-based ID
 (``struct dsa_bridge :: num``, ``struct dsa_lag :: id``) for the purposes of
 refcounting addresses on shared ports. Drivers may piggyback on DSA's numbering
-scheme (the ID is readable through ``db->bridge.num`` and ``db->lag.id`` or may
+scheme (the ID is readable through ``db->bridge.num`` and ``db->lag.id``) or may
 implement their own.
 
 Only the drivers which declare support for FDB isolation are notified of FDB
-- 
2.55.0


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

* [PATCH] docs: ethtool: Fix bracket
  2026-09-16 12:34 [PATCH] Documentation: net: dsa: Fix bracket Manuel Ebner
@ 2026-09-16 12:34 ` Manuel Ebner
  2026-09-16 12:47   ` Andrew Lunn
  2026-09-16 12:34 ` [PATCH] docs: networking: radiotap: " Manuel Ebner
                   ` (2 subsequent siblings)
  3 siblings, 1 reply; 8+ messages in thread
From: Manuel Ebner @ 2026-09-16 12:34 UTC (permalink / raw)
  To: Andrew Lunn, Jakub Kicinski, David S. Miller, Eric Dumazet,
	Paolo Abeni, Simon Horman, Jonathan Corbet, Shuah Khan,
	Randy Dunlap
  Cc: Manuel Ebner, netdev, linux-doc, linux-kernel

Add missing ')' to sentence.

Signed-off-by: Manuel Ebner <manuelebnerli@mailbox.org>
---
 Documentation/networking/ethtool-netlink.rst | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Documentation/networking/ethtool-netlink.rst b/Documentation/networking/ethtool-netlink.rst
index e92abf45f..393f2b6d3 100644
--- a/Documentation/networking/ethtool-netlink.rst
+++ b/Documentation/networking/ethtool-netlink.rst
@@ -131,7 +131,7 @@ For compact form, ``ETHTOOL_A_BITSET_SIZE`` and ``ETHTOOL_A_BITSET_VALUE`` are
 mandatory. ``ETHTOOL_A_BITSET_MASK`` attribute is mandatory if
 ``ETHTOOL_A_BITSET_NOMASK`` is not set (bitset represents a value/mask pair);
 if ``ETHTOOL_A_BITSET_NOMASK`` is not set, ``ETHTOOL_A_BITSET_MASK`` is not
-allowed (bitset represents a single bitmap.
+allowed (bitset represents a single bitmap).
 
 Kernel bit set length may differ from userspace length if older application is
 used on newer kernel or vice versa. If userspace bitmap is longer, an error is
-- 
2.55.0


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

* [PATCH] docs: networking: radiotap: Fix bracket
  2026-09-16 12:34 [PATCH] Documentation: net: dsa: Fix bracket Manuel Ebner
  2026-09-16 12:34 ` [PATCH] docs: ethtool: " Manuel Ebner
@ 2026-09-16 12:34 ` Manuel Ebner
  2026-09-16 16:52   ` Randy Dunlap
  2026-09-16 12:35 ` [PATCH] scsi: docs: aic: Fix brackets Manuel Ebner
  2026-09-16 17:02 ` [PATCH] Documentation: net: dsa: Fix bracket Randy Dunlap
  3 siblings, 1 reply; 8+ messages in thread
From: Manuel Ebner @ 2026-09-16 12:34 UTC (permalink / raw)
  To: David S. Miller, Eric Dumazet, Jakub Kicinski, Paolo Abeni,
	Simon Horman, Jonathan Corbet, Shuah Khan, Randy Dunlap
  Cc: Manuel Ebner, netdev, linux-doc, linux-kernel

Remove needless ')' from code.

Signed-off-by: Manuel Ebner <manuelebnerli@mailbox.org>
---
 Documentation/networking/radiotap-headers.rst | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Documentation/networking/radiotap-headers.rst b/Documentation/networking/radiotap-headers.rst
index 1a1bd1ec0..aadc7bc6b 100644
--- a/Documentation/networking/radiotap-headers.rst
+++ b/Documentation/networking/radiotap-headers.rst
@@ -133,7 +133,7 @@ its prototypes available in include/net/cfg80211.h.  You use it like this::
 
 		    case IEEE80211_RADIOTAP_ANTENNA:
 			    /* radiotap uses 0 for 1st ant */
-			    antenna = *iterator.this_arg);
+			    antenna = *iterator.this_arg;
 			    break;
 
 		    case IEEE80211_RADIOTAP_DBM_TX_POWER:
-- 
2.55.0


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

* [PATCH] scsi: docs: aic: Fix brackets
  2026-09-16 12:34 [PATCH] Documentation: net: dsa: Fix bracket Manuel Ebner
  2026-09-16 12:34 ` [PATCH] docs: ethtool: " Manuel Ebner
  2026-09-16 12:34 ` [PATCH] docs: networking: radiotap: " Manuel Ebner
@ 2026-09-16 12:35 ` Manuel Ebner
  2026-09-16 16:58   ` Randy Dunlap
  2026-09-16 17:02 ` [PATCH] Documentation: net: dsa: Fix bracket Randy Dunlap
  3 siblings, 1 reply; 8+ messages in thread
From: Manuel Ebner @ 2026-09-16 12:35 UTC (permalink / raw)
  To: Jonathan Corbet, Shuah Khan, Randy Dunlap
  Cc: Manuel Ebner, linux-doc, linux-kernel

Add missing '}' to lines of code.

Signed-off-by: Manuel Ebner <manuelebnerli@mailbox.org>
---
Would it be better to remove the '{' and replace '{{'/'}}' with '{'/'}'?
---
 Documentation/scsi/aic79xx.rst | 2 +-
 Documentation/scsi/aic7xxx.rst | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/Documentation/scsi/aic79xx.rst b/Documentation/scsi/aic79xx.rst
index 071ff5111..ba5299cb9 100644
--- a/Documentation/scsi/aic79xx.rst
+++ b/Documentation/scsi/aic79xx.rst
@@ -292,7 +292,7 @@ tag_info:{{value[,value...]}[,{value[,value...]}...]}
 
 	::
 
-	    tag_info:{{16,32,32,64,8,8,,32,32,32,32,32,32,32,32,32}
+	    tag_info:{{16,32,32,64,8,8,,32,32,32,32,32,32,32,32,32}}
 
 	On Controller 0
 
diff --git a/Documentation/scsi/aic7xxx.rst b/Documentation/scsi/aic7xxx.rst
index bad0e5567..117cd55fd 100644
--- a/Documentation/scsi/aic7xxx.rst
+++ b/Documentation/scsi/aic7xxx.rst
@@ -308,7 +308,7 @@ tag_info:{{value[,value...]}[,{value[,value...]}...]}
 
 	    ::
 
-	        tag_info:{{16,32,32,64,8,8,,32,32,32,32,32,32,32,32,32}
+	        tag_info:{{16,32,32,64,8,8,,32,32,32,32,32,32,32,32,32}}
 
 	    On Controller 0:
 
-- 
2.55.0


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

* Re: [PATCH] docs: ethtool: Fix bracket
  2026-09-16 12:34 ` [PATCH] docs: ethtool: " Manuel Ebner
@ 2026-09-16 12:47   ` Andrew Lunn
  0 siblings, 0 replies; 8+ messages in thread
From: Andrew Lunn @ 2026-09-16 12:47 UTC (permalink / raw)
  To: Manuel Ebner
  Cc: Jakub Kicinski, David S. Miller, Eric Dumazet, Paolo Abeni,
	Simon Horman, Jonathan Corbet, Shuah Khan, Randy Dunlap, netdev,
	linux-doc, linux-kernel

On Wed, Sep 16, 2026 at 02:34:55PM +0200, Manuel Ebner wrote:
> Add missing ')' to sentence.
> 
> Signed-off-by: Manuel Ebner <manuelebnerli@mailbox.org>

Reviewed-by: Andrew Lunn <andrew@lunn.ch>

    Andrew

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

* Re: [PATCH] docs: networking: radiotap: Fix bracket
  2026-09-16 12:34 ` [PATCH] docs: networking: radiotap: " Manuel Ebner
@ 2026-09-16 16:52   ` Randy Dunlap
  0 siblings, 0 replies; 8+ messages in thread
From: Randy Dunlap @ 2026-09-16 16:52 UTC (permalink / raw)
  To: Manuel Ebner, David S. Miller, Eric Dumazet, Jakub Kicinski,
	Paolo Abeni, Simon Horman, Jonathan Corbet, Shuah Khan
  Cc: netdev, linux-doc, linux-kernel



On 9/16/26 5:34 AM, Manuel Ebner wrote:
> Remove needless ')' from code.
> 
> Signed-off-by: Manuel Ebner <manuelebnerli@mailbox.org>

Acked-by: Randy Dunlap <rdunlap@infradead.org>

> ---
>  Documentation/networking/radiotap-headers.rst | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/Documentation/networking/radiotap-headers.rst b/Documentation/networking/radiotap-headers.rst
> index 1a1bd1ec0..aadc7bc6b 100644
> --- a/Documentation/networking/radiotap-headers.rst
> +++ b/Documentation/networking/radiotap-headers.rst
> @@ -133,7 +133,7 @@ its prototypes available in include/net/cfg80211.h.  You use it like this::
>  
>  		    case IEEE80211_RADIOTAP_ANTENNA:
>  			    /* radiotap uses 0 for 1st ant */
> -			    antenna = *iterator.this_arg);
> +			    antenna = *iterator.this_arg;
>  			    break;
>  
>  		    case IEEE80211_RADIOTAP_DBM_TX_POWER:

-- 
~Randy

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

* Re: [PATCH] scsi: docs: aic: Fix brackets
  2026-09-16 12:35 ` [PATCH] scsi: docs: aic: Fix brackets Manuel Ebner
@ 2026-09-16 16:58   ` Randy Dunlap
  0 siblings, 0 replies; 8+ messages in thread
From: Randy Dunlap @ 2026-09-16 16:58 UTC (permalink / raw)
  To: Manuel Ebner, Jonathan Corbet, Shuah Khan; +Cc: linux-doc, linux-kernel



On 9/16/26 5:35 AM, Manuel Ebner wrote:
> Add missing '}' to lines of code.
> 
> Signed-off-by: Manuel Ebner <manuelebnerli@mailbox.org>

Acked-by: Randy Dunlap <rdunlap@infradead.org>

> ---
> Would it be better to remove the '{' and replace '{{'/'}}' with '{'/'}'?

Not IMO. Looks to me like the patch now matches the definition
of tag_info.

> ---
>  Documentation/scsi/aic79xx.rst | 2 +-
>  Documentation/scsi/aic7xxx.rst | 2 +-
>  2 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/Documentation/scsi/aic79xx.rst b/Documentation/scsi/aic79xx.rst
> index 071ff5111..ba5299cb9 100644
> --- a/Documentation/scsi/aic79xx.rst
> +++ b/Documentation/scsi/aic79xx.rst
> @@ -292,7 +292,7 @@ tag_info:{{value[,value...]}[,{value[,value...]}...]}
>  
>  	::
>  
> -	    tag_info:{{16,32,32,64,8,8,,32,32,32,32,32,32,32,32,32}
> +	    tag_info:{{16,32,32,64,8,8,,32,32,32,32,32,32,32,32,32}}
>  
>  	On Controller 0
>  
> diff --git a/Documentation/scsi/aic7xxx.rst b/Documentation/scsi/aic7xxx.rst
> index bad0e5567..117cd55fd 100644
> --- a/Documentation/scsi/aic7xxx.rst
> +++ b/Documentation/scsi/aic7xxx.rst
> @@ -308,7 +308,7 @@ tag_info:{{value[,value...]}[,{value[,value...]}...]}
>  
>  	    ::
>  
> -	        tag_info:{{16,32,32,64,8,8,,32,32,32,32,32,32,32,32,32}
> +	        tag_info:{{16,32,32,64,8,8,,32,32,32,32,32,32,32,32,32}}
>  
>  	    On Controller 0:
>  

-- 
~Randy

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

* Re: [PATCH] Documentation: net: dsa: Fix bracket
  2026-09-16 12:34 [PATCH] Documentation: net: dsa: Fix bracket Manuel Ebner
                   ` (2 preceding siblings ...)
  2026-09-16 12:35 ` [PATCH] scsi: docs: aic: Fix brackets Manuel Ebner
@ 2026-09-16 17:02 ` Randy Dunlap
  3 siblings, 0 replies; 8+ messages in thread
From: Randy Dunlap @ 2026-09-16 17:02 UTC (permalink / raw)
  To: Manuel Ebner, David S. Miller, Eric Dumazet, Jakub Kicinski,
	Paolo Abeni, Simon Horman, Jonathan Corbet, Shuah Khan
  Cc: netdev, linux-doc, linux-kernel



On 9/16/26 5:34 AM, Manuel Ebner wrote:
> Add missing ')'.
> 
> Signed-off-by: Manuel Ebner <manuelebnerli@mailbox.org>

Acked-by: Randy Dunlap <rdunlap@infradead.org>
Thanks.

> ---
>  Documentation/networking/dsa/dsa.rst | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/Documentation/networking/dsa/dsa.rst b/Documentation/networking/dsa/dsa.rst
> index 42a99f5df..7edfdd555 100644
> --- a/Documentation/networking/dsa/dsa.rst
> +++ b/Documentation/networking/dsa/dsa.rst
> @@ -864,7 +864,7 @@ The drivers which act upon the ``dsa_db`` argument in ``port_fdb_add``,
>  DSA associates each offloaded bridge and each offloaded LAG with a one-based ID
>  (``struct dsa_bridge :: num``, ``struct dsa_lag :: id``) for the purposes of
>  refcounting addresses on shared ports. Drivers may piggyback on DSA's numbering
> -scheme (the ID is readable through ``db->bridge.num`` and ``db->lag.id`` or may
> +scheme (the ID is readable through ``db->bridge.num`` and ``db->lag.id``) or may
>  implement their own.
>  
>  Only the drivers which declare support for FDB isolation are notified of FDB

-- 
~Randy

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

end of thread, other threads:[~2026-09-16 17:03 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2026-09-16 12:34 [PATCH] Documentation: net: dsa: Fix bracket Manuel Ebner
2026-09-16 12:34 ` [PATCH] docs: ethtool: " Manuel Ebner
2026-09-16 12:47   ` Andrew Lunn
2026-09-16 12:34 ` [PATCH] docs: networking: radiotap: " Manuel Ebner
2026-09-16 16:52   ` Randy Dunlap
2026-09-16 12:35 ` [PATCH] scsi: docs: aic: Fix brackets Manuel Ebner
2026-09-16 16:58   ` Randy Dunlap
2026-09-16 17:02 ` [PATCH] Documentation: net: dsa: Fix bracket Randy Dunlap

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

all inboxes | Powered by JetHome®