Re: [PATCH] nand: Add a watch command

[Miquel Raynal <miquel.raynal@bootlin.com>]

Hi Michael,

> On Tue, Nov 28, 2023 at 11:56 AM Miquel Raynal
> <miquel.raynal@bootlin.com> wrote:
>>
>> This is a debug command to monitor the retention state of the data on
>> the array. The command needs a duplication of the mtd_read_oob()
>> function to actually return the maximum number of bitflips encountered
>> while reading the page. We could write a specific implementation for the
>> Sunxi driver but this is probably enough.
>>
>>     nand watch <off> <size> - check an area for bitflips
>>     nand watch.part <part> - check a partition for bitflips
>>     nand watch.chip - check the whole device for bitflips
>>
>> The output may be a bit verbose and could look like:
>>
>
> I have rebased and I will take a look on this patch tomorrow. I think having it
> is ok.

The patch has been in the mainline tree for quite some time now, but I
see today that the authorship has changed:

    commit 1fac577207196e12932c2c733b58e3b14013b1ce
    Author: Michael Trimarchi <michael@amarulasolutions.com>

    nand: Add a watch command

    [...]

    Signed-off-by: Miquel Raynal <miquel.raynal@bootlin.com>
    Signed-off-by: Michael Trimarchi <michael@amarulasolutions.com>

It is probably a mistake but please be careful in the future :)

Thanks,
Miquèl