Troy Kisky wrote:
Dirk Behme wrote:

This patch adds NAND driver to recent git. Tested on DM6446 DVEVM.

It is mainly based on Sander Huijsen <[EMAIL PROTECTED]> work

http://linux.omap.com/pipermail/davinci-linux-open-source/2007-December/004788.html

and ECC changes from Troy Kisky <[EMAIL PROTECTED]>

http://linux.omap.com/pipermail/davinci-linux-open-source/2008-January/004895.html

Signed-off-by: Dirk Behme <[EMAIL PROTECTED]>

I'm a little disappointed that you didn't pick one of the replacement routines, but I can understand wanting to maintain compatibility with software ecc. So with the aim of speeding up the code without changing formats, here's a new routine to kick the tire.
(nand_davinci_gen_true_ecc_compatible,nand_davinci_compare_ecc_compatible)

Sorry, I didn't want to disappoint you. And, no, I didn't want anything special like maintain compatibility.

I just tried to understand which parts of your user space nandtest.c you want me to put into the driver and seems that due to missing ECC knowledge I failed. Sorry.

It would help if you kindly would send exactly and only the code part you like to see in the driver. Then I can simply copy & paste it and hopefully I will get this right ;) Best format would be a patch on top of my v3 patch or a complete file davinci_nand.c with what you want to have in it, based on my v3 as well. I then will re-package it to v4 version of my patch.

Btw: Maybe this question is stupid again, but your code mentions "256 or 512 bytes/ecc". Whats about 2048byte pages? We had a request on the list whether the driver will work with 2k page NANDs as well.

Many thanks for your help

Dirk
_______________________________________________
Davinci-linux-open-source mailing list
Davinci-linux-open-source@linux.davincidsp.com
http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source

Reply via email to