public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
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

  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