[U-Boot] [PATCH] Olimex SAM9-L9260 board support

[Jean-Christophe PLAGNIOL-VILLARD]

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.