Re: [PATCH 0/2] mtd: oobtest: Add parameter to ignore bitflip errors within specified limit

[Roger Quadros <rogerq@ti.com>]

Hi,

On 10/21/2014 04:53 PM, Roger Quadros wrote:
> Hi,
> 
> The oobtest case uses raw NAND read/writes to OOB area bypassing the error correction
> mechanism and hence is bound to be affected by bitflip errors which are normal
> in NAND memories. (e.g. we can never get DRA7-evm's NAND to fully pass
> mtd_oobtest).
> 
> In these patches we add a module parameter "bitflip_limit" to specify how many
> bitflips per page are tolerable. Not specifiing the parameter defaults to old
> behaviour (i.e. zero bitflips tolerated).
> Specifying bitflip_limit=1 makes us pass on DRA7-evm with 0 errors.
> 
> Introduce a new memcmpshow() function that shows the data byte where comparison failed.
> This is useful for debugging. The same function is also used to calculate number of
> bitflip errors over the data block.

Any comments on this series?

cheers,
-roger

> 
> Roger Quadros (2):
>   mtd: mtd_oobtest: Show the verification error location and data
>   mtd: mtd_oobtest: add bitflip_limit parameter
> 
>  drivers/mtd/tests/oobtest.c | 77 +++++++++++++++++++++++++++++++++++++--------
>  1 file changed, 64 insertions(+), 13 deletions(-)
>