From mboxrd@z Thu Jan  1 00:00:00 1970
From: Troy Kisky <troy.kisky@boundarydevices.com>
Date: Wed, 22 Aug 2012 11:30:57 -0700
Subject: [U-Boot] [PATCH V2 1/1] mx31/mx35/mx51/mx53/mx6: add watchdog
In-Reply-To: <503488A6.2030300@denx.de>
References: <1345503784-18559-1-git-send-email-troy.kisky@boundarydevices.com>
	<50332699.6040701@denx.de> <5033CAAF.90303@boundarydevices.com>
	<503488A6.2030300@denx.de>
Message-ID: <50352561.4020807@boundarydevices.com>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

On 8/22/2012 12:22 AM, Stefano Babic wrote:
> On 21/08/2012 19:51, Troy Kisky wrote:
>> On 8/20/2012 11:11 PM, Stefano Babic wrote:
>>> On 21/08/2012 01:03, Troy Kisky wrote:
>>>
>> So, you are saying CONFIG_ options don't belong in imx-regs.h, or
>> you didn't notice I stuck it there, or both???
> I didn't notice, but CONFIG_ options do not belong to imx-regs.h. They
> must be define only inside the board configuration file.
>
> Best regards,
> Stefano
>
Would it be acceptable to put
#if defined(CONFIG_MX31) || defined(CONFIG_MX35) || defined(CONFIG_MX51) \
     || defined(CONFIG_MX53) || defined(CONFIG_MX6Q)
#define CONFIG_IMX_WATCHDOG    /* cpu_reset implemented in watchdog */
#endif


at the bottom of config_cmd_default.h or do you have a better place in mind?

Thanks
Troy