[U-Boot] [PATCH V2 1/2] imx: spl: enable CONFIG_SPL_WATCHDOG_SUPPORT

[Stefano Babic]

Hi Marek, Peng,

On 04/01/2016 13:45, Marek Vasut wrote:
> On Monday, January 04, 2016 at 01:40:10 PM, Peng Fan wrote:
>> Hi Marek,
>>
>> On Mon, Jan 04, 2016 at 01:38:23PM +0100, Marek Vasut wrote:
>>> On Monday, January 04, 2016 at 01:27:27 PM, Peng Fan wrote:
>>>> This commit 4bdcbe60a142b08eefccb0e326a37ba81d3389e8 removes reset_cpu
>>>> which breaks SPL build when DEBUG macro defined.
>>>>
>>>> "
>>>> arch/arm/lib/built-in.o: In function `do_reset':
>>>> ~/uboot/arch/arm/lib/reset.c:45: undefined reference to `reset_cpu'
>>>> scripts/Makefile.spl:244: recipe for target 'spl/u-boot-spl' failed
>>>> "
>>>> Enable CONFIG_SPL_WATCHDOG_SUPPORT to fix this issue.
>>>
>>> Why don't you implement dummy reset_cpu() {} instead ?
>>
>> Do you mean this, https://patchwork.ozlabs.org/patch/562232/?
> 
> Yes, I'd prefer that, since I do not want to see watchdog support becoming
> mandatory part of the SPL build. Watchdog support should be optional.

Yes, enabling watchdog should be not be mandatory. Anyway, this happens
only if CONFIG_IMX_WATCHDOG is set, else CONFIG_SPL_WATCHDOG_SUPPORT
lets only to link reset_cpu() to the build. This is also what happens
for all MX6 boards that do not enable the watchdog but need reset_cpu().

Adding another dummy function looks nasty to me, and Fabio sent some
time ago patches to clean up this and drop empty reset_cpu() inside
board files. We should not create this mess again.

Best regards,
Stefano Babic

-- 
=====================================================================
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de
=====================================================================