* [PATCH] cdrom: fix brackets
@ 2026-09-02 12:54 Manuel Ebner
2026-09-02 16:46 ` Randy Dunlap
0 siblings, 1 reply; 3+ messages in thread
From: Manuel Ebner @ 2026-09-02 12:54 UTC (permalink / raw)
To: Phillip Potter, Jonathan Corbet, Shuah Khan, Mauro Carvalho Chehab
Cc: Manuel Ebner, linux-doc, linux-kernel
From: Manuel Ebner <manuelebner@mailbox.org>
Fix missing ')' and add '()' to function.
Fixes: e327cfcb2542 ("docs: cdrom-standard.tex: convert from LaTeX to ReST")
Signed-off-by: Manuel Ebner <manuelebner@mailbox.org>
---
Documentation/cdrom/cdrom-standard.rst | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/Documentation/cdrom/cdrom-standard.rst b/Documentation/cdrom/cdrom-standard.rst
index b97a4e9b9bd3..b6dde1d1eaaa 100644
--- a/Documentation/cdrom/cdrom-standard.rst
+++ b/Documentation/cdrom/cdrom-standard.rst
@@ -1014,7 +1014,7 @@ How to update your driver
just calls to the routines you adapted in the previous step.
- You may remove all remaining memory checking code in the
*audio_ioctl()* function that deals with audio commands (these are
- listed in the second part of cdrom_ioctl_. There is no
+ listed in the second part of cdrom_ioctl()). There is no
need for memory allocation either, so most *case*s in the *switch*
statement look similar to::
--
2.54.0
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] cdrom: fix brackets
2026-09-02 12:54 [PATCH] cdrom: fix brackets Manuel Ebner
@ 2026-09-02 16:46 ` Randy Dunlap
2026-09-02 18:30 ` Phillip Potter
0 siblings, 1 reply; 3+ messages in thread
From: Randy Dunlap @ 2026-09-02 16:46 UTC (permalink / raw)
To: Manuel Ebner, Phillip Potter, Jonathan Corbet, Shuah Khan,
Mauro Carvalho Chehab
Cc: Manuel Ebner, linux-doc, linux-kernel
On 9/2/26 5:54 AM, Manuel Ebner wrote:
> From: Manuel Ebner <manuelebner@mailbox.org>
>
> Fix missing ')' and add '()' to function.
>
> Fixes: e327cfcb2542 ("docs: cdrom-standard.tex: convert from LaTeX to ReST")
> Signed-off-by: Manuel Ebner <manuelebner@mailbox.org>
Reviewed-by: Randy Dunlap <rdunlap@infradead.org>
Tested-by: Randy Dunlap <rdunlap@infradead.org>
Thanks.
> ---
> Documentation/cdrom/cdrom-standard.rst | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/Documentation/cdrom/cdrom-standard.rst b/Documentation/cdrom/cdrom-standard.rst
> index b97a4e9b9bd3..b6dde1d1eaaa 100644
> --- a/Documentation/cdrom/cdrom-standard.rst
> +++ b/Documentation/cdrom/cdrom-standard.rst
> @@ -1014,7 +1014,7 @@ How to update your driver
> just calls to the routines you adapted in the previous step.
> - You may remove all remaining memory checking code in the
> *audio_ioctl()* function that deals with audio commands (these are
> - listed in the second part of cdrom_ioctl_. There is no
> + listed in the second part of cdrom_ioctl()). There is no
> need for memory allocation either, so most *case*s in the *switch*
> statement look similar to::
>
--
~Randy
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] cdrom: fix brackets
2026-09-02 16:46 ` Randy Dunlap
@ 2026-09-02 18:30 ` Phillip Potter
0 siblings, 0 replies; 3+ messages in thread
From: Phillip Potter @ 2026-09-02 18:30 UTC (permalink / raw)
To: Manuel Ebner
Cc: Phillip Potter, Jonathan Corbet, Shuah Khan,
Mauro Carvalho Chehab, Manuel Ebner, linux-doc, linux-kernel
On Wed, Sep 02, 2026 at 09:46:58AM -0700, Randy Dunlap wrote:
>
>
> On 9/2/26 5:54 AM, Manuel Ebner wrote:
> > From: Manuel Ebner <manuelebner@mailbox.org>
> >
> > Fix missing ')' and add '()' to function.
> >
> > Fixes: e327cfcb2542 ("docs: cdrom-standard.tex: convert from LaTeX to ReST")
> > Signed-off-by: Manuel Ebner <manuelebner@mailbox.org>
>
> Reviewed-by: Randy Dunlap <rdunlap@infradead.org>
> Tested-by: Randy Dunlap <rdunlap@infradead.org>
> Thanks.
>
> > ---
> > Documentation/cdrom/cdrom-standard.rst | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/Documentation/cdrom/cdrom-standard.rst b/Documentation/cdrom/cdrom-standard.rst
> > index b97a4e9b9bd3..b6dde1d1eaaa 100644
> > --- a/Documentation/cdrom/cdrom-standard.rst
> > +++ b/Documentation/cdrom/cdrom-standard.rst
> > @@ -1014,7 +1014,7 @@ How to update your driver
> > just calls to the routines you adapted in the previous step.
> > - You may remove all remaining memory checking code in the
> > *audio_ioctl()* function that deals with audio commands (these are
> > - listed in the second part of cdrom_ioctl_. There is no
> > + listed in the second part of cdrom_ioctl()). There is no
> > need for memory allocation either, so most *case*s in the *switch*
> > statement look similar to::
> >
>
> --
> ~Randy
Hi Manuel,
I concur with Randy, looks good to me. Will send along for inclusion.
Thanks for the patch.
Regards,
Phil
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2026-09-02 18:30 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2026-09-02 12:54 [PATCH] cdrom: fix brackets Manuel Ebner
2026-09-02 16:46 ` Randy Dunlap
2026-09-02 18:30 ` Phillip Potter
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®