Linux-mtd Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Lukasz Majewski <lukma@denx.de>
To: Fabio Estevam <festevam@gmail.com>,
	linux-imx@nxp.com, linux-mtd@lists.infradead.org,
	Boris Brezillon <bbrezillon@kernel.org>,
	Frieder Schrempf <frieder.schrempf@kontron.de>,
	Yogesh Gaur <yogeshnarayan.gaur@nxp.com>,
	Suresh Gupta <suresh.gupta@nxp.com>
Subject: [imx][vybrid][qspi] Regression notification - vybrid vf610 QUADSPI - BK4 board
Date: Thu, 25 Jul 2019 00:11:14 +0200	[thread overview]
Message-ID: <20190725001114.0ceff19c@jawa> (raw)


[-- Attachment #1.1: Type: text/plain, Size: 2227 bytes --]

Dear All,

I'd like to report that after moving the fsl-quadspi.c driver
from ./drivers/mtd to ./drivers/spi (around 5.1-rc1) [1] the 
Vybrid's QUADSPI (even with single SPI-NOR memory connected) is not
working anymore (the chip ID read is FF FF FF).

This wouldn't be a problem per-se, but not all necessary code was moved
- especially for vybrid which has issues with some HW bugs [2]:

--------------->8----------------
 * The IC guy suggests we use the "AHB Command Read" which is faster
 * then the "IP Command Read". (What's more is that there is a bug in
 * the "IP Command Read" in the Vybrid.)
---------------8<----------------

I've explicitly asked NXP support for this HW issue [3] - but till now
there was no reply.

The official linux-imx (from meta-freescale) seems not to support vybrid
(vf610) anymore - and the newest available now kernel is 4.19 (with the
old fsl-quadspi.c driver available).



Hence the question - has anybody noticed this issue and (maybe) is
working on it?


My quick-hack is to revert the changes from [1] and apply some
"fixes" (which causes this driver to work on Vybrid) on the old driver
[4].


Maybe somebody from NXP (linux-imx@nxp.com ?) can ask HW guys what is
the exact problem for Vybrid's QUADSPI controller, so it can be fixed
properly in the new kernel?
It would be great is somebody from Linux/HW team could reply to the
original post [3].




Thanks in advance for _any_ help.


Note:

[1] - SHA1 IDs for the commits:

84d043185dbe0d1b4f6db575bd91c834d37e2f78
80261459804507a349daf754d6e5d835bb8578ae
78df30808961cd32f0517c7469886386b0680852
50f1242c674226dd866949f24043f5a1076ee242
fcf85e5c2ac0e57430f065c77407e33c7b036a9d

[2] -
https://elixir.bootlin.com/linux/v4.19-rc4/source/drivers/mtd/spi-nor/fsl-quadspi.c#L671

[3] - https://community.nxp.com/thread/485139

[4] -
https://github.com/lmajewski/y2038_kernel/commits/v5.2-vybrid-old-quadspi

Best regards,

Lukasz Majewski

--

DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-59 Fax: (+49)-8142-66989-80 Email: lukma@denx.de

[-- Attachment #1.2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

[-- Attachment #2: Type: text/plain, Size: 144 bytes --]

______________________________________________________
Linux MTD discussion mailing list
http://lists.infradead.org/mailman/listinfo/linux-mtd/

             reply	other threads:[~2019-07-24 22:11 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-07-24 22:11 Lukasz Majewski [this message]
2019-07-29 19:09 ` [imx][vybrid][qspi] Regression notification - vybrid vf610 QUADSPI - BK4 board Fabio Estevam
2019-07-29 20:43   ` Lukasz Majewski
2019-07-29 21:17     ` Fabio Estevam
2019-07-29 21:54       ` Lukasz Majewski
2019-07-30  3:48         ` Fabio Estevam
2019-07-30 15:14           ` Lukasz Majewski
2019-07-30  3:07     ` Andrey Smirnov

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=20190725001114.0ceff19c@jawa \
    --to=lukma@denx.de \
    --cc=bbrezillon@kernel.org \
    --cc=festevam@gmail.com \
    --cc=frieder.schrempf@kontron.de \
    --cc=linux-imx@nxp.com \
    --cc=linux-mtd@lists.infradead.org \
    --cc=suresh.gupta@nxp.com \
    --cc=yogeshnarayan.gaur@nxp.com \
    /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