Re: powerpc: dts: replace 'linux,stdout-path' with 'stdout-path'

[Michael Ellerman]

On Wed, 2018-02-28 at 22:44:06 UTC, Rob Herring wrote:
> 'linux,stdout-path' has been deprecated for some time in favor of
> 'stdout-path'. Now dtc will warn on occurrences of 'linux,stdout-path'.
> Search and replace all the of occurrences with 'stdout-path'.
> 
> Signed-off-by: Rob Herring 
> Cc: Mark Rutland 
> Cc: Benjamin Herrenschmidt 
> Cc: Paul Mackerras 
> Cc: Michael Ellerman 
> Cc: linuxppc-dev@lists.ozlabs.org

Applied to powerpc next, thanks.

https://git.kernel.org/powerpc/c/78e5dfea84dc15d69940831b3981b3

cheers

Re: powerpc: dts: replace 'linux,stdout-path' with 'stdout-path'

[Michael Ellerman]

On Wed, 2018-02-28 at 22:44:06 UTC, Rob Herring wrote:
> 'linux,stdout-path' has been deprecated for some time in favor of
> 'stdout-path'. Now dtc will warn on occurrences of 'linux,stdout-path'.
> Search and replace all the of occurrences with 'stdout-path'.
> 
> Signed-off-by: Rob Herring 
> Cc: Mark Rutland 
> Cc: Benjamin Herrenschmidt 
> Cc: Paul Mackerras 
> Cc: Michael Ellerman 
> Cc: linuxppc-dev@lists.ozlabs.org

Applied to powerpc next, thanks.

https://git.kernel.org/powerpc/c/d79a02b3e03b103d84536fb19ae7a7

cheers

Re: [PATCH] powerpc: dts: replace 'linux, stdout-path' with 'stdout-path'

[Michael Ellerman]

Rob Herring  writes:

> On Wed, Feb 28, 2018 at 7:33 PM, Michael Ellerman  wrote:
>> Rob Herring  writes:
>>
>>> 'linux,stdout-path' has been deprecated for some time in favor of
>>> 'stdout-path'. Now dtc will warn on occurrences of 'linux,stdout-path'.
>>> Search and replace all the of occurrences with 'stdout-path'.
>>
>> This patch looks OK.
>>
>> But please remember that not all device trees are generated with dtc, we
>> still have machines in the wild that have firmware which use
>> "linux,stdout-path" and may never be updated.
>
> Absolutely. The core code still supports both.

Phew ;)

It has prompted us to look at various firmware pieces and some we can
update to start using the new property or both. It's still going to be
many years before we can actually remove support for linux,stdout-path
though.

> The only scenario I could come up with is someone has an old
> bootloader that only understands linux,stdout-path and modifies it and
> they update the dtb. We may end up with both properties with
> stdout-path taking preference.

Yeah I think this is fairly safe, and in the unlikely case it does cause
a problem we can always back the change out for an individual file.

cheers

Re: [PATCH] powerpc: dts: replace 'linux, stdout-path' with 'stdout-path'

[Rob Herring]

On Wed, Feb 28, 2018 at 7:33 PM, Michael Ellerman  wrote:
> Rob Herring  writes:
>
>> 'linux,stdout-path' has been deprecated for some time in favor of
>> 'stdout-path'. Now dtc will warn on occurrences of 'linux,stdout-path'.
>> Search and replace all the of occurrences with 'stdout-path'.
>
> This patch looks OK.
>
> But please remember that not all device trees are generated with dtc, we
> still have machines in the wild that have firmware which use
> "linux,stdout-path" and may never be updated.

Absolutely. The core code still supports both.

The only scenario I could come up with is someone has an old
bootloader that only understands linux,stdout-path and modifies it and
they update the dtb. We may end up with both properties with
stdout-path taking preference.

Rob

Re: [PATCH] powerpc: dts: replace 'linux, stdout-path' with 'stdout-path'

[Michael Ellerman]

Rob Herring  writes:

> 'linux,stdout-path' has been deprecated for some time in favor of
> 'stdout-path'. Now dtc will warn on occurrences of 'linux,stdout-path'.
> Search and replace all the of occurrences with 'stdout-path'.

This patch looks OK.

But please remember that not all device trees are generated with dtc, we
still have machines in the wild that have firmware which use
"linux,stdout-path" and may never be updated.

cheers

[PATCH] powerpc: dts: replace 'linux,stdout-path' with 'stdout-path'

[Rob Herring]

'linux,stdout-path' has been deprecated for some time in favor of
'stdout-path'. Now dtc will warn on occurrences of 'linux,stdout-path'.
Search and replace all the of occurrences with 'stdout-path'.

