All of lore.kernel.org
 help / color / mirror / Atom feed
* [U-Boot] Antwort: Re: QSPI "sf probe ...", "sf read ..." on Altera SoC FPGA
@ 2017-10-09 14:17 Goldschmidt Simon
  0 siblings, 0 replies; 12+ messages in thread
From: Goldschmidt Simon @ 2017-10-09 14:17 UTC (permalink / raw)
  To: u-boot

Hi Clémént,

> Did you also test the saveenv and sf unlock ?

I did test saveenv and it works. I did not test sf protection.

> Did you get some strange behaviors after a "warm reboot" from linux ?

Indeed, warm reboot fails. When rebooting via "reboot" command from
linux, the last thing I see is SPL writing "Trying to boot from SPI".

I haven't been able to debug this further, yet.

Also, I still can't sf read without disabling the data cache :-(

Regards,
Simon

^ permalink raw reply	[flat|nested] 12+ messages in thread
* [U-Boot] Antwort: Re: QSPI "sf probe ...", "sf read ..." on Altera SoC FPGA
@ 2017-09-29  9:23 Goldschmidt Simon
  2017-10-04 14:28 ` Clément Péron
  0 siblings, 1 reply; 12+ messages in thread
From: Goldschmidt Simon @ 2017-09-29  9:23 UTC (permalink / raw)
  To: u-boot

Hi Clement,

> Did you also test the saveenv and sf unlock ?

Not yet.

> Did you get some strange behaviors after a "warm reboot" from linux ?

Unfortunately, I'm not that far, yet. My Linux image only uses the sd-card for now.

Regards,
Simon

^ permalink raw reply	[flat|nested] 12+ messages in thread
* [U-Boot] Antwort: Re: QSPI "sf probe ...", "sf read ..." on Altera SoC FPGA
@ 2017-09-29  5:23 Goldschmidt Simon
  2017-09-29  8:44 ` Clément Péron
  0 siblings, 1 reply; 12+ messages in thread
From: Goldschmidt Simon @ 2017-09-29  5:23 UTC (permalink / raw)
  To: u-boot


On 09/27/2017 06:54 AM, Hannes Schmelzer wrote:
> On 09/22/2017 02:20 PM, Clément Péron wrote:
>> Sorry these are my local commits you can find them here :
>>
>> https://patchwork.ozlabs.org/patch/765992/
>> https://patchwork.ozlabs.org/patch/765996/
>> https://patchwork.ozlabs.org/patch/765997/
>> https://patchwork.ozlabs.org/patch/765998/
> Hi,
> just tested this on my cyclone5 board, but unfortunately without success.
> 
> ---
> U-Boot SPL 2017.09-00354-g824def8 (Sep 27 2017 - 06:47:19)
> (....)
> Hit any key to stop autoboot:  0
> => sf probe
> SF: Detected n25q512 with page size 256 Bytes, erase size 64 KiB, total
> 64 MiB
> ### ERROR ### Please RESET the board ###
> ----
> 
> same behavior as before.

Have you compared your dts to the other socfpga_cyclone5_*.dts files?

I have tested this on the socrates board and it would also give me the
above error until I change the "compatible" string of the flash chip
from "n25q00" to "spi-flash".

Without that, the sf code seems to try to use the first child node as
flash chip which results in a divide-by-zero error because it passes a
frequency of 0 Hz to the set_speed callback of the qspi driver, which
it is not prepared to handle:

If cadence_qspi_apb_config_baudrate_div is called with sclk_hz == 0,
the DIV_ROUND_UP macro fails to check the denominator for zero.

I wonder where this should be fixed: in the core sf code or in this
driver...

Regards,
Simon

^ permalink raw reply	[flat|nested] 12+ messages in thread
* [U-Boot] Antwort: Re: QSPI "sf probe ...", "sf read ..." on Altera SoC FPGA
@ 2017-09-29  5:23 Goldschmidt Simon
  0 siblings, 0 replies; 12+ messages in thread
From: Goldschmidt Simon @ 2017-09-29  5:23 UTC (permalink / raw)
  To: u-boot

On 09/22/2017 02:20 PM, Clément Péron wrote:
> Sorry these are my local commits you can find them here :
> 
> https://patchwork.ozlabs.org/patch/765992/
> https://patchwork.ozlabs.org/patch/765996/
> https://patchwork.ozlabs.org/patch/765997/
> https://patchwork.ozlabs.org/patch/765998/

Tested on socfpga_cyclone5_socrates by applying these 4 patches to
2017.09. Works as expected.
I had to disable the data cache to actually get the data from qspi to
ram, but that's a totally different issue.

Tested-by: Simon Goldschmidt <sgoldschmidt@de.pepperl-fuchs.com>

As I'm rather new to this list and project, when can we expect to have
this patch committed?

Regards,
Simon

^ permalink raw reply	[flat|nested] 12+ messages in thread
* [U-Boot] QSPI "sf probe ...", "sf read ..." on Altera SoC FPGA
@ 2017-09-01 11:53 Hannes Schmelzer
  2017-09-01 14:39 ` Jagan Teki
  0 siblings, 1 reply; 12+ messages in thread
From: Hannes Schmelzer @ 2017-09-01 11:53 UTC (permalink / raw)
  To: u-boot

Hi Eldor,

just found your post in the mailinglist.

https://lists.denx.de/pipermail/u-boot/2016-December/276491.html

Reason why i'm searched there is, that i've now excactly same problem as 
you.

----

----

I'm using most current mainline u-boot.

did you ever found some solution for your trouble ?

cheers,
Hannes

^ permalink raw reply	[flat|nested] 12+ messages in thread

end of thread, other threads:[~2017-10-09 14:17 UTC | newest]

Thread overview: 12+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-10-09 14:17 [U-Boot] Antwort: Re: QSPI "sf probe ...", "sf read ..." on Altera SoC FPGA Goldschmidt Simon
  -- strict thread matches above, loose matches on Subject: below --
2017-09-29  9:23 Goldschmidt Simon
2017-10-04 14:28 ` Clément Péron
2017-09-29  5:23 Goldschmidt Simon
2017-09-29  8:44 ` Clément Péron
2017-09-29  8:50   ` Hannes Schmelzer
2017-09-29  5:23 Goldschmidt Simon
2017-09-01 11:53 [U-Boot] " Hannes Schmelzer
2017-09-01 14:39 ` Jagan Teki
2017-09-04  6:22   ` [U-Boot] Antwort: " Hannes Schmelzer
2017-09-06  6:10     ` Hannes Schmelzer
2017-09-22 12:12       ` Clément Péron
2017-09-22 12:20         ` Clément Péron
2017-09-27  4:54           ` Hannes Schmelzer

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.