<p>John E. Kabat would like Marshall Dawson to <strong>review</strong> this change.</p><p><a href="https://review.coreboot.org/21044">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">amd/padmelon: Remove board ID file<br><br>Padmelon does not use an identifying scheme like Bettong.  Remove<br>the support associated with determining the ID.<br><br>Change-Id: I34fb1d61016e84b8bd12b21de65fed40c5e2e4d9<br>Signed-off-by: Marshall Dawson <marshalldawson3rd@gmail.com><br>---<br>M src/mainboard/amd/padmelon/Makefile.inc<br>D src/mainboard/amd/padmelon/boardid.c<br>2 files changed, 0 insertions(+), 53 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://review.coreboot.org:29418/coreboot refs/changes/44/21044/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">diff --git a/src/mainboard/amd/padmelon/Makefile.inc b/src/mainboard/amd/padmelon/Makefile.inc<br>index cfcc9c0..72cd042 100644<br>--- a/src/mainboard/amd/padmelon/Makefile.inc<br>+++ b/src/mainboard/amd/padmelon/Makefile.inc<br>@@ -15,9 +15,7 @@<br> <br> romstage-y += BiosCallOuts.c<br> romstage-y += OemCustomize.c<br>-romstage-y += boardid.c<br> <br> ramstage-y += BiosCallOuts.c<br> ramstage-y += OemCustomize.c<br> ramstage-$(CONFIG_HUDSON_IMC_FWM) += fchec.c<br>-ramstage-y += boardid.c<br>diff --git a/src/mainboard/amd/padmelon/boardid.c b/src/mainboard/amd/padmelon/boardid.c<br>deleted file mode 100644<br>index 5cbb1a7..0000000<br>--- a/src/mainboard/amd/padmelon/boardid.c<br>+++ /dev/null<br>@@ -1,51 +0,0 @@<br>-/*<br>- * This file is part of the coreboot project.<br>- *<br>- * Copyright (C) 2015 Advanced Micro Devices, Inc.<br>- *<br>- * This program is free software; you can redistribute it and/or modify<br>- * it under the terms of the GNU General Public License as published by<br>- * the Free Software Foundation; version 2 of the License.<br>- *<br>- * This program is distributed in the hope that it will be useful,<br>- * but WITHOUT ANY WARRANTY; without even the implied warranty of<br>- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the<br>- * GNU General Public License for more details.<br>- */<br>-<br>-#include <stdint.h><br>-#include <arch/io.h><br>-#include <southbridge/amd/common/amd_defs.h><br>-#include <boardid.h><br>-<br>-/**<br>- *todo: remove this file<br>- *<br>- *Bettong uses 3 GPIO(5-7) pins to identify board.<br>- *The GPIO ports are mapped to MMIO space.<br>- *The GPIO value and board version are mapped as follow:<br>- *GPIO5 GPIO6 GPIO7 Version<br>- *  0     0     0      A<br>- *  0     0     1      B<br>- *  ......<br>- *  1     1     1      H<br>- */<br>-uint8_t board_id(void)<br>-{<br>-       void *gpiommioaddr;<br>-  u8  value = 0;<br>-       u8  boardrev = 0;<br>-    char boardid;<br>-<br>-     gpiommioaddr = (void *)AMD_SB_ACPI_MMIO_ADDR + 0x1500;<br>-       value = read8(gpiommioaddr + (7 << 2) + 2); /* agpio7: board_id2 */<br>-    boardrev = value & 1;<br>-    value = read8(gpiommioaddr + (6 << 2) + 2); /* agpio6: board_id1 */<br>-    boardrev |= (value & 1) << 1;<br>-      value = read8(gpiommioaddr + (5 << 2) + 2); /* agpio5: board_id0 */<br>-    boardrev |= (value & 1) << 2;<br>-<br>-   boardid = 'A' + boardrev;<br>-<br>- return boardid;<br>-}<br></pre><p>To view, visit <a href="https://review.coreboot.org/21044">change 21044</a>. To unsubscribe, 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/21044"/><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: I34fb1d61016e84b8bd12b21de65fed40c5e2e4d9 </div>
<div style="display:none"> Gerrit-Change-Number: 21044 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: John E. Kabat <sljkrr@gmail.com> </div>
<div style="display:none"> Gerrit-Reviewer: John E. Kabat Jr. <john.kabat@scarletltd.com> </div>
<div style="display:none"> Gerrit-Reviewer: Marshall Dawson <marshalldawson3rd@gmail.com> </div>