[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CAMuHMdV3Z-BH+c1B7wB7PBn+WEih35H77OZi=FarBiB9GkdGOQ@mail.gmail.com>
Date: Sun, 6 Jul 2025 12:27:31 +0200
From: Geert Uytterhoeven <geert@...ux-m68k.org>
To: Finn Thain <fthain@...ux-m68k.org>
Cc: linux-m68k@...ts.linux-m68k.org, stable@...r.kernel.org,
linux-kernel@...r.kernel.org, Andreas Schwab <schwab@...ux-m68k.org>
Subject: Re: [PATCH v3 1/3] m68k: Fix lost column on framebuffer debug console
On Thu, 27 Mar 2025 at 23:44, Finn Thain <fthain@...ux-m68k.org> wrote:
> Move the cursor position rightward after rendering the character,
> not before. This avoids complications that arise when the recursive
> console_putc call has to wrap the line and/or scroll the display.
> This also fixes the linewrap bug that crops off the rightmost column.
>
> When the cursor is at the bottom of the display, a linefeed will not
> move the cursor position further downward. Instead, the display scrolls
> upward. Avoid the repeated add/subtract sequence by way of a single
> subtraction at the initialization of console_struct_num_rows.
>
> Fixes: 1da177e4c3f4 ("Linux-2.6.12-rc2")
> Cc: stable@...r.kernel.org
> Signed-off-by: Finn Thain <fthain@...ux-m68k.org>
> ---
> Changed since v2:
> - addil and subil are now addql and subql resp.
Reviewed-by: Geert Uytterhoeven <geert@...ux-m68k.org>
i.e. will queue in the m68k tree for v6.17.
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@...ux-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
Powered by blists - more mailing lists