[coreboot-gerrit] Change in coreboot[master]: lib/edid: Save the display ASCII string

[Arthur Heymans (Code Review)]

Arthur Heymans has uploaded a new change for review. ( https://review.coreboot.org/19499 )

Change subject: lib/edid: Save the display ASCII string
......................................................................

lib/edid: Save the display ASCII string

Change-Id: Ic31af53dcb9947e2264c809ee8f80ea4f89f347d
Signed-off-by: Arthur Heymans <arthur at aheymans.xyz>
---
M src/include/edid.h
M src/lib/edid.c
2 files changed, 5 insertions(+), 3 deletions(-)


  git pull ssh://review.coreboot.org:29418/coreboot refs/changes/99/19499/1

diff --git a/src/include/edid.h b/src/include/edid.h
index 100dbe9..834f5f2 100644
--- a/src/include/edid.h
+++ b/src/include/edid.h
@@ -89,6 +89,7 @@
 	u32 bytes_per_line;
 
 	int hdmi_monitor_detected;
+	char ascii_string[13];
 };
 
 enum edid_status {
diff --git a/src/lib/edid.c b/src/lib/edid.c
index 9de3a24..43449d9 100644
--- a/src/lib/edid.c
+++ b/src/lib/edid.c
@@ -470,9 +470,9 @@
 			 * slots, seems to be specified by SPWG:
 			 * http://www.spwg.org/
 			 */
-			printk(BIOS_SPEW, "ASCII string: %s\n",
-			       extract_string(x + 5,
-			       &c->has_valid_string_termination, 13));
+			memcpy(result_edid->ascii_string, extract_string(x + 5,
+				&c->has_valid_string_termination, 13), 13);
+			printk(BIOS_SPEW, "ASCII string: %s\n", result_edid->ascii_string);
 			return 1;
 		case 0xFF:
 			printk(BIOS_SPEW, "Serial number: %s\n",
@@ -1481,6 +1481,7 @@
 		c.nonconformant_extension +=
 				parse_extension(out, &edid[i], &c);
 
+	printk(BIOS_DEBUG,"ASCII string: %s\n", out->ascii_string);
 	if (c.claims_one_point_four) {
 		if (c.nonconformant_digital_display ||
 		    !c.has_valid_string_termination ||

-- 
To view, visit https://review.coreboot.org/19499
To unsubscribe, visit https://review.coreboot.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ic31af53dcb9947e2264c809ee8f80ea4f89f347d
Gerrit-PatchSet: 1
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Owner: Arthur Heymans <arthur at aheymans.xyz>