All of lore.kernel.org
 help / color / mirror / Atom feed
From: Frieder Schrempf <frieder.schrempf@kontron.de>
To: linux-mtd@lists.infradead.org, boris.brezillon@bootlin.com,
	linux-spi@vger.kernel.org
Cc: dwmw2@infradead.org, computersforpeace@gmail.com,
	marek.vasut@gmail.com, richard@nod.at, miquel.raynal@bootlin.com,
	broonie@kernel.org, david.wolfe@nxp.com, fabio.estevam@nxp.com,
	prabhakar.kushwaha@nxp.com, yogeshnarayan.gaur@nxp.com,
	han.xu@nxp.com, shawnguo@kernel.org,
	Frieder Schrempf <frieder.schrempf@exceet.de>,
	Rob Herring <robh+dt@kernel.org>,
	Mark Rutland <mark.rutland@arm.com>,
	devicetree@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: [PATCH v4 03/10] dt-bindings: spi: Adjust the bindings for the FSL QSPI driver
Date: Wed,  7 Nov 2018 15:43:20 +0100	[thread overview]
Message-ID: <1541601809-16950-4-git-send-email-frieder.schrempf@kontron.de> (raw)
In-Reply-To: <1541601809-16950-1-git-send-email-frieder.schrempf@kontron.de>

From: Frieder Schrempf <frieder.schrempf@exceet.de>

Adjust the documentation of the new SPI memory interface based
driver to reflect the new drivers settings.

The "old" driver was using the "fsl,qspi-has-second-chip" property to
select one of two dual chip setups (two chips on one bus or two chips
on separate buses). And it used the order in which the subnodes are
defined in the dt to select the CS, the chip is connected to.

Both methods are wrong and in fact the "reg" property should be used to
determine which bus and CS a chip is connected to. This also enables us
to use different setups than just single chip, or symmetric dual chip.

So the porting of the driver from the MTD to the SPI framework actually
enforces the use of the "reg" properties and makes
"fsl,qspi-has-second-chip" superfluous.

As all boards that have "fsl,qspi-has-second-chip" set, also have
correct "reg" properties, the removal of this property shouldn't lead to
any incompatibilities.

The only compatibility issues I can see are with imx6sx-sdb.dts and
imx6sx-sdb-reva.dts, which have their reg properties set incorrectly
(see explanation here: [2]), all other boards should stay compatible.

Also the "big-endian" flag was removed, as this setting is now selected
by the driver, depending on which SoC is in use.

Signed-off-by: Frieder Schrempf <frieder.schrempf@exceet.de>
---
 .../devicetree/bindings/spi/spi-fsl-qspi.txt    | 21 +++++++++-----------
 1 file changed, 9 insertions(+), 12 deletions(-)

diff --git a/Documentation/devicetree/bindings/spi/spi-fsl-qspi.txt b/Documentation/devicetree/bindings/spi/spi-fsl-qspi.txt
index 483e9cf..6d7c9ec 100644
--- a/Documentation/devicetree/bindings/spi/spi-fsl-qspi.txt
+++ b/Documentation/devicetree/bindings/spi/spi-fsl-qspi.txt
@@ -3,9 +3,8 @@
 Required properties:
   - compatible : Should be "fsl,vf610-qspi", "fsl,imx6sx-qspi",
 		 "fsl,imx7d-qspi", "fsl,imx6ul-qspi",
-		 "fsl,ls1021a-qspi"
+		 "fsl,ls1021a-qspi", "fsl,ls2080a-qspi"
 		 or
-		 "fsl,ls2080a-qspi" followed by "fsl,ls1021a-qspi",
 		 "fsl,ls1043a-qspi" followed by "fsl,ls1021a-qspi"
   - reg : the first contains the register location and length,
           the second contains the memory mapping address and length
@@ -14,15 +13,13 @@ Required properties:
   - clocks : The clocks needed by the QuadSPI controller
   - clock-names : Should contain the name of the clocks: "qspi_en" and "qspi".
 
-Optional properties:
-  - fsl,qspi-has-second-chip: The controller has two buses, bus A and bus B.
-                              Each bus can be connected with two NOR flashes.
-			      Most of the time, each bus only has one NOR flash
-			      connected, this is the default case.
-			      But if there are two NOR flashes connected to the
-			      bus, you should enable this property.
-			      (Please check the board's schematic.)
-  - big-endian : That means the IP register is big endian
+Required SPI slave node properties:
+  - reg: There are two buses (A and B) with two chip selects each.
+	 This encodes to which bus and CS the flash is connected:
+		<0>: Bus A, CS 0
+		<1>: Bus A, CS 1
+		<2>: Bus B, CS 0
+		<3>: Bus B, CS 1
 
 Example:
 
@@ -40,7 +37,7 @@ qspi0: quadspi@40044000 {
 	};
 };
 
