This is the mail archive of the ecos-patches@sources.redhat.com mailing list for the eCos project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

RE: EB40 flash driver


Hi Andrew,

it seems something is wrong with your changes, because now
both the old at91 and the new eb40 flash driver are included
when building the redboot template. This of course leads to
build errors.

tk
----------------------------------------------- 
Thomas Koeller, Software Development 

Basler Vision Technologies 
An der Strusbek 60-62 
22926 Ahrensburg 
Germany 

Tel +49 (4102) 463-390 
Fax +49 (4102) 463-46390

mailto:Thomas.Koeller@baslerweb.com 
http://www.baslerweb.com 




> -----Original Message-----
> From: Andrew Lunn [mailto:andrew.lunn@ascom.ch]
> Sent: Tuesday, August 05, 2003 11:14 AM
> To: Thomas Koeller
> Cc: ecos-patches@sources.redhat.com
> Subject: Re: EB40 flash driver
> 
> 
> On Thu, Jul 31, 2003 at 12:22:52PM +0200, Thomas Koeller wrote:
> > Hi,
> > 
> > the ecos repository contains two drivers for the Atmel 
> AT29LV1024 flash
> > chip: 'at91' and 'at29cxxxx'. The 'at91' is only used by 
> the 'eb40' target and
> > has some deficiencies, such as not using '.2ram' sections. 
> The included
> > patch changes the 'eb40' target to use the newer 
> 'at29cxxxx' driver. After
> > applying it, the 'at91' flash driver can be removed.
> 
> I've applied this patch with a few minor changes:
> 
> 1) There were two references to eb40a which you missed to 
> change to eb40.
> 2) I added ChangeLog entries for ecos.db and the HAL *.ecm files
> 3) I've not removed the old driver from the tree. Maybe 
> somebody relies on
>    its broken behavior. 
> 
> I don't have this hardware, so all i've been able to do it check it
> compiles cleanly.
> 
> Thanks for the patch
> 
>        Andrew
> 


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]