lkml.org 
[lkml]   [2011]   [Nov]   [12]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH v2 00/16] DocG3 fixes and write support
On 11/10/2011 12:05 AM, Robert Jarzmik wrote:
> nandtest/nanddump/nandwrite testing has been
> done, with and without bitflipped data.


Yay!


> Moreover, 3 new patches have been added:
> - fix reading oob+data without correction
> This was triggered by the bitflipped pages tests, and
> fixes cases where we want to read corrupted data.
> - add docg3 modes support
> This actually fixes the read part, as we now read in
> "normal" mode by default rather than in "reliable"
> mode. This is mandatory for nandtest to work, because of
> the "even/odd pages logical AND".
> - add protection support
> This is an evolution to be able to access the whole chip
> in protected chips, as they are in smartphones.


How did you figure out these operating modes and the protected part stuff? From
reverse-engineering a disassembled binary? I'm impressed. You don't get that
level of detail from just monitoring cpu accesses to the device during normal
operation.

BTW, I'm coming around to your thinking that the nand interface is not
appropriate for these chips. Even though they are nand, the lack of any
standard nand interface means the nand base does not do much for you except
obfuscate. Memory based bbt maintenance is handled in nand base, maybe a couple
other minor things. I hope I change my mind; otherwise I'll have to rework the
G4 driver :-(

This would also eliminate my main argument for combining G3/G4 drivers: to keep
the nand interface code consistent. That and more device differences seen in
your latest patches argue against combining. I still strongly suspect that the
G3 is very similiar to the P3 in my Treo 650. At some point down the road I'll
test it out on the P3. Device capacity might be the only difference between the
two devices. If so, the G3 driver might even work on the P3 right out of the box.

Thanks,
Mike




\
 
 \ /
  Last update: 2011-11-12 20:05    [from the cache]
©2003-2011 Jasper Spaans