All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH v2 1/2] dt-bindings: tegra: Remove 0, prefix from unit-addresses
@ 2016-04-12 15:07 Thierry Reding
       [not found] ` <1460473656-27209-1-git-send-email-thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
  0 siblings, 1 reply; 6+ messages in thread
From: Thierry Reding @ 2016-04-12 15:07 UTC (permalink / raw)
  To: Rob Herring
  Cc: Stephen Warren, Alexandre Courbot,
	devicetree-u79uwXL29TY76Z2rM5mHXA,
	linux-tegra-u79uwXL29TY76Z2rM5mHXA

From: Thierry Reding <treding-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>

When Tegra124 support was first merged the unit-addresses of all devices
were listed with a "0," prefix to encode the reg property's second cell.
It turns out that this notation is not correct, and the "," separator is
only used to separate fields in the unit address (such as the device and
function number in PCI devices), not individual cells for addresses
with more than one cell.

Acked-by: Stephen Warren <swarren-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
Signed-off-by: Thierry Reding <treding-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
---
 Documentation/devicetree/bindings/clock/nvidia,tegra124-dfll.txt    | 2 +-
 Documentation/devicetree/bindings/gpu/nvidia,gk20a.txt              | 2 +-
 .../devicetree/bindings/memory-controllers/nvidia,tegra-mc.txt      | 6 +++---
 Documentation/devicetree/bindings/memory-controllers/tegra-emc.txt  | 4 ++--
 .../devicetree/bindings/pinctrl/nvidia,tegra124-xusb-padctl.txt     | 6 +++---
 Documentation/devicetree/bindings/sound/nvidia,tegra30-hda.txt      | 2 +-
 Documentation/devicetree/bindings/thermal/tegra-soctherm.txt        | 2 +-
 7 files changed, 12 insertions(+), 12 deletions(-)

diff --git a/Documentation/devicetree/bindings/clock/nvidia,tegra124-dfll.txt b/Documentation/devicetree/bindings/clock/nvidia,tegra124-dfll.txt
index ee7e5fd4a50b..63f9d8277d48 100644
--- a/Documentation/devicetree/bindings/clock/nvidia,tegra124-dfll.txt
+++ b/Documentation/devicetree/bindings/clock/nvidia,tegra124-dfll.txt
@@ -50,7 +50,7 @@ Required properties for I2C mode:
 
 Example:
 
-clock@0,70110000 {
+clock@70110000 {
         compatible = "nvidia,tegra124-dfll";
         reg = <0 0x70110000 0 0x100>, /* DFLL control */
               <0 0x70110000 0 0x100>, /* I2C output control */
diff --git a/Documentation/devicetree/bindings/gpu/nvidia,gk20a.txt b/Documentation/devicetree/bindings/gpu/nvidia,gk20a.txt
index 23bfe8e1f7cc..9d47a2c25e2d 100644
--- a/Documentation/devicetree/bindings/gpu/nvidia,gk20a.txt
+++ b/Documentation/devicetree/bindings/gpu/nvidia,gk20a.txt
@@ -26,7 +26,7 @@ Required properties:
 
 Example:
 
-	gpu@0,57000000 {
+	gpu@57000000 {
 		compatible = "nvidia,gk20a";
 		reg = <0x0 0x57000000 0x0 0x01000000>,
 		      <0x0 0x58000000 0x0 0x01000000>;
diff --git a/Documentation/devicetree/bindings/memory-controllers/nvidia,tegra-mc.txt b/Documentation/devicetree/bindings/memory-controllers/nvidia,tegra-mc.txt
index 3338a2834ad7..8dbe47013c2b 100644
--- a/Documentation/devicetree/bindings/memory-controllers/nvidia,tegra-mc.txt
+++ b/Documentation/devicetree/bindings/memory-controllers/nvidia,tegra-mc.txt
@@ -61,7 +61,7 @@ specified, according to the board documentation:
 Example SoC include file:
 
 / {
-	mc: memory-controller@0,70019000 {
+	mc: memory-controller@70019000 {
 		compatible = "nvidia,tegra124-mc";
 		reg = <0x0 0x70019000 0x0 0x1000>;
 		clocks = <&tegra_car TEGRA124_CLK_MC>;
@@ -72,7 +72,7 @@ Example SoC include file:
 		#iommu-cells = <1>;
 	};
 
-	sdhci@0,700b0000 {
+	sdhci@700b0000 {
 		compatible = "nvidia,tegra124-sdhci";
 		...
 		iommus = <&mc TEGRA_SWGROUP_SDMMC1A>;
@@ -82,7 +82,7 @@ Example SoC include file:
 Example board file:
 
 / {
-	memory-controller@0,70019000 {
+	memory-controller@70019000 {
 		emc-timings-3 {
 			nvidia,ram-code = <3>;
 
diff --git a/Documentation/devicetree/bindings/memory-controllers/tegra-emc.txt b/Documentation/devicetree/bindings/memory-controllers/tegra-emc.txt
index b59c625d6336..ba0bc3f12419 100644
--- a/Documentation/devicetree/bindings/memory-controllers/tegra-emc.txt
+++ b/Documentation/devicetree/bindings/memory-controllers/tegra-emc.txt
@@ -190,7 +190,7 @@ be specified, according to the board documentation:
 Example SoC include file:
 
 / {
-	emc@0,7001b000 {
+	emc@7001b000 {
 		compatible = "nvidia,tegra124-emc";
 		reg = <0x0 0x7001b000 0x0 0x1000>;
 
@@ -201,7 +201,7 @@ Example SoC include file:
 Example board file:
 
 / {
-	emc@0,7001b000 {
+	emc@7001b000 {
 		emc-timings-3 {
 			nvidia,ram-code = <3>;
 
diff --git a/Documentation/devicetree/bindings/pinctrl/nvidia,tegra124-xusb-padctl.txt b/Documentation/devicetree/bindings/pinctrl/nvidia,tegra124-xusb-padctl.txt
index 30676ded85bb..9c8ddd547a99 100644
--- a/Documentation/devicetree/bindings/pinctrl/nvidia,tegra124-xusb-padctl.txt
+++ b/Documentation/devicetree/bindings/pinctrl/nvidia,tegra124-xusb-padctl.txt
@@ -79,7 +79,7 @@ Example:
 SoC file extract:
 -----------------
 
-	padctl@0,7009f000 {
+	padctl@7009f000 {
 		compatible = "nvidia,tegra124-xusb-padctl";
 		reg = <0x0 0x7009f000 0x0 0x1000>;
 		resets = <&tegra_car 142>;
@@ -91,7 +91,7 @@ SoC file extract:
 Board file extract:
 -------------------
 
-	pcie-controller@0,01003000 {
+	pcie-controller@01003000 {
 		...
 
 		phys = <&padctl 0>;
@@ -102,7 +102,7 @@ Board file extract:
 
 	...
 
-	padctl: padctl@0,7009f000 {
+	padctl: padctl@7009f000 {
 		pinctrl-0 = <&padctl_default>;
 		pinctrl-names = "default";
 
diff --git a/Documentation/devicetree/bindings/sound/nvidia,tegra30-hda.txt b/Documentation/devicetree/bindings/sound/nvidia,tegra30-hda.txt
index 275c6ea356f6..44d27456e8a4 100644
--- a/Documentation/devicetree/bindings/sound/nvidia,tegra30-hda.txt
+++ b/Documentation/devicetree/bindings/sound/nvidia,tegra30-hda.txt
@@ -15,7 +15,7 @@ Required properties:
 
 Example:
 
-hda@0,70030000 {
+hda@70030000 {
 	compatible = "nvidia,tegra124-hda", "nvidia,tegra30-hda";
 	reg = <0x0 0x70030000 0x0 0x10000>;
 	interrupts = <GIC_SPI 81 IRQ_TYPE_LEVEL_HIGH>;
diff --git a/Documentation/devicetree/bindings/thermal/tegra-soctherm.txt b/Documentation/devicetree/bindings/thermal/tegra-soctherm.txt
index 351a7376baa8..edebfa0a985e 100644
--- a/Documentation/devicetree/bindings/thermal/tegra-soctherm.txt
+++ b/Documentation/devicetree/bindings/thermal/tegra-soctherm.txt
@@ -33,7 +33,7 @@ than it, the system will be shutdown or reset by hardware.
 
 Example :
 
-	soctherm@0,700e2000 {
+	soctherm@700e2000 {
 		compatible = "nvidia,tegra124-soctherm";
 		reg = <0x0 0x700e2000 0x0 0x1000>;
 		interrupts = <GIC_SPI 48 IRQ_TYPE_LEVEL_HIGH>;
-- 
2.8.0

^ permalink raw reply related	[flat|nested] 6+ messages in thread

* [PATCH v2 2/2] dt-bindings: tegra: Rename some bindings for consistency
       [not found] ` <1460473656-27209-1-git-send-email-thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
@ 2016-04-12 15:07   ` Thierry Reding
       [not found]     ` <1460473656-27209-2-git-send-email-thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
  2016-04-15 13:41   ` [PATCH v2 1/2] dt-bindings: tegra: Remove 0, prefix from unit-addresses Thierry Reding
  2016-04-19 22:24   ` Rob Herring
  2 siblings, 1 reply; 6+ messages in thread
From: Thierry Reding @ 2016-04-12 15:07 UTC (permalink / raw)
  To: Rob Herring
  Cc: Stephen Warren, Alexandre Courbot,
	devicetree-u79uwXL29TY76Z2rM5mHXA,
	linux-tegra-u79uwXL29TY76Z2rM5mHXA

From: Thierry Reding <treding-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>

Device tree binding for NVIDIA Tegra have traditionally carried the
"nvidia," vendor prefix in the filename. A couple of odd ones don't, so
fix them up for consistency.

Also rename existing bindings to reflect the first compatible value that
they document. This wasn't done consistently either.

Acked-by: Stephen Warren <swarren-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
Signed-off-by: Thierry Reding <treding-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
---
Changes in v2:
- rename a couple more binding documents whose name didn't reflect the
  compatible string that they describe
- generate patch with -M to make obvious that there weren't any changes
  besides the renaming

 .../devicetree/bindings/ata/{tegra-sata.txt => nvidia,tegra124-ahci.txt}  | 0
 .../cpufreq/{tegra124-cpufreq.txt => nvidia,tegra124-cpufreq.txt}         | 0
 .../bindings/dma/{tegra20-apbdma.txt => nvidia,tegra20-apbdma.txt}        | 0
 .../{nvidia,tegra-ictlr.txt => nvidia,tegra20-ictlr.txt}                  | 0
 .../memory-controllers/{tegra-emc.txt => nvidia,tegra124-emc.txt}         | 0
 .../memory-controllers/{nvidia,tegra-mc.txt => nvidia,tegra30-mc.txt}     | 0
 .../bindings/thermal/{tegra-soctherm.txt => nvidia,tegra124-soctherm.txt} | 0
 7 files changed, 0 insertions(+), 0 deletions(-)
 rename Documentation/devicetree/bindings/ata/{tegra-sata.txt => nvidia,tegra124-ahci.txt} (100%)
 rename Documentation/devicetree/bindings/cpufreq/{tegra124-cpufreq.txt => nvidia,tegra124-cpufreq.txt} (100%)
 rename Documentation/devicetree/bindings/dma/{tegra20-apbdma.txt => nvidia,tegra20-apbdma.txt} (100%)
 rename Documentation/devicetree/bindings/interrupt-controller/{nvidia,tegra-ictlr.txt => nvidia,tegra20-ictlr.txt} (100%)
 rename Documentation/devicetree/bindings/memory-controllers/{tegra-emc.txt => nvidia,tegra124-emc.txt} (100%)
 rename Documentation/devicetree/bindings/memory-controllers/{nvidia,tegra-mc.txt => nvidia,tegra30-mc.txt} (100%)
 rename Documentation/devicetree/bindings/thermal/{tegra-soctherm.txt => nvidia,tegra124-soctherm.txt} (100%)

diff --git a/Documentation/devicetree/bindings/ata/tegra-sata.txt b/Documentation/devicetree/bindings/ata/nvidia,tegra124-ahci.txt
similarity index 100%
rename from Documentation/devicetree/bindings/ata/tegra-sata.txt
rename to Documentation/devicetree/bindings/ata/nvidia,tegra124-ahci.txt
diff --git a/Documentation/devicetree/bindings/cpufreq/tegra124-cpufreq.txt b/Documentation/devicetree/bindings/cpufreq/nvidia,tegra124-cpufreq.txt
similarity index 100%
rename from Documentation/devicetree/bindings/cpufreq/tegra124-cpufreq.txt
rename to Documentation/devicetree/bindings/cpufreq/nvidia,tegra124-cpufreq.txt
diff --git a/Documentation/devicetree/bindings/dma/tegra20-apbdma.txt b/Documentation/devicetree/bindings/dma/nvidia,tegra20-apbdma.txt
similarity index 100%
rename from Documentation/devicetree/bindings/dma/tegra20-apbdma.txt
rename to Documentation/devicetree/bindings/dma/nvidia,tegra20-apbdma.txt
diff --git a/Documentation/devicetree/bindings/interrupt-controller/nvidia,tegra-ictlr.txt b/Documentation/devicetree/bindings/interrupt-controller/nvidia,tegra20-ictlr.txt
similarity index 100%
rename from Documentation/devicetree/bindings/interrupt-controller/nvidia,tegra-ictlr.txt
rename to Documentation/devicetree/bindings/interrupt-controller/nvidia,tegra20-ictlr.txt
diff --git a/Documentation/devicetree/bindings/memory-controllers/tegra-emc.txt b/Documentation/devicetree/bindings/memory-controllers/nvidia,tegra124-emc.txt
similarity index 100%
rename from Documentation/devicetree/bindings/memory-controllers/tegra-emc.txt
rename to Documentation/devicetree/bindings/memory-controllers/nvidia,tegra124-emc.txt
diff --git a/Documentation/devicetree/bindings/memory-controllers/nvidia,tegra-mc.txt b/Documentation/devicetree/bindings/memory-controllers/nvidia,tegra30-mc.txt
similarity index 100%
rename from Documentation/devicetree/bindings/memory-controllers/nvidia,tegra-mc.txt
rename to Documentation/devicetree/bindings/memory-controllers/nvidia,tegra30-mc.txt
diff --git a/Documentation/devicetree/bindings/thermal/tegra-soctherm.txt b/Documentation/devicetree/bindings/thermal/nvidia,tegra124-soctherm.txt
similarity index 100%
rename from Documentation/devicetree/bindings/thermal/tegra-soctherm.txt
rename to Documentation/devicetree/bindings/thermal/nvidia,tegra124-soctherm.txt
-- 
2.8.0

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH v2 1/2] dt-bindings: tegra: Remove 0, prefix from unit-addresses
       [not found] ` <1460473656-27209-1-git-send-email-thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
  2016-04-12 15:07   ` [PATCH v2 2/2] dt-bindings: tegra: Rename some bindings for consistency Thierry Reding
@ 2016-04-15 13:41   ` Thierry Reding
       [not found]     ` <20160415134102.GB3472-EkSeR96xj6Pcmrwk2tT4+A@public.gmane.org>
  2016-04-19 22:24   ` Rob Herring
  2 siblings, 1 reply; 6+ messages in thread
From: Thierry Reding @ 2016-04-15 13:41 UTC (permalink / raw)
  To: Rob Herring
  Cc: Stephen Warren, Alexandre Courbot,
	devicetree-u79uwXL29TY76Z2rM5mHXA,
	linux-tegra-u79uwXL29TY76Z2rM5mHXA

[-- Attachment #1: Type: text/plain, Size: 1497 bytes --]

On Tue, Apr 12, 2016 at 05:07:35PM +0200, Thierry Reding wrote:
> From: Thierry Reding <treding-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
> 
> When Tegra124 support was first merged the unit-addresses of all devices
> were listed with a "0," prefix to encode the reg property's second cell.
> It turns out that this notation is not correct, and the "," separator is
> only used to separate fields in the unit address (such as the device and
> function number in PCI devices), not individual cells for addresses
> with more than one cell.
> 
> Acked-by: Stephen Warren <swarren-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
> Signed-off-by: Thierry Reding <treding-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
> ---
>  Documentation/devicetree/bindings/clock/nvidia,tegra124-dfll.txt    | 2 +-
>  Documentation/devicetree/bindings/gpu/nvidia,gk20a.txt              | 2 +-
>  .../devicetree/bindings/memory-controllers/nvidia,tegra-mc.txt      | 6 +++---
>  Documentation/devicetree/bindings/memory-controllers/tegra-emc.txt  | 4 ++--
>  .../devicetree/bindings/pinctrl/nvidia,tegra124-xusb-padctl.txt     | 6 +++---
>  Documentation/devicetree/bindings/sound/nvidia,tegra30-hda.txt      | 2 +-
>  Documentation/devicetree/bindings/thermal/tegra-soctherm.txt        | 2 +-
>  7 files changed, 12 insertions(+), 12 deletions(-)

Hi Rob,

Is this (and patch 2/2) something that you could pick up into the DT
tree, or would you rather have me take it through the Tegra tree?

Thierry

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 819 bytes --]

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH v2 1/2] dt-bindings: tegra: Remove 0, prefix from unit-addresses
       [not found]     ` <20160415134102.GB3472-EkSeR96xj6Pcmrwk2tT4+A@public.gmane.org>
@ 2016-04-15 16:46       ` Rob Herring
  0 siblings, 0 replies; 6+ messages in thread
From: Rob Herring @ 2016-04-15 16:46 UTC (permalink / raw)
  To: Thierry Reding
  Cc: Stephen Warren, Alexandre Courbot,
	devicetree-u79uwXL29TY76Z2rM5mHXA,
	linux-tegra-u79uwXL29TY76Z2rM5mHXA

On Fri, Apr 15, 2016 at 8:41 AM, Thierry Reding
<thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> wrote:
> On Tue, Apr 12, 2016 at 05:07:35PM +0200, Thierry Reding wrote:
>> From: Thierry Reding <treding-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
>>
>> When Tegra124 support was first merged the unit-addresses of all devices
>> were listed with a "0," prefix to encode the reg property's second cell.
>> It turns out that this notation is not correct, and the "," separator is
>> only used to separate fields in the unit address (such as the device and
>> function number in PCI devices), not individual cells for addresses
>> with more than one cell.
>>
>> Acked-by: Stephen Warren <swarren-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
>> Signed-off-by: Thierry Reding <treding-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
>> ---
>>  Documentation/devicetree/bindings/clock/nvidia,tegra124-dfll.txt    | 2 +-
>>  Documentation/devicetree/bindings/gpu/nvidia,gk20a.txt              | 2 +-
>>  .../devicetree/bindings/memory-controllers/nvidia,tegra-mc.txt      | 6 +++---
>>  Documentation/devicetree/bindings/memory-controllers/tegra-emc.txt  | 4 ++--
>>  .../devicetree/bindings/pinctrl/nvidia,tegra124-xusb-padctl.txt     | 6 +++---
>>  Documentation/devicetree/bindings/sound/nvidia,tegra30-hda.txt      | 2 +-
>>  Documentation/devicetree/bindings/thermal/tegra-soctherm.txt        | 2 +-
>>  7 files changed, 12 insertions(+), 12 deletions(-)
>
> Hi Rob,
>
> Is this (and patch 2/2) something that you could pick up into the DT
> tree, or would you rather have me take it through the Tegra tree?

I have them queued up.

Rob
>
> Thierry

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH v2 1/2] dt-bindings: tegra: Remove 0, prefix from unit-addresses
       [not found] ` <1460473656-27209-1-git-send-email-thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
  2016-04-12 15:07   ` [PATCH v2 2/2] dt-bindings: tegra: Rename some bindings for consistency Thierry Reding
  2016-04-15 13:41   ` [PATCH v2 1/2] dt-bindings: tegra: Remove 0, prefix from unit-addresses Thierry Reding
