[ECOS] Flash support for ea2648 target

[ECOS] Flash support for ea2648 target

[Sergei Gavrikov]

Hi Uwe,

If you have a time... Uwe, you are an author/contributor of an eCos
HAL for the Embedded Artists LPC2468 OEM board. This month some guys
reported here that they have not success with a fitted FLASH on the
board, in particular they had not got FLASH support in RedBoot.

Can you, please, guide here, how add FLASH support for RedBoot for the
target? I saw no redboot*.ecm files in HAL tree. I'm sorry, I have not
the same hardware to look on the issue more deeper, but, at the least,
Shu Higashiyama reported that FLASH ID codes read itself properly and
flash initialization code failed.

More recent thread on the issue
http://ecos.sourceware.org/ml/ecos-discuss/2010-02/msg00081.html

Thanks. Best regards,

Sergei
-- 



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

[ECOS] Re: Flash support for ea2648 target

[cetoni GmbH - Uwe Kindler]

Hi,

the EA2468 target uses the generic CYGPKG_DEVS_FLASH_SST_39VF3201 flash 
driver. You should check, if the flash device returns the proper 
manufacturer ID (0x00BF) and device ID (0x235B) in function sst_init() 
in flash_sst_39vfxxx.inl. Maybe the flash part changed?

Uwe

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

Re: [ECOS] Re: Flash support for ea2648 target

[viren.dobariya]

hi,
  I check in sst_init function but my prog don't reach there.
so, i debug whole code and findout from where it generates error.

it returns from cyg_"flash_init function" from
"ecos-3.0/packages/io/flash/v3_0/src/flash.c".

In cyg_flash_init() function:

#if (1 == CYGHWR_IO_FLASH_DEVICE)
  // Make sure there is one device, otherwise we could end up
  // accessing a non-existent cyg_flash_dev structure.

   if (&(cyg_flashdevtab[0]) == &cyg_flashdevtab_end) {
        return CYG_FLASH_ERR_INVALID;
  }

From above loop, it always return CYG_FLASH_ERR_INVALID.

so that's why i get error given below.
FLASH: driver init failed: Invalid FLASH address.

how to solve this error?
please reply me.

Thanks,
Viren






> Hi,
>
> the EA2468 target uses the generic CYGPKG_DEVS_FLASH_SST_39VF3201 flash
> driver. You should check, if the flash device returns the proper
> manufacturer ID (0x00BF) and device ID (0x235B) in function sst_init()
> in flash_sst_39vfxxx.inl. Maybe the flash part changed?
>
> Uwe
>
> --
> Before posting, please read the FAQ: http://ecos.sourceware.org/fom/ecos
> and search the list archive: http://ecos.sourceware.org/ml/ecos-discuss
>
>



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

Re: [ECOS] Re: Flash support for ea2648 target

[Sergei Gavrikov]

[change dead email address in CC]

Uwe Kindler wrote:
> Hi,
> 
> the EA2468 target uses the generic CYGPKG_DEVS_FLASH_SST_39VF3201
> flash driver. You should check, if the flash device returns the
> proper manufacturer ID (0x00BF) and device ID (0x235B) in function
> sst_init() in flash_sst_39vfxxx.inl. Maybe the flash part changed?
> 
> Uwe

Hi Uwe,

Thank you for your quick and kindly reply! I hope this point will help
the followers of the "ea2468" targets.

Regards,

Sergei
-- 

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

Re: [ECOS] Re: Flash support for ea2648 target

[Sergei Gavrikov]

Please, do not top-post!
http://en.wikipedia.org/wiki/Top_posting

On Wed, Feb 24, 2010 at 05:02:09AM -0600, viren.dobariya@aftek.com wrote:
> hi,
>   I check in sst_init function but my prog don't reach there.
> so, i debug whole code and findout from where it generates error.
> 
> it returns from cyg_"flash_init function" from
> "ecos-3.0/packages/io/flash/v3_0/src/flash.c".
> 
> In cyg_flash_init() function:
> 
> #if (1 == CYGHWR_IO_FLASH_DEVICE)
>   // Make sure there is one device, otherwise we could end up
>   // accessing a non-existent cyg_flash_dev structure.
> 
>    if (&(cyg_flashdevtab[0]) == &cyg_flashdevtab_end) {
>         return CYG_FLASH_ERR_INVALID;
>   }
> 
> >From above loop, it always return CYG_FLASH_ERR_INVALID.
> 
> so that's why i get error given below.
> FLASH: driver init failed: Invalid FLASH address.
> 
> how to solve this error?