Signed-off-by: Rob Herring 
Cc: Mark Rutland 
Cc: Benjamin Herrenschmidt 
Cc: Paul Mackerras 
Cc: Michael Ellerman 
Cc: linuxppc-dev@lists.ozlabs.org
---
 arch/powerpc/boot/dts/acadia.dts   | 2 +-
 arch/powerpc/boot/dts/adder875-redboot.dts | 2 +-
 arch/powerpc/boot/dts/adder875-uboot.dts   | 2 +-
 arch/powerpc/boot/dts/akebono.dts  | 2 +-
 arch/powerpc/boot/dts/amigaone.dts | 2 +-
 arch/powerpc/boot/dts/asp834x-redboot.dts  | 2 +-
 arch/powerpc/boot/dts/bamboo.dts   | 2 +-
 arch/powerpc/boot/dts/c2k.dts  | 2 +-
 arch/powerpc/boot/dts/currituck.dts| 2 +-
 arch/powerpc/boot/dts/ebony.dts| 2 +-
 arch/powerpc/boot/dts/eiger.dts| 2 +-
 arch/powerpc/boot/dts/ep405.dts| 2 +-
 arch/powerpc/boot/dts/fsl/mvme7100.dts | 2 +-
 arch/powerpc/boot/dts/fsp2.dts | 2 +-
 arch/powerpc/boot/dts/holly.dts| 2 +-
 arch/powerpc/boot/dts/hotfoot.dts  | 2 +-
 arch/powerpc/boot/dts/icon.dts | 2 +-
 arch/powerpc/boot/dts/iss4xx-mpic.dts  | 2 +-
 arch/powerpc/boot/dts/iss4xx.dts   | 2 +-
 arch/powerpc/boot/dts/katmai.dts   | 2 +-
 arch/powerpc/boot/dts/klondike.dts | 2 +-
 arch/powerpc/boot/dts/ksi8560.dts  | 2 +-
 arch/powerpc/boot/dts/media5200.dts| 2 +-
 arch/powerpc/boot/dts/mpc8272ads.dts   | 2 +-
 arch/powerpc/boot/dts/mpc866ads.dts| 2 +-
 arch/powerpc/boot/dts/mpc885ads.dts| 2 +-
 arch/powerpc/boot/dts/mvme5100.dts | 2 +-
 arch/powerpc/boot/dts/obs600.dts   | 2 +-
 arch/powerpc/boot/dts/pq2fads.dts  | 2 +-
 arch/powerpc/boot/dts/rainier.dts  | 2 +-
 arch/powerpc/boot/dts/redwood.dts  | 2 +-
 arch/powerpc/boot/dts/sam440ep.dts | 2 +-
 arch/powerpc/boot/dts/sequoia.dts  | 2 +-
 arch/powerpc/boot/dts/storcenter.dts   | 2 +-
 arch/powerpc/boot/dts/taishan.dts  | 2 +-
 arch/powerpc/boot/dts/virtex440-ml507.dts  | 2 +-
 arch/powerpc/boot/dts/virtex440-ml510.dts  | 2 +-
 arch/powerpc/boot/dts/walnut.dts   | 2 +-
 arch/powerpc/boot/dts/warp.dts | 2 +-
 arch/powerpc/boot/dts/xpedite5200_xmon.dts | 2 +-
 arch/powerpc/boot/dts/yosemite.dts | 2 +-
 41 files changed, 41 insertions(+), 41 deletions(-)

diff --git a/arch/powerpc/boot/dts/acadia.dts b/arch/powerpc/boot/dts/acadia.dts
index 86266159521e..deb52e41ab84 100644
--- a/arch/powerpc/boot/dts/acadia.dts
+++ b/arch/powerpc/boot/dts/acadia.dts
@@ -219,6 +219,6 @@
};
 
chosen {
-   linux,stdout-path = "/plb/opb/serial@ef600300";
+   stdout-path = "/plb/opb/serial@ef600300";
};
 };
diff --git a/arch/powerpc/boot/dts/adder875-redboot.dts 
b/arch/powerpc/boot/dts/adder875-redboot.dts
index 083984720b2f..7f5ff4168482 100644
--- a/arch/powerpc/boot/dts/adder875-redboot.dts
+++ b/arch/powerpc/boot/dts/adder875-redboot.dts
@@ -178,6 +178,6 @@
};
 
chosen {
-   linux,stdout-path = 
+   stdout-path = 
};
 };
diff --git a/arch/powerpc/boot/dts/adder875-uboot.dts 
b/arch/powerpc/boot/dts/adder875-uboot.dts
index e4554caf8f8d..bd9f33c57737 100644
--- a/arch/powerpc/boot/dts/adder875-uboot.dts
+++ b/arch/powerpc/boot/dts/adder875-uboot.dts
@@ -177,6 +177,6 @@
};
 
chosen {
-   linux,stdout-path = 
+   stdout-path = 
};
 };
diff --git a/arch/powerpc/boot/dts/akebono.dts 
b/arch/powerpc/boot/dts/akebono.dts
index 746779202a12..8a7a10139bc9 100644
--- a/arch/powerpc/boot/dts/akebono.dts
+++ b/arch/powerpc/boot/dts/akebono.dts
@@ -410,6 +410,6 @@
};
 
chosen {
-   linux,stdout-path = 
+   stdout-path = 
};
 };
diff --git a/arch/powerpc/boot/dts/amigaone.dts 
b/arch/powerpc/boot/dts/amigaone.dts
index 49ac36b16dd7..712430155b99 100644
--- a/arch/powerpc/boot/dts/amigaone.dts
+++ b/arch/powerpc/boot/dts/amigaone.dts
@@ -168,6 +168,6 @@
};
 
chosen {
-   linux,stdout-path = "/pci@8000/isa@7/serial@3f8";
+   stdout-path = "/pci@8000/isa@7/serial@3f8";
};
 };
diff --git a/arch/powerpc/boot/dts/asp834x-redboot.dts 
b/arch/powerpc/boot/dts/asp834x-redboot.dts
index 9198745f45fb..e987b5af9326 100644
--- a/arch/powerpc/boot/dts/asp834x-redboot.dts
+++ b/arch/powerpc/boot/dts/asp834x-redboot.dts
@@ -304,7 +304,7 @@
 
chosen {
bootargs = "console=ttyS0,38400 root=/dev/mtdblock3 
rootfstype=jffs2";
-   linux,stdout-path = 
+   stdout-path = 
};
 
 };
diff --git a/arch/powerpc/boot/dts/bamboo.dts