@ 2016-04-19 22:24   ` Rob Herring
  2 siblings, 0 replies; 6+ messages in thread
From: Rob Herring @ 2016-04-19 22:24 UTC (permalink / raw)
  To: Thierry Reding
  Cc: Stephen Warren, Alexandre Courbot,
	devicetree-u79uwXL29TY76Z2rM5mHXA,
	linux-tegra-u79uwXL29TY76Z2rM5mHXA

On Tue, Apr 12, 2016 at 05:07:35PM +0200, Thierry Reding wrote:
> From: Thierry Reding <treding-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
> 
> When Tegra124 support was first merged the unit-addresses of all devices
> were listed with a "0," prefix to encode the reg property's second cell.
> It turns out that this notation is not correct, and the "," separator is
> only used to separate fields in the unit address (such as the device and
> function number in PCI devices), not individual cells for addresses
> with more than one cell.
> 
> Acked-by: Stephen Warren <swarren-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
> Signed-off-by: Thierry Reding <treding-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>

Applied, thanks.

Rob

> ---
>  Documentation/devicetree/bindings/clock/nvidia,tegra124-dfll.txt    | 2 +-
>  Documentation/devicetree/bindings/gpu/nvidia,gk20a.txt              | 2 +-
>  .../devicetree/bindings/memory-controllers/nvidia,tegra-mc.txt      | 6 +++---
>  Documentation/devicetree/bindings/memory-controllers/tegra-emc.txt  | 4 ++--
>  .../devicetree/bindings/pinctrl/nvidia,tegra124-xusb-padctl.txt     | 6 +++---
>  Documentation/devicetree/bindings/sound/nvidia,tegra30-hda.txt      | 2 +-
>  Documentation/devicetree/bindings/thermal/tegra-soctherm.txt        | 2 +-
>  7 files changed, 12 insertions(+), 12 deletions(-)
> 
> diff --git a/Documentation/devicetree/bindings/clock/nvidia,tegra124-dfll.txt b/Documentation/devicetree/bindings/clock/nvidia,tegra124-dfll.txt
> index ee7e5fd4a50b..63f9d8277d48 100644
> --- a/Documentation/devicetree/bindings/clock/nvidia,tegra124-dfll.txt
> +++ b/Documentation/devicetree/bindings/clock/nvidia,tegra124-dfll.txt
> @@ -50,7 +50,7 @@ Required properties for I2C mode:
>  
>  Example:
>  
> -clock@0,70110000 {
> +clock@70110000 {
>          compatible = "nvidia,tegra124-dfll";
>          reg = <0 0x70110000 0 0x100>, /* DFLL control */
>                <0 0x70110000 0 0x100>, /* I2C output control */
> diff --git a/Documentation/devicetree/bindings/gpu/nvidia,gk20a.txt b/Documentation/devicetree/bindings/gpu/nvidia,gk20a.txt
> index 23bfe8e1f7cc..9d47a2c25e2d 100644
> --- a/Documentation/devicetree/bindings/gpu/nvidia,gk20a.txt
> +++ b/Documentation/devicetree/bindings/gpu/nvidia,gk20a.txt
> @@ -26,7 +26,7 @@ Required properties:
>  
>  Example:
>  
> -	gpu@0,57000000 {
> +	gpu@57000000 {
>  		compatible = "nvidia,gk20a";
>  		reg = <0x0 0x57000000 0x0 0x01000000>,
>  		      <0x0 0x58000000 0x0 0x01000000>;
> diff --git a/Documentation/devicetree/bindings/memory-controllers/nvidia,tegra-mc.txt b/Documentation/devicetree/bindings/memory-controllers/nvidia,tegra-mc.txt
> index 3338a2834ad7..8dbe47013c2b 100644
> --- a/Documentation/devicetree/bindings/memory-controllers/nvidia,tegra-mc.txt
> +++ b/Documentation/devicetree/bindings/memory-controllers/nvidia,tegra-mc.txt
> @@ -61,7 +61,7 @@ specified, according to the board documentation:
>  Example SoC include file:
>  
>  / {
> -	mc: memory-controller@0,70019000 {
> +	mc: memory-controller@70019000 {
>  		compatible = "nvidia,tegra124-mc";
>  		reg = <0x0 0x70019000 0x0 0x1000>;
>  		clocks = <&tegra_car TEGRA124_CLK_MC>;
> @@ -72,7 +72,7 @@ Example SoC include file:
>  		#iommu-cells = <1>;
>  	};
>  
> -	sdhci@0,700b0000 {
> +	sdhci@700b0000 {
>  		compatible = "nvidia,tegra124-sdhci";
>  		...
>  		iommus = <&mc TEGRA_SWGROUP_SDMMC1A>;
> @@ -82,7 +82,7 @@ Example SoC include file:
>  Example board file:
>  
>  / {
> -	memory-controller@0,70019000 {
> +	memory-controller@70019000 {
>  		emc-timings-3 {
>  			nvidia,ram-code = <3>;
>  
> diff --git a/Documentation/devicetree/bindings/memory-controllers/tegra-emc.txt b/Documentation/devicetree/bindings/memory-controllers/tegra-emc.txt
> index b59c625d6336..ba0bc3f12419 100644
> --- a/Documentation/devicetree/bindings/memory-controllers/tegra-emc.txt
> +++ b/Documentation/devicetree/bindings/memory-controllers/tegra-emc.txt
> @@ -190,7 +190,7 @@ be specified, according to the board documentation:
>  Example SoC include file:
>  
>  / {
> -	emc@0,7001b000 {
> +	emc@7001b000 {
>  		compatible = "nvidia,tegra124-emc";
>  		reg = <0x0 0x7001b000 0x0 0x1000>;
>  
> @@ -201,7 +201,7 @@ Example SoC include file:
>  Example board file:
>  
>  / {
> -	emc@0,7001b000 {
> +	emc@7001b000 {
>  		emc-timings-3 {
>  			nvidia,ram-code = <3>;
>  
> diff --git a/Documentation/devicetree/bindings/pinctrl/nvidia,tegra124-xusb-padctl.txt b/Documentation/devicetree/bindings/pinctrl/nvidia,tegra124-xusb-padctl.txt
> index 30676ded85bb..9c8ddd547a99 100644
> --- a/Documentation/devicetree/bindings/pinctrl/nvidia,tegra124-xusb-padctl.txt
> +++ b/Documentation/devicetree/bindings/pinctrl/nvidia,tegra124-xusb-padctl.txt
> @@ -79,7 +79,7 @@ Example:
>  SoC file extract:
>  -----------------
>  
> -	padctl@0,7009f000 {
> +	padctl@7009f000 {
>  		compatible = "nvidia,tegra124-xusb-padctl";
>  		reg = <0x0 0x7009f000 0x0 0x1000>;
>  		resets = <&tegra_car 142>;
> @@ -91,7 +91,7 @@ SoC file extract:
>  Board file extract:
>  -------------------
>  
> -	pcie-controller@0,01003000 {
> +	pcie-controller@01003000 {
>  		...
>  
>  		phys = <&padctl 0>;
> @@ -102,7 +102,7 @@ Board file extract:
>  
>  	...
>  
> -	padctl: padctl@0,7009f000 {
> +	padctl: padctl@7009f000 {
>  		pinctrl-0 = <&padctl_default>;
>  		pinctrl-names = "default";
>  
> diff --git a/Documentation/devicetree/bindings/sound/nvidia,tegra30-hda.txt b/Documentation/devicetree/bindings/sound/nvidia,tegra30-hda.txt
> index 275c6ea356f6..44d27456e8a4 100644
> --- a/Documentation/devicetree/bindings/sound/nvidia,tegra30-hda.txt
> +++ b/Documentation/devicetree/bindings/sound/nvidia,tegra30-hda.txt
> @@ -15,7 +15,7 @@ Required properties:
>  
>  Example:
>  
> -hda@0,70030000 {
> +hda@70030000 {
>  	compatible = "nvidia,tegra124-hda", "nvidia,tegra30-hda";
>  	reg = <0x0 0x70030000 0x0 0x10000>;
>  	interrupts = <GIC_SPI 81 IRQ_TYPE_LEVEL_HIGH>;
> diff --git a/Documentation/devicetree/bindings/thermal/tegra-soctherm.txt b/Documentation/devicetree/bindings/thermal/tegra-soctherm.txt
> index 351a7376baa8..edebfa0a985e 100644
> --- a/Documentation/devicetree/bindings/thermal/tegra-soctherm.txt
> +++ b/Documentation/devicetree/bindings/thermal/tegra-soctherm.txt
> @@ -33,7 +33,7 @@ than it, the system will be shutdown or reset by hardware.
>  
>  Example :
>  
> -	soctherm@0,700e2000 {
> +	soctherm@700e2000 {
>  		compatible = "nvidia,tegra124-soctherm";
>  		reg = <0x0 0x700e2000 0x0 0x1000>;
>  		interrupts = <GIC_SPI 48 IRQ_TYPE_LEVEL_HIGH>;
> -- 
> 2.8.0
> 

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH v2 2/2] dt-bindings: tegra: Rename some bindings for consistency
       [not found]     ` <1460473656-27209-2-git-send-email-thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
