[U-Boot-Users] patch for OMAP1610 board

Gupta, Kshitij kshitij at ti.com
Tue Feb 24 07:04:20 CET 2004


hi Wolfgang,
	We shifted some IRQ specific code (which is more board specific) to
the platform.S file. The watchdog reset code was already there in platform.S
file, so just removed the duplicate code from start.S :).  This makes the
cpu specific file more usable for other boards based on arm926.
regards
-kshitij

-----Original Message-----
From: u-boot-users-admin at lists.sourceforge.net
[mailto:u-boot-users-admin at lists.sourceforge.net]On Behalf Of Wolfgang
Denk
Sent: Tuesday, February 24, 2004 4:53 AM
To: Rahul Shanbhag
Cc: u-boot-users at lists.sourceforge.net
Subject: Re: [U-Boot-Users] patch for OMAP1610 board 


Dear Rahul,

in message <000001c3f6bc$85c39da0$0953579d at a0875497> you wrote:
> 
> I am submitting a patch for u-boot on 1610 innovator with some corrections
> in start.S file(arm926ejs cpu) and platform.S file (1610 innovator board).
> This patch moves some board specific initializations from the cpu specific
> file to the board specific file.

I see that you not only shift code, but  also  removed  the  code  to
disable  the  watchdog. Is this intentionally? Shouldn't we keep this
part of the code somewhere, at  least  when  CONFIG_WATCHDOG  is  not
defined?

Best regards,

Wolfgang Denk

-- 
Software Engineering:  Embedded and Realtime Systems,  Embedded Linux
Phone: (+49)-8142-4596-87  Fax: (+49)-8142-4596-88  Email: wd at denx.de
The more we disagree, the more chance there is that at least  one  of
us is right.


-------------------------------------------------------
SF.Net is sponsored by: Speed Start Your Linux Apps Now.
Build and deploy apps & Web services for Linux with
a free DVD software kit from IBM. Click Now!
http://ads.osdn.com/?ad_id=1356&alloc_id=3438&op=click
_______________________________________________
U-Boot-Users mailing list
U-Boot-Users at lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users




More information about the U-Boot mailing list