<p>PraveenX Hodagatta Pranesh has uploaded this change for <strong>review</strong>.</p><p><a href="https://review.coreboot.org/c/coreboot/+/30306">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">mb/intel/kblrvp: Add helper function to get Board Id<br><br>Add 2 helper function get_board_id() & get_spd_index()<br>to read board id & spd index from EC.<br>Rename the get_board_id() function to get_ec_boardinfo().<br><br>BUG=None<br>TEST= Tested on KBL RVP11, able to read the Board id (0x44)<br>      and verified in serial logs. not verified on KBL RVP8.<br><br>Signed-off-by: Praveen hodagatta pranesh <praveenx.hodagatta.pranesh@intel.com><br>Change-Id: Ie20bf0d45a3568c2c433e5b844bea86aac07c47d<br>---<br>M src/mainboard/intel/kblrvp/board_id.c<br>M src/mainboard/intel/kblrvp/board_id.h<br>2 files changed, 24 insertions(+), 4 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://review.coreboot.org:29418/coreboot refs/changes/06/30306/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/src/mainboard/intel/kblrvp/board_id.c b/src/mainboard/intel/kblrvp/board_id.c</span><br><span>index a362b08..8d0cad3 100644</span><br><span>--- a/src/mainboard/intel/kblrvp/board_id.c</span><br><span>+++ b/src/mainboard/intel/kblrvp/board_id.c</span><br><span>@@ -1,7 +1,7 @@</span><br><span> /*</span><br><span>  * This file is part of the coreboot project.</span><br><span>  *</span><br><span style="color: hsl(0, 100%, 40%);">- * Copyright (C) 2016 Intel Corporation.</span><br><span style="color: hsl(120, 100%, 40%);">+ * Copyright (C) 2016-2018 Intel Corporation.</span><br><span>  *</span><br><span>  * This program is free software; you can redistribute it and/or modify</span><br><span>  * it under the terms of the GNU General Public License as published by</span><br><span>@@ -17,9 +17,9 @@</span><br><span> #include <stdint.h></span><br><span> </span><br><span> /*</span><br><span style="color: hsl(0, 100%, 40%);">- * Get Board ID via EC I/O port write/read</span><br><span style="color: hsl(120, 100%, 40%);">+ * Get Board info via EC I/O port write/read</span><br><span>  */</span><br><span style="color: hsl(0, 100%, 40%);">-int get_board_id(void)</span><br><span style="color: hsl(120, 100%, 40%);">+int get_ec_boardinfo(void)</span><br><span> {</span><br><span>         uint8_t buffer[2];</span><br><span>   uint8_t index;</span><br><span>@@ -30,3 +30,15 @@</span><br><span>  }</span><br><span>    return -1;</span><br><span> }</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+/* Get spd index */</span><br><span style="color: hsl(120, 100%, 40%);">+int get_spd_index(void)</span><br><span style="color: hsl(120, 100%, 40%);">+{</span><br><span style="color: hsl(120, 100%, 40%);">+    return ((get_ec_boardinfo() >> 5) & 0x7);</span><br><span style="color: hsl(120, 100%, 40%);">+}</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+/* Get Board Id */</span><br><span style="color: hsl(120, 100%, 40%);">+int get_board_id(void)</span><br><span style="color: hsl(120, 100%, 40%);">+{</span><br><span style="color: hsl(120, 100%, 40%);">+       return ((get_ec_boardinfo() >> 8) & 0xff);</span><br><span style="color: hsl(120, 100%, 40%);">+}</span><br><span>diff --git a/src/mainboard/intel/kblrvp/board_id.h b/src/mainboard/intel/kblrvp/board_id.h</span><br><span>index 881866f..4405587 100644</span><br><span>--- a/src/mainboard/intel/kblrvp/board_id.h</span><br><span>+++ b/src/mainboard/intel/kblrvp/board_id.h</span><br><span>@@ -1,7 +1,7 @@</span><br><span> /*</span><br><span>  * This file is part of the coreboot project.</span><br><span>  *</span><br><span style="color: hsl(0, 100%, 40%);">- * Copyright (C) 2016 Intel Corporation.</span><br><span style="color: hsl(120, 100%, 40%);">+ * Copyright (C) 2016-2018 Intel Corporation.</span><br><span>  *</span><br><span>  * This program is free software; you can redistribute it and/or modify</span><br><span>  * it under the terms of the GNU General Public License as published by</span><br><span>@@ -19,6 +19,8 @@</span><br><span> /* Mobile Board Id 0x00 - 0xFF */</span><br><span> #define BOARD_ID_SKL_A0_RVP3     0x04</span><br><span> #define BOARD_ID_SKL_RVP7       0x0B</span><br><span style="color: hsl(120, 100%, 40%);">+#define BOARD_ID_KBL_RVP8 0x42</span><br><span style="color: hsl(120, 100%, 40%);">+#define BOARD_ID_KBL_RVP11        0x44</span><br><span> </span><br><span> /* 60-6F reserved for KBL RVPs */</span><br><span> #define BOARD_ID_KBL_LPDDR3_RVP3       0x60</span><br><span>@@ -31,6 +33,12 @@</span><br><span>  * Returns board information (board id[15:8] and</span><br><span>  * Fab info[7:0]) on success and < 0 on error</span><br><span>  */</span><br><span style="color: hsl(120, 100%, 40%);">+int get_ec_boardinfo(void);</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+int get_spd_index(void);</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+/* Board id[15:8] */</span><br><span> int get_board_id(void);</span><br><span> </span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span> #endif /* _MAINBOARD_BOARD_ID_H_ */</span><br><span></span><br></pre><p>To view, visit <a href="https://review.coreboot.org/c/coreboot/+/30306">change 30306</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://review.coreboot.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://review.coreboot.org/c/coreboot/+/30306"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: coreboot </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: Ie20bf0d45a3568c2c433e5b844bea86aac07c47d </div>
<div style="display:none"> Gerrit-Change-Number: 30306 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: PraveenX Hodagatta Pranesh <praveenx.hodagatta.pranesh@intel.com> </div>
<div style="display:none"> Gerrit-MessageType: newchange </div>