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] Does U-boot supports Spansion S29GL512P NOR Flash?
Date: Tue, 13 Apr 2010 14:16:01 +0200	[thread overview]
Message-ID: <201004131416.01575.sr@denx.de> (raw)
In-Reply-To: <q2hb9c1f3ac1004130446h96cf8625ib66d42dee0728cdf@mail.gmail.com>

Hi Prakash,

On Tuesday 13 April 2010 13:46:11 prakash bedge wrote:
> >> Are you not using the mainline version of cfi_flash.c? If not, which
> >> "fixup code" are you referring to (link)?
> 
> I am using U-Boot 2009.08 version.

This version is outdated. I suggest to switch to current code.

> I used the fixup for M29W128GH chip from the patch you suggested earlier.
> In that code, I changed the condition to check chipwidth and it works.

You shouldn't need any additional code to support the Spansion S29GL chips. 
Please use the latest code from mainline U-Boot.
 
> But I have a concern now, as it works with by changing chipwidth to 16Bit
> or portwidth to 16Bit. This is making me a little bit confuse.
>  if(info->chipwidth == FLASH_CFI_16BIT)
>   info->portwidth = FLASH_CFI_8BIT;
> 
> if(info->chipwidth == FLASH_CFI_8BIT)
> info->portwidth = FLASH_CFI_16BIT;

???
 
> Can you please tell what is chipwidth and portwidth and how we can find
> that?

chipwidth: Width of the NOR FLASH chip, meaning count of data lines
	actively used.

portwidth: Width of the external bus (data lines).
 
> Also I am not getting the correct manufacturer and device ids.
> 
> From the flash control register I can see that my device is 16 Bit.It
> reprsent a bit set 1 saying it is a 16 bit device.
> Now is this flash chipwidth or flash portwidth?

chipwidth.
 
> To my flash chip 16 data lines and 26 address lines are connected.

So you have chipwidth *and* portwidth of 16! This is the most common use case. 
Again, please use the mainline CFI driver and give it another try.
 
> >> Perhaps a problem of incorrect unlocking addresses (byte vs. word
> 
> address)?
> Does it mean that unlocking addresses are not correct?
> Supoose if chip is in word mode then what should be the portwidth and
> chipwidth? What will be the ublocking address then?
> 
> I am using an utilty to erase flash and to load uboot image and it is
> working fine, only that it copies data at multple locations.

What kind of "utility"? Don't you use the BDI3000 for FLASH programming? This 
mirroring might a a result of an incorrect EBC setup.

Cheers,
Stefan

--
DENX Software Engineering GmbH,      MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich,  Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-0 Fax: (+49)-8142-66989-80 Email: office at denx.de

  reply	other threads:[~2010-04-13 12:16 UTC|newest]

Thread overview: 31+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-04-09  5:26 [U-Boot] Does U-boot supports Spansion S29GL512P NOR Flash? prakash bedge
2010-04-09  6:10 ` Stefan Roese
2010-04-09  7:22   ` prakash bedge
2010-04-09  7:35     ` Stefan Roese
2010-04-09  7:55       ` Frank Svendsbøe
2010-04-09  8:05       ` prakash bedge
2010-04-09  8:09         ` Stefan Roese
2010-04-09  9:10           ` Frank Svendsbøe
2010-04-09 12:11             ` prakash bedge
2010-04-09 13:27               ` Frank Svendsbøe
2010-04-12  4:13                 ` prakash bedge
2010-04-12 17:01                   ` Frank Svendsbøe
2010-04-12 17:15                   ` Frank Svendsbøe
2010-04-13  6:31                     ` prakash bedge
2010-04-13  8:34                       ` Stefan Roese
2010-04-13 11:46                         ` prakash bedge
2010-04-13 12:16                           ` Stefan Roese [this message]
2010-04-14  5:03                             ` prakash bedge
2010-04-14  7:38                               ` Frank Svendsbøe
2010-04-14 12:05                               ` Stefan Roese
2010-04-16 14:38                                 ` prakash bedge
2010-04-20  9:25                                   ` prakash bedge
2010-04-09 13:35               ` Stefan Roese
2010-04-12  4:18                 ` prakash bedge
2010-04-12  7:59                   ` Stefan Roese
2012-03-10 10:47   ` Balaji Sivakumar
2012-03-10 18:29     ` Dirk Behme
2012-03-10 19:10     ` Wolfgang Denk
2010-04-09  7:48 ` Frank Svendsbøe
2010-04-09  8:07   ` prakash bedge
2010-04-09  8:23     ` Frank Svendsbøe

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=201004131416.01575.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