As Uwe said, you have to check the real Manufacturer/Device IDs for your
flash parts.

So, first, take a glass and know what is "your" SST flash IC variant
is fitted on the board. Download data sheet and know the Device ID code
for the IC. If the codes will be different from the "Uwe's" parts, you
have to change the SST flash define here:

devs/flash/arm/ea2468/current/src/flash_ea2468.c

The available SST 39VFXXX parts (=defines) are described here:

devs/flash/sst/39vfxxx/current/include/flash_sst_39vfxxx.inl


I hope this help.

Sergei
--

> please reply me.
> 
> Thanks,
> Viren

> > Hi,
> >
> > the EA2468 target uses the generic CYGPKG_DEVS_FLASH_SST_39VF3201 flash
> > driver. You should check, if the flash device returns the proper
> > manufacturer ID (0x00BF) and device ID (0x235B) in function sst_init()
> > in flash_sst_39vfxxx.inl. Maybe the flash part changed?
> >
> > Uwe
> >
> > --
> > Before posting, please read the FAQ: http://ecos.sourceware.org/fom/ecos
> > and search the list archive: http://ecos.sourceware.org/ml/ecos-discuss

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

Re: [ECOS] Re: Flash support for ea2648 target

[Sergei Gavrikov]

Please keep your replies on the mailing list so that all may benefit.

On Wed, Feb 24, 2010 at 08:11:07AM -0600, Viren wrote:
> hi,
> In my board, SST39VF3201 flash is available.
> and entry of this part is also in this file -->flash_sst_39vfxxx.inl
> Then also it gives same error.

Just a guess (I doubt, but). It may be bus width issue or it may be a
bus timing...

It seems for me, there is an additional place to get info in HAL setup:
hal/arm/lpc24xx/ea2468/current/src/ea2468_misc.c

