public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Frank Smith <smith@amirix.com>
To: u-boot@lists.denx.de
Subject: [U-Boot-Users] strataflash.c
Date: Sun, 11 May 2003 20:50:35 -0300	[thread overview]
Message-ID: <03May11.204944adt.7308@dragon.appliedmicro.ns.ca> (raw)
In-Reply-To: 92E171C34A30D51193CA000103CE74F00379059A@CAREX

Hi,

The more I look into strataflash.c, the more I'm starting to think
that it doesn't support x16 devices in x8 mode.

The problem I see is that the 'portwidth' value is used to drive
two things -- offset multiplier, and side of data reads/writes
to the flash device.

On a pure x16 device, the offset multiplier is 2 and the size of the
data read/writes is 2.  So portwidth is 2.

On an x16 device in x8 mode, however, the offset multiplier is 2,
but the size of data read/writes is 1.  What should portwidth be...?

I think the offset multiplier and size of data read/writes should be
driven by *different* values.  Or am I overlooking something obvious?


Thanks,
Frank.

----- Original Message -----
From: "Tom Guilliams" <tguilliams@sbs.com>
To: "'Frank Smith'" <smith@amirix.com>; <u-boot-users@lists.sourceforge.net>
Sent: Friday, May 09, 2003 7:14 PM
Subject: RE: [U-Boot-Users] strataflash.c


> Frank,
>
> I can tell you I've had some problems with a x16 in x8 mode in
> strataflash.c.  Basically, the flash_detect_cfi() routine detects my
> E28F128J3A chip as portwidth 2 (x16) and chipwidth 1 (x8).  All the CFI
info
> is read correctly.  However, any write accesses to chip (programming or
> erasing) fail upon the calculated offset (it's incorrectly doubled because
> of portwidth being 2).
>
> I had to manually force the portwidth to 1 (x8) after the flash_get_size()
> routine to make everything behave for any flash operations.
>
> I don't think the "detect" logic is correct in terms of port and chip
width
> for "some" devices.  However, the author - Brad Kemp - has said he's had
> success on numerous other flash width devices.
>
> Your problem sounds exactly like mine.  For a hack fix, set
>
> info->portwidth = 1;
>
> for the last thing flash_get_size() does before returning the size.
>
> Tom
>
> -----Original Message-----
> From: Frank Smith [mailto:smith at amirix.com]
> Sent: Friday, May 09, 2003 1:00 PM
> To: u-boot-users at lists.sourceforge.net
> Subject: [U-Boot-Users] strataflash.c
>
>
>
> Hello,
>
> I have a board with a couple of Intel RC28F320J3A Strataflash devices.
One
> device is wired to x16 mode, the other to x8 mode.
>
> I'm trying to use the strataflash.c driver with these flash chips.  The
> driver detects the flash that's in x16 mode, but not
> the one in x8 mode.  So I'm digging through the code to try to figure
> out what's up.  I've been successful in avoiding dealing with flash
> programming code up until now in my life.....
>
> If anyone knows anything about whether strataflash.c was designed to work
> with x16 devices in x8 mode, I'd love to hear from you! From what I've
seen
> so far, I'm tending to think that it doesn't.
>
>
> Thanks,
> Frank.
>
>
> -------------------------------------------------------
> Enterprise Linux Forum Conference & Expo, June 4-6, 2003, Santa Clara The
> only event dedicated to issues related to Linux enterprise solutions
> www.enterpriselinuxforum.com
>
> _______________________________________________
> U-Boot-Users mailing list
> U-Boot-Users at lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/u-boot-users
>
>
> -------------------------------------------------------
> Enterprise Linux Forum Conference & Expo, June 4-6, 2003, Santa Clara
> The only event dedicated to issues related to Linux enterprise solutions
> www.enterpriselinuxforum.com
>
> _______________________________________________
> U-Boot-Users mailing list
> U-Boot-Users at lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/u-boot-users
>

  reply	other threads:[~2003-05-11 23:50 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2003-05-09 22:14 [U-Boot-Users] strataflash.c Tom Guilliams
2003-05-11 23:50 ` Frank Smith [this message]
  -- strict thread matches above, loose matches on Subject: below --
2003-05-27 16:09 Brad Kemp
2003-05-13 18:51 Tom Guilliams
2003-05-09 20:00 Frank Smith

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=03May11.204944adt.7308@dragon.appliedmicro.ns.ca \
    --to=smith@amirix.com \
    --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