Hi Christophe On 2/22/22 17:38, Christophe Kerello wrote: > This patch adds the support of the WP# signal. WP will be disabled > before the first access to the NAND flash. > > Signed-off-by: Christophe Kerello <[email protected]> > --- > > drivers/mtd/nand/raw/stm32_fmc2_nand.c | 9 +++++++++ > 1 file changed, 9 insertions(+) > > diff --git a/drivers/mtd/nand/raw/stm32_fmc2_nand.c > b/drivers/mtd/nand/raw/stm32_fmc2_nand.c > index eee65949d77..fb3279b405e 100644 > --- a/drivers/mtd/nand/raw/stm32_fmc2_nand.c > +++ b/drivers/mtd/nand/raw/stm32_fmc2_nand.c > @@ -12,6 +12,7 @@ > #include <log.h> > #include <nand.h> > #include <reset.h> > +#include <asm/gpio.h> > #include <dm/device_compat.h> > #include <linux/bitfield.h> > #include <linux/bitops.h> > @@ -149,6 +150,7 @@ struct stm32_fmc2_timings { > struct stm32_fmc2_nand { > struct nand_chip chip; > struct stm32_fmc2_timings timings; > + struct gpio_desc wp_gpio; > int ncs; > int cs_used[FMC2_MAX_CE]; > }; > @@ -824,6 +826,9 @@ static int stm32_fmc2_nfc_parse_child(struct > stm32_fmc2_nfc *nfc, ofnode node) > nand->cs_used[i] = cs[i]; > } > > + gpio_request_by_name_nodev(node, "wp-gpios", 0, &nand->wp_gpio, > + GPIOD_IS_OUT | GPIOD_IS_OUT_ACTIVE); > + > nand->chip.flash_node = node; > > return 0; > @@ -972,6 +977,10 @@ static int stm32_fmc2_nfc_probe(struct udevice *dev) > chip->ecc.size = FMC2_ECC_STEP_SIZE; > chip->ecc.strength = FMC2_ECC_BCH8; > > + /* Disable Write Protect */ > + if (dm_gpio_is_valid(&nand->wp_gpio)) > + dm_gpio_set_value(&nand->wp_gpio, 0); > + > ret = nand_scan_ident(mtd, nand->ncs, NULL); > if (ret) > return ret; Reviewed-by: Patrice Chotard <[email protected]>
Thanks Patrice
