On 14:49 Sat 01 Nov , Cristian Birsan wrote: > 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. > It's supposed to be done in an other patch which I've post btw > > > +#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. I've merge it at the same time of all CFG_
Best Regards, J. _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
