From 3db7653aabb98b02b9dbea0231fa68eacbbb5991 Mon Sep 17 00:00:00 2001 From: Nico Huber Date: Thu, 18 May 2017 18:07:34 +0200 Subject: lib/edid: Split out fill_lb_framebuffer() Place it into new edid_fill_fb.c, and invert the logic of the Kconfig guard (NATIVE_VGA_INIT_USE_EDID is now !NO_EDID_FILL_FB). It has to be selected by all drivers that use MAINBOARD_DO_NATIVE_VGA_INIT but pro- vide their own fill_lb_framebuffer() implementation. Change-Id: I90634b835bd8e2d150b1c714328a5b2774d891bd Signed-off-by: Nico Huber Reviewed-on: https://review.coreboot.org/19764 Tested-by: build bot (Jenkins) Reviewed-by: Aaron Durbin Reviewed-by: Julius Werner --- src/lib/Kconfig | 7 ++++ src/lib/Makefile.inc | 3 ++ src/lib/edid.c | 70 -------------------------------------- src/lib/edid_fill_fb.c | 92 ++++++++++++++++++++++++++++++++++++++++++++++++++ 4 files changed, 102 insertions(+), 70 deletions(-) create mode 100644 src/lib/Kconfig create mode 100644 src/lib/edid_fill_fb.c (limited to 'src/lib') diff --git a/src/lib/Kconfig b/src/lib/Kconfig new file mode 100644 index 0000000000..6d5f0344f0 --- /dev/null +++ b/src/lib/Kconfig @@ -0,0 +1,7 @@ +config NO_EDID_FILL_FB + bool + default y if !MAINBOARD_DO_NATIVE_VGA_INIT + help + Don't include default fill_lb_framebuffer() implementation. Select + this if your drivers uses MAINBOARD_DO_NATIVE_VGA_INIT but provides + its own fill_lb_framebuffer() implementation. diff --git a/src/lib/Makefile.inc b/src/lib/Makefile.inc index 55f5960103..c4d4f6ab40 100644 --- a/src/lib/Makefile.inc +++ b/src/lib/Makefile.inc @@ -131,6 +131,9 @@ ramstage-$(CONFIG_TRACE) += trace.c ramstage-$(CONFIG_COLLECT_TIMESTAMPS) += timestamp.c ramstage-$(CONFIG_COVERAGE) += libgcov.c ramstage-y += edid.c +ifneq ($(CONFIG_NO_EDID_FILL_FB),y) +ramstage-y += edid_fill_fb.c +endif ramstage-y += memrange.c ramstage-$(CONFIG_COOP_MULTITASKING) += thread.c ramstage-$(CONFIG_TIMER_QUEUE) += timer_queue.c diff --git a/src/lib/edid.c b/src/lib/edid.c index 2216690963..7ad4136814 100644 --- a/src/lib/edid.c +++ b/src/lib/edid.c @@ -94,9 +94,6 @@ static struct { static struct edid tmp_edid; -static int vbe_valid; -static struct lb_framebuffer edid_fb; - static char *manufacturer_name(unsigned char *x) { extra_info.manuf_name[0] = ((x[0] & 0x7C) >> 2) + '@'; @@ -1693,70 +1690,3 @@ void edid_set_framebuffer_bits_per_pixel(struct edid *edid, int fb_bpp, edid->x_resolution = edid->bytes_per_line / (fb_bpp / 8); edid->y_resolution = edid->mode.va; } - -/* - * Take an edid, and create a framebuffer. Set vbe_valid to 1. - */ - -void set_vbe_mode_info_valid(const struct edid *edid, uintptr_t fb_addr) -{ - edid_fb.physical_address = fb_addr; - edid_fb.x_resolution = edid->x_resolution; - edid_fb.y_resolution = edid->y_resolution; - edid_fb.bytes_per_line = edid->bytes_per_line; - /* In the case of (e.g.) 24 framebuffer bits per pixel, the convention - * nowadays seems to be to round it up to the nearest reasonable - * boundary, because otherwise the byte-packing is hideous. - * So, for example, in RGB with no alpha, the bytes are still - * packed into 32-bit words, the so-called 32bpp-no-alpha mode. - * Or, in 5:6:5 mode, the bytes are also packed into 32-bit words, - * and in 4:4:4 mode, they are packed into 16-bit words. - * Good call on the hardware guys part. - * It's not clear we're covering all cases here, but - * I'm not sure with grahpics you ever can. - */ - edid_fb.bits_per_pixel = edid->framebuffer_bits_per_pixel; - edid_fb.reserved_mask_pos = 0; - edid_fb.reserved_mask_size = 0; - switch (edid->framebuffer_bits_per_pixel) { - case 32: - case 24: - /* packed into 4-byte words */ - edid_fb.reserved_mask_pos = 24; - edid_fb.reserved_mask_size = 8; - edid_fb.red_mask_pos = 16; - edid_fb.red_mask_size = 8; - edid_fb.green_mask_pos = 8; - edid_fb.green_mask_size = 8; - edid_fb.blue_mask_pos = 0; - edid_fb.blue_mask_size = 8; - break; - case 16: - /* packed into 2-byte words */ - edid_fb.red_mask_pos = 11; - edid_fb.red_mask_size = 5; - edid_fb.green_mask_pos = 5; - edid_fb.green_mask_size = 6; - edid_fb.blue_mask_pos = 0; - edid_fb.blue_mask_size = 5; - break; - default: - printk(BIOS_SPEW, "%s: unsupported BPP %d\n", __func__, - edid->framebuffer_bits_per_pixel); - return; - } - - vbe_valid = 1; -} - -#if IS_ENABLED(CONFIG_NATIVE_VGA_INIT_USE_EDID) -int fill_lb_framebuffer(struct lb_framebuffer *framebuffer) -{ - if (!vbe_valid) - return -1; - - *framebuffer = edid_fb; - - return 0; -} -#endif diff --git a/src/lib/edid_fill_fb.c b/src/lib/edid_fill_fb.c new file mode 100644 index 0000000000..210c727224 --- /dev/null +++ b/src/lib/edid_fill_fb.c @@ -0,0 +1,92 @@ +/* + * Copyright 2013 Google Inc. + * + * Permission is hereby granted, free of charge, to any person obtaining a + * copy of this software and associated documentation files (the "Software"), + * to deal in the Software without restriction, including without limitation + * on the rights to use, copy, modify, merge, publish, distribute, sub + * license, and/or sell copies of the Software, and to permit persons to whom + * the Software is furnished to do so, subject to the following conditions: + * + * The above copyright notice and this permission notice (including the next + * paragraph) shall be included in all copies or substantial portions of the + * Software. + * + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, + * FITNESS FOR A PARTICULAR PURPOSE AND NON-INFRINGEMENT. IN NO EVENT SHALL + * THE AUTHORS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER + * IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN + * CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE. + */ + +#include +#include +#include + +static int fb_valid; +static struct lb_framebuffer edid_fb; + +/* + * Take an edid, and create a framebuffer. Set fb_valid to 1. + */ +void set_vbe_mode_info_valid(const struct edid *edid, uintptr_t fb_addr) +{ + edid_fb.physical_address = fb_addr; + edid_fb.x_resolution = edid->x_resolution; + edid_fb.y_resolution = edid->y_resolution; + edid_fb.bytes_per_line = edid->bytes_per_line; + /* In the case of (e.g.) 24 framebuffer bits per pixel, the convention + * nowadays seems to be to round it up to the nearest reasonable + * boundary, because otherwise the byte-packing is hideous. + * So, for example, in RGB with no alpha, the bytes are still + * packed into 32-bit words, the so-called 32bpp-no-alpha mode. + * Or, in 5:6:5 mode, the bytes are also packed into 32-bit words, + * and in 4:4:4 mode, they are packed into 16-bit words. + * Good call on the hardware guys part. + * It's not clear we're covering all cases here, but + * I'm not sure with grahpics you ever can. + */ + edid_fb.bits_per_pixel = edid->framebuffer_bits_per_pixel; + edid_fb.reserved_mask_pos = 0; + edid_fb.reserved_mask_size = 0; + switch (edid->framebuffer_bits_per_pixel) { + case 32: + case 24: + /* packed into 4-byte words */ + edid_fb.reserved_mask_pos = 24; + edid_fb.reserved_mask_size = 8; + edid_fb.red_mask_pos = 16; + edid_fb.red_mask_size = 8; + edid_fb.green_mask_pos = 8; + edid_fb.green_mask_size = 8; + edid_fb.blue_mask_pos = 0; + edid_fb.blue_mask_size = 8; + break; + case 16: + /* packed into 2-byte words */ + edid_fb.red_mask_pos = 11; + edid_fb.red_mask_size = 5; + edid_fb.green_mask_pos = 5; + edid_fb.green_mask_size = 6; + edid_fb.blue_mask_pos = 0; + edid_fb.blue_mask_size = 5; + break; + default: + printk(BIOS_SPEW, "%s: unsupported BPP %d\n", __func__, + edid->framebuffer_bits_per_pixel); + return; + } + + fb_valid = 1; +} + +int fill_lb_framebuffer(struct lb_framebuffer *framebuffer) +{ + if (!fb_valid) + return -1; + + *framebuffer = edid_fb; + + return 0; +} -- cgit v1.2.3