From mboxrd@z Thu Jan  1 00:00:00 1970
From: Jean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com>
Date: Mon, 9 Mar 2009 09:32:09 +0100
Subject: [U-Boot] [PATCH 1/2] at91sam9: add watchdog support
In-Reply-To: <20090308231759.126DE832E8B8@gemini.denx.de>
References: <1232623377-26351-1-git-send-email-plagnioj@jcrosoft.com>
	<1232623377-26351-2-git-send-email-plagnioj@jcrosoft.com>
	<20090308231759.126DE832E8B8@gemini.denx.de>
Message-ID: <20090309083209.GC24237@game.jcrosoft.org>
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 00:17 Mon 09 Mar     , Wolfgang Denk wrote:
> Dear Jean-Christophe PLAGNIOL-VILLARD,
> 
> In message <1232623377-26351-2-git-send-email-plagnioj@jcrosoft.com> you wrote:
> > Signed-off-by: Jean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com>
> > ---
> >  Makefile                             |    1 +
> >  doc/README.at91                      |    9 ++++
> >  drivers/serial/atmel_usart.c         |    4 +-
> >  drivers/watchdog/Makefile            |   46 ++++++++++++++++++++
> >  drivers/watchdog/at91sam9_wdt.c      |   79 ++++++++++++++++++++++++++++++++++
> >  include/asm-arm/arch-at91/at91_wdt.h |   38 ++++++++++++++++
> >  6 files changed, 176 insertions(+), 1 deletions(-)
> >  create mode 100644 drivers/watchdog/Makefile
> >  create mode 100644 drivers/watchdog/at91sam9_wdt.c
> >  create mode 100644 include/asm-arm/arch-at91/at91_wdt.h
> > 
> > diff --git a/Makefile b/Makefile
> > index 4a3a488..c3f089d 100644
> > --- a/Makefile
> > +++ b/Makefile
> > @@ -266,6 +266,7 @@ LIBS += drivers/rtc/librtc.a
> >  LIBS += drivers/serial/libserial.a
> >  LIBS += drivers/usb/libusb.a
> >  LIBS += drivers/video/libvideo.a
> > +LIBS += drivers/watchdog/libwatchdog.a
> >  LIBS += common/libcommon.a
> >  LIBS += libfdt/libfdt.a
> >  LIBS += api/libapi.a
> > diff --git a/doc/README.at91 b/doc/README.at91
> > index 4e3928a..1ef3b2a 100644
> > --- a/doc/README.at91
> > +++ b/doc/README.at91
> > @@ -2,6 +2,7 @@ Atmel AT91 Evaluation kits
> >  
> >  http://atmel.com/dyn/products/tools.asp?family_id=605#1443
> >  
> > +I. Board mapping & boot media
> >  ------------------------------------------------------------------------------
> >  AT91SAM9260EK & AT91SAM9XEEK
> >  ------------------------------------------------------------------------------
> > @@ -86,3 +87,11 @@ Environment variables
> >  		make at91sam9263ek_config		- use data flash (spi cs0) (default)
> >  		make at91sam9263ek_nandflash_config	- use nand flash
> >  		make at91sam9263ek_dataflash_cs0_config	- use data flash (spi cs0)
> > +
> > +II. Watchdog support
> > +
> > +	The watchdog can only be activate once so you need to activate in the
> > +	bootloader and in the bootstrap if you use it
> 
> This sentence makes no sense to me.
> 
> If I can activate it only once, then why do you claim I have to
> activate it twice ("in the bootloader [1] and in the bootstrap [2]").
you must activate it support in the twice
> 
> And what exaxctly is "the bootstrap"?
U-Boot can be the First or the Second level bootloader on AT91SAM9 or CAP9
In this second case the Bootstrap is the first bootloader (when booting on
dataflash as example)

Best Regards,
J.