This is the mail archive of the ecos-discuss@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]

Re: minor bug in flash_28fxxx.inl


>>>>> "Christoph" == Christoph Csebits <christoph.csebits@frequentis.com> writes:

Christoph> hi maybe i found a bug in flash_28fxxx.inl.

Christoph> a 28F160 have a block size of 32k ( 0x8000 ) the whole
Christoph> flash block size is multiplied with the number of
Christoph> interleaved devices.  (thats clear)

Christoph> but this multiplication is done twice:

Christoph> # define FLASH_BLOCK_SIZE (0x8000*CYGNUM_FLASH_INTERLEAVE)

Christoph> // Hard wired for now flash_info.block_size =
Christoph> FLASH_BLOCK_SIZE * CYGNUM_FLASH_INTERLEAVE;

The latter should be CYGNUM_FLASH_SERIES, I think. I hope to get time
to rewrite that driver to match what I did for the AMD driver soonish
- but I'll commit this fix for now. Thanks.

Jesper


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