Hi,

in the file yaffs_tagsmarshall.c on the line 121-125:

if (tags && ecc_result == -YAFFS_ECC_RESULT_FIXED) {
if (tags->ecc_result <= YAFFS_ECC_RESULT_NO_ERROR)
tags->ecc_result = YAFFS_ECC_RESULT_FIXED;
dev->n_ecc_fixed++;
}

Should not be checking for YAFFS_ECC_RESULT_FIXED (without the minus sign)
instead? I cannot find in the code any other instance of negative
ecc_result.

Best regards
Michele Artoni