-Example showing the usage of two SPI NOR devices:
+Example showing the usage of two SPI NOR devices on bus A:
 
 &qspi2 {
 	pinctrl-names = "default";
-- 
2.7.4


  parent reply	other threads:[~2018-11-07 14:44 UTC|newest]

Thread overview: 66+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-11-07 14:43 [PATCH v4 00/10] Port the FSL QSPI driver to the SPI framework Frieder Schrempf
2018-11-07 14:43 ` Frieder Schrempf
2018-11-07 14:43 ` [PATCH v4 01/10] spi: Add a driver for the Freescale/NXP QuadSPI controller Frieder Schrempf
2018-11-13  8:22   ` Yogesh Narayan Gaur
2018-11-13  8:22     ` Yogesh Narayan Gaur
2018-11-13  8:30     ` Yogesh Narayan Gaur
2018-11-13 13:56     ` Schrempf Frieder
2018-11-07 14:43 ` [PATCH v4 02/10] dt-bindings: spi: Move the bindings for the FSL QSPI driver Frieder Schrempf
2018-11-07 14:43   ` Frieder Schrempf
2018-11-08  8:37   ` Boris Brezillon
2018-11-08  8:54     ` Schrempf Frieder
2018-11-08  8:54       ` Schrempf Frieder
2018-11-08  8:54       ` Schrempf Frieder
2018-11-07 14:43 ` Frieder Schrempf [this message]
2018-11-08  8:41   ` [PATCH v4 03/10] dt-bindings: spi: Adjust " Boris Brezillon
2018-11-08  8:48     ` Schrempf Frieder
2018-11-08  8:48       ` Schrempf Frieder
2018-11-08  8:48       ` Schrempf Frieder
2018-11-07 14:43 ` [PATCH v4 04/10] ARM: dts: Reflect change of FSL QSPI driver and remove unused properties Frieder Schrempf
2018-11-07 14:43   ` Frieder Schrempf
2018-11-07 14:43 ` [PATCH v4 05/10] arm64: " Frieder Schrempf
2018-11-07 14:43   ` Frieder Schrempf
2018-11-07 14:43 ` [PATCH v4 06/10] ARM: defconfig: Use the new FSL QSPI driver under the SPI framework Frieder Schrempf
2018-11-07 14:43   ` Frieder Schrempf
2018-11-07 14:43   ` Frieder Schrempf
2018-11-07 16:20   ` Olof Johansson
2018-11-07 16:20     ` Olof Johansson
2018-11-07 16:20     ` Olof Johansson
2018-11-07 16:36     ` Schrempf Frieder
2018-11-07 16:36       ` Schrempf Frieder
2018-11-07 16:36       ` Schrempf Frieder
2018-11-07 23:08       ` Olof Johansson
2018-11-07 23:08         ` Olof Johansson
2018-11-07 23:08         ` Olof Johansson
2018-11-08  8:34       ` Boris Brezillon
2018-11-08  8:34         ` Boris Brezillon
2018-11-08  8:34         ` Boris Brezillon
2018-11-12 10:46         ` Schrempf Frieder
2018-11-12 10:46           ` Schrempf Frieder
2018-11-12 10:46           ` Schrempf Frieder
2018-11-12 10:56           ` Boris Brezillon
2018-11-12 10:56             ` Boris Brezillon
2018-11-12 10:56             ` Boris Brezillon
2018-11-12 11:24             ` Schrempf Frieder
2018-11-12 11:24               ` Schrempf Frieder
2018-11-12 11:24               ` Schrempf Frieder
2018-11-07 14:43 ` [PATCH v4 07/10] mtd: fsl-quadspi: Remove the driver as it was replaced by spi-fsl-qspi.c Frieder Schrempf
2018-11-07 14:43   ` Frieder Schrempf
2018-11-07 14:43 ` [PATCH v4 08/10] ARM: dts: ls1021a: Remove fsl,qspi-has-second-chip as it is not used Frieder Schrempf
2018-11-07 14:43   ` [PATCH v4 08/10] ARM: dts: ls1021a: Remove fsl, qspi-has-second-chip " Frieder Schrempf
2018-11-07 14:43   ` Frieder Schrempf
2018-11-07 14:43   ` Frieder Schrempf
2018-11-07 14:43 ` [PATCH v4 09/10] ARM64: dts: ls1046a: Remove fsl,qspi-has-second-chip " Frieder Schrempf
2018-11-07 14:43   ` [PATCH v4 09/10] ARM64: dts: ls1046a: Remove fsl, qspi-has-second-chip " Frieder Schrempf
2018-11-07 14:43   ` Frieder Schrempf
2018-11-07 14:43   ` Frieder Schrempf
2018-11-07 14:43 ` [PATCH v4 10/10] MAINTAINERS: Move the Freescale QSPI driver to the SPI framework Frieder Schrempf
2018-11-07 14:43   ` Frieder Schrempf
2018-11-07 14:53 ` [PATCH v4 00/10] Port the FSL " Boris Brezillon
2018-11-07 14:53   ` Boris Brezillon
2018-11-07 15:07   ` Schrempf Frieder
2018-11-07 15:07     ` Schrempf Frieder
2018-11-07 15:10     ` Boris Brezillon
2018-11-07 15:10       ` Boris Brezillon
2018-11-08  7:59       ` Schrempf Frieder
2018-11-08  7:59         ` Schrempf Frieder

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=1541601809-16950-4-git-send-email-frieder.schrempf@kontron.de \
    --to=frieder.schrempf@kontron.de \
    --cc=boris.brezillon@bootlin.com \
    --cc=broonie@kernel.org \
    --cc=computersforpeace@gmail.com \
    --cc=david.wolfe@nxp.com \
    --cc=devicetree@vger.kernel.org \
    --cc=dwmw2@infradead.org \
    --cc=fabio.estevam@nxp.com \
    --cc=frieder.schrempf@exceet.de \
    --cc=han.xu@nxp.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mtd@lists.infradead.org \
    --cc=linux-spi@vger.kernel.org \
    --cc=marek.vasut@gmail.com \
    --cc=mark.rutland@arm.com \
    --cc=miquel.raynal@bootlin.com \
    --cc=prabhakar.kushwaha@nxp.com \
    --cc=richard@nod.at \
    --cc=robh+dt@kernel.org \
    --cc=shawnguo@kernel.org \
    --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 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.