* [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
* 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
* [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
* 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
* [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] 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®