Re: [PATCH] mtd: nand: brcmnand: fix mtd corrected bits stat

[Miquel Raynal]

On 29/05/2025 at 20:46:59 -07, David Regan <dregan@xxxxxxxxxxxx> wrote:

> Currently we attempt to get the amount of flipped bits from a hardware
> location which is reset on every subpage. Instead obtain total flipped
> bits stat from hardware accumulator. In addition identify the correct
> maximum subpage corrected bits.

This change does not feel correct. We gather two statistics:
- the maximum number of bitflips corrected in a single ECC chunk
- the total number of bitflips among the whole page

The most important one is the former, because it may trigger wear
levelling from the top layer (UBI, usually). It feels like you are
breaking this, am I wrong? Would you mind to be more explicit otherwise?

Thanks,
Miquèl