> -----Original Message----- > From: Linus Walleij <[email protected]> > Sent: Wednesday, September 11, 2024 12:11 AM > To: [email protected]; Dario Binacchi > <[email protected]>; Michael Trimarchi > <[email protected]>; Anand Gore > <[email protected]>; William Zhang > <[email protected]>; Kursad Oney > <[email protected]>; Philippe Reynes > <[email protected]> > Cc: Linus Walleij <[email protected]>; Florian Fainelli > <[email protected]>; Kamal Dasu > <[email protected]>; David Regan <[email protected]>; > Miquel Raynal <[email protected]> > Subject: [PATCH 6/7] mtd: rawnand: brcmnand: Support write protection > setting from dts > > From: William Zhang <[email protected]> > > Backport of upstream Linux > commit 8e7daa85641c9559c113f6b217bdc923397de77c > "mtd: rawnand: brcmnand: Support write protection setting from dts" > > The write protection feature is controlled by the module parameter wp_on > with default set to enabled. But not all the board use this feature > especially in BCMBCA broadband board. And module parameter is not > sufficient as different board can have different option. Add a device > tree property and allow this feature to be configured through the board > dts on per board basis. > > Signed-off-by: William Zhang <[email protected]> > Reviewed-by: Florian Fainelli <[email protected]> > Reviewed-by: Kamal Dasu <[email protected]> > Reviewed-by: David Regan <[email protected]> > Signed-off-by: Miquel Raynal <[email protected]> > Link: https://lore.kernel.org/linux-mtd/20240223034758.13753-14- > [email protected] > Signed-off-by: Linus Walleij <[email protected]> > --- > drivers/mtd/nand/raw/brcmnand/brcmnand.c | 8 ++++++++ > 1 file changed, 8 insertions(+) > > diff --git a/drivers/mtd/nand/raw/brcmnand/brcmnand.c > b/drivers/mtd/nand/raw/brcmnand/brcmnand.c > index 2f786584a1ae..55d5d27438a8 100644 > --- a/drivers/mtd/nand/raw/brcmnand/brcmnand.c > +++ b/drivers/mtd/nand/raw/brcmnand/brcmnand.c > @@ -2847,6 +2847,14 @@ int brcmnand_probe(struct udevice *dev, > struct brcmnand_soc *soc) > /* Read the write-protect configuration in the device tree */ > wp_on = dev_read_u32_default(dev, "write-protect", wp_on); > There is some dts file using this "write-protect" property. I believe Phiippe originally added but it was not upstreamed to linux. The new one in the line below is upstreamed and documented in brcmnand.yaml. For new dts, we should use this new property. But we should still honor the old flag to make sure old dtb still works instead of overriding it here. > + /* Check if the board connects the WP pin */ > +#ifndef __UBOOT__ > + if (of_property_read_bool(dn, "brcm,wp-not-connected")) > +#else > + if (dev_read_bool(ctrl->dev, "brcm,wp-not-connected")) > +#endif /* __UBOOT__ */ > + wp_on = 0; > + > if (ctrl->features & BRCMNAND_HAS_WP) { > /* Permanently disable write protection */ > if (wp_on == 2) > > -- > 2.46.0
smime.p7s
Description: S/MIME Cryptographic Signature
