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
prev parent 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