* [PATCH v4] Tegra: DT: add device tree binding doc for QSPI
@ 2015-10-26 22:22 Tom Warren
[not found] ` <1445898140-3561-1-git-send-email-twarren-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
0 siblings, 1 reply; 5+ messages in thread
From: Tom Warren @ 2015-10-26 22:22 UTC (permalink / raw)
To: devicetree-u79uwXL29TY76Z2rM5mHXA
Cc: swarren-DDmLM1+adcrQT0dZR+AlfA,
tomcwarren3959-Re5JQEeQqe8AvxtiuMwx3w,
jteki-oRp2ZoJdM/RWk0Htik3J/w, yelin-DDmLM1+adcrQT0dZR+AlfA,
robh+dt-DgEjT+Ai2ygdnm+yROfE0A, pawel.moll-5wv7dgnIgG8,
mark.rutland-5wv7dgnIgG8, ijc+devicetree-KcIKpvwj1kUDXYZnReoRVg,
galak-sgV2jX0FEOL9JmXXK+q4OQ, treding-DDmLM1+adcrQT0dZR+AlfA,
linux-tegra-u79uwXL29TY76Z2rM5mHXA,
acourbot-DDmLM1+adcrQT0dZR+AlfA,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, Tom Warren
This patch adds the device tree binding doc for the Tegra
QSPI controller on Tegra210.
Signed-off-by: Tom Warren <twarren-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
---
Changes in v2:
- based it more on kernel's nvidia,tegra114-spi.txt binding
- changes based on prelim review by swarren-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org
Changes in v3:
- renamed to 'nvidia,tegra210-qspi.txt' to match kernel bindings
- more changes based on review from swarren-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org
Changes in v4:
- move 'clock' up under 'clock-names' entry
Documentation/devicetree/bindings/spi/nvidia,tegra210-qspi.txt | 42 ++++++++++++++++++++++++++++++
1 file changed, 42 insertions(+)
create mode 100644 Documentation/devicetree/bindings/spi/nvidia,tegra210-qspi.txt
diff --git a/Documentation/devicetree/bindings/spi/nvidia,tegra210-qspi.txt b/Documentation/devicetree/bindings/spi/nvidia,tegra210-qspi.txt
new file mode 100644
index 0000000..fe1558f
--- /dev/null
+++ b/Documentation/devicetree/bindings/spi/nvidia,tegra210-qspi.txt
@@ -0,0 +1,42 @@
+NVIDIA Tegra QSPI controller.
+
+Required properties:
+- compatible : for Tegra210, must contain "nvidia,tegra210-qspi".
+- reg: Should contain QSPI registers location and length.
+- interrupts: Should contain QSPI interrupt.
+- clock-names : Must include the following entries:
+ - qspi
+- 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:
+ - qspi
+
+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@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 = "qspi";
+ resets = <&tegra_car 211>;
+ reset-names = "qspi";
+ dmas = <&apbdma 16>, <&apbdma 16>;
+ dma-names = "rx", "tx";
+ status = "disabled";
+};
--
1.8.2.1.610.g562af5b
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH v4] Tegra: DT: add device tree binding doc for QSPI
[not found] ` <1445898140-3561-1-git-send-email-twarren-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
@ 2015-10-26 23:02 ` Stephen Warren
[not found] ` <562EB11D.5040706-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>
2015-10-26 23:09 ` Rob Herring
1 sibling, 1 reply; 5+ messages in thread
From: Stephen Warren @ 2015-10-26 23:02 UTC (permalink / raw)
To: Tom Warren
Cc: devicetree-u79uwXL29TY76Z2rM5mHXA,
swarren-DDmLM1+adcrQT0dZR+AlfA,
tomcwarren3959-Re5JQEeQqe8AvxtiuMwx3w,
jteki-oRp2ZoJdM/RWk0Htik3J/w, yelin-DDmLM1+adcrQT0dZR+AlfA,
robh+dt-DgEjT+Ai2ygdnm+yROfE0A, pawel.moll-5wv7dgnIgG8,
mark.rutland-5wv7dgnIgG8, ijc+devicetree-KcIKpvwj1kUDXYZnReoRVg,
galak-sgV2jX0FEOL9JmXXK+q4OQ, treding-DDmLM1+adcrQT0dZR+AlfA,
linux-tegra-u79uwXL29TY76Z2rM5mHXA,
acourbot-DDmLM1+adcrQT0dZR+AlfA,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r
On 10/26/2015 04:22 PM, Tom Warren wrote:
> This patch adds the device tree binding doc for the Tegra
> QSPI controller on Tegra210.
Acked-by: Stephen Warren <swarren-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
(I assume if others approve the binding, Thierry will take it through
the Tegra tree)
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH v4] Tegra: DT: add device tree binding doc for QSPI
[not found] ` <1445898140-3561-1-git-send-email-twarren-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
2015-10-26 23:02 ` Stephen Warren
@ 2015-10-26 23:09 ` Rob Herring
1 sibling, 0 replies; 5+ messages in thread
From: Rob Herring @ 2015-10-26 23:09 UTC (permalink / raw)
To: Tom Warren
Cc: devicetree-u79uwXL29TY76Z2rM5mHXA, Stephen Warren,
tomcwarren3959-Re5JQEeQqe8AvxtiuMwx3w, Jagan Teki,
yelin-DDmLM1+adcrQT0dZR+AlfA, Pawel Moll, Mark Rutland,
Ian Campbell, Kumar Gala, Thierry Reding,
linux-tegra-u79uwXL29TY76Z2rM5mHXA,
acourbot-DDmLM1+adcrQT0dZR+AlfA,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r
On Mon, Oct 26, 2015 at 5:22 PM, Tom Warren <twarren-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org> wrote:
> This patch adds the device tree binding doc for the Tegra
> QSPI controller on Tegra210.
>
> Signed-off-by: Tom Warren <twarren-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
Acked-by: Rob Herring <robh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
> ---
> Changes in v2:
> - based it more on kernel's nvidia,tegra114-spi.txt binding
> - changes based on prelim review by swarren-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org
> Changes in v3:
> - renamed to 'nvidia,tegra210-qspi.txt' to match kernel bindings
> - more changes based on review from swarren-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org
> Changes in v4:
> - move 'clock' up under 'clock-names' entry
>
> Documentation/devicetree/bindings/spi/nvidia,tegra210-qspi.txt | 42 ++++++++++++++++++++++++++++++
> 1 file changed, 42 insertions(+)
> create mode 100644 Documentation/devicetree/bindings/spi/nvidia,tegra210-qspi.txt
>
> diff --git a/Documentation/devicetree/bindings/spi/nvidia,tegra210-qspi.txt b/Documentation/devicetree/bindings/spi/nvidia,tegra210-qspi.txt
> new file mode 100644
> index 0000000..fe1558f
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/spi/nvidia,tegra210-qspi.txt
> @@ -0,0 +1,42 @@
> +NVIDIA Tegra QSPI controller.
> +
> +Required properties:
> +- compatible : for Tegra210, must contain "nvidia,tegra210-qspi".
> +- reg: Should contain QSPI registers location and length.
> +- interrupts: Should contain QSPI interrupt.
> +- clock-names : Must include the following entries:
> + - qspi
> +- 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:
> + - qspi
> +
> +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@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 = "qspi";
> + resets = <&tegra_car 211>;
> + reset-names = "qspi";
> + dmas = <&apbdma 16>, <&apbdma 16>;
> + dma-names = "rx", "tx";
> + status = "disabled";
> +};
> --
> 1.8.2.1.610.g562af5b
>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH v4] Tegra: DT: add device tree binding doc for QSPI
[not found] ` <562EB11D.5040706-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>
@ 2015-11-09 12:45 ` Thierry Reding
[not found] ` <20151109124545.GD23941-AwZRO8vwLAwmlAP/+Wk3EA@public.gmane.org>
0 siblings, 1 reply; 5+ messages in thread
From: Thierry Reding @ 2015-11-09 12:45 UTC (permalink / raw)
To: Stephen Warren
Cc: Tom Warren, devicetree-u79uwXL29TY76Z2rM5mHXA,
swarren-DDmLM1+adcrQT0dZR+AlfA,
tomcwarren3959-Re5JQEeQqe8AvxtiuMwx3w,
jteki-oRp2ZoJdM/RWk0Htik3J/w, yelin-DDmLM1+adcrQT0dZR+AlfA,
robh+dt-DgEjT+Ai2ygdnm+yROfE0A, pawel.moll-5wv7dgnIgG8,
mark.rutland-5wv7dgnIgG8, ijc+devicetree-KcIKpvwj1kUDXYZnReoRVg,
galak-sgV2jX0FEOL9JmXXK+q4OQ, linux-tegra-u79uwXL29TY76Z2rM5mHXA,
acourbot-DDmLM1+adcrQT0dZR+AlfA,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r
[-- Attachment #1: Type: text/plain, Size: 567 bytes --]
On Mon, Oct 26, 2015 at 05:02:53PM -0600, Stephen Warren wrote:
> On 10/26/2015 04:22 PM, Tom Warren wrote:
> >This patch adds the device tree binding doc for the Tegra
> >QSPI controller on Tegra210.
>
> Acked-by: Stephen Warren <swarren-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
>
> (I assume if others approve the binding, Thierry will take it through the
> Tegra tree)
Do we have a kernel driver and hardware where this can be tested? I'm
slightly reluctant to merge a binding through the Tegra tree without a
driver that implements it.
Thierry
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 819 bytes --]
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH v4] Tegra: DT: add device tree binding doc for QSPI
[not found] ` <20151109124545.GD23941-AwZRO8vwLAwmlAP/+Wk3EA@public.gmane.org>
@ 2015-11-09 15:17 ` Stephen Warren
0 siblings, 0 replies; 5+ messages in thread
From: Stephen Warren @ 2015-11-09 15:17 UTC (permalink / raw)
To: Thierry Reding
Cc: Tom Warren, devicetree-u79uwXL29TY76Z2rM5mHXA,
swarren-DDmLM1+adcrQT0dZR+AlfA,
tomcwarren3959-Re5JQEeQqe8AvxtiuMwx3w,
jteki-oRp2ZoJdM/RWk0Htik3J/w, yelin-DDmLM1+adcrQT0dZR+AlfA,
robh+dt-DgEjT+Ai2ygdnm+yROfE0A, pawel.moll-5wv7dgnIgG8,
mark.rutland-5wv7dgnIgG8, ijc+devicetree-KcIKpvwj1kUDXYZnReoRVg,
galak-sgV2jX0FEOL9JmXXK+q4OQ, linux-tegra-u79uwXL29TY76Z2rM5mHXA,
acourbot-DDmLM1+adcrQT0dZR+AlfA,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r
On 11/09/2015 05:45 AM, Thierry Reding wrote:
> On Mon, Oct 26, 2015 at 05:02:53PM -0600, Stephen Warren wrote:
>> On 10/26/2015 04:22 PM, Tom Warren wrote:
>>> This patch adds the device tree binding doc for the Tegra
>>> QSPI controller on Tegra210.
>>
>> Acked-by: Stephen Warren <swarren-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
>>
>> (I assume if others approve the binding, Thierry will take it through the
>> Tegra tree)
>
> Do we have a kernel driver and hardware where this can be tested? I'm
> slightly reluctant to merge a binding through the Tegra tree without a
> driver that implements it.
AFAIK, the only HW this can be tested on is a one-off modified version
of p2571. The standard boards don't have this. I don't believe there's
an (upstream at least) kernel driver for this. I'm sure downstream has a
driver.
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2015-11-09 15:17 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-10-26 22:22 [PATCH v4] Tegra: DT: add device tree binding doc for QSPI Tom Warren
[not found] ` <1445898140-3561-1-git-send-email-twarren-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
2015-10-26 23:02 ` Stephen Warren
[not found] ` <562EB11D.5040706-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>
2015-11-09 12:45 ` Thierry Reding
[not found] ` <20151109124545.GD23941-AwZRO8vwLAwmlAP/+Wk3EA@public.gmane.org>
2015-11-09 15:17 ` Stephen Warren
2015-10-26 23:09 ` Rob Herring
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).