diff mbox series

x86/efi: Remove references of EFI earlyprintk from documentation

Message ID 20220321045853.3324-1-akihiko.odaki@gmail.com
State Accepted
Commit 82850028aab5b44923e2c8a2fe875ecd32590835
Headers show
Series x86/efi: Remove references of EFI earlyprintk from documentation | expand

Commit Message

Akihiko Odaki March 21, 2022, 4:58 a.m. UTC
x86 EFI earlyprink was removed with commit 69c1f396f25b ("efi/x86:
Convert x86 EFI earlyprintk into generic earlycon implementation").

Signed-off-by: Akihiko Odaki <akihiko.odaki@gmail.com>
---
 Documentation/admin-guide/kernel-parameters.txt | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comments

Jonathan Corbet April 5, 2022, 4:26 p.m. UTC | #1
Akihiko Odaki <akihiko.odaki@gmail.com> writes:

> x86 EFI earlyprink was removed with commit 69c1f396f25b ("efi/x86:
> Convert x86 EFI earlyprintk into generic earlycon implementation").

So I will confess that I don't know this area well enough to know
whether this patch makes sense or not.  To help with that, it would have
been good to CC Ard, who is the author of the patch you cite above.
Adding him now.

Ard, do you agree with this change (preserved below)?

Thanks,

jon

> Signed-off-by: Akihiko Odaki <akihiko.odaki@gmail.com>
> ---
>  Documentation/admin-guide/kernel-parameters.txt | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/Documentation/admin-guide/kernel-parameters.txt b/Documentation/admin-guide/kernel-parameters.txt
> index f5a27f067db9..fb39337f4123 100644
> --- a/Documentation/admin-guide/kernel-parameters.txt
> +++ b/Documentation/admin-guide/kernel-parameters.txt
> @@ -1252,7 +1252,7 @@
>  			Append ",keep" to not disable it when the real console
>  			takes over.
>  
> -			Only one of vga, efi, serial, or usb debug port can
> +			Only one of vga, serial, or usb debug port can
>  			be used at a time.
>  
>  			Currently only ttyS0 and ttyS1 may be specified by
> @@ -1267,7 +1267,7 @@
>  			Interaction with the standard serial driver is not
>  			very good.
>  
> -			The VGA and EFI output is eventually overwritten by
> +			The VGA output is eventually overwritten by
>  			the real console.
>  
>  			The xen option can only be used in Xen domains.
> -- 
> 2.35.1
Ard Biesheuvel April 5, 2022, 6 p.m. UTC | #2
On Tue, 5 Apr 2022 at 18:26, Jonathan Corbet <corbet@lwn.net> wrote:
>
> Akihiko Odaki <akihiko.odaki@gmail.com> writes:
>
> > x86 EFI earlyprink was removed with commit 69c1f396f25b ("efi/x86:
> > Convert x86 EFI earlyprintk into generic earlycon implementation").
>
> So I will confess that I don't know this area well enough to know
> whether this patch makes sense or not.  To help with that, it would have
> been good to CC Ard, who is the author of the patch you cite above.
> Adding him now.
>
> Ard, do you agree with this change (preserved below)?
>

Yes, it looks fine to me.

Reviewed-by: Ard Biesheuvel <ardb@kernel.org>

>
> > Signed-off-by: Akihiko Odaki <akihiko.odaki@gmail.com>
> > ---
> >  Documentation/admin-guide/kernel-parameters.txt | 4 ++--
> >  1 file changed, 2 insertions(+), 2 deletions(-)
> >
> > diff --git a/Documentation/admin-guide/kernel-parameters.txt b/Documentation/admin-guide/kernel-parameters.txt
> > index f5a27f067db9..fb39337f4123 100644
> > --- a/Documentation/admin-guide/kernel-parameters.txt
> > +++ b/Documentation/admin-guide/kernel-parameters.txt
> > @@ -1252,7 +1252,7 @@
> >                       Append ",keep" to not disable it when the real console
> >                       takes over.
> >
> > -                     Only one of vga, efi, serial, or usb debug port can
> > +                     Only one of vga, serial, or usb debug port can
> >                       be used at a time.
> >
> >                       Currently only ttyS0 and ttyS1 may be specified by
> > @@ -1267,7 +1267,7 @@
> >                       Interaction with the standard serial driver is not
> >                       very good.
> >
> > -                     The VGA and EFI output is eventually overwritten by
> > +                     The VGA output is eventually overwritten by
> >                       the real console.
> >
> >                       The xen option can only be used in Xen domains.
> > --
> > 2.35.1
Jonathan Corbet April 16, 2022, 8:48 a.m. UTC | #3
Ard Biesheuvel <ardb@kernel.org> writes:

> On Tue, 5 Apr 2022 at 18:26, Jonathan Corbet <corbet@lwn.net> wrote:
>>
>> Akihiko Odaki <akihiko.odaki@gmail.com> writes:
>>
>> > x86 EFI earlyprink was removed with commit 69c1f396f25b ("efi/x86:
>> > Convert x86 EFI earlyprintk into generic earlycon implementation").
>>
>> So I will confess that I don't know this area well enough to know
>> whether this patch makes sense or not.  To help with that, it would have
>> been good to CC Ard, who is the author of the patch you cite above.
>> Adding him now.
>>
>> Ard, do you agree with this change (preserved below)?
>>
>
> Yes, it looks fine to me.
>
> Reviewed-by: Ard Biesheuvel <ardb@kernel.org>

Great.  Applied, thanks.

jon
diff mbox series

Patch

diff --git a/Documentation/admin-guide/kernel-parameters.txt b/Documentation/admin-guide/kernel-parameters.txt
index f5a27f067db9..fb39337f4123 100644
--- a/Documentation/admin-guide/kernel-parameters.txt
+++ b/Documentation/admin-guide/kernel-parameters.txt
@@ -1252,7 +1252,7 @@ 
 			Append ",keep" to not disable it when the real console
 			takes over.
 
-			Only one of vga, efi, serial, or usb debug port can
+			Only one of vga, serial, or usb debug port can
 			be used at a time.
 
 			Currently only ttyS0 and ttyS1 may be specified by
@@ -1267,7 +1267,7 @@ 
 			Interaction with the standard serial driver is not
 			very good.
 
-			The VGA and EFI output is eventually overwritten by
+			The VGA output is eventually overwritten by
 			the real console.
 
 			The xen option can only be used in Xen domains.