[U-Boot] [PATCH v2] spi: Tegra: add device tree binding doc for SPI/QSPI
This patch adds the device tree binding doc for the Tegra SPI/QSPI controllers on Tegra114 and Tegra210.
Signed-off-by: Tom Warren twarren@nvidia.com --- Changes in v2: - based it more on kernel's nvidia,tegra114-spi.txt binding - changes based on prelim review by swarren@nvidia.com
doc/device-tree-bindings/spi/spi-tegra.txt | 60 ++++++++++++++++++++++++++++++ 1 file changed, 60 insertions(+) create mode 100644 doc/device-tree-bindings/spi/spi-tegra.txt
diff --git a/doc/device-tree-bindings/spi/spi-tegra.txt b/doc/device-tree-bindings/spi/spi-tegra.txt new file mode 100644 index 0000000..3664267 --- /dev/null +++ b/doc/device-tree-bindings/spi/spi-tegra.txt @@ -0,0 +1,60 @@ +NVIDIA Tegra SPI/QSPI controller. + +Required properties: +- compatible : should be one of the following: + "nvidia,tegra114-spi" (for Tegra114) + "nvidia,tegra210-qspi" (for Tegra210) +- reg: Should contain SPI registers location and length. +- interrupts: Should contain SPI interrupts. +- clock-names : Must include the following entries: + - spi +- clocks : Must contain an entry for each entry in clock-names. + See ../clocks/clock-bindings.txt for details. +- resets : Must contain an entry for each entry in reset-names. + See ../reset/reset.txt for details. +- reset-names : Must include the following entries: + - spi + +Optional properties: +- dmas : Must contain an entry for each entry in clock-names. + See ../dma/dma.txt for details. +- dma-names : Must include the following entries: + - rx + - tx + +Recommended properties: +- spi-max-frequency: Definition as per spi-bus.txt + +Example: + +spi@7000d600 { + compatible = "nvidia,tegra114-spi"; + reg = <0x7000d600 0x200>; + interrupts = <0 82 0x04>; + spi-max-frequency = <25000000>; + #address-cells = <1>; + #size-cells = <0>; + clocks = <&tegra_car 44>; + clock-names = "spi"; + resets = <&tegra_car 44>; + reset-names = "spi"; + dmas = <&apbdma 16>, <&apbdma 16>; + dma-names = "rx", "tx"; + status = "disabled"; +}; + +spi@70410000 { + compatible = "nvidia,tegra210-qspi"; + reg = <0x0 0x70410000 0x0 0x1000>; + interrupts = <0 10 0x04>; + spi-max-frequency = <24000000>; + #address-cells = <1>; + #size-cells = <0>; + clocks = <&tegra_car 211>; + clock-names = "spi"; + resets = <&tegra_car 211>; + reset-names = "spi"; + dmas = <&apbdma 16>, <&apbdma 16>; + dma-names = "rx", "tx"; + status = "disabled"; +};
On 10/23/2015 05:30 PM, Tom Warren wrote:
This patch adds the device tree binding doc for the Tegra SPI/QSPI controllers on Tegra114 and Tegra210.
diff --git a/doc/device-tree-bindings/spi/spi-tegra.txt b/doc/device-tree-bindings/spi/spi-tegra.txt
Since this is a patch to the Linux kernel, it should use the path and file names from the Linux kernel, so that should be:
Documentation/devicetree/bindings/spi/nvidia,tegra210-qspi.txt
(or perhaps an earlier SoC number if this HW block first appeared before Tegra210)
+NVIDIA Tegra SPI/QSPI controller.
There's already a binding document for the regular SPI controller. The plain SPI entries should be removed from this file, leaving it covering the QSPI controller only.
Can you confirm that there physically is a separate QSPI controller in HW and that it has different registers to the regular SPI controller? Or, is QSPI simply a new mode of operation for the existing HW?
+Required properties: +- compatible : should be one of the following:
- "nvidia,tegra114-spi" (for Tegra114)
- "nvidia,tegra210-qspi" (for Tegra210)
+- reg: Should contain SPI registers location and length. +- interrupts: Should contain SPI interrupts.
Are there multiple interrupts or just one? I imagine just one, in which case just "interrupt" at the end of that description.
Stephen,
-----Original Message----- From: Stephen Warren [mailto:swarren@wwwdotorg.org] Sent: Monday, October 26, 2015 9:33 AM To: Tom Warren TWarren@nvidia.com; u-boot@lists.denx.de Cc: Stephen Warren swarren@nvidia.com; tomcwarren3959@gmail.com; jteki@openedev.com; robh+dt@kernel.org; pawel.moll@arm.com; mark.rutland@arm.com; ijc+devicetree@hellion.org.uk; galak@codeaurora.org; Thierry Reding treding@nvidia.com; linux- tegra@vger.kernel.org; Alex Courbot acourbot@nvidia.com; linux-arm- kernel@lists.infradead.org Subject: Re: [PATCH v2] spi: Tegra: add device tree binding doc for SPI/QSPI
On 10/23/2015 05:30 PM, Tom Warren wrote:
This patch adds the device tree binding doc for the Tegra SPI/QSPI controllers on Tegra114 and Tegra210.
diff --git a/doc/device-tree-bindings/spi/spi-tegra.txt b/doc/device-tree-bindings/spi/spi-tegra.txt
Since this is a patch to the Linux kernel, it should use the path and file names from the Linux kernel, so that should be:
Documentation/devicetree/bindings/spi/nvidia,tegra210-qspi.txt
I thought I was adding a binding doc to U-Boot, so the QSPI driver for T210 can go in. I wasn't aware that I had to submit a changelist against the kernel's binding first. I'll redo the file path.
(or perhaps an earlier SoC number if this HW block first appeared before Tegra210)
AFAICT, QSPI first appeared in Tegra210/X1.
+NVIDIA Tegra SPI/QSPI controller.
There's already a binding document for the regular SPI controller. The plain SPI entries should be removed from this file, leaving it covering the QSPI controller only.
There is no spi binding for Tegra114 in U-Boot at present, hence my duplication. I'll remove the SPI entries from this file, and perhaps copy over the kernel's tegra114 spi binding for U-Boot at a later date.
Can you confirm that there physically is a separate QSPI controller in HW and that it has different registers to the regular SPI controller? Or, is QSPI simply a new mode of operation for the existing HW?
QSPI is its own controller, separate from the traditional 4 Tegra SPI controllers. It has its own address space, and some differences in bits in some registers, and adds misc and timing registers that don't exist in the SPI controller. It supports Master mode only (no slave mode), and adds Dual (x2) and Quad mode (x4) transfer options. It's basically a superset of the Tegra SPI controller.
+Required properties: +- compatible : should be one of the following:
- "nvidia,tegra114-spi" (for Tegra114)
- "nvidia,tegra210-qspi" (for Tegra210)
+- reg: Should contain SPI registers location and length. +- interrupts: Should contain SPI interrupts.
Are there multiple interrupts or just one? I imagine just one, in which case just "interrupt" at the end of that description.
Only 1 interrupt for QSPI. Note that this was copied verbatim from the kernel's current nvidia,tegra114-spi.txt binding, so if it's wrong, it's wrong there, too.
Tom -- nvpublic
participants (3)
-
Stephen Warren -
Tom Warren -
Tom Warren