From: Stefan Roese <sr@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot-Users] flash protection code in cfi_flash
Date: Fri, 17 Mar 2006 10:00:43 +0100 [thread overview]
Message-ID: <200603171000.44433.sr@denx.de> (raw)
In-Reply-To: <4dd15d180603161159x323e7293h5111f8777f77dc97@mail.gmail.com>
Hi David,
On Thursday, 16. March 2006 20:59, David Ho wrote:
> Some explanation is in order.
>
> While Intel flashes K3/C3 use the that command sequence as block
> unlock, the same command sequence (0x60 0xD0) is used to "clear all
> blocks bits" on the J3. This is restrictly speaking not an intel
> flash bug. The datasheet had not mentioned there is a change to the
> lock/unlock command in the datasheet revision history so it appears to
> have been there in the J3 datasheet from the start. Just that there
> is an inconsistency in the behaviour of the command sequence. They
> refer to it as Legacy lock/unlock in the Primary Vendor Specifc
> Exended Query Table. So information can be extracted from CFI
> attributes.
After digging through some Intel manuals it seems that you are correct here.
Good catch.
> With the evidence gathered thus far, it is simply small detail
> overlooked by the implementor of the original code, which is perfectly
> acceptable. Just that I would have hoped this thread elicited
> discussion that led to this discovery.
>
> I wonder how I can better present myself to make others more co-operative.
You did quite well. Sometimes you just need some patience (and endurance). ;-)
The best way to proceed (if nobody of the CFI experts objects) would be, if
you could create a patch to fix this problem using the "Legacy lock/unlock"
bit from the query table.
Best regards,
Stefan
next prev parent reply other threads:[~2006-03-17 9:00 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-03-14 19:15 [U-Boot-Users] flash protection code in cfi_flash David Ho
2006-03-14 20:58 ` Wolfgang Denk
2006-03-15 16:07 ` David Ho
2006-03-16 1:13 ` Tolunay Orkun
2006-03-16 17:17 ` David Ho
2006-03-16 17:57 ` Tolunay Orkun
2006-03-16 18:55 ` David Ho
2006-03-16 19:10 ` David Ho
2006-03-16 19:59 ` David Ho
2006-03-17 9:00 ` Stefan Roese [this message]
2006-03-30 22:43 ` David Ho
2006-03-17 22:38 ` Tolunay Orkun
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=200603171000.44433.sr@denx.de \
--to=sr@denx.de \
--cc=u-boot@lists.denx.de \
/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