public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Tolunay Orkun <listmember@orkun.us>
To: u-boot@lists.denx.de
Subject: [U-Boot-Users] Where does U-Boot's CFI driver check for top/bottom boot?
Date: Sun, 05 Nov 2006 01:56:12 -0600	[thread overview]
Message-ID: <454D991C.5080602@orkun.us> (raw)
In-Reply-To: <200611041104.36057.sr@denx.de>

Hi Stefan,

Stefan Roese wrote:
>> I just looked at the datasheet of your flash part as well as datasheet
>> of a couple of intel flash parts as well as the current code. As I
>> suspected for your particular part, it looks like they are using the
>> same values for "Erase Bank Area 1" and "Erase Bank Area 2" irrespective
>> of top boot or bottom boot flash. I think, this is fundamentally wrong
>> and non-compliant with the general CFI standard.
>>
>> I will look at some AMD part datasheets. If this is generally available
>> on all AMD and AMD like parts, we can add it as a patch for AMD only.
>> Otherwise, we will either add CONFIG_GEOMETRY_REVERSED definition or
>> restrict a patch to specific vendor ids (and possibly part ids) which is
>> then a pain to manage.
>>     
>
> I suggest that we also look at the linux mtd cfi driver to see, if and how 
> those devices are handled. I remember seeing something like "broken CFI 
> table" in the linux bootlog on some boards.
>
>   

I will check MTD CFI driver when I have a bit of free time. I just sent 
an email to the list regarding what I learned from AMD CFI 
specifications. It looks like if we can enumerate those early top/bottom 
devices that use AMD CFI 1.0 specification, we can come up with a patch 
that can handle this case transparently. AMD specification specifies two 
devices. Perhaps we can prepare patch based on these and if there 
happens to be more like these we can expand the list using future patches.

Tolunay

  reply	other threads:[~2006-11-05  7:56 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-11-03 16:15 [U-Boot-Users] Where does U-Boot's CFI driver check for top/bottom boot? Timur Tabi
     [not found] ` <454BC9B3.8030806@orkun.us>
2006-11-04 10:03   ` Stefan Roese
2006-11-05  7:56     ` Tolunay Orkun [this message]
2006-11-05  9:31       ` Wolfgang Denk
2006-11-05  5:51 ` Tolunay Orkun
2006-11-05  7:44   ` Tolunay Orkun
2006-11-06 19:13   ` Timur Tabi
2006-11-06 22:00   ` Timur Tabi
  -- strict thread matches above, loose matches on Subject: below --
2006-11-06 22:27 Yogi

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=454D991C.5080602@orkun.us \
    --to=listmember@orkun.us \
    --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