<p>Raul Rangel has uploaded this change for <strong>review</strong>.</p><p><a href="https://review.coreboot.org/26235">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">stonyridge: Add TP_Perf_STRUCT struct<br><br>The TP_Perf_STRUCT was missing from pi/00670F00. So I copied the file<br>from src/vendorcode/amd/pi/00630F01/Include/IdsPerf.h and removed<br>everything that we don't need. I did have to change<br>MAX_PERFORMANCE_UNIT_NUM so it matches the size used by pi/00670F00.<br><br>This struct is used to extract the timestamps from AGESA.<br><br>BUG=b:64549506<br>TEST=built on grunt<br><br>Change-Id: I06ec82348e3d10f2430c1192a925a49389ae4414<br>Signed-off-by: Raul E Rangel <rrangel@chromium.org><br>---<br>A src/vendorcode/amd/pi/00670F00/Proc/IDS/IdsLib.h<br>M src/vendorcode/amd/pi/00670F00/agesa_headers.h<br>2 files changed, 67 insertions(+), 0 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://review.coreboot.org:29418/coreboot refs/changes/35/26235/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/src/vendorcode/amd/pi/00670F00/Proc/IDS/IdsLib.h b/src/vendorcode/amd/pi/00670F00/Proc/IDS/IdsLib.h</span><br><span>new file mode 100644</span><br><span>index 0000000..d53878f</span><br><span>--- /dev/null</span><br><span>+++ b/src/vendorcode/amd/pi/00670F00/Proc/IDS/IdsLib.h</span><br><span>@@ -0,0 +1,66 @@</span><br><span style="color: hsl(120, 100%, 40%);">+/* $NoKeywords:$ */</span><br><span style="color: hsl(120, 100%, 40%);">+/**</span><br><span style="color: hsl(120, 100%, 40%);">+ * @file</span><br><span style="color: hsl(120, 100%, 40%);">+ *</span><br><span style="color: hsl(120, 100%, 40%);">+ * AMD IDS Routines</span><br><span style="color: hsl(120, 100%, 40%);">+ *</span><br><span style="color: hsl(120, 100%, 40%);">+ * Contains AMD AGESA Integrated Debug Macros</span><br><span style="color: hsl(120, 100%, 40%);">+ *</span><br><span style="color: hsl(120, 100%, 40%);">+ * @xrefitem bom "File Content Label" "Release Content"</span><br><span style="color: hsl(120, 100%, 40%);">+ * @e project: AGESA</span><br><span style="color: hsl(120, 100%, 40%);">+ * @e sub-project: IDS</span><br><span style="color: hsl(120, 100%, 40%);">+ * @e \$Revision: 281181 $ @e \$Date: 2013-12-18 02:18:55 -0600 (Wed, 18 Dec 2013) $</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%);">+ *</span><br><span style="color: hsl(120, 100%, 40%);">+ * Copyright (c) 2008 - 2013, Advanced Micro Devices, Inc.</span><br><span style="color: hsl(120, 100%, 40%);">+ * All rights reserved.</span><br><span style="color: hsl(120, 100%, 40%);">+ *</span><br><span style="color: hsl(120, 100%, 40%);">+ * Redistribution and use in source and binary forms, with or without</span><br><span style="color: hsl(120, 100%, 40%);">+ * modification, are permitted provided that the following conditions are met:</span><br><span style="color: hsl(120, 100%, 40%);">+ * * Redistributions of source code must retain the above copyright</span><br><span style="color: hsl(120, 100%, 40%);">+ * notice, this list of conditions and the following disclaimer.</span><br><span style="color: hsl(120, 100%, 40%);">+ * * Redistributions in binary form must reproduce the above copyright</span><br><span style="color: hsl(120, 100%, 40%);">+ * notice, this list of conditions and the following disclaimer in the</span><br><span style="color: hsl(120, 100%, 40%);">+ * documentation and/or other materials provided with the distribution.</span><br><span style="color: hsl(120, 100%, 40%);">+ * * Neither the name of Advanced Micro Devices, Inc. nor the names of</span><br><span style="color: hsl(120, 100%, 40%);">+ * its contributors may be used to endorse or promote products derived</span><br><span style="color: hsl(120, 100%, 40%);">+ * from this software without specific prior written permission.</span><br><span style="color: hsl(120, 100%, 40%);">+ *</span><br><span style="color: hsl(120, 100%, 40%);">+ * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS" AND</span><br><span style="color: hsl(120, 100%, 40%);">+ * ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED</span><br><span style="color: hsl(120, 100%, 40%);">+ * WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE</span><br><span style="color: hsl(120, 100%, 40%);">+ * DISCLAIMED. IN NO EVENT SHALL ADVANCED MICRO DEVICES, INC. BE LIABLE FOR ANY</span><br><span style="color: hsl(120, 100%, 40%);">+ * DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES</span><br><span style="color: hsl(120, 100%, 40%);">+ * (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES;</span><br><span style="color: hsl(120, 100%, 40%);">+ * LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND</span><br><span style="color: hsl(120, 100%, 40%);">+ * ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT</span><br><span style="color: hsl(120, 100%, 40%);">+ * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS</span><br><span style="color: hsl(120, 100%, 40%);">+ * SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.</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%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+#include <check_for_wrapper.h></span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+#ifndef _IDS_LIB_H_</span><br><span style="color: hsl(120, 100%, 40%);">+#define _IDS_LIB_H_</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+/// Data Structure of Parameters for TestPoint_TSC.</span><br><span style="color: hsl(120, 100%, 40%);">+typedef struct {</span><br><span style="color: hsl(120, 100%, 40%);">+ UINT32 LineInFile; ///< Line of current time counter</span><br><span style="color: hsl(120, 100%, 40%);">+ UINT64 Description; ///<Description ID</span><br><span style="color: hsl(120, 100%, 40%);">+ UINT64 StartTsc; ///< The StartTimer of TestPoint_TSC</span><br><span style="color: hsl(120, 100%, 40%);">+} TestPoint_TSC;</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+#define RESERVED_TP_NUMER 0x20</span><br><span style="color: hsl(120, 100%, 40%);">+#define MAX_PERFORMANCE_UNIT_NUM (IDS_TP_END - TP_BEGINPROCAMDINITEARLY + 1 + RESERVED_TP_NUMER)</span><br><span style="color: hsl(120, 100%, 40%);">+/// Data Structure of Parameters for TP_Perf_STRUCT.</span><br><span style="color: hsl(120, 100%, 40%);">+typedef struct {</span><br><span style="color: hsl(120, 100%, 40%);">+ UINT32 Signature; ///< "PERF"</span><br><span style="color: hsl(120, 100%, 40%);">+ UINT32 Version; ///< version</span><br><span style="color: hsl(120, 100%, 40%);">+ UINT32 Index; ///< The Index of TP_Perf_STRUCT</span><br><span style="color: hsl(120, 100%, 40%);">+ UINT32 TscInMhz; ///< Tsc counter in 1 mhz</span><br><span style="color: hsl(120, 100%, 40%);">+ TestPoint_TSC TP[MAX_PERFORMANCE_UNIT_NUM]; ///< The TP of TP_Perf_STRUCT</span><br><span style="color: hsl(120, 100%, 40%);">+} TP_Perf_STRUCT;</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+#endif //_IDS_LIB_H_</span><br><span>diff --git a/src/vendorcode/amd/pi/00670F00/agesa_headers.h b/src/vendorcode/amd/pi/00670F00/agesa_headers.h</span><br><span>index a8deae5..fb17927 100644</span><br><span>--- a/src/vendorcode/amd/pi/00670F00/agesa_headers.h</span><br><span>+++ b/src/vendorcode/amd/pi/00670F00/agesa_headers.h</span><br><span>@@ -25,6 +25,7 @@</span><br><span> #include "Include/Ids.h"</span><br><span> #include "Include/PlatformMemoryConfiguration.h"</span><br><span> #include "Proc/Fch/FchPlatform.h"</span><br><span style="color: hsl(120, 100%, 40%);">+#include "Proc/IDS/IdsLib.h"</span><br><span> #include "Proc/Psp/PspBaseLib/PspBaseLib.h"</span><br><span> #pragma pack(pop)</span><br><span> #undef AGESA_HEADERS_ARE_WRAPPED</span><br><span></span><br></pre><p>To view, visit <a href="https://review.coreboot.org/26235">change 26235</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/26235"/><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-MessageType: newchange </div>
<div style="display:none"> Gerrit-Change-Id: I06ec82348e3d10f2430c1192a925a49389ae4414 </div>
<div style="display:none"> Gerrit-Change-Number: 26235 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Raul Rangel <rrangel@chromium.org> </div>