Re: [ECOS] adding user defined flash variables as cdl_option to inl file problems

[Brian Austin <brian dot austin at cirrus dot com>]

maybe I can simplify this a bit.

I have been looking at how others do their flash implementation and
wonder why the values for Flash are hard coded.

It seems that ecos is based on development boards and not chip types per
say.

In my case, I have several dev boards with a defined flash type and
flash variables, but people make their own boards with different flash
types than what we put on our dev boards.  So when they need to get
redboot running on them, they have to make their own cdl, inl files to
match their flash types based on our edb93xx device.  I know most people
on this list will tell me that they should do it, but from my
experience, if I can make it easy for them to just select flash types,
and variables from a menu, they can actually get it to work.  (customer
support standpoint)

With ecos being so configurable, I do not see why flash variables have
hard coded values.  Is there an example of a configurable flash
implementation  for a certain chip flavor, or am I gonna have to come up
with a solution myself?




On Sun, 2006-11-26 at 19:00 -0600, Brian Austin wrote:
> hello all,
> 
> I am trying to make my flash configuration as such that a user can
> modify the following using cdl_options.
> 
> CYGNUM_FLASH_INTERLEAVE 
> CYGNUM_FLASH_SERIES    
> CYGNUM_FLASH_WIDTH
> CYGNUM_FLASH_BASE
> CYGNUM_FLASH_BASE_MASK
> CYGNUM_FLASH_DEVICES
> CYGNUM_FLASH_BLANK
> 
> these values are then passed to the STRATA inl file and the amd.c file
> as follows...
> 
> #define CYGNUM_FLASH_DEVICES  CYGHWR_DEVS_FLASH_EDB93XX_DEVICES 	
> 
> #define CYGNUM_FLASH_BASE    CYGHWR_DEVS_FLASH_EDB93XX_BASE
> 
> #define CYGNUM_FLASH_BASE_MASK CYGHWR_DEVS_FLASH_EDB93XX_BASE_MASK 
> 
> #define CYGNUM_FLASH_WIDTH  CYGHWR_DEVS_FLASH_EDB93XX_WIDTH
> 
> #define CYGNUM_FLASH_BLANK    CYGHWR_DEVS_FLASH_EDB93XX_BLANK
> 
> These EDB93XX values are set as cdl_options like so.
> 
> cdl_option CYGHWR_DEVS_FLASH_EDB93XX_BASE_MASK {
> 		default_value "0xFF000000"
> 	}
> 
> but these values are not getting into the .inl files or the amd .c files
> where they are needed.
> 
> Is there any way to make these Flash variables defined as cdl_options?
> 
> Am I missing something here.  This seems like it should not be hard to
> do, but I am stuck.
> 
> thanks,
> 
> Brian
> 
> 
> 



-- 
Before posting, please read the FAQ: http://ecos.sourceware.org/fom/ecos
and search the list archive: http://ecos.sourceware.org/ml/ecos-discuss