<p>John E. Kabat would like Marshall Dawson to <strong>review</strong> this change.</p><p><a href="https://review.coreboot.org/21037">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">amd/padmelon: Enable the console<br><br>The system has two RS-232 ports connected to a Fintek F81803.  Set<br>up the console on the first one.  The superio's CLKIN is driven by<br>the APU's OSCOUT2 so it must be enabled.  The APU's internal<br>UART signals are NCs so remove the configure_hudson_uart().<br><br>Change-Id: I482146b9f0acb1bb5468f8bc92c231f76bd3cb04<br>Signed-off-by: Marshall Dawson <marshalldawson3rd@gmail.com><br>---<br>M src/mainboard/amd/padmelon/Kconfig<br>M src/mainboard/amd/padmelon/romstage.c<br>2 files changed, 8 insertions(+), 3 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://review.coreboot.org:29418/coreboot refs/changes/37/21037/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">diff --git a/src/mainboard/amd/padmelon/Kconfig b/src/mainboard/amd/padmelon/Kconfig<br>index 9d89b07..7f75dc4 100644<br>--- a/src/mainboard/amd/padmelon/Kconfig<br>+++ b/src/mainboard/amd/padmelon/Kconfig<br>@@ -26,6 +26,8 @@<br>       select HAVE_ACPI_TABLES<br>       select BOARD_ROMSIZE_KB_8192<br>  select GFXUMA<br>+        select SUPERIO_FINTEK_F81803A<br>+        select SUPERIO_FINTEK_COMMON_ROMSTAGE<br> <br> config MAINBOARD_DIR<br>       string<br>diff --git a/src/mainboard/amd/padmelon/romstage.c b/src/mainboard/amd/padmelon/romstage.c<br>index 29cd8a7..7f29002 100644<br>--- a/src/mainboard/amd/padmelon/romstage.c<br>+++ b/src/mainboard/amd/padmelon/romstage.c<br>@@ -23,6 +23,10 @@<br> #include <northbridge/amd/pi/agesawrapper.h><br> #include <northbridge/amd/pi/agesawrapper_call.h><br> #include <southbridge/amd/pi/hudson/hudson.h><br>+#include <superio/fintek/common/fintek.h><br>+#include <superio/fintek/f81803a/f81803a.h><br>+<br>+#define SERIAL_DEV PNP_DEV(0x4e, F81803A_SP1)<br> <br> void cache_as_ram_main(unsigned long bist, unsigned long cpu_init_detectedx)<br> {<br>@@ -35,10 +39,9 @@<br> <br>    if (!cpu_init_detectedx && boot_cpu()) {<br>              post_code(0x30);<br>+             hudson_clk_output_48Mhz(2);<br>+          fintek_enable_serial(SERIAL_DEV, CONFIG_TTYS0_BASE);<br> <br>-#if IS_ENABLED(CONFIG_HUDSON_UART)<br>-         configure_hudson_uart();<br>-#endif<br>             post_code(0x31);<br>              console_init();<br>       }<br></pre><p>To view, visit <a href="https://review.coreboot.org/21037">change 21037</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/21037"/><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: I482146b9f0acb1bb5468f8bc92c231f76bd3cb04 </div>
<div style="display:none"> Gerrit-Change-Number: 21037 </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>