[U-Boot] AT91 clock and timer cleanups (was: ARM relocation, probably trivial mistake - back to original problem)

Wolfgang Denk
2010-10-04T08:32:31Z

Dear Reinhard Meyer,

In message <4CA985DC.4000208@emk-elektronik.de> you wrote:
>
> The Qs that remain now are:
>
> 1. currently I see about 8 CONFIG_AT91SAMxxx defines required
> (not only 2 like above). The method is still undecided how to
> handle this. Add "CONFIG_AT91SAM9_FAMILY" to "common.h"? Or put
> all 8 defined() into "global_data.h" (would give 4 lines there)?

You wille ventually need this in other locations as well, so please
add a single 4-line-#if to some appropriate header file; if there is
no better file for AT91 I will accept this for common.h

> 2. How many patches would this change need, and who would collect them?
> It affects at worst 3 areas:
> common.h, arm/global_data.h, at91/clock.c+timer.c ---
> Or is it ok to put that into one patch and add it later to my atmel tree?

I see two patches:

Patch 1 will change the #if and introduce CONFIG_AT91SAM9_FAMILY
instead.

Patch 2 will fix at91/clock.c+timer.c

Best regards,

Wolfgang Denk

--
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
They say a little knowledge is a dangerous thing, but it is not one
half so bad as a lot of ignorance. - Terry Pratchett, _Equal Rites_