<blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">Hello, you were involved in the commit I partially revert, so I'm<br>adding you.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Also I could do it by adding an config option to not reintroduce<br>the function by default.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Best Regards</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">The rationale behind removing this was that nobody seems to use it, that it is confusing to have both a cmos setting and a Kconfig to do the same thing and that when fetching the baud rate from cmos it actually tried to print stuff before baud rate was set which could result in very long delays in the boot time.</p><p style="white-space: pre-wrap; word-wrap: break-word;">To avoid having a something override the value defined in Kconfig (to avoid confusion) I think it would be better to not have it show this option in Kconfig at all. something like MAINBOARD_SET_KCONFIG which is mutually exlusive with TTYS0_BAUD...</p><p><a href="https://review.coreboot.org/23713">View Change</a></p><ul style="list-style: none; padding: 0;"></ul><p>To view, visit <a href="https://review.coreboot.org/23713">change 23713</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/23713"/><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: comment </div>
<div style="display:none"> Gerrit-Change-Id: I970ee788bf90b9e1a8c6ccdc5eee8029d9af0ecc </div>
<div style="display:none"> Gerrit-Change-Number: 23713 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Julien Viard de Galbert <jviarddegalbert@online.net> </div>
<div style="display:none"> Gerrit-Reviewer: Arthur Heymans <arthur@aheymans.xyz> </div>
<div style="display:none"> Gerrit-Reviewer: Julien Viard de Galbert <jviarddegalbert@online.net> </div>
<div style="display:none"> Gerrit-Reviewer: Nico Huber <nico.h@gmx.de> </div>
<div style="display:none"> Gerrit-Reviewer: Patrick Rudolph <siro@das-labor.org> </div>
<div style="display:none"> Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org> </div>
<div style="display:none"> Gerrit-Comment-Date: Mon, 12 Feb 2018 15:45:04 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>
<div style="display:none"> Gerrit-HasLabels: No </div>