BTW, you can build eCos `flash1' test and walk through out SST flash
init sections in GDB (using the built RedBoot without FLASH support):

;# chdir in a suitable place
ecosconfig new ea2468
ecosconfig add flash
ecosconfig tree
make
make -C io/flash/current tests 

arm-eabi-nm install/tests/io/flash/current/tests/flash1|grep flash

It's good to know for break points (useful bps: cyg_flash_get_info,
cyg_flash_init).

Then try to investigate
arm-eabi-gdb -nx install/tests/io/flash/current/tests/flash1


Sergei
-- 

> Thanks,
> Viren
> 
> 
> 
> 
> > Please, do not top-post!
> > http://en.wikipedia.org/wiki/Top_posting
> >
> > On Wed, Feb 24, 2010 at 05:02:09AM -0600, viren.dobariya@aftek.com wrote:
> >> hi,
> >>   I check in sst_init function but my prog don't reach there.
> >> so, i debug whole code and findout from where it generates error.
> >>
> >> it returns from cyg_"flash_init function" from
> >> "ecos-3.0/packages/io/flash/v3_0/src/flash.c".
> >>
> >> In cyg_flash_init() function:
> >>
> >> #if (1 == CYGHWR_IO_FLASH_DEVICE)
> >>   // Make sure there is one device, otherwise we could end up
> >>   // accessing a non-existent cyg_flash_dev structure.
> >>
> >>    if (&(cyg_flashdevtab[0]) == &cyg_flashdevtab_end) {
> >>         return CYG_FLASH_ERR_INVALID;
> >>   }
> >>
> >> >From above loop, it always return CYG_FLASH_ERR_INVALID.
> >>
> >> so that's why i get error given below.
> >> FLASH: driver init failed: Invalid FLASH address.
> >>
> >> how to solve this error?
> >
> > As Uwe said, you have to check the real Manufacturer/Device IDs for your
> > flash parts.
> >
> > So, first, take a glass and know what is "your" SST flash IC variant
> > is fitted on the board. Download data sheet and know the Device ID code
> > for the IC. If the codes will be different from the "Uwe's" parts, you
> > have to change the SST flash define here:
> >
> > devs/flash/arm/ea2468/current/src/flash_ea2468.c
> >
> > The available SST 39VFXXX parts (=defines) are described here:
> >
> > devs/flash/sst/39vfxxx/current/include/flash_sst_39vfxxx.inl
> >
> >
> > I hope this help.
> >
> > Sergei
> > --
> >
> >> please reply me.
> >>
> >> Thanks,
> >> Viren
> >
> >> > Hi,
> >> >
> >> > the EA2468 target uses the generic CYGPKG_DEVS_FLASH_SST_39VF3201
> >> flash
> >> > driver. You should check, if the flash device returns the proper
> >> > manufacturer ID (0x00BF) and device ID (0x235B) in function sst_init()
> >> > in flash_sst_39vfxxx.inl. Maybe the flash part changed?
> >> >
> >> > Uwe
> >> >
> >> > --
> >> > Before posting, please read the FAQ:
> >> http://ecos.sourceware.org/fom/ecos
> >> > and search the list archive:
> >> http://ecos.sourceware.org/ml/ecos-discuss
> >
> > --
> > Before posting, please read the FAQ: http://ecos.sourceware.org/fom/ecos
> > and search the list archive: http://ecos.sourceware.org/ml/ecos-discuss
> >
> >
> 

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

Re: [ECOS] Re: Flash support for ea2648 target

[Sergei Gavrikov]

On Wed, 24 Feb 2010, Sergei Gavrikov wrote:
> Uwe Kindler wrote:
>> Hi,
>>
>> the EA2468 target uses the generic CYGPKG_DEVS_FLASH_SST_39VF3201
>> flash driver. You should check, if the flash device returns the
>> proper manufacturer ID (0x00BF) and device ID (0x235B) in function
>> sst_init() in flash_sst_39vfxxx.inl. Maybe the flash part changed?
>>
>> Uwe
>
> Hi Uwe,
>
> Thank you for your quick and kindly reply! I hope this point will help
> the followers of the "ea2468" targets.

Hello Uwe,

FYI: Youqiang Zou sent me an information and his own snippet. He added 
CYG_FLASH_DRIVER definition to flash_ae2468.c. May be that's it, because 
then I saw such an entry in SST ChangeLog

2008-11-17  Jonathan Larmour  <drop his email address>

         * cdl/flash_sst_39vf400.cdl:
         Don't implement CYGHWR_IO_FLASH_DEVICE here. Leave it for
         the chip driver.


... and this was happened after that you had distributed your port.

I asked him to send a patch in the eCos patches list. Thus I did not see 
Zou on the eCos lists. In any case that is something likes a thing is 
under "10-lines" limit.

Regards,

Sergei
-- 

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

RE: [ECOS] Re: Flash support for ea2648 target

[viren.dobariya]

Hi, sergei,
      I am still struggling with flash problem in EA-2468 board.
      In this mail, you mention that Youqiang Zou added 
CYG_FLASH_DRIVER definition to flash_ea2468.c. Can you explain me detail?
Should I add same definition for my board?? 

Thanks,
Viren

-----Original Message-----
From: ecos-discuss-owner@ecos.sourceware.org
[mailto:ecos-discuss-owner@ecos.sourceware.org] On Behalf Of Sergei Gavrikov
Sent: Saturday, February 27, 2010 11:41 PM
To: Sergei Gavrikov
Cc: cetoni GmbH - Uwe Kindler; eCos discuss list
Subject: Re: [ECOS] Re: Flash support for ea2648 target

On Wed, 24 Feb 2010, Sergei Gavrikov wrote:
> Uwe Kindler wrote:
>> Hi,
>>
>> the EA2468 target uses the generic CYGPKG_DEVS_FLASH_SST_39VF3201
>> flash driver. You should check, if the flash device returns the
>> proper manufacturer ID (0x00BF) and device ID (0x235B) in function
>> sst_init() in flash_sst_39vfxxx.inl. Maybe the flash part changed?
>>
>> Uwe
>
> Hi Uwe,
>
> Thank you for your quick and kindly reply! I hope this point will help
> the followers of the "ea2468" targets.

Hello Uwe,

FYI: Youqiang Zou sent me an information and his own snippet. He added 
CYG_FLASH_DRIVER definition to flash_ae2468.c. May be that's it, because 
then I saw such an entry in SST ChangeLog

2008-11-17  Jonathan Larmour  <drop his email address>

         * cdl/flash_sst_39vf400.cdl:
         Don't implement CYGHWR_IO_FLASH_DEVICE here. Leave it for
         the chip driver.


... and this was happened after that you had distributed your port.

I asked him to send a patch in the eCos patches list. Thus I did not see 
Zou on the eCos lists. In any case that is something likes a thing is 
under "10-lines" limit.

Regards,

Sergei
-- 

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





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

RE: [ECOS] Re: Flash support for ea2648 target

[Sergei Gavrikov]

On Tue, 2 Mar 2010, viren.dobariya wrote:
> Hi, sergei,
>      I am still struggling with flash problem in EA-2468 board.
>      In this mail, you mention that Youqiang Zou added
> CYG_FLASH_DRIVER definition to flash_ea2468.c. Can you explain me detail?
> Should I add same definition for my board??

As I could understand, it is possible the same definition should be added
to current platform specific driver. But, Youqiang also pointed that he
use other flash parts, so, do not apply his lines as is. To know what the
macro is (does), please, refer to eCos documentation:

http://ecos.sourceware.org/docs-latest/ref/io-flash.html
http://ecos.sourceware.org/docs-latest/ref/ecos-flash-v2-dev.html

Sergei

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

RE: [ECOS] Re: Flash support for ea2648 target

[viren.dobariya]

> On Tue, 2 Mar 2010, viren.dobariya wrote:
>> Hi, sergei,
>>      I am still struggling with flash problem in EA-2468 board.
>>      In this mail, you mention that Youqiang Zou added
>> CYG_FLASH_DRIVER definition to flash_ea2468.c. Can you explain me
>> detail?
>> Should I add same definition for my board??
>
> As I could understand, it is possible the same definition should be added
> to current platform specific driver. But, Youqiang also pointed that he
> use other flash parts, so, do not apply his lines as is. To know what the
> macro is (does), please, refer to eCos documentation:
>
> http://ecos.sourceware.org/docs-latest/ref/io-flash.html
> http://ecos.sourceware.org/docs-latest/ref/ecos-flash-v2-dev.html
>
> Sergei
>
> --
 ---------------------------------

Hi, Sergei,
   Thanks for documentation.
   Can you ask Youqiang Zou that how did he add CYG_FLASH_DRIVER defination??
If he gives us example then i will add according to my device.


Thanks,
Viren




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

RE: [ECOS] Re: Flash support for ea2648 target

[Sergei Gavrikov]

[-- Attachment #1: Type: TEXT/PLAIN, Size: 3741 bytes --]

On Tue, 2 Mar 2010, viren.dobariya@aftek.com wrote:
>> On Tue, 2 Mar 2010, viren.dobariya wrote:
>>> Hi, sergei,
>>>      I am still struggling with flash problem in EA-2468 board.
>>>      In this mail, you mention that Youqiang Zou added
>>> CYG_FLASH_DRIVER definition to flash_ea2468.c. Can you explain me
>>> detail?
>>> Should I add same definition for my board??
>>
>> As I could understand, it is possible the same definition should be added
>> to current platform specific driver. But, Youqiang also pointed that he
>> use other flash parts, so, do not apply his lines as is. To know what the
>> macro is (does), please, refer to eCos documentation:
>>
>> http://ecos.sourceware.org/docs-latest/ref/io-flash.html
>> http://ecos.sourceware.org/docs-latest/ref/ecos-flash-v2-dev.html
>>
>> Sergei
>
> Hi, Sergei,
>   Thanks for documentation.
>   Can you ask Youqiang Zou that how did he add CYG_FLASH_DRIVER 
> defination?? If he gives us example then i will add according to my 
> device.
>
>
> Thanks,
> Viren

I was looking the cludges when I got his e-mail and I'd seen that he did
CC to the list, but, may be he is not subscribed to the list, so, you have
got no copy. Well, he wrote

---------------------------------------------------------------------->8
From elitechip@gmail.com Sun Feb 28 18:42:54 2010
Date: Mon, 1 Mar 2010 00:42:54 +0800
From: youqiang zou <elitechip@gmail.com>
To: sergei.gavrikov@gmail.com
Cc: ecos-discuss@ecos.sourceware.org
Subject: Re: [ECOS] Re: Re: Flash support for ea2648 target

Refered to the arm_aim711_flash.c, I changed the flash_ea2468.c as the below
list.
 
// Device properties
#include <pkgconf/devs_flash_ea2468.h>

// The EA LPC2468 OEM board has one SST 39VF3201 part
#define CYGNUM_FLASH_INTERLEAVE (1)
#define CYGNUM_FLASH_SERIES     (1)
#define CYGNUM_FLASH_WIDTH      (16)
#define CYGNUM_FLASH_BASE       (0x80000000)
//--------------------------------------------------------------------------
// Platform specific extras
#define CYGPKG_DEVS_FLASH_SST_39VF3201
#ifdef CYGINT_DEVS_FLASH_SST_39VFXXX_REQUIRED
//--------------------------------------------------------------------------
// Now include the driver code.
#include "cyg/io/flash_sst_39vfxxx.inl"
static const cyg_flash_block_info_t cyg_flash_sst_block_info[1] = {
    { FLASH_BLOCK_SIZE, FLASH_NUM_REGIONS * CYGNUM_FLASH_SERIES }
};
CYG_FLASH_DRIVER(cyg_flash_sst_flashdev,
                 &cyg_sst_funs,
                 0,                     // Flags
                 CYGNUM_FLASH_BASE,     // Start
                 CYGNUM_FLASH_BASE + (FLASH_BLOCK_SIZE * FLASH_NUM_REGIONS *
CYGNUM_FLASH_SERIES) - 1,    // End
                 1,                     // Number of block infos
                 cyg_flash_sst_block_info,
                 NULL                   // priv
    );
#endif
// ------------------------------------------------------------------------
// EOF flash_ea2468.c
 
I had not the EA2468 board, my platform is:
    Platform: LPC2478 Board (ARM7TDMI-S)
    RAM: 0xa0000000-0xa2000000 [0xa000aac8-0xa1ffc000 available]
    FLASH: 0x81000000-0x811fffff, 512 x 0x1000 blocks
 
Should change the MACRO
    #define CYGPKG_DEVS_FLASH_SST_39VF3201
to
    #define CYGPKG_DEVS_FLASH_SST_39VF1601
---------------------------------------------------------------------->8

My comments: as I can see there is fitted other flash parts on his board,
so, I do not think that you must change own definition, just define
CYG_FLASH_DRIVER properly acording the used FLASH's "geometry".


Sergei
-- 

[-- Attachment #2: Type: text/plain, Size: 148 bytes --]

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

RE: [ECOS] Re: Flash support for ea2648 target

[Sergei Gavrikov]

[-- Attachment #1: Type: TEXT/PLAIN, Size: 2585 bytes --]

On Tue, 2 Mar 2010, Sergei Gavrikov wrote:

[Add Youqiang Zou to CC]

> On Tue, 2 Mar 2010, viren.dobariya@aftek.com wrote:
>>> On Tue, 2 Mar 2010, viren.dobariya wrote:
>>>> Hi, sergei,
>>>>      I am still struggling with flash problem in EA-2468 board.
>>>>      In this mail, you mention that Youqiang Zou added
>>>> CYG_FLASH_DRIVER definition to flash_ea2468.c. Can you explain me
>>>> detail?
>>>> Should I add same definition for my board??
>>> 
>>> As I could understand, it is possible the same definition should be added
>>> to current platform specific driver. But, Youqiang also pointed that he
>>> use other flash parts, so, do not apply his lines as is. To know what the
>>> macro is (does), please, refer to eCos documentation:
>>> 
>>> http://ecos.sourceware.org/docs-latest/ref/io-flash.html
>>> http://ecos.sourceware.org/docs-latest/ref/ecos-flash-v2-dev.html
>>> 
>>> Sergei
>> 
>> Hi, Sergei,
>>   Thanks for documentation.
>>   Can you ask Youqiang Zou that how did he add CYG_FLASH_DRIVER 
>> defination?? If he gives us example then i will add according to my device.
>> 
>> 
>> Thanks,
>> Viren
>
> I was looking the cludges when I got his e-mail and I'd seen that he did
> CC to the list, but, may be he is not subscribed to the list, so, you have
> got no copy. Well, he wrote

         Youqiang> Refered to the arm_aim711_flash.c, I changed the
         Youqiang> flash_ea2468.c as the below list.

         Youqiang> // The EA LPC2468 OEM board has one SST 39VF3201 part
         Youqiang> #define CYGNUM_FLASH_INTERLEAVE (1)
         Youqiang> #define CYGNUM_FLASH_SERIES     (1)
         Youqiang> #define CYGNUM_FLASH_WIDTH      (16)
         Youqiang> #define CYGNUM_FLASH_BASE       (0x80000000)

[drop some implementation details]

         Youqiang> I had not the EA2468 board, my platform is:
         Youqiang> Platform: LPC2478 Board (ARM7TDMI-S)
         Youqiang> RAM: 0xa0000000-0xa2000000 [0xa000aac8-0xa1ffc000 available]
         Youqiang> FLASH: 0x81000000-0x811fffff, 512 x 0x1000 blocks

I assume that is the real RedBoot's output and that looks like that
CYGNUM_FLASH_BASE address was pointed on 0x81000000. Other chip select
pin? So, check that too. Also, CYGNUM_FLASH_WIDTH says about 16-bit.

They said here
http://ecos.sourceware.org/docs-latest/ref/ecos-flash-library.html 
Will Robinson!


Sergei
-- 

> My comments: as I can see there is fitted other flash parts on his board,
> so, I do not think that you must change own definition, just define
> CYG_FLASH_DRIVER properly acording the used FLASH's "geometry".
>
>
> Sergei

[-- Attachment #2: Type: text/plain, Size: 148 bytes --]

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