Re: [PATCH v3 02/16] term/terminfo: for ppc, reset console display attr when clear screen

[Avnish Chouhan <avnish@linux.ibm.com>]

> Message: 4
> Date: Thu, 10 Oct 2024 15:43:20 -0600
> From: Leo Sandoval <lsandova@redhat.com>
> To: grub-devel@gnu.org
> Subject: [PATCH v3 02/16] term/terminfo: for ppc, reset console
> 	display attr when clear screen
> Message-ID: <20241010214334.1749167-3-lsandova@redhat.com>
> Content-Type: text/plain; charset="US-ASCII"; x-default=true
> 
> From: Paulo Flabiano Smorigo <pfsmorigo@br.ibm.com>
> 
> v2: Also use \x0c instead of a literal ^L to make future patches less
> awkward.
> 
> This should fix this bugzilla:
> https://bugzilla.redhat.com/show_bug.cgi?id=908519
> 
> Signed-off-by: Peter Jones <pjones@redhat.com>
> Signed-off-by: Paulo Flabiano Smorigo <pfsmorigo@br.ibm.com>
> Signed-off-by: Robbie Harwood <rharwood@redhat.com>
> ---
>  grub-core/term/terminfo.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/grub-core/term/terminfo.c b/grub-core/term/terminfo.c
> index 4e534c683..3dbe88e89 100644
> --- a/grub-core/term/terminfo.c
> +++ b/grub-core/term/terminfo.c
> @@ -151,7 +151,7 @@ grub_terminfo_set_current (struct grub_term_output 
> *term,
>        /* Clear the screen.  Using serial console, screen(1) only 
> recognizes the
>         * ANSI escape sequence.  Using video console, Apple Open 
> Firmware
>         * (version 3.1.1) only recognizes the literal ^L.  So use both. 
>  */
> -      data->cls               = grub_strdup ("\f\e[2J");
> +      data->cls               = grub_strdup ("\x0c\e[2J\e[m");
>        data->reverse_video_on  = grub_strdup ("\e[7m");
>        data->reverse_video_off = grub_strdup ("\e[m");
>        if (grub_strcmp ("ieee1275", str) == 0)
> --
> 2.46.2
> 
> 
> 
> 
> ------------------------------


Reviewed-by: Avnish Chouhan <avnish@linux.ibm.com>

_______________________________________________
Grub-devel mailing list
Grub-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/grub-devel