From: "Alexander Belyakov" <abelyako@googlemail.com>
To: "Uwe Kleine-König" <Uwe.Kleine-Koenig@digi.com>
Cc: linux-mtd@lists.infradead.org
Subject: Re: [PATCH 2/4] [RFC] [MTD] cfi_probe: remove Intel chip workaround
Date: Mon, 28 Jul 2008 11:21:13 +0400 [thread overview]
Message-ID: <fe810cf0807280021y798b91f0h63bae54cd0082bb1@mail.gmail.com> (raw)
In-Reply-To: <1216712341-25869-3-git-send-email-Uwe.Kleine-Koenig@digi.com>
On Tue, Jul 22, 2008 at 11:38 AM, Uwe Kleine-König
<Uwe.Kleine-Koenig@digi.com> wrote:
> For me this workaround breaks detection of a M29W128GL flash because it
> doesn't switch to CFI Query mode after writing 0xff.
>
> This workaround was introduced in 2004 between v2.6.8-rc1 and
> v2.6.8-rc2 in CVS revision 1.77 of cfi_probe.c. ("Send 0xFF to switch Intel
> chips back to read mode too. Some of them ignore 0xF0")
>
> Alternatively writing 0xff *before* 0xf0 works with the M29W128GL.
>
As far as I know M29W128GL is only chip having troubles with 0xFF.
Even M29DW128GL works fine with that.
Writing 0xFF before 0xF0 works fine both M29W128GL, M29DW128GL and
Intel's flash (we have tried with M18). Not sure though if it would
work for *all* flash parts.
Thank you,
Alexander
next prev parent reply other threads:[~2008-07-28 7:21 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-07-22 7:38 [PATCH 0/4] some mtd fixes Uwe Kleine-König
2008-07-22 7:38 ` [PATCH 1/4] [MTD] remove bogus warning about missing boot bank location Uwe Kleine-König
2008-07-22 7:38 ` [PATCH 2/4] [RFC] [MTD] cfi_probe: remove Intel chip workaround Uwe Kleine-König
2008-07-22 7:39 ` [PATCH 3/4] [MTD] physmap: Fix suspend/resume/shutdown bugs Uwe Kleine-König
2008-07-22 7:39 ` [PATCH 4/4] [MTD] physmap: resume already suspended chips on failure to suspend Uwe Kleine-König
2008-07-25 13:52 ` [PATCH 2/4] [RFC] [MTD] cfi_probe: remove Intel chip workaround David Woodhouse
2008-07-25 20:45 ` Uwe Kleine-König
2008-07-28 7:21 ` Alexander Belyakov [this message]
2008-07-28 14:05 ` Alexey Korolev
2008-08-05 14:41 ` Alexey Korolev
2008-10-19 5:33 ` [PATCH 1/4] [MTD] remove bogus warning about missing boot bank location Christopher Moore
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=fe810cf0807280021y798b91f0h63bae54cd0082bb1@mail.gmail.com \
--to=abelyako@googlemail.com \
--cc=Uwe.Kleine-Koenig@digi.com \
--cc=linux-mtd@lists.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox