Re: [PATCH 1/7] mtd: replace name of 'rfree' field with 'free' in struct mtd_ooblayout_ops to better match it's description

Hi

On Fri, Oct 7, 2022 at 1:58 PM Fabio Estevam festevam@gmail.com wrote:

On Fri, Oct 7, 2022 at 8:34 AM Michael Nazzareno Trimarchi michael@amarulasolutions.com wrote:

...

About this change, there was a commit from Simon 8d38a8459b0 that was renamed. You ask basically to revert it

Correct, what about fixing it like this?

diff --git a/include/linux/mtd/mtd.h b/include/linux/mtd/mtd.h index ff635bd716..09f5269887 100644 --- a/include/linux/mtd/mtd.h +++ b/include/linux/mtd/mtd.h @@ -122,7 +122,7 @@ struct mtd_oob_region {

• @ecc: function returning an ECC region in the OOB area.

•  Should return -ERANGE if %section exceeds the total number of
  

•  ECC sections.
  

• • @free: function returning a free region in the OOB area.

• • @rfree: function returning a free region in the OOB area.

  •   Should return -ERANGE if %section exceeds the total number of
    

  •   free sections.
    

  */

That's is anyway a fix

Michael