[coreboot-gerrit] Change in coreboot[master]: soc/intel/coffeelake: Add Intel Bootsplash Logo

[Kin Wai Ng (Code Review)]

Hello Naresh Solanki, Subrata Banik, Balaji Manigandan, Aamir Bohra, Maulik V Vaghela, Rizwan Qureshi,

I'd like you to do a code review. Please visit

    https://review.coreboot.org/25243

to review the following change.


Change subject: soc/intel/coffeelake: Add Intel Bootsplash Logo
......................................................................

soc/intel/coffeelake: Add Intel Bootsplash Logo

Change-Id: I826104805497ed30c60e5bc72b803d057ddc2c99
Signed-off-by: Ng Kin Wai <kin.wai.ng at intel.com>
---
M src/soc/intel/coffeelake/chip.c
1 file changed, 16 insertions(+), 0 deletions(-)



  git pull ssh://review.coreboot.org:29418/coreboot refs/changes/43/25243/1

diff --git a/src/soc/intel/coffeelake/chip.c b/src/soc/intel/coffeelake/chip.c
index 42926fa..1e9ecc4 100755
--- a/src/soc/intel/coffeelake/chip.c
+++ b/src/soc/intel/coffeelake/chip.c
@@ -12,6 +12,7 @@
  * GNU General Public License for more details.
  */
 
+#include <cbfs.h>
 #include <chip.h>
 #include <console/console.h>
 #include <device/device.h>
@@ -285,6 +286,21 @@
 	 * 3 = GT unsliced,  4 = GT sliced */
 	for (i = 0; i < ARRAY_SIZE(config->domain_vr_config); i++)
 		fill_vr_domain_config(params, i, &config->domain_vr_config[i]);
+
+#if CONFIG_BOOTSPLASH_IMAGE
+	size_t sz;
+	uint16_t BITMAP_HEADER_SIGNATURE = 0x4D42;
+
+	params->LogoPtr = (uint32_t) cbfs_boot_map_with_leak(\
+						"bootsplash.jpg", \
+						CBFS_TYPE_BOOTSPLASH, &sz);
+	if((*((uint16_t*)(params->LogoPtr))) == BITMAP_HEADER_SIGNATURE) {
+		params->LogoSize = (uint32_t) sz;
+	}
+	else {
+		params->LogoPtr = (uint32_t) NULL;
+	}
+#endif
 }
 
 /* Mainboard GPIO Configuration */

-- 
To view, visit https://review.coreboot.org/25243
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I826104805497ed30c60e5bc72b803d057ddc2c99
Gerrit-Change-Number: 25243
Gerrit-PatchSet: 1
Gerrit-Owner: Kin Wai Ng <kin.wai.ng at intel.com>
Gerrit-Reviewer: Aamir Bohra <aamir.bohra at intel.com>
Gerrit-Reviewer: Balaji Manigandan <balaji.manigandan at intel.com>
Gerrit-Reviewer: Maulik V Vaghela <maulik.v.vaghela at intel.com>
Gerrit-Reviewer: Naresh Solanki <naresh.solanki at intel.com>
Gerrit-Reviewer: Rizwan Qureshi <rizwan.qureshi at intel.com>
Gerrit-Reviewer: Subrata Banik <subrata.banik at intel.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.coreboot.org/pipermail/coreboot-gerrit/attachments/20180316/84ce33ab/attachment-0001.html>