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 
---
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 ||