All of lore.kernel.org
 help / color / mirror / Atom feed
From: Reinhard Meyer <u-boot@emk-elektronik.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] Non standard CFI detection tweaks
Date: Thu, 12 Aug 2010 15:14:20 +0200	[thread overview]
Message-ID: <4C63F3AC.4070201@emk-elektronik.de> (raw)
In-Reply-To: <201008121440.01113.sr@denx.de>

Stefan Roese wrote:
> Hi Rogan,
> 
> On Thursday 12 August 2010 14:07:03 Rogan Dawes wrote:
>> I found the following configuration snippet for OpenOCD for the DNS323
>> at http://wiki.dns323.info/hardware:jtag:
>>
>> #          driver   addr       size     chip_width  bus_width  options
>> flash bank cfi      0xff800000 0x800000 1           2          0
>>
>> It seems that the key here is that the bus_width is set to 2, even
>> though the chip_width is only 1.
> 
> Seems not to be so uncommon: 8bit device on a 16bit external bus.

It is mostly likely a 8/16 bit wide device in 8 bit mode on an 8 bit
external bus. Then all CFI data appears twice.

Reason: the chip presents CFI data correctly (from the flash point of view)
in 16 bit mode (flash A0 connected to CPU A1). If connected in 8 bit mode
(flash A0 connected to CPU A1, flash A-1 connected to CPU A0) it still
presents the data at the same adresses but the CFI driver gets misled
because its told to try for 8 bit wide flash.

Since that was an issue long ago, I made the board/emk/common/flash.c
(CFI) driver. Anyone is welcome to add that special handling for 16 bit
flash connected in 8 bit mode to the general CFI driver :)

Best Regards,
Reinhard

  reply	other threads:[~2010-08-12 13:14 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-07-13 20:00 [U-Boot] Non standard CFI detection tweaks Rogan Dawes
2010-07-19  9:34 ` Stefan Roese
2010-07-19 10:18   ` Rogan Dawes
2010-07-19 10:49     ` Stefan Roese
2010-07-19 11:05       ` Rogan Dawes
2010-07-19 11:50         ` Stefan Roese
2010-07-19 12:18           ` Rogan Dawes
2010-07-19 13:33             ` Stefan Roese
2010-07-19 14:37               ` Rogan Dawes
2010-07-20  7:58                 ` Stefan Roese
2010-07-20  8:38                   ` Rogan Dawes
2010-07-20 10:00                     ` Rogan Dawes
2010-07-20 10:06                       ` Reinhard Meyer
2010-07-20 10:22                       ` Reinhard Meyer
2010-07-20 10:29                         ` Rogan Dawes
2010-07-20 10:53                           ` Reinhard Meyer
2010-07-20 11:01                             ` Rogan Dawes
2010-07-23 10:17                               ` Albert ARIBAUD
2010-08-12 12:07                   ` Rogan Dawes
2010-08-12 12:40                     ` Stefan Roese
2010-08-12 13:14                       ` Reinhard Meyer [this message]
2010-08-12 13:26                         ` Rogan Dawes

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=4C63F3AC.4070201@emk-elektronik.de \
    --to=u-boot@emk-elektronik.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.