@ 2016-04-19 22:24       ` Rob Herring
  0 siblings, 0 replies; 6+ messages in thread
From: Rob Herring @ 2016-04-19 22:24 UTC (permalink / raw)
  To: Thierry Reding
  Cc: Stephen Warren, Alexandre Courbot,
	devicetree-u79uwXL29TY76Z2rM5mHXA,
	linux-tegra-u79uwXL29TY76Z2rM5mHXA

On Tue, Apr 12, 2016 at 05:07:36PM +0200, Thierry Reding wrote:
> From: Thierry Reding <treding-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
> 
> Device tree binding for NVIDIA Tegra have traditionally carried the
> "nvidia," vendor prefix in the filename. A couple of odd ones don't, so
> fix them up for consistency.
> 
> Also rename existing bindings to reflect the first compatible value that
> they document. This wasn't done consistently either.
> 
> Acked-by: Stephen Warren <swarren-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
> Signed-off-by: Thierry Reding <treding-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>

Applied, thanks.

Rob

> ---
> Changes in v2:
> - rename a couple more binding documents whose name didn't reflect the
>   compatible string that they describe
> - generate patch with -M to make obvious that there weren't any changes
>   besides the renaming
> 
>  .../devicetree/bindings/ata/{tegra-sata.txt => nvidia,tegra124-ahci.txt}  | 0
>  .../cpufreq/{tegra124-cpufreq.txt => nvidia,tegra124-cpufreq.txt}         | 0
>  .../bindings/dma/{tegra20-apbdma.txt => nvidia,tegra20-apbdma.txt}        | 0
>  .../{nvidia,tegra-ictlr.txt => nvidia,tegra20-ictlr.txt}                  | 0
>  .../memory-controllers/{tegra-emc.txt => nvidia,tegra124-emc.txt}         | 0
>  .../memory-controllers/{nvidia,tegra-mc.txt => nvidia,tegra30-mc.txt}     | 0
>  .../bindings/thermal/{tegra-soctherm.txt => nvidia,tegra124-soctherm.txt} | 0
>  7 files changed, 0 insertions(+), 0 deletions(-)
>  rename Documentation/devicetree/bindings/ata/{tegra-sata.txt => nvidia,tegra124-ahci.txt} (100%)
>  rename Documentation/devicetree/bindings/cpufreq/{tegra124-cpufreq.txt => nvidia,tegra124-cpufreq.txt} (100%)
>  rename Documentation/devicetree/bindings/dma/{tegra20-apbdma.txt => nvidia,tegra20-apbdma.txt} (100%)
>  rename Documentation/devicetree/bindings/interrupt-controller/{nvidia,tegra-ictlr.txt => nvidia,tegra20-ictlr.txt} (100%)
>  rename Documentation/devicetree/bindings/memory-controllers/{tegra-emc.txt => nvidia,tegra124-emc.txt} (100%)
>  rename Documentation/devicetree/bindings/memory-controllers/{nvidia,tegra-mc.txt => nvidia,tegra30-mc.txt} (100%)
>  rename Documentation/devicetree/bindings/thermal/{tegra-soctherm.txt => nvidia,tegra124-soctherm.txt} (100%)
> 
> diff --git a/Documentation/devicetree/bindings/ata/tegra-sata.txt b/Documentation/devicetree/bindings/ata/nvidia,tegra124-ahci.txt
> similarity index 100%
> rename from Documentation/devicetree/bindings/ata/tegra-sata.txt
> rename to Documentation/devicetree/bindings/ata/nvidia,tegra124-ahci.txt
> diff --git a/Documentation/devicetree/bindings/cpufreq/tegra124-cpufreq.txt b/Documentation/devicetree/bindings/cpufreq/nvidia,tegra124-cpufreq.txt
> similarity index 100%
> rename from Documentation/devicetree/bindings/cpufreq/tegra124-cpufreq.txt
> rename to Documentation/devicetree/bindings/cpufreq/nvidia,tegra124-cpufreq.txt
> diff --git a/Documentation/devicetree/bindings/dma/tegra20-apbdma.txt b/Documentation/devicetree/bindings/dma/nvidia,tegra20-apbdma.txt
> similarity index 100%
> rename from Documentation/devicetree/bindings/dma/tegra20-apbdma.txt
> rename to Documentation/devicetree/bindings/dma/nvidia,tegra20-apbdma.txt
> diff --git a/Documentation/devicetree/bindings/interrupt-controller/nvidia,tegra-ictlr.txt b/Documentation/devicetree/bindings/interrupt-controller/nvidia,tegra20-ictlr.txt
> similarity index 100%
> rename from Documentation/devicetree/bindings/interrupt-controller/nvidia,tegra-ictlr.txt
> rename to Documentation/devicetree/bindings/interrupt-controller/nvidia,tegra20-ictlr.txt
> diff --git a/Documentation/devicetree/bindings/memory-controllers/tegra-emc.txt b/Documentation/devicetree/bindings/memory-controllers/nvidia,tegra124-emc.txt
> similarity index 100%
> rename from Documentation/devicetree/bindings/memory-controllers/tegra-emc.txt
> rename to Documentation/devicetree/bindings/memory-controllers/nvidia,tegra124-emc.txt
> diff --git a/Documentation/devicetree/bindings/memory-controllers/nvidia,tegra-mc.txt b/Documentation/devicetree/bindings/memory-controllers/nvidia,tegra30-mc.txt
> similarity index 100%
> rename from Documentation/devicetree/bindings/memory-controllers/nvidia,tegra-mc.txt
> rename to Documentation/devicetree/bindings/memory-controllers/nvidia,tegra30-mc.txt
> diff --git a/Documentation/devicetree/bindings/thermal/tegra-soctherm.txt b/Documentation/devicetree/bindings/thermal/nvidia,tegra124-soctherm.txt
> similarity index 100%
> rename from Documentation/devicetree/bindings/thermal/tegra-soctherm.txt
> rename to Documentation/devicetree/bindings/thermal/nvidia,tegra124-soctherm.txt
> -- 
> 2.8.0
> 

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2016-04-19 22:24 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-04-12 15:07 [PATCH v2 1/2] dt-bindings: tegra: Remove 0, prefix from unit-addresses Thierry Reding
     [not found] ` <1460473656-27209-1-git-send-email-thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2016-04-12 15:07   ` [PATCH v2 2/2] dt-bindings: tegra: Rename some bindings for consistency Thierry Reding
     [not found]     ` <1460473656-27209-2-git-send-email-thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2016-04-19 22:24       ` Rob Herring
2016-04-15 13:41   ` [PATCH v2 1/2] dt-bindings: tegra: Remove 0, prefix from unit-addresses Thierry Reding
     [not found]     ` <20160415134102.GB3472-EkSeR96xj6Pcmrwk2tT4+A@public.gmane.org>
2016-04-15 16:46       ` Rob Herring
2016-04-19 22:24   ` Rob Herring

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.