diff mbox

m25p80: fix detection of m25p16 flashes

Message ID 1225396786-23679-1-git-send-email-vapier@gentoo.org
State Accepted
Commit 9168ab861ae3eb8942da61d884a5c1980ba98a5f
Headers show

Commit Message

Mike Frysinger Oct. 30, 2008, 7:59 p.m. UTC 
Commit d0e8c47c58575b9131e786edb488fd029eba443e added support for extended ids
but seems to break on flashes which don't have an extended id defined.  If the
table does not have an extid defined, then we should ignore it.

Signed-off-by: Mike Frysinger <vapier@gentoo.org>
Signed-off-by: Michael Hennerich <Michael.Hennerich@analog.com>
---
 drivers/mtd/devices/m25p80.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

Comments

Andrew Morton Nov. 3, 2008, 9:37 p.m. UTC | #1 
On Thu, 30 Oct 2008 15:59:46 -0400
Mike Frysinger <vapier@gentoo.org> wrote:

> Commit d0e8c47c58575b9131e786edb488fd029eba443e added support for extended ids

Quoting a bare hash in the changelog is a bit reader-hostile and I
believe there's some git-related issue which can cause people to not be
able to find the commit at all (I'm not sure what the circumstances
are, but Linus has mentioned it a couple of times when asking people to
not quote bare commit IDs in changelogs).

So when referencing a previous commit please use this form:

   Commit d0e8c47c58575b9131e786edb488fd029eba443e ("m25p80.c
   extended jedec support").

Thanks.
diff mbox

Patch

diff --git a/drivers/mtd/devices/m25p80.c b/drivers/mtd/devices/m25p80.c
index a298fed..6188fd4 100644
--- a/drivers/mtd/devices/m25p80.c
+++ b/drivers/mtd/devices/m25p80.c
@@ -575,7 +575,7 @@  static struct flash_info *__devinit jedec_probe(struct spi_device *spi)
 			tmp < ARRAY_SIZE(m25p_data);
 			tmp++, info++) {
 		if (info->jedec_id == jedec) {
-			if (ext_jedec != 0 && info->ext_id != ext_jedec)
+			if (info->ext_id != 0 && info->ext_id != ext_jedec)
 				continue;
 			return info;
 		}