From mboxrd@z Thu Jan  1 00:00:00 1970
From: Stijn Souffriau <stijn.souffriau@essensium.com>
Date: Sat, 02 Jun 2012 17:30:25 +0200
Subject: [U-Boot] [PATCH] Added watchdog support for davinchi_dm365evm
In-Reply-To: <4FB9DDE6.5010501@denx.de>
References: <1337532640-1310-1-git-send-email-stijn.souffriau@essensium.com>
	<4FB9DDE6.5010501@denx.de>
Message-ID: <4FCA3191.9090306@essensium.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

Hello Heiko,

First of all, sorry it took so long for me to reply.

Because the code in arch/arm/cpu/arm926ejs/davinci/timer.c didn't work 
for the davinci dm365evm, I tried. If you look at the code you will find 
that the operations are similar but different.

Maybe this code works for more chips than the dm365 but I was hoping 
someone from TI might be able to tell me that. In any case, it's better 
to have support for one chip than for none.

Best Regards,

Stijn


On 05/21/2012 08:17 AM, Heiko Schocher wrote:
> Hello Stijn,
>
> Stijn Souffriau wrote:
>> ---
>>   board/davinci/dm365evm/dm365evm.c  |   68 ++++++++++++++++++++++++++++++++++++
>>   include/configs/davinci_dm365evm.h |   11 ++++++
>>   2 files changed, 79 insertions(+), 0 deletions(-)
> Why you add this here board specific? Why you do not use existing watchdog code
> in arch/arm/cpu/arm926ejs/davinci/timer.c ?
>
> Ok, it needs some cleanup, but this should work for dm365 too ...
>
> bye,
> Heiko
>
> -- 
> Stijn Souffriau
> Embedded Software Developer - Mind Embedded Software Division
>
> ESSENSIUM nv
> Mind - Embedded Software Division
> Gaston Geenslaan 9 - B-3001 Leuven
> email : stijn.souffriau at essensium.com
> Web: www.essensium.com   /   www.mind.be
> BE 872 984 063 RPR Leuven