Dear Jean-Christophe PLAGNIOL-VILLARD,

I will remove all unnecessary white spaces from the previous patch. I
have now a patch version for u-boot u-boot v2008.10

> >   "
> > 
> >   #########################################################################
> > diff --git a/Makefile b/Makefile
> > index 7c13ce8..5b0acae 100644
> > --- a/Makefile
> > +++ b/Makefile
> > @@ -2487,15 +2487,6 @@ shannon_config       :       unconfig
> >   at91rm9200dk_config       :       unconfig
> >     @$(MKCONFIG) $(@:_config=) arm arm920t at91rm9200dk atmel at91rm9200
> > 
> > -at91sam9261ek_config       :       unconfig
> > -   @$(MKCONFIG) $(@:_config=) arm arm926ejs at91sam9261ek atmel at91
> > -
> > -at91sam9263ek_config       :       unconfig
> > -   @$(MKCONFIG) $(@:_config=) arm arm926ejs at91sam9263ek atmel at91
> > -
> > -at91sam9rlek_config        :       unconfig
> > -   @$(MKCONFIG) $(@:_config=) arm arm926ejs at91sam9rlek atmel at91
> > -
> >   cmc_pu2_config    :       unconfig
> >     @$(MKCONFIG) $(@:_config=) arm arm920t cmc_pu2 NULL at91rm9200
> > 
> > @@ -2521,6 +2512,18 @@ at91cap9adk_config   :       unconfig
> >   at91sam9260ek_config      :       unconfig
> >     @$(MKCONFIG) $(@:_config=) arm arm926ejs at91sam9260ek atmel at91
> > 
> > +at91sam9261ek_config       :       unconfig
> > +   @$(MKCONFIG) $(@:_config=) arm arm926ejs at91sam9261ek atmel at91
> > +
> > +at91sam9263ek_config       :       unconfig
> > +   @$(MKCONFIG) $(@:_config=) arm arm926ejs at91sam9263ek atmel at91
> > +
> > +at91sam9rlek_config        :       unconfig
> > +   @$(MKCONFIG) $(@:_config=) arm arm926ejs at91sam9rlek atmel at91
> > +
> > +sam9l9260_config   :       unconfig
> > +   @$(MKCONFIG) $(@:_config=) arm arm926ejs sam9l9260 olimex at91
> > +
> why do you touch other at91 config?

I touched other configs because i tried to put correctly boards with
arm920t and arm926ejs in their sections. I belive that at91sam9261ek
at91sam9263ek should be placed in the arm926ejs core section. Please
correct me if i'm wrong.

> > +#ifndef __CONFIG_H
> > +#define __CONFIG_H
> > +
> > +/* Hostname */
> > +#define    CONFIG_HOSTNAME                 sam9l9260
> > +
> > +/* ARM asynchronous clock */
> > +#define AT91_MAIN_CLOCK            198656000       /* from 18.432 MHz 
> > crystal */
> > +#define AT91_MASTER_CLOCK  99328000        /* peripheral = main / 2 */
> > +#define CFG_HZ                     1000000         /* 1us resolution */
> please update to CONFIG_SYS_

> Note that I've plan and announce that I'll generate a common code for
> at91sam9260 and other to reduce duplicate code.

I can change CFG_HZ to CONFIG_SYS_HZ but that macro is used
in /cpu/arm926ejs/at91/timer.c. If I change the macro from timer.c I
should update all at91 configurations but I can test only on Olimex
(at91sam9260) board. I do not know if it's safe to modify other
configurations without appropriate testing.

Regards,
Cristian

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to