On Mon, 2019-06-17 at 21:10 +0530, Sagar Kadam wrote: > On Sun, Jun 16, 2019 at 6:35 PM Vignesh Raghavendra <vigneshr@xxxxxx> wrote: [] > > > +static int issi_unlock(struct spi_nor *nor, loff_t ofs, uint64_t len) > > > +{ [] > > > + if (ret > 0 && !(ret & mask)) { > > > + dev_info(nor->dev, > > > + "ISSI Block Protection Bits cleared SR=0x%x", ret); Please use '\n' terminations on formats > > > + ret = 0; > > > + } else { > > > + dev_err(nor->dev, "ISSI Block Protection Bits not cleared\n"); like this one > > > + ret = -EINVAL; > > > + } > > > + return ret; > > > +} > > > + > > > +/** > > > * spansion_quad_enable() - set QE bit in Configuraiton Register. s/Configuraiton/Configuration/ ______________________________________________________ Linux MTD discussion mailing list http://lists.infradead.org/mailman/listinfo/linux-mtd/