All of lore.kernel.org
 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 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.