* [PATCH v2 0/5] arm64: tegra: fix DT definitions for GM20B GPU
@ 2016-03-15 2:58 Alexandre Courbot
2016-03-15 2:58 ` [PATCH v2 2/5] dt-bindings: gk20a: Document iommus property Alexandre Courbot
` (2 more replies)
0 siblings, 3 replies; 14+ messages in thread
From: Alexandre Courbot @ 2016-03-15 2:58 UTC (permalink / raw)
To: Stephen Warren, Thierry Reding
Cc: devicetree-u79uwXL29TY76Z2rM5mHXA,
linux-kernel-u79uwXL29TY76Z2rM5mHXA,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
linux-tegra-u79uwXL29TY76Z2rM5mHXA,
gnurou-Re5JQEeQqe8AvxtiuMwx3w, Alexandre Courbot
Small series that fixes/completes DT bindings for Tegra GPUs and add two
missing entries required for the Tegra210 GPU to operate properly.
Changes since v1:
- Renamed "pllg_ref" clock to "ref" in DT bindings
Alexandre Courbot (5):
dt-bindings: gk20a: Fix typo in compatible name
dt-bindings: gk20a: Document iommus property
dt-bindings: Add documentation for GM20B GPU
arm64: tegra210: Add reference clock to GM20B
arm64: tegra210: Add IOMMU node to GM20B
.../devicetree/bindings/gpu/nvidia,gk20a.txt | 35 +++++++++++++++++++---
arch/arm64/boot/dts/nvidia/tegra210.dtsi | 8 +++--
2 files changed, 37 insertions(+), 6 deletions(-)
--
2.7.2
^ permalink raw reply [flat|nested] 14+ messages in thread
* [PATCH v2 1/5] dt-bindings: gk20a: Fix typo in compatible name
[not found] ` <1458010724-10945-1-git-send-email-acourbot-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
@ 2016-03-15 2:58 ` Alexandre Courbot
[not found] ` <1458010724-10945-2-git-send-email-acourbot-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
2016-03-15 2:58 ` [PATCH v2 3/5] dt-bindings: Add documentation for GM20B GPU Alexandre Courbot
2016-03-15 2:58 ` [PATCH v2 5/5] arm64: tegra210: Add IOMMU node to GM20B Alexandre Courbot
2 siblings, 1 reply; 14+ messages in thread
From: Alexandre Courbot @ 2016-03-15 2:58 UTC (permalink / raw)
To: Stephen Warren, Thierry Reding
Cc: devicetree-u79uwXL29TY76Z2rM5mHXA,
linux-kernel-u79uwXL29TY76Z2rM5mHXA,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
linux-tegra-u79uwXL29TY76Z2rM5mHXA,
gnurou-Re5JQEeQqe8AvxtiuMwx3w, Alexandre Courbot
The correct compatible name is "nvidia,gk20a".
Signed-off-by: Alexandre Courbot <acourbot-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
---
Documentation/devicetree/bindings/gpu/nvidia,gk20a.txt | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/Documentation/devicetree/bindings/gpu/nvidia,gk20a.txt b/Documentation/devicetree/bindings/gpu/nvidia,gk20a.txt
index 23bfe8e1f7cc..914f0ff4020e 100644
--- a/Documentation/devicetree/bindings/gpu/nvidia,gk20a.txt
+++ b/Documentation/devicetree/bindings/gpu/nvidia,gk20a.txt
@@ -1,9 +1,9 @@
NVIDIA GK20A Graphics Processing Unit
Required properties:
-- compatible: "nvidia,<chip>-<gpu>"
+- compatible: "nvidia,<gpu>"
Currently recognized values:
- - nvidia,tegra124-gk20a
+ - nvidia,gk20a
- reg: Physical base address and length of the controller's registers.
Must contain two entries:
- first entry for bar0
--
2.7.2
^ permalink raw reply related [flat|nested] 14+ messages in thread
* [PATCH v2 2/5] dt-bindings: gk20a: Document iommus property
2016-03-15 2:58 [PATCH v2 0/5] arm64: tegra: fix DT definitions for GM20B GPU Alexandre Courbot
@ 2016-03-15 2:58 ` Alexandre Courbot
2016-03-18 20:41 ` Rob Herring
2016-03-15 2:58 ` [PATCH v2 4/5] arm64: tegra210: Add reference clock to GM20B Alexandre Courbot
[not found] ` <1458010724-10945-1-git-send-email-acourbot-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
2 siblings, 1 reply; 14+ messages in thread
From: Alexandre Courbot @ 2016-03-15 2:58 UTC (permalink / raw)
To: Stephen Warren, Thierry Reding
Cc: devicetree, linux-kernel, linux-arm-kernel, linux-tegra, gnurou,
Alexandre Courbot
GK20A can optionally make use of an IOMMU.
Signed-off-by: Alexandre Courbot <acourbot@nvidia.com>
---
Documentation/devicetree/bindings/gpu/nvidia,gk20a.txt | 4 ++++
1 file changed, 4 insertions(+)
diff --git a/Documentation/devicetree/bindings/gpu/nvidia,gk20a.txt b/Documentation/devicetree/bindings/gpu/nvidia,gk20a.txt
index 914f0ff4020e..1e3748337319 100644
--- a/Documentation/devicetree/bindings/gpu/nvidia,gk20a.txt
+++ b/Documentation/devicetree/bindings/gpu/nvidia,gk20a.txt
@@ -24,6 +24,9 @@ Required properties:
- reset-names: Must include the following entries:
- gpu
+Optional properties:
+- iommus: A reference to the IOMMU. See ../iommu/iommu.txt for details.
+
Example:
gpu@0,57000000 {
@@ -39,5 +42,6 @@ Example:
clock-names = "gpu", "pwr";
resets = <&tegra_car 184>;
reset-names = "gpu";
+ iommus = <&mc TEGRA_SWGROUP_GPU>;
status = "disabled";
};
--
2.7.2
^ permalink raw reply related [flat|nested] 14+ messages in thread
* [PATCH v2 3/5] dt-bindings: Add documentation for GM20B GPU
[not found] ` <1458010724-10945-1-git-send-email-acourbot-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
2016-03-15 2:58 ` [PATCH v2 1/5] dt-bindings: gk20a: Fix typo in compatible name Alexandre Courbot
@ 2016-03-15 2:58 ` Alexandre Courbot
[not found] ` <1458010724-10945-4-git-send-email-acourbot-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
2016-03-15 2:58 ` [PATCH v2 5/5] arm64: tegra210: Add IOMMU node to GM20B Alexandre Courbot
2 siblings, 1 reply; 14+ messages in thread
From: Alexandre Courbot @ 2016-03-15 2:58 UTC (permalink / raw)
To: Stephen Warren, Thierry Reding
Cc: devicetree-u79uwXL29TY76Z2rM5mHXA,
linux-kernel-u79uwXL29TY76Z2rM5mHXA,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
linux-tegra-u79uwXL29TY76Z2rM5mHXA,
gnurou-Re5JQEeQqe8AvxtiuMwx3w, Alexandre Courbot
GM20B's definition is mostly similar to GK20A's, but requires an
additional clock.
Signed-off-by: Alexandre Courbot <acourbot-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
---
.../devicetree/bindings/gpu/nvidia,gk20a.txt | 27 ++++++++++++++++++++--
1 file changed, 25 insertions(+), 2 deletions(-)
diff --git a/Documentation/devicetree/bindings/gpu/nvidia,gk20a.txt b/Documentation/devicetree/bindings/gpu/nvidia,gk20a.txt
index 1e3748337319..d9ad6b87fbbc 100644
--- a/Documentation/devicetree/bindings/gpu/nvidia,gk20a.txt
+++ b/Documentation/devicetree/bindings/gpu/nvidia,gk20a.txt
@@ -1,9 +1,10 @@
-NVIDIA GK20A Graphics Processing Unit
+NVIDIA Tegra Graphics Processing Units
Required properties:
- compatible: "nvidia,<gpu>"
Currently recognized values:
- nvidia,gk20a
+ - nvidia,gm20b
- reg: Physical base address and length of the controller's registers.
Must contain two entries:
- first entry for bar0
@@ -19,6 +20,9 @@ Required properties:
- clock-names: Must include the following entries:
- gpu
- pwr
+If the compatible string is "nvidia,gm20b", then the following clock
+is also required:
+ - ref
- resets: Must contain an entry for each entry in reset-names.
See ../reset/reset.txt for details.
- reset-names: Must include the following entries:
@@ -27,7 +31,7 @@ Required properties:
Optional properties:
- iommus: A reference to the IOMMU. See ../iommu/iommu.txt for details.
-Example:
+Example for GK20A:
gpu@0,57000000 {
compatible = "nvidia,gk20a";
@@ -45,3 +49,22 @@ Example:
iommus = <&mc TEGRA_SWGROUP_GPU>;
status = "disabled";
};
+
+Example for GM20B:
+
+ gpu@0,57000000 {
+ compatible = "nvidia,gm20b";
+ reg = <0x0 0x57000000 0x0 0x01000000>,
+ <0x0 0x58000000 0x0 0x01000000>;
+ interrupts = <GIC_SPI 157 IRQ_TYPE_LEVEL_HIGH>,
+ <GIC_SPI 158 IRQ_TYPE_LEVEL_HIGH>;
+ interrupt-names = "stall", "nonstall";
+ clocks = <&tegra_car TEGRA210_CLK_GPU>,
+ <&tegra_car TEGRA210_CLK_PLL_P_OUT5>,
+ <&tegra_car TEGRA210_CLK_PLL_G_REF>;
+ clock-names = "gpu", "pwr", "ref";
+ resets = <&tegra_car 184>;
+ reset-names = "gpu";
+ iommus = <&mc TEGRA_SWGROUP_GPU>;
+ status = "disabled";
+ };
--
2.7.2
^ permalink raw reply related [flat|nested] 14+ messages in thread
* [PATCH v2 4/5] arm64: tegra210: Add reference clock to GM20B
2016-03-15 2:58 [PATCH v2 0/5] arm64: tegra: fix DT definitions for GM20B GPU Alexandre Courbot
2016-03-15 2:58 ` [PATCH v2 2/5] dt-bindings: gk20a: Document iommus property Alexandre Courbot
@ 2016-03-15 2:58 ` Alexandre Courbot
[not found] ` <1458010724-10945-1-git-send-email-acourbot-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
2 siblings, 0 replies; 14+ messages in thread
From: Alexandre Courbot @ 2016-03-15 2:58 UTC (permalink / raw)
To: Stephen Warren, Thierry Reding
Cc: devicetree, linux-kernel, linux-arm-kernel, linux-tegra, gnurou,
Alexandre Courbot
This clock is required for the GPU to operate.
Signed-off-by: Alexandre Courbot <acourbot@nvidia.com>
---
arch/arm64/boot/dts/nvidia/tegra210.dtsi | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/arch/arm64/boot/dts/nvidia/tegra210.dtsi b/arch/arm64/boot/dts/nvidia/tegra210.dtsi
index 362c269946ff..04898cb25f0c 100644
--- a/arch/arm64/boot/dts/nvidia/tegra210.dtsi
+++ b/arch/arm64/boot/dts/nvidia/tegra210.dtsi
@@ -309,8 +309,9 @@
<GIC_SPI 158 IRQ_TYPE_LEVEL_HIGH>;
interrupt-names = "stall", "nonstall";
clocks = <&tegra_car TEGRA210_CLK_GPU>,
- <&tegra_car TEGRA210_CLK_PLL_P_OUT5>;
- clock-names = "gpu", "pwr";
+ <&tegra_car TEGRA210_CLK_PLL_P_OUT5>,
+ <&tegra_car TEGRA210_CLK_PLL_G_REF>;
+ clock-names = "gpu", "pwr", "ref";
resets = <&tegra_car 184>;
reset-names = "gpu";
status = "disabled";
--
2.7.2
^ permalink raw reply related [flat|nested] 14+ messages in thread
* [PATCH v2 5/5] arm64: tegra210: Add IOMMU node to GM20B
[not found] ` <1458010724-10945-1-git-send-email-acourbot-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
2016-03-15 2:58 ` [PATCH v2 1/5] dt-bindings: gk20a: Fix typo in compatible name Alexandre Courbot
2016-03-15 2:58 ` [PATCH v2 3/5] dt-bindings: Add documentation for GM20B GPU Alexandre Courbot
@ 2016-03-15 2:58 ` Alexandre Courbot
2 siblings, 0 replies; 14+ messages in thread
From: Alexandre Courbot @ 2016-03-15 2:58 UTC (permalink / raw)
To: Stephen Warren, Thierry Reding
Cc: devicetree-u79uwXL29TY76Z2rM5mHXA,
linux-kernel-u79uwXL29TY76Z2rM5mHXA,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
linux-tegra-u79uwXL29TY76Z2rM5mHXA,
gnurou-Re5JQEeQqe8AvxtiuMwx3w, Alexandre Courbot
Nouveau can take advantage of this declaration to remove the need for
contiguous memory.
Signed-off-by: Alexandre Courbot <acourbot-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
---
arch/arm64/boot/dts/nvidia/tegra210.dtsi | 3 +++
1 file changed, 3 insertions(+)
diff --git a/arch/arm64/boot/dts/nvidia/tegra210.dtsi b/arch/arm64/boot/dts/nvidia/tegra210.dtsi
index 04898cb25f0c..478543f74863 100644
--- a/arch/arm64/boot/dts/nvidia/tegra210.dtsi
+++ b/arch/arm64/boot/dts/nvidia/tegra210.dtsi
@@ -314,6 +314,9 @@
clock-names = "gpu", "pwr", "ref";
resets = <&tegra_car 184>;
reset-names = "gpu";
+
+ iommus = <&mc TEGRA_SWGROUP_GPU>;
+
status = "disabled";
};
--
2.7.2
^ permalink raw reply related [flat|nested] 14+ messages in thread
* Re: [PATCH v2 1/5] dt-bindings: gk20a: Fix typo in compatible name
[not found] ` <1458010724-10945-2-git-send-email-acourbot-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
@ 2016-03-18 20:31 ` Rob Herring
2016-03-20 6:55 ` Alexandre Courbot
0 siblings, 1 reply; 14+ messages in thread
From: Rob Herring @ 2016-03-18 20:31 UTC (permalink / raw)
To: Alexandre Courbot
Cc: Stephen Warren, Thierry Reding,
devicetree-u79uwXL29TY76Z2rM5mHXA,
linux-kernel-u79uwXL29TY76Z2rM5mHXA,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
linux-tegra-u79uwXL29TY76Z2rM5mHXA,
gnurou-Re5JQEeQqe8AvxtiuMwx3w
On Tue, Mar 15, 2016 at 11:58:40AM +0900, Alexandre Courbot wrote:
> The correct compatible name is "nvidia,gk20a".
>
> Signed-off-by: Alexandre Courbot <acourbot-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
> ---
> Documentation/devicetree/bindings/gpu/nvidia,gk20a.txt | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
Acked-by: Rob Herring <robh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
This may be what is used, but having the SoC name would be preferred
assuming gk20a is in multiple SoCs.
Rob
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH v2 2/5] dt-bindings: gk20a: Document iommus property
2016-03-15 2:58 ` [PATCH v2 2/5] dt-bindings: gk20a: Document iommus property Alexandre Courbot
@ 2016-03-18 20:41 ` Rob Herring
0 siblings, 0 replies; 14+ messages in thread
From: Rob Herring @ 2016-03-18 20:41 UTC (permalink / raw)
To: Alexandre Courbot
Cc: Stephen Warren, Thierry Reding, devicetree, linux-kernel,
linux-arm-kernel, linux-tegra, gnurou
On Tue, Mar 15, 2016 at 11:58:41AM +0900, Alexandre Courbot wrote:
> GK20A can optionally make use of an IOMMU.
>
> Signed-off-by: Alexandre Courbot <acourbot@nvidia.com>
> ---
> Documentation/devicetree/bindings/gpu/nvidia,gk20a.txt | 4 ++++
> 1 file changed, 4 insertions(+)
Acked-by: Rob Herring <robh@kernel.org>
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH v2 3/5] dt-bindings: Add documentation for GM20B GPU
[not found] ` <1458010724-10945-4-git-send-email-acourbot-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
@ 2016-03-18 20:47 ` Rob Herring
2016-03-20 6:55 ` Alexandre Courbot
0 siblings, 1 reply; 14+ messages in thread
From: Rob Herring @ 2016-03-18 20:47 UTC (permalink / raw)
To: Alexandre Courbot
Cc: Stephen Warren, Thierry Reding,
devicetree-u79uwXL29TY76Z2rM5mHXA,
linux-kernel-u79uwXL29TY76Z2rM5mHXA,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
linux-tegra-u79uwXL29TY76Z2rM5mHXA,
gnurou-Re5JQEeQqe8AvxtiuMwx3w
On Tue, Mar 15, 2016 at 11:58:42AM +0900, Alexandre Courbot wrote:
> GM20B's definition is mostly similar to GK20A's, but requires an
> additional clock.
>
> Signed-off-by: Alexandre Courbot <acourbot-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
> ---
> .../devicetree/bindings/gpu/nvidia,gk20a.txt | 27 ++++++++++++++++++++--
> 1 file changed, 25 insertions(+), 2 deletions(-)
Acked-by: Rob Herring <robh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
One nit below.
>
> diff --git a/Documentation/devicetree/bindings/gpu/nvidia,gk20a.txt b/Documentation/devicetree/bindings/gpu/nvidia,gk20a.txt
> index 1e3748337319..d9ad6b87fbbc 100644
> --- a/Documentation/devicetree/bindings/gpu/nvidia,gk20a.txt
> +++ b/Documentation/devicetree/bindings/gpu/nvidia,gk20a.txt
> @@ -1,9 +1,10 @@
> -NVIDIA GK20A Graphics Processing Unit
> +NVIDIA Tegra Graphics Processing Units
>
> Required properties:
> - compatible: "nvidia,<gpu>"
> Currently recognized values:
> - nvidia,gk20a
> + - nvidia,gm20b
> - reg: Physical base address and length of the controller's registers.
> Must contain two entries:
> - first entry for bar0
> @@ -19,6 +20,9 @@ Required properties:
> - clock-names: Must include the following entries:
> - gpu
> - pwr
> +If the compatible string is "nvidia,gm20b", then the following clock
> +is also required:
> + - ref
> - resets: Must contain an entry for each entry in reset-names.
> See ../reset/reset.txt for details.
> - reset-names: Must include the following entries:
> @@ -27,7 +31,7 @@ Required properties:
> Optional properties:
> - iommus: A reference to the IOMMU. See ../iommu/iommu.txt for details.
>
> -Example:
> +Example for GK20A:
>
> gpu@0,57000000 {
> compatible = "nvidia,gk20a";
> @@ -45,3 +49,22 @@ Example:
> iommus = <&mc TEGRA_SWGROUP_GPU>;
> status = "disabled";
> };
> +
> +Example for GM20B:
> +
> + gpu@0,57000000 {
Drop the comma and leading zero.
> + compatible = "nvidia,gm20b";
> + reg = <0x0 0x57000000 0x0 0x01000000>,
> + <0x0 0x58000000 0x0 0x01000000>;
> + interrupts = <GIC_SPI 157 IRQ_TYPE_LEVEL_HIGH>,
> + <GIC_SPI 158 IRQ_TYPE_LEVEL_HIGH>;
> + interrupt-names = "stall", "nonstall";
> + clocks = <&tegra_car TEGRA210_CLK_GPU>,
> + <&tegra_car TEGRA210_CLK_PLL_P_OUT5>,
> + <&tegra_car TEGRA210_CLK_PLL_G_REF>;
> + clock-names = "gpu", "pwr", "ref";
> + resets = <&tegra_car 184>;
> + reset-names = "gpu";
> + iommus = <&mc TEGRA_SWGROUP_GPU>;
> + status = "disabled";
> + };
> --
> 2.7.2
>
> --
> To unsubscribe from this list: send the line "unsubscribe devicetree" in
> the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH v2 1/5] dt-bindings: gk20a: Fix typo in compatible name
2016-03-18 20:31 ` Rob Herring
@ 2016-03-20 6:55 ` Alexandre Courbot
0 siblings, 0 replies; 14+ messages in thread
From: Alexandre Courbot @ 2016-03-20 6:55 UTC (permalink / raw)
To: Rob Herring
Cc: Alexandre Courbot, Stephen Warren, Thierry Reding,
devicetree-u79uwXL29TY76Z2rM5mHXA, Linux Kernel Mailing List,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
linux-tegra-u79uwXL29TY76Z2rM5mHXA
On Sat, Mar 19, 2016 at 5:31 AM, Rob Herring <robh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org> wrote:
> On Tue, Mar 15, 2016 at 11:58:40AM +0900, Alexandre Courbot wrote:
>> The correct compatible name is "nvidia,gk20a".
>>
>> Signed-off-by: Alexandre Courbot <acourbot-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
>> ---
>> Documentation/devicetree/bindings/gpu/nvidia,gk20a.txt | 4 ++--
>> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> Acked-by: Rob Herring <robh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
>
> This may be what is used, but having the SoC name would be preferred
> assuming gk20a is in multiple SoCs.
gk20a is exclusive to t124 (and so is gm20b to t210), so there can be
no confusion here. This is why we eventually went for the shorter
compatible string (and forgot to update the DT documentation).
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH v2 3/5] dt-bindings: Add documentation for GM20B GPU
2016-03-18 20:47 ` Rob Herring
@ 2016-03-20 6:55 ` Alexandre Courbot
[not found] ` <CAAVeFuKQvjb20aEh9_MBnrzEUjKoVc8yQu3a4Ego6Qaqe70M6A-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
0 siblings, 1 reply; 14+ messages in thread
From: Alexandre Courbot @ 2016-03-20 6:55 UTC (permalink / raw)
To: Rob Herring
Cc: Alexandre Courbot, Stephen Warren, Thierry Reding,
devicetree-u79uwXL29TY76Z2rM5mHXA, Linux Kernel Mailing List,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
linux-tegra-u79uwXL29TY76Z2rM5mHXA
On Sat, Mar 19, 2016 at 5:47 AM, Rob Herring <robh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org> wrote:
> On Tue, Mar 15, 2016 at 11:58:42AM +0900, Alexandre Courbot wrote:
>> GM20B's definition is mostly similar to GK20A's, but requires an
>> additional clock.
>>
>> Signed-off-by: Alexandre Courbot <acourbot-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
>> ---
>> .../devicetree/bindings/gpu/nvidia,gk20a.txt | 27 ++++++++++++++++++++--
>> 1 file changed, 25 insertions(+), 2 deletions(-)
>
> Acked-by: Rob Herring <robh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
>
> One nit below.
>
>>
>> diff --git a/Documentation/devicetree/bindings/gpu/nvidia,gk20a.txt b/Documentation/devicetree/bindings/gpu/nvidia,gk20a.txt
>> index 1e3748337319..d9ad6b87fbbc 100644
>> --- a/Documentation/devicetree/bindings/gpu/nvidia,gk20a.txt
>> +++ b/Documentation/devicetree/bindings/gpu/nvidia,gk20a.txt
>> @@ -1,9 +1,10 @@
>> -NVIDIA GK20A Graphics Processing Unit
>> +NVIDIA Tegra Graphics Processing Units
>>
>> Required properties:
>> - compatible: "nvidia,<gpu>"
>> Currently recognized values:
>> - nvidia,gk20a
>> + - nvidia,gm20b
>> - reg: Physical base address and length of the controller's registers.
>> Must contain two entries:
>> - first entry for bar0
>> @@ -19,6 +20,9 @@ Required properties:
>> - clock-names: Must include the following entries:
>> - gpu
>> - pwr
>> +If the compatible string is "nvidia,gm20b", then the following clock
>> +is also required:
>> + - ref
>> - resets: Must contain an entry for each entry in reset-names.
>> See ../reset/reset.txt for details.
>> - reset-names: Must include the following entries:
>> @@ -27,7 +31,7 @@ Required properties:
>> Optional properties:
>> - iommus: A reference to the IOMMU. See ../iommu/iommu.txt for details.
>>
>> -Example:
>> +Example for GK20A:
>>
>> gpu@0,57000000 {
>> compatible = "nvidia,gk20a";
>> @@ -45,3 +49,22 @@ Example:
>> iommus = <&mc TEGRA_SWGROUP_GPU>;
>> status = "disabled";
>> };
>> +
>> +Example for GM20B:
>> +
>> + gpu@0,57000000 {
>
> Drop the comma and leading zero.
Even though this is how it appears in the actual DT?
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH v2 3/5] dt-bindings: Add documentation for GM20B GPU
[not found] ` <CAAVeFuKQvjb20aEh9_MBnrzEUjKoVc8yQu3a4Ego6Qaqe70M6A-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
@ 2016-03-22 1:41 ` Rob Herring
[not found] ` <CAL_Jsq+SEOC1s_j8u3hMyDj5i9ROyAAN_-qmbsLrNvR3v2cXwA-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
0 siblings, 1 reply; 14+ messages in thread
From: Rob Herring @ 2016-03-22 1:41 UTC (permalink / raw)
To: Alexandre Courbot
Cc: Alexandre Courbot, Stephen Warren, Thierry Reding,
devicetree-u79uwXL29TY76Z2rM5mHXA, Linux Kernel Mailing List,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
linux-tegra-u79uwXL29TY76Z2rM5mHXA
On Sun, Mar 20, 2016 at 1:55 AM, Alexandre Courbot <gnurou-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> wrote:
> On Sat, Mar 19, 2016 at 5:47 AM, Rob Herring <robh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org> wrote:
>> On Tue, Mar 15, 2016 at 11:58:42AM +0900, Alexandre Courbot wrote:
>>> GM20B's definition is mostly similar to GK20A's, but requires an
>>> additional clock.
[...]
>>> gpu@0,57000000 {
>>> compatible = "nvidia,gk20a";
>>> @@ -45,3 +49,22 @@ Example:
>>> iommus = <&mc TEGRA_SWGROUP_GPU>;
>>> status = "disabled";
>>> };
>>> +
>>> +Example for GM20B:
>>> +
>>> + gpu@0,57000000 {
>>
>> Drop the comma and leading zero.
>
> Even though this is how it appears in the actual DT?
Yes, those will need to get fixed, too.
Rob
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH v2 3/5] dt-bindings: Add documentation for GM20B GPU
[not found] ` <CAL_Jsq+SEOC1s_j8u3hMyDj5i9ROyAAN_-qmbsLrNvR3v2cXwA-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
@ 2016-03-22 4:00 ` Alexandre Courbot
[not found] ` <56F0C375.1030907-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
0 siblings, 1 reply; 14+ messages in thread
From: Alexandre Courbot @ 2016-03-22 4:00 UTC (permalink / raw)
To: Rob Herring, Alexandre Courbot
Cc: Stephen Warren, Thierry Reding,
devicetree-u79uwXL29TY76Z2rM5mHXA, Linux Kernel Mailing List,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
linux-tegra-u79uwXL29TY76Z2rM5mHXA
On 03/22/2016 10:41 AM, Rob Herring wrote:
> On Sun, Mar 20, 2016 at 1:55 AM, Alexandre Courbot <gnurou-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> wrote:
>> On Sat, Mar 19, 2016 at 5:47 AM, Rob Herring <robh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org> wrote:
>>> On Tue, Mar 15, 2016 at 11:58:42AM +0900, Alexandre Courbot wrote:
>>>> GM20B's definition is mostly similar to GK20A's, but requires an
>>>> additional clock.
>
> [...]
>
>>>> gpu@0,57000000 {
>>>> compatible = "nvidia,gk20a";
>>>> @@ -45,3 +49,22 @@ Example:
>>>> iommus = <&mc TEGRA_SWGROUP_GPU>;
>>>> status = "disabled";
>>>> };
>>>> +
>>>> +Example for GM20B:
>>>> +
>>>> + gpu@0,57000000 {
>>>
>>> Drop the comma and leading zero.
>>
>> Even though this is how it appears in the actual DT?
>
> Yes, those will need to get fixed, too.
Sorry, I just want to confirm that I understand why this needs to be
fixed. The parent node has #address-cells = <2>, and the practice of
specifying two cells in the node name is consistent with what I see in
http://www.devicetree.org/Device_Tree_Usage.
However in the device tree usage example one can interpret the two cells
as being two different components of the address, whereas in our case we
are using two cells because the address is 64-bit - hence we should
specify it in the name as a single entity. Is this correct?
Thanks,
Alex.
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH v2 3/5] dt-bindings: Add documentation for GM20B GPU
[not found] ` <56F0C375.1030907-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
@ 2016-03-22 12:44 ` Rob Herring
0 siblings, 0 replies; 14+ messages in thread
From: Rob Herring @ 2016-03-22 12:44 UTC (permalink / raw)
To: Alexandre Courbot
Cc: Alexandre Courbot, Stephen Warren, Thierry Reding,
devicetree-u79uwXL29TY76Z2rM5mHXA, Linux Kernel Mailing List,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
linux-tegra-u79uwXL29TY76Z2rM5mHXA
On Mon, Mar 21, 2016 at 11:00 PM, Alexandre Courbot <acourbot-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org> wrote:
> On 03/22/2016 10:41 AM, Rob Herring wrote:
>>
>> On Sun, Mar 20, 2016 at 1:55 AM, Alexandre Courbot <gnurou-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
>> wrote:
>>>
>>> On Sat, Mar 19, 2016 at 5:47 AM, Rob Herring <robh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org> wrote:
>>>>
>>>> On Tue, Mar 15, 2016 at 11:58:42AM +0900, Alexandre Courbot wrote:
>>>>>
>>>>> GM20B's definition is mostly similar to GK20A's, but requires an
>>>>> additional clock.
>>
>>
>> [...]
>>
>>>>> gpu@0,57000000 {
>>>>> compatible = "nvidia,gk20a";
>>>>> @@ -45,3 +49,22 @@ Example:
>>>>> iommus = <&mc TEGRA_SWGROUP_GPU>;
>>>>> status = "disabled";
>>>>> };
>>>>> +
>>>>> +Example for GM20B:
>>>>> +
>>>>> + gpu@0,57000000 {
>>>>
>>>>
>>>> Drop the comma and leading zero.
>>>
>>>
>>> Even though this is how it appears in the actual DT?
>>
>>
>> Yes, those will need to get fixed, too.
>
>
> Sorry, I just want to confirm that I understand why this needs to be fixed.
> The parent node has #address-cells = <2>, and the practice of specifying two
> cells in the node name is consistent with what I see in
> http://www.devicetree.org/Device_Tree_Usage.
>
> However in the device tree usage example one can interpret the two cells as
> being two different components of the address, whereas in our case we are
> using two cells because the address is 64-bit - hence we should specify it
> in the name as a single entity. Is this correct?
Exactly, commas are for separating distinct fields like chip select
and offset in the wiki example. A 64-bit address is a single field.
The other reason to fix it is dtc is going to start warning for this.
Rob
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 14+ messages in thread
end of thread, other threads:[~2016-03-22 12:44 UTC | newest]
Thread overview: 14+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-03-15 2:58 [PATCH v2 0/5] arm64: tegra: fix DT definitions for GM20B GPU Alexandre Courbot
2016-03-15 2:58 ` [PATCH v2 2/5] dt-bindings: gk20a: Document iommus property Alexandre Courbot
2016-03-18 20:41 ` Rob Herring
2016-03-15 2:58 ` [PATCH v2 4/5] arm64: tegra210: Add reference clock to GM20B Alexandre Courbot
[not found] ` <1458010724-10945-1-git-send-email-acourbot-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
2016-03-15 2:58 ` [PATCH v2 1/5] dt-bindings: gk20a: Fix typo in compatible name Alexandre Courbot
[not found] ` <1458010724-10945-2-git-send-email-acourbot-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
2016-03-18 20:31 ` Rob Herring
2016-03-20 6:55 ` Alexandre Courbot
2016-03-15 2:58 ` [PATCH v2 3/5] dt-bindings: Add documentation for GM20B GPU Alexandre Courbot
[not found] ` <1458010724-10945-4-git-send-email-acourbot-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
2016-03-18 20:47 ` Rob Herring
2016-03-20 6:55 ` Alexandre Courbot
[not found] ` <CAAVeFuKQvjb20aEh9_MBnrzEUjKoVc8yQu3a4Ego6Qaqe70M6A-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2016-03-22 1:41 ` Rob Herring
[not found] ` <CAL_Jsq+SEOC1s_j8u3hMyDj5i9ROyAAN_-qmbsLrNvR3v2cXwA-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2016-03-22 4:00 ` Alexandre Courbot
[not found] ` <56F0C375.1030907-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
2016-03-22 12:44 ` Rob Herring
2016-03-15 2:58 ` [PATCH v2 5/5] arm64: tegra210: Add IOMMU node to GM20B Alexandre Courbot
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).