linux-kernel - RE: [PATCH 1/1] mtd m25p80: fix bug - ATmel spi flash fails to be copied to

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <8A42379416420646B9BFAC9682273B6D0183AE38@limkexm3.ad.analog.com>
Date:	Mon, 19 May 2008 10:12:46 +0100
From:	"Hennerich, Michael" <Michael.Hennerich@...log.com>
To:	"Bryan Wu" <cooloney@...nel.org>, <linux-mtd@...ts.infradead.org>,
	<linux-kernel@...r.kernel.org>, <mike@...roidmicros.com>
Cc:	"Michael Hennerich" <michael.hennerich@...log.com>
Subject: RE: [PATCH 1/1] mtd m25p80: fix bug - ATmel spi flash fails to be copied to

>+	spi_write(flash->spi, flash->command, 2);
>+
>+	return spi_write(flash->spi, flash->command, 2);


I just noticed that write_sr contains a duplicated spi_write.
Bryan sends out a new patch.

-Michael

>-----Original Message-----
>From: Bryan Wu [mailto:cooloney.lkml@...il.com] On Behalf Of Bryan Wu
>Sent: Samstag, 17. Mai 2008 12:52
>To: linux-mtd@...ts.infradead.org; linux-kernel@...r.kernel.org;
>mike@...roidmicros.com
>Cc: Michael Hennerich; Bryan Wu
>Subject: [PATCH 1/1] mtd m25p80: fix bug - ATmel spi flash fails to be
>copied to
>
>From: Michael Hennerich <michael.hennerich@...log.com>
>
>Atmel serial flash tends to power up with the protection status bits
set.
>http://blackfin.uclinux.org/gf/project/uclinux-
>dist/tracker/?action=TrackerItemEdit&tracker_item_id=4089
>
>Signed-off-by: Michael Hennerich <michael.hennerich@...log.com>
>Signed-off-by: Bryan Wu <cooloney@...nel.org>
>---
> drivers/mtd/devices/m25p80.c |   24 ++++++++++++++++++++++++
> 1 files changed, 24 insertions(+), 0 deletions(-)
>
>diff --git a/drivers/mtd/devices/m25p80.c
b/drivers/mtd/devices/m25p80.c
>index 25efd33..b10649b 100644
>--- a/drivers/mtd/devices/m25p80.c
>+++ b/drivers/mtd/devices/m25p80.c
>@@ -33,6 +33,7 @@
> /* Flash opcodes. */
> #define	OPCODE_WREN		0x06	/* Write enable */
> #define	OPCODE_RDSR		0x05	/* Read status register
*/
>+#define	OPCODE_WRSR		0x01	/* Write status register
1 byte */
> #define	OPCODE_NORM_READ	0x03	/* Read data bytes (low
frequency) */
> #define	OPCODE_FAST_READ	0x0b	/* Read data bytes (high
frequency) */
> #define	OPCODE_PP		0x02	/* Page program (up to
256 bytes) */
>@@ -112,6 +113,19 @@ static int read_sr(struct m25p *flash)
> 	return val;
> }
>
>+/*
>+ * Write status register 1 byte
>+ * Returns negative if error occurred.
>+ */
>+static int write_sr(struct m25p *flash, u8 val)
>+{
>+	flash->command[0] = OPCODE_WRSR;
>+	flash->command[1] = val;
>+
>+	spi_write(flash->spi, flash->command, 2);
>+
>+	return spi_write(flash->spi, flash->command, 2);
>+}
>
> /*
>  * Set write enable latch with Write Enable command.
>@@ -587,6 +601,16 @@ static int __devinit m25p_probe(struct spi_device
>*spi)
> 	mutex_init(&flash->lock);
> 	dev_set_drvdata(&spi->dev, flash);
>
>+	/*
>+	 * Atmel serial flash tend to power up
>+	 * with the software protection bits set
>+	 */
>+
>+	if (info->jedec_id >> 16 == 0x1f) {
>+		write_enable(flash);
>+		write_sr(flash, 0);
>+	}
>+
> 	if (data && data->name)
> 		flash->mtd.name = data->name;
> 	else
>--
>1.5.5
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/