Thread (7 messages) 7 messages, 3 authors, 2019-06-21

Re: [PATCH] efifb: BGRT: Add check for new BGRT status field rotation bits

flat view

From: Ard Biesheuvel <hidden>
Date: 2019-06-10 15:12:50
Also in: dri-devel, linux-efi

On Wed, 29 May 2019 at 17:46, Hans de Goede [off-list ref] wrote:
Starting with ACPI 6.2 bits 1 and 2 of the BGRT status field are no longer
reserved. These bits are now used to indicate if the image needs to be
rotated before being displayed.

The efifb code does not support rotating the image before copying it to
the screen.

This commit adds a check for these new bits and if they are set leaves the
fb contents as is instead of trying to use the un-rotated BGRT image.

Signed-off-by: Hans de Goede <redacted>
Acked-by: Ard Biesheuvel <redacted>
quoted hunk ↗ jump to hunk
---
 drivers/video/fbdev/efifb.c | 5 +++++
 1 file changed, 5 insertions(+)
diff --git a/drivers/video/fbdev/efifb.c b/drivers/video/fbdev/efifb.c
index 9f39f0c360e0..dfa8dd47d19d 100644
--- a/drivers/video/fbdev/efifb.c
+++ b/drivers/video/fbdev/efifb.c
@@ -169,6 +169,11 @@ static void efifb_show_boot_graphics(struct fb_info *info)
                return;
        }

+       if (bgrt_tab.status & 0x06) {
+               pr_info("efifb: BGRT rotation bits set, not showing boot graphics\n");
+               return;
+       }
+
        /* Avoid flashing the logo if we're going to print std probe messages */
        if (console_loglevel > CONSOLE_LOGLEVEL_QUIET)
                return;
--
2.21.0
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help