This is the mail archive of the ecos-patches@sourceware.org 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: Support for additional AMD-type Flash Devices


Rene Nielsen wrote:
Can you confirm the -B and -T parts really have different
bufsiz and that these values are correct?
I tried looking it up myself in the datasheet[1]
and the only mention of the buffer size claimed it was unsupported.


Thanks for your scepticism, Jonathan. Indeed, there's a flaw in the
previous patch. Never trust other people's work :-)

A lot of stuff we have to take on trust (not having hardware for everything) but I just happened to notice. I guess it was a fairly benign issue this way round making it fractionally slower (rather than claiming it was bigger than it was which would have messed up the programming).


And if you can include a ChangeLog entry with your reply that makes

things easier too :-). Sorry, I've never tried to post here before. Slowly learning... Included in the attached and updated patch.

Looks fine, now committed, thanks!


Jifl
--
*See us at Embedded World 2009, Nürnberg, Germany, 3-5 Mar, Stand 11-300*
eCosCentric Limited      http://www.eCosCentric.com/     The eCos experts
Barnwell House, Barnwell Drive, Cambridge, UK.       Tel: +44 1223 245571
Registered in England and Wales: Reg No 4422071.
------["Si fractum non sit, noli id reficere"]------       Opinions==mine


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