Hi Tudor,
On Tue, May 7, 2019 at 3:25 PM <Tudor.Ambarus@xxxxxxxxxxxxx> wrote:
> On 05/07/2019 04:15 PM, Tudor Ambarus wrote:
> > diff --git a/drivers/mtd/spi-nor/spi-nor.c b/drivers/mtd/spi-nor/spi-nor.c
> > index 73172d7f512b..b94a6eaaaca5 100644
> > --- a/drivers/mtd/spi-nor/spi-nor.c
> > +++ b/drivers/mtd/spi-nor/spi-nor.c
> > @@ -1551,6 +1551,7 @@ static int spansion_read_cr_quad_enable(struct spi_nor *nor)
> > u8 sr_cr[2];
> > int ret;
> >
> > + dev_err(dev, "%s\n", __FUNCTION__);
This one isn't triggered.
> > /* Check current Quad Enable bit value. */
> > ret = read_cr(nor);
> > if (ret < 0) {
> > @@ -3911,6 +3912,12 @@ static int spi_nor_setup(struct spi_nor *nor,
> > static int spi_nor_init(struct spi_nor *nor)
> > {
> > int err;
> > + u8 val;
> > + u8 mask = SR_BP2 | SR_BP1 | SR_BP0;
> > +
> > + /* Check current Quad Enable bit value. */
> > + val = read_cr(nor);
> this should have been:
> val = read_sr(nor);
> sorry.
>
> > + dev_err(nor->dev, "%s val = %02x\n", val);
> and here
> dev_err(nor->dev, "%s val = %02x\n", __FUNCTION__, val);
m25p80 spi0.0: spi_nor_init val = 00
so the masking trick doesn't help :-(
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
