From: Marek Vasut <marex@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [RFC PATCH 4/4 v3] mtd: sf: Add CONFIG_SPI_N25Q256A_RESET for software-reset
Date: Thu, 2 Oct 2014 13:23:30 +0200 [thread overview]
Message-ID: <201410021323.30310.marex@denx.de> (raw)
In-Reply-To: <20141002084052.GA10339@pollux.denx.de>
On Thursday, October 02, 2014 at 10:40:52 AM, Pavel Machek wrote:
> On Thu 2014-10-02 04:47:23, Marek Vasut wrote:
> > On Wednesday, October 01, 2014 at 09:04:48 PM, Jagan Teki wrote:
> > > On 2 October 2014 00:27, Stefan Roese <sr@denx.de> wrote:
> > > > On 01.10.2014 20:25, Marek Vasut wrote:
> > > >> On Wednesday, October 01, 2014 at 05:13:11 PM, Stefan Roese wrote:
> > > >>> This is needed for the SoCFPGA booting from SPI NOR flash
> > > >>> e.g. (N25Q256A). With these changes, the SoCrates can boot and
> > > >>> re-boot (reset) from SPI NOR flash without any problems.
> > > >>
> > > >> Seems like your SPI NOR reset logic is buggy. Does any of [1] apply
> > > >> to your
> > > >> board please?
> > > >>
> > > >> [1]
> > > >> http://www.rocketboards.org/foswiki/Documentation/SocBoardQspiBoot
> > > >
> > > > Yes. This seems to be that case. But I can't change it right now. So
> > > > this "solution" with the soft-reset is better than nothing.
> > >
> > > If this is some think that must require, any possibility to this
> > > resetting prior to u-boot?
> > > like preloader or in first stage boot loader or something.
> >
> > You do understand, that this is a hardware bug on one particular board,
> > right ? This can _not_ be reliably solved in software, not ever. I keep
> > seeing people implementing one such workaround after the other in
> > linux-mtd list, but sooner or later, they discover that their workaround
> > is not reliable. Without proper reset logic in place, a system simply
> > cannot reliably reboot, since it has no way to put all the hardware into
> > defined state.
>
> Well, if you have >16M flash and if you need bootrom to work with it.
>
> AFAICT, as long as you avoid using SPI from bootrom (socrates will happily
> run from SD card, for example), reliable operation should be possible. And
> you can still use SPI from Linux and u-boot...
SD has the same problem if you don't have proper reset logic for it ;-)
Best regards,
Marek Vasut
next prev parent reply other threads:[~2014-10-02 11:23 UTC|newest]
Thread overview: 46+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-10-01 15:13 [U-Boot] [PATCH 0/2 v3] arm: socfpga: Add Cadence QSPI support Stefan Roese
2014-10-01 15:13 ` [U-Boot] [PATCH 1/4 v3] spi: Add Cadence QSPI driver used by SoCFPGA Stefan Roese
2014-10-01 15:13 ` [U-Boot] [PATCH 2/4 v3] arm: socfpga: Add Cadence QSPI support to config header Stefan Roese
2014-10-01 15:13 ` [U-Boot] [PATCH 3/4 v3] arm: socfpga: Don't define CONFIG_SPI_FLASH_QUAD Stefan Roese
2014-10-01 15:13 ` [U-Boot] [RFC PATCH 4/4 v3] mtd: sf: Add CONFIG_SPI_N25Q256A_RESET for software-reset Stefan Roese
2014-10-01 18:25 ` Marek Vasut
2014-10-01 18:57 ` Stefan Roese
2014-10-01 19:04 ` Jagan Teki
2014-10-01 19:25 ` Stefan Roese
2014-10-01 23:07 ` Pavel Machek
2014-10-02 6:13 ` Stefan Roese
2014-10-02 2:47 ` Marek Vasut
2014-10-02 8:40 ` Pavel Machek
2014-10-02 11:23 ` Marek Vasut [this message]
2015-04-25 19:48 ` Pavel Machek
2015-04-27 16:35 ` Marek Vasut
2015-05-01 9:01 ` [U-Boot] [PATCH] " Pavel Machek
2015-05-01 14:24 ` Marek Vasut
2015-05-01 14:49 ` Pavel Machek
2015-05-01 17:26 ` Marek Vasut
2015-05-10 9:07 ` Pavel Machek
2015-05-10 9:15 ` [U-Boot] [PATCHv2] " Pavel Machek
2015-05-10 9:48 ` Marek Vasut
2015-05-10 10:49 ` [U-Boot] [PATCHv3] " Pavel Machek
2015-05-10 12:24 ` Jagan Teki
2015-05-10 16:25 ` Marek Vasut
2015-05-10 17:43 ` Pavel Machek
2015-05-11 7:47 ` Marek Vasut
2015-05-10 17:53 ` Pavel Machek
2015-05-11 7:48 ` Marek Vasut
2015-05-11 8:05 ` Jagan Teki
2015-05-11 8:29 ` Pavel Machek
2015-05-11 8:33 ` Jagan Teki
2015-05-11 8:39 ` Pavel Machek
2015-05-11 8:44 ` Jagan Teki
2015-05-11 8:50 ` Pavel Machek
2015-05-11 9:05 ` Jagan Teki
2015-05-11 9:56 ` Pavel Machek
2015-05-11 10:03 ` Jagan Teki
2015-05-15 8:47 ` Pavel Machek
2015-05-15 9:36 ` Marek Vasut
2015-05-15 9:55 ` Pavel Machek
2015-05-10 9:48 ` [U-Boot] [PATCH] " Marek Vasut
2015-04-25 19:44 ` [U-Boot] [RFC PATCH 4/4 v3] " Pavel Machek
2014-10-01 18:43 ` Jagan Teki
2014-10-03 20:58 ` [U-Boot] [PATCH 0/2 v3] arm: socfpga: Add Cadence QSPI support Marek Vasut
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=201410021323.30310.marex@denx.de \
--to=marex@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