Re: [PATCH 4/4] Reset ieee1275 console display attribute upon screen clearing

[Vladimir 'φ-coder/phcoder' Serbinenko]

On 20.01.2014 22:29, Lubomir Rintel wrote:
> From: Paulo Flabiano Smorigo <address@hidden>
> 
> On IBM Power it could be hooked to a serial terminal where the reverse video
> attribute would bleed into loaded kernel output.
> 
Not a right solution. Right solution would be to have some handoff
function, not to squeeze it in cls.
> Link: https://bugzilla.redhat.com/show_bug.cgi?id=908519
> 
> address@hidden: Clarified the commit message]
> address@hidden: Add Changelog]
> ---
>  ChangeLog                 | 5 +++++
>  grub-core/term/terminfo.c | 2 +-
>  2 files changed, 6 insertions(+), 1 deletion(-)
> 
> diff --git a/ChangeLog b/ChangeLog
> index 23beb14..f2272d1 100644
> --- a/ChangeLog
> +++ b/ChangeLog
> @@ -1,5 +1,10 @@
>  2014-01-20  Paulo Flabiano Smorigo  <address@hidden>
>  
> +     * grub-core/term/terminfo.c (grub_terminfo_set_current): Add attribute
> +     reset to ieee1275 terminal cls sequence.
> +
> +2014-01-20  Paulo Flabiano Smorigo  <address@hidden>
> +
>       * grub-core/kern/ieee1275/init.c (grub_machine_get_bootlocation): Call
>       grub_ieee1275_parse_net_options.
>       * grub-core/kern/ieee1275/openfw.c: Include grub/env.h.
> diff --git a/grub-core/term/terminfo.c b/grub-core/term/terminfo.c
> index f0d3e3d..7cb7909 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 ("\e[2J");
> +      data->cls               = grub_strdup ("\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)
>

signature.asc
Description: OpenPGP digital signature