[PATCH] [net] dibs: Improve DIBS prompts and help texts

Geert Uytterhoeven posted 1 patch 1 month ago
There is a newer version of this series
drivers/dibs/Kconfig | 18 +++++++++---------
1 file changed, 9 insertions(+), 9 deletions(-)
[PATCH] [net] dibs: Improve DIBS prompts and help texts
Posted by Geert Uytterhoeven 1 month ago
The Kconfig prompts for the DIBS options read:

    DIBS support (DIBS) [N/m/y/?]
      intra-OS shortcut with dibs loopback (DIBS_LO) [N/y/?]

Clarify the DIBS prompt by expanding the acronym.
Capitalize the first character of the DIBS_LO prompt, and always write
DIBS in full caps.
While at it, join the first two lines of the DIBS help text into a real
sentence.

Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
---
 drivers/dibs/Kconfig | 18 +++++++++---------
 1 file changed, 9 insertions(+), 9 deletions(-)

diff --git a/drivers/dibs/Kconfig b/drivers/dibs/Kconfig
index 5dc347b9b2350667..f2dd4c3e7c5f6d72 100644
--- a/drivers/dibs/Kconfig
+++ b/drivers/dibs/Kconfig
@@ -1,23 +1,23 @@
 # SPDX-License-Identifier: GPL-2.0
 config DIBS
-	tristate "DIBS support"
+	tristate "Direct Internal Buffer Sharing support"
 	default n
 	help
-	  Direct Internal Buffer Sharing (DIBS)
-	  A communication method that uses common physical (internal) memory
-	  for synchronous direct access into a remote buffer.
+	  Direct Internal Buffer Sharing (DIBS) is a communication method that
+	  uses common physical (internal) memory for synchronous direct access
+	  into a remote buffer.
 
 	  Select this option to provide the abstraction layer between
-	  dibs devices and dibs clients like the SMC protocol.
-	  The module name is dibs.
+	  DIBS devices and DIBS clients like the SMC protocol.
+	  The module name is DIBS.
 
 config DIBS_LO
-	bool "intra-OS shortcut with dibs loopback"
+	bool "Intra-OS shortcut with DIBS loopback"
 	depends on DIBS
 	default n
 	help
-	  DIBS_LO enables the creation of an software-emulated dibs device
+	  DIBS_LO enables the creation of an software-emulated DIBS device
 	  named lo which can be used for transferring data when communication
 	  occurs within the same OS. This helps in convenient testing of
-	  dibs clients, since dibs loopback is independent of architecture or
+	  DIBS clients, since DIBS loopback is independent of architecture or
 	  hardware.
-- 
2.43.0
Re: [PATCH] [net] dibs: Improve DIBS prompts and help texts
Posted by Alexandra Winter 1 month ago

On 13.05.26 15:29, Geert Uytterhoeven wrote:
> The Kconfig prompts for the DIBS options read:
> 
>     DIBS support (DIBS) [N/m/y/?]
>       intra-OS shortcut with dibs loopback (DIBS_LO) [N/y/?]
> 
> Clarify the DIBS prompt by expanding the acronym.

Thank you, yes that is an improvement.


> Capitalize the first character of the DIBS_LO prompt, and always write
> DIBS in full caps.

dibs is a shim layer between other kernel components and code pieces are
typically lowercase, same for sysfs paths. It is not some formal standard
that is documented anywhere in uppercase. So I'd like to keep it lowercase
in the help text.

> While at it, join the first two lines of the DIBS help text into a real
> sentence.
> 
> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
> ---
>  drivers/dibs/Kconfig | 18 +++++++++---------
>  1 file changed, 9 insertions(+), 9 deletions(-)
> 
> diff --git a/drivers/dibs/Kconfig b/drivers/dibs/Kconfig
> index 5dc347b9b2350667..f2dd4c3e7c5f6d72 100644
> --- a/drivers/dibs/Kconfig
> +++ b/drivers/dibs/Kconfig
> @@ -1,23 +1,23 @@
>  # SPDX-License-Identifier: GPL-2.0
>  config DIBS
> -	tristate "DIBS support"
> +	tristate "Direct Internal Buffer Sharing support"
>  	default n
>  	help
> -	  Direct Internal Buffer Sharing (DIBS)
> -	  A communication method that uses common physical (internal) memory
> -	  for synchronous direct access into a remote buffer.
> +	  Direct Internal Buffer Sharing (DIBS) is a communication method that

I could consider changing this to (dibs), if people think that is easier to read.
But I thought it more obvious this way.

> +	  uses common physical (internal) memory for synchronous direct access
> +	  into a remote buffer.
>  
>  	  Select this option to provide the abstraction layer between
> -	  dibs devices and dibs clients like the SMC protocol.
> -	  The module name is dibs.
> +	  DIBS devices and DIBS clients like the SMC protocol.
> +	  The module name is DIBS.

The module name is definitively dibs in lowercase.


>  
>  config DIBS_LO
> -	bool "intra-OS shortcut with dibs loopback"
> +	bool "Intra-OS shortcut with DIBS loopback"
>  	depends on DIBS
>  	default n
>  	help
> -	  DIBS_LO enables the creation of an software-emulated dibs device
> +	  DIBS_LO enables the creation of an software-emulated DIBS device
>  	  named lo which can be used for transferring data when communication
>  	  occurs within the same OS. This helps in convenient testing of
> -	  dibs clients, since dibs loopback is independent of architecture or
> +	  DIBS clients, since DIBS loopback is independent of architecture or
>  	  hardware.
Re: [PATCH] [net] dibs: Improve DIBS prompts and help texts
Posted by Geert Uytterhoeven 1 month ago
Hi Alexandra,

On Wed, 13 May 2026 at 16:20, Alexandra Winter <wintera@linux.ibm.com> wrote:
> On 13.05.26 15:29, Geert Uytterhoeven wrote:
> > The Kconfig prompts for the DIBS options read:
> >
> >     DIBS support (DIBS) [N/m/y/?]
> >       intra-OS shortcut with dibs loopback (DIBS_LO) [N/y/?]
> >
> > Clarify the DIBS prompt by expanding the acronym.
>
> Thank you, yes that is an improvement.
>
> > Capitalize the first character of the DIBS_LO prompt, and always write
> > DIBS in full caps.
>
> dibs is a shim layer between other kernel components and code pieces are
> typically lowercase, same for sysfs paths. It is not some formal standard
> that is documented anywhere in uppercase. So I'd like to keep it lowercase
> in the help text.
>
> > While at it, join the first two lines of the DIBS help text into a real
> > sentence.
> >
> > Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>

> > --- a/drivers/dibs/Kconfig
> > +++ b/drivers/dibs/Kconfig
> > @@ -1,23 +1,23 @@
> >  # SPDX-License-Identifier: GPL-2.0
> >  config DIBS
> > -     tristate "DIBS support"
> > +     tristate "Direct Internal Buffer Sharing support"
> >       default n
> >       help
> > -       Direct Internal Buffer Sharing (DIBS)
> > -       A communication method that uses common physical (internal) memory
> > -       for synchronous direct access into a remote buffer.
> > +       Direct Internal Buffer Sharing (DIBS) is a communication method that
>
> I could consider changing this to (dibs), if people think that is easier to read.
> But I thought it more obvious this way.
>
> > +       uses common physical (internal) memory for synchronous direct access
> > +       into a remote buffer.
> >
> >         Select this option to provide the abstraction layer between
> > -       dibs devices and dibs clients like the SMC protocol.
> > -       The module name is dibs.
> > +       DIBS devices and DIBS clients like the SMC protocol.
> > +       The module name is DIBS.
>
> The module name is definitively dibs in lowercase.

My fault, that is indeed a silly mistake.
Will send v2 later, taking your comments into accoun.

Gr{oetje,eeting}s,

                        Geert

-- 
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds