All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Johan Adolfsson" <johan.adolfsson@axis.com>
To: "David Woodhouse" <dwmw2@infradead.org>,
	"Johan Adolfsson" <johan.adolfsson@axis.com>
Cc: "Jonas Holmberg" <jonas.holmberg@axis.com>, <mtd@infradead.org>
Subject: Re: Problems with cfi_cmdset_0002.c
Date: Mon, 12 Feb 2001 18:33:04 +0100	[thread overview]
Message-ID: <024701c09519$db036e10$0a070d0a@axis.se> (raw)
In-Reply-To: 21475.981998659@redhat.com


----- Original Message -----
From: David Woodhouse <dwmw2@infradead.org>
To: Johan Adolfsson <johana@axis.com>
Cc: Jonas Holmberg <jonashg@axis.com>; <mtd@infradead.org>
Sent: Monday, February 12, 2001 18:24
Subject: Re: Problems with cfi_cmdset_0002.c


>
> johan.adolfsson@axis.com said:
> > Wouldn't it be possible to check both the bootloc and the erase
> > regions and do something like this? :
>
> Would that help? I thought that _all_ the AMD chips had the smallest
> regions listed first - that's the whole problem. But the top-boot chips
you
> have don't have bootloc == CFI_BOTTOM_BOOT. Or am I misunderstanding?

It's probably mee...:-)
Jonas has those flash chips, I dont.
What does the bootloc say? Does it says something other the 2 and 3?

> If the erase regions are listed the wrong way round _and_ the byte in the
> extended header isn't set appropriately to tell you so, these AMD chips
> really are broken. We need to work round them explicitly with a table of
> JEDEC IDs for the broken chips, or just reject them and buy Intel chips
> instead.

We need a table like that for other AMD compatible non CFI chips as well
(e.g. Toshiba).
Jonas has a solution that seems to work here, I guess it should go into the
mtd tree soner or later?
We probably want the check against the list before doing the CFI probe.

> --
> dwmw2

/Johan




To unsubscribe, send "unsubscribe mtd" to majordomo@infradead.org

  reply	other threads:[~2001-02-12 17:38 UTC|newest]

Thread overview: 24+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2001-02-12 16:31 Problems with cfi_cmdset_0002.c Jonas Holmberg
2001-02-12 16:36 ` David Woodhouse
2001-02-12 17:14   ` Johan Adolfsson
2001-02-12 17:24     ` David Woodhouse
2001-02-12 17:33       ` Johan Adolfsson [this message]
  -- strict thread matches above, loose matches on Subject: below --
2001-02-15 15:49 Jonas Holmberg
2001-02-15 13:02 Jonas Holmberg
2001-02-15 12:44 Jonas Holmberg
2001-02-15 12:56 ` David Woodhouse
2001-02-15 11:01 Jonas Holmberg
2001-02-15 11:10 ` David Woodhouse
2001-02-14 15:41 Jonas Holmberg
2001-02-14 15:45 ` David Woodhouse
2001-02-13 12:53 Jamey Hicks
2001-02-13 13:31 ` David Woodhouse
2001-02-13 10:04 Jonas Holmberg
2001-02-13 10:15 ` David Woodhouse
2001-02-13  8:59 Jonas Holmberg
2001-02-13  8:31 Jonas Holmberg
2001-02-13  8:30 ` Johan Adolfsson
2001-02-13  9:17 ` David Woodhouse
2001-02-13  9:27   ` Johan Adolfsson
2001-02-12 12:08 Jonas Holmberg
2001-02-12 13:24 ` David Woodhouse

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to='024701c09519$db036e10$0a070d0a@axis.se' \
    --to=johan.adolfsson@axis.com \
    --cc=dwmw2@infradead.org \
    --cc=jonas.holmberg@axis.com \
    --cc=mtd@infradead.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.