[coreboot-gerrit] New patch to review for coreboot: FSP2.0: Add framebuffer graphics support

Andrey Petrov (andrey.petrov@intel.com) gerrit at coreboot.org
Thu Feb 25 23:34:14 CET 2016


Andrey Petrov (andrey.petrov at intel.com) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/13802

-gerrit

commit a25b97e89b125a9a8ec1d964e848f1df2e5e25f7
Author: Andrey Petrov <andrey.petrov at intel.com>
Date:   Thu Feb 25 14:22:03 2016 -0800

    FSP2.0: Add framebuffer graphics support
    
    This adds a few helper functions that are intended to assist setting
    up framebuffer.
    
    Change-Id: Id8ed4de1f9de32e9222b0120c15a6d33676346e7
    Signed-off-by: Andrey Petrov <andrey.petrov at intel.com>
---
 src/drivers/intel/fsp2_0/graphics.c | 105 ++++++++++++++++++++++++++++++++++++
 1 file changed, 105 insertions(+)

diff --git a/src/drivers/intel/fsp2_0/graphics.c b/src/drivers/intel/fsp2_0/graphics.c
new file mode 100644
index 0000000..1480179
--- /dev/null
+++ b/src/drivers/intel/fsp2_0/graphics.c
@@ -0,0 +1,105 @@
+/*
+ * This file is part of the coreboot project.
+ *
+ * Copyright (C) 2015 Intel Corp.
+ * (Written by Alexandru Gagniuc <alexandrux.gagniuc at intel.com> for Intel Corp.)
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; either version 2 of the License, or
+ * (at your option) any later version.
+ */
+
+#include <cbfs.h>
+#include <console/console.h>
+#include <fsp/util.h>
+
+enum pixel_format {
+	pixel_rgbx_8bpc = 0,
+	pixel_bgrx_8bpc = 1,
+	pixel_bitmask = 2,		/* defined by <rgb>_mask values */
+};
+
+static const uint8_t uuid_graphics_info[16] = {
+	0xce, 0x2c, 0xf6, 0x39, 0x25, 0x68, 0x69, 0x46,
+	0xbb, 0x56, 0x54, 0x1a, 0xba, 0x75, 0x3a, 0x07
+};
+
+struct hob_graphics_info {
+	uint64_t framebuffer_base;
+	uint32_t framebuffer_size;
+	uint32_t version;
+	uint32_t horizontal_resolution;
+	uint32_t vertical_resolution;
+	uint32_t pixel_format;		/* See enum pixel_format */
+	uint32_t red_mask;
+	uint32_t green_mask;
+	uint32_t blue_mask;
+	uint32_t reserved_mask;
+	uint32_t pixels_per_scanline;
+} __attribute__((packed));
+
+struct pixel {
+	uint8_t pos;
+	uint8_t size;
+};
+
+static const struct fsp_framebuffer {
+	struct pixel red;
+	struct pixel green;
+	struct pixel blue;
+	struct pixel rsvd;
+} fsp_framebuffer_format_map[] = {
+	[pixel_rgbx_8bpc] = { {0, 8}, {8, 8}, {16, 8}, {24, 8} },
+	[pixel_bgrx_8bpc] = { {16, 8}, {8, 8}, {0, 8}, {24, 8} },
+};
+
+enum cb_err fsp_fill_lb_framebuffer(struct lb_framebuffer *framebuffer)
+{
+	size_t size;
+	const struct hob_graphics_info *ginfo;
+	const struct fsp_framebuffer *fbinfo;
+
+	ginfo = fsp_find_extension_hob_by_uuid(uuid_graphics_info, &size);
+
+	if (!ginfo) {
+		printk(BIOS_ALERT, "Graphics hand-off block not found\n");
+		return CB_ERR;
+	}
+
+	if (ginfo->pixel_format > ARRAY_SIZE(fsp_framebuffer_format_map)) {
+		printk(BIOS_ALERT, "FSP set unknown framebuffer format: %d\n",
+		       ginfo->pixel_format);
+		return CB_ERR;
+	}
+
+	fbinfo = fsp_framebuffer_format_map + ginfo->pixel_format;
+
+	framebuffer->physical_address = ginfo->framebuffer_base;
+	framebuffer->x_resolution = ginfo->horizontal_resolution;
+	framebuffer->y_resolution = ginfo->vertical_resolution;
+	framebuffer->bytes_per_line = ginfo->pixels_per_scanline * 4;
+	framebuffer->bits_per_pixel = 32;
+	framebuffer->red_mask_pos = fbinfo->red.pos;
+	framebuffer->red_mask_size = fbinfo->red.size;
+	framebuffer->green_mask_pos = fbinfo->green.pos;
+	framebuffer->green_mask_size = fbinfo->green.size;
+	framebuffer->blue_mask_pos = fbinfo->blue.pos;
+	framebuffer->blue_mask_size = fbinfo->blue.size;
+	framebuffer->reserved_mask_pos = fbinfo->rsvd.pos;
+	framebuffer->reserved_mask_size = fbinfo->rsvd.pos;
+	framebuffer->tag = LB_TAG_FRAMEBUFFER;
+	framebuffer->size = sizeof(*framebuffer);
+	return CB_SUCCESS;
+}
+
+uintptr_t fsp_load_vbt(void)
+{
+	void *vbt;
+
+	vbt = cbfs_boot_map_with_leak("vbt.bin", CBFS_TYPE_RAW, NULL);
+	if (!vbt)
+		printk(BIOS_NOTICE, "Could not locate a VBT file in CBFS\n");
+
+	return (uintptr_t)vbt;
+}



More information about the coreboot-gerrit mailing list