public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Stijn Souffriau <stijn.souffriau@essensium.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH] Added watchdog support for davinchi_dm365evm
Date: Sat, 02 Jun 2012 17:30:25 +0200	[thread overview]
Message-ID: <4FCA3191.9090306@essensium.com> (raw)
In-Reply-To: <4FB9DDE6.5010501@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

      reply	other threads:[~2012-06-02 15:30 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-05-20 16:50 [U-Boot] [PATCH] Added watchdog support for davinchi_dm365evm Stijn Souffriau
2012-05-20 18:59 ` Mike Frysinger
2012-06-02 15:32   ` Stijn Souffriau
2012-07-19 13:28     ` Mike Frysinger
2012-05-21  6:17 ` Heiko Schocher
2012-06-02 15:30   ` Stijn Souffriau [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4FCA3191.9090306@essensium.com \
    --to=stijn.souffriau@essensium.com \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox