| Message ID | 20191216204949.5688-1-oscar.ravadilla@alliedtelesis.co.nz |
|---|---|
| State | Changes Requested |
| Delegated to: | Miquel Raynal |
| Headers | show |
| Series | mtd: rawnand: brcmnand: Add bad block marker option. | expand |
+Kamal, On 12/16/19 12:49 PM, Oscar Ravadilla wrote: > In commit 04649ec1335f ("mtd: rawnand: Always store info about bad > block markers in chip struct") changed where the information of the > bad block markers are stored. It is now stored in nand_chip.options > and nand_chip.badblockpos. However brcmnand driver wasn't updated > so it now fails to handle bad blocks. > > So update brcmnand driver to add bad block markers to nand_chip.options. > > Signed-off-by: Oscar Ravadilla <oscar.ravadilla@alliedtelesis.co.nz> Please copy the maintainer of the driver, should not this have a Fixes: 04649ec1335f ("mtd: rawnand: Always store info about bad block markers in chip struct") tag to the commit message? > --- > drivers/mtd/nand/raw/brcmnand/brcmnand.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/drivers/mtd/nand/raw/brcmnand/brcmnand.c b/drivers/mtd/nand/raw/brcmnand/brcmnand.c > index 1a66b1cd51c0..68d37d1636ae 100644 > --- a/drivers/mtd/nand/raw/brcmnand/brcmnand.c > +++ b/drivers/mtd/nand/raw/brcmnand/brcmnand.c > @@ -2360,6 +2360,8 @@ static int brcmnand_attach_chip(struct nand_chip *chip) > * needed. > */ > chip->options |= NAND_USE_BOUNCE_BUFFER; > + chip->options |= NAND_BBM_FIRSTPAGE | NAND_BBM_SECONDPAGE | > + NAND_BBM_LASTPAGE; > > if (chip->bbt_options & NAND_BBT_USE_FLASH) > chip->bbt_options |= NAND_BBT_NO_OOB; >
diff --git a/drivers/mtd/nand/raw/brcmnand/brcmnand.c b/drivers/mtd/nand/raw/brcmnand/brcmnand.c index 1a66b1cd51c0..68d37d1636ae 100644 --- a/drivers/mtd/nand/raw/brcmnand/brcmnand.c +++ b/drivers/mtd/nand/raw/brcmnand/brcmnand.c @@ -2360,6 +2360,8 @@ static int brcmnand_attach_chip(struct nand_chip *chip) * needed. */ chip->options |= NAND_USE_BOUNCE_BUFFER; + chip->options |= NAND_BBM_FIRSTPAGE | NAND_BBM_SECONDPAGE | + NAND_BBM_LASTPAGE; if (chip->bbt_options & NAND_BBT_USE_FLASH) chip->bbt_options |= NAND_BBT_NO_OOB;
In commit 04649ec1335f ("mtd: rawnand: Always store info about bad block markers in chip struct") changed where the information of the bad block markers are stored. It is now stored in nand_chip.options and nand_chip.badblockpos. However brcmnand driver wasn't updated so it now fails to handle bad blocks. So update brcmnand driver to add bad block markers to nand_chip.options. Signed-off-by: Oscar Ravadilla <oscar.ravadilla@alliedtelesis.co.nz> --- drivers/mtd/nand/raw/brcmnand/brcmnand.c | 2 ++ 1 file changed, 2 insertions(+)