<div dir="ltr">On Wed, Feb 6, 2013 at 3:01 PM, Peter Stuge <span dir="ltr"><<a href="mailto:peter@stuge.se" target="_blank">peter@stuge.se</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div class="im">David Hendricks wrote:<br>
> David Hendricks (<a href="mailto:dhendrix@chromium.org">dhendrix@chromium.org</a>) just uploaded a new patch set to gerrit, which you can find at <a href="http://review.coreboot.org/2297" target="_blank">http://review.coreboot.org/2297</a><br>


><br>
> -gerrit<br>
><br>
> commit b115b35c581af7892739afd7c4aaf2250e260f51<br>
> Author: Hung-Te Lin <<a href="mailto:hungte@chromium.org">hungte@chromium.org</a>><br>
> Date:   Wed Feb 6 17:48:20 2013 +0800<br>
><br>
>     armv7: Clean up: remove deprecated SPL.<br>
><br>
>     "SPL" from U-Boot is deprecated by bootblock in coreboot/arm, so we don't need<br>
>     it anymore.<br>
<br>
</div>..<br>
<div class="im"><br>
> diff --git a/src/cpu/samsung/exynos5250/power.c b/src/cpu/samsung/exynos5250/power.c<br>
> index 7c61655..60003c4 100644<br>
> --- a/src/cpu/samsung/exynos5250/power.c<br>
> +++ b/src/cpu/samsung/exynos5250/power.c<br>
> @@ -23,12 +23,12 @@<br>
>   */<br>
><br>
>  #include <common.h><br>
> +#include <arch/hlt.h><br>
>  #include <arch/io.h><br>
>  #include <console/console.h><br>
>  #include <cpu/samsung/exynos5250/cpu.h><br>
>  #include <cpu/samsung/exynos5250/power.h><br>
>  #include <cpu/samsung/exynos5250/sysreg.h><br>
> -#include <cpu/samsung/exynos5-common/spl.h><br>
>  #include <drivers/maxim/max77686/max77686.h><br>
><br>
>  #include "device/i2c.h"<br>
> @@ -62,7 +62,7 @@ void power_shutdown(void)<br>
><br>
>       clrbits_le32(&power->ps_hold_ctrl, POWER_PS_HOLD_CONTROL_DATA_HIGH);<br>
><br>
> -     hang();<br>
> +     hlt();<br>
>  }<br>
><br>
>  void power_enable_dp_phy(void)<br>
<br>
</div>Most of the above hunk has nothing to do with this commit. It should<br>
of course have been part of the hang() removal commit but failing<br>
that at the very least put in a separate commit.<br></blockquote><div><br></div><div style>Yes and no... This hunk broke when the rest of this patch was applied because it relied on a header which was included by another header which was included by cpu/samsung/exynos5-common/spl.h. So the problem went undetected until this patch was applied. This abysmal code architecture is something we're trying to fix in the coreboot port.</div>

<div style><br></div></div>-- <br>David Hendricks (dhendrix)<br>Systems Software Engineer, Google Inc.
</div></div>