linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 0/9] ARM: omap: Add empty chosen node in SoCs top level DTSI
@ 2016-12-19 14:44 Javier Martinez Canillas
  2016-12-19 14:44 ` [PATCH 1/9] ARM: dts: omap2: Add an empty chosen node to " Javier Martinez Canillas
                   ` (9 more replies)
  0 siblings, 10 replies; 12+ messages in thread
From: Javier Martinez Canillas @ 2016-12-19 14:44 UTC (permalink / raw)
  To: linux-kernel
  Cc: Pali Rohar, Javier Martinez Canillas, Benoît Cousson,
	devicetree, Rob Herring, Tony Lindgren, Mark Rutland, linux-omap,
	Russell King, linux-arm-kernel

Hello Tony,

As discussed in [0], there's a regression when booting a kernel with a DTB
that doesn't have a pre-existing chosen node. This is usually not an issue
for most boards since u-boot creates an empty chosen node if isn't present
in the DTB.

But it can be an issue for others bootloaders as Pali pointed out with the
N9/900/950 phones and the Nokia Loader (NoLo).

This patch series add chosen nodes in the top level DTSI for all OMAP SoCs.

[0]: http://www.mail-archive.com/linux-kernel@vger.kernel.org/msg1294379.html

Best regards,
Javier


Javier Martinez Canillas (9):
  ARM: dts: omap2: Add an empty chosen node to top level DTSI
  ARM: dts: omap3: Add an empty chosen node to top level DTSI
  ARM: dts: omap4: Add an empty chosen node to top level DTSI
  ARM: dts: omap5: Add an empty chosen node to top level DTSI
  ARM: dts: am33xx: Add an empty chosen node to top level DTSI
  ARM: dts: am4372: Add an empty chosen node to top level DTSI
  ARM: dts: dm814x: Add an empty chosen node to top level DTSI
  ARM: dts: dm816x: Add an empty chosen node to top level DTSI
  ARM: dts: dra7: Add an empty chosen node to top level DTSI

 arch/arm/boot/dts/am33xx.dtsi | 1 +
 arch/arm/boot/dts/am4372.dtsi | 1 +
 arch/arm/boot/dts/dm814x.dtsi | 1 +
 arch/arm/boot/dts/dm816x.dtsi | 1 +
 arch/arm/boot/dts/dra7.dtsi   | 1 +
 arch/arm/boot/dts/omap2.dtsi  | 1 +
 arch/arm/boot/dts/omap3.dtsi  | 1 +
 arch/arm/boot/dts/omap4.dtsi  | 1 +
 arch/arm/boot/dts/omap5.dtsi  | 1 +
 9 files changed, 9 insertions(+)

-- 
2.7.4

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

* [PATCH 1/9] ARM: dts: omap2: Add an empty chosen node to top level DTSI
  2016-12-19 14:44 [PATCH 0/9] ARM: omap: Add empty chosen node in SoCs top level DTSI Javier Martinez Canillas
@ 2016-12-19 14:44 ` Javier Martinez Canillas
  2016-12-19 14:44 ` [PATCH 2/9] ARM: dts: omap3: " Javier Martinez Canillas
                   ` (8 subsequent siblings)
  9 siblings, 0 replies; 12+ messages in thread
From: Javier Martinez Canillas @ 2016-12-19 14:44 UTC (permalink / raw)
  To: linux-kernel
  Cc: Pali Rohar, Javier Martinez Canillas, Benoît Cousson,
	devicetree, Rob Herring, Tony Lindgren, Mark Rutland, linux-omap,
	Russell King, linux-arm-kernel

Commit d1f3156fc8c7 ("ARM: dts: omap2: Remove skeleton.dtsi usage")
removed the skeleton.dtsi usage since we want to get rid of it.

But this can cause issues when booting a kernel with a boot-loader
that doesn't create a chosen node if this isn't present in the DTB
since the decompressor relies on a pre-existing chosen node to be
available to insert the command line and merge other ATAGS info.

Fixes: d1f3156fc8c7 ("ARM: dts: omap2: Remove skeleton.dtsi usage")
Reported-by: Pali Rohar <pali.rohar@gmail.com>
Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com>
---

 arch/arm/boot/dts/omap2.dtsi | 1 +
 1 file changed, 1 insertion(+)

diff --git a/arch/arm/boot/dts/omap2.dtsi b/arch/arm/boot/dts/omap2.dtsi
index 4f793a025a72..f1d6de8b3c19 100644
--- a/arch/arm/boot/dts/omap2.dtsi
+++ b/arch/arm/boot/dts/omap2.dtsi
@@ -17,6 +17,7 @@
 	interrupt-parent = <&intc>;
 	#address-cells = <1>;
 	#size-cells = <1>;
+	chosen { };
 
 	aliases {
 		serial0 = &uart1;
-- 
2.7.4

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

* [PATCH 2/9] ARM: dts: omap3: Add an empty chosen node to top level DTSI
  2016-12-19 14:44 [PATCH 0/9] ARM: omap: Add empty chosen node in SoCs top level DTSI Javier Martinez Canillas
  2016-12-19 14:44 ` [PATCH 1/9] ARM: dts: omap2: Add an empty chosen node to " Javier Martinez Canillas
@ 2016-12-19 14:44 ` Javier Martinez Canillas
  2017-01-02 18:01   ` Pali Rohár
  2016-12-19 14:44 ` [PATCH 3/9] ARM: dts: omap4: " Javier Martinez Canillas
                   ` (7 subsequent siblings)
  9 siblings, 1 reply; 12+ messages in thread
From: Javier Martinez Canillas @ 2016-12-19 14:44 UTC (permalink / raw)
  To: linux-kernel
  Cc: Pali Rohar, Javier Martinez Canillas, Benoît Cousson,
	devicetree, Rob Herring, Tony Lindgren, Mark Rutland, linux-omap,
	Russell King, linux-arm-kernel

Commit 008a2ebcd677 ("ARM: dts: omap3: Remove skeleton.dtsi usage")
removed the skeleton.dtsi usage since we want to get rid of it.

But this can cause issues when booting a kernel with a boot-loader
that doesn't create a chosen node if this isn't present in the DTB
since the decompressor relies on a pre-existing chosen node to be
available to insert the command line and merge other ATAGS info.

Fixes: 008a2ebcd677 ("ARM: dts: omap3: Remove skeleton.dtsi usage")
Reported-by: Pali Rohar <pali.rohar@gmail.com>
Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com>
---

 arch/arm/boot/dts/omap3.dtsi | 1 +
 1 file changed, 1 insertion(+)

diff --git a/arch/arm/boot/dts/omap3.dtsi b/arch/arm/boot/dts/omap3.dtsi
index ecf5eb584c75..a3ff4933dbc1 100644
--- a/arch/arm/boot/dts/omap3.dtsi
+++ b/arch/arm/boot/dts/omap3.dtsi
@@ -17,6 +17,7 @@
 	interrupt-parent = <&intc>;
 	#address-cells = <1>;
 	#size-cells = <1>;
+	chosen { };
 
 	aliases {
 		i2c0 = &i2c1;
-- 
2.7.4

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

* [PATCH 3/9] ARM: dts: omap4: Add an empty chosen node to top level DTSI
  2016-12-19 14:44 [PATCH 0/9] ARM: omap: Add empty chosen node in SoCs top level DTSI Javier Martinez Canillas
  2016-12-19 14:44 ` [PATCH 1/9] ARM: dts: omap2: Add an empty chosen node to " Javier Martinez Canillas
  2016-12-19 14:44 ` [PATCH 2/9] ARM: dts: omap3: " Javier Martinez Canillas
@ 2016-12-19 14:44 ` Javier Martinez Canillas
  2016-12-19 14:44 ` [PATCH 4/9] ARM: dts: omap5: " Javier Martinez Canillas
                   ` (6 subsequent siblings)
  9 siblings, 0 replies; 12+ messages in thread
From: Javier Martinez Canillas @ 2016-12-19 14:44 UTC (permalink / raw)
  To: linux-kernel
  Cc: Pali Rohar, Javier Martinez Canillas, Benoît Cousson,
	devicetree, Rob Herring, Tony Lindgren, Mark Rutland, linux-omap,
	Russell King, linux-arm-kernel

Commit da6269e7e3dd ("ARM: dts: omap4: Remove skeleton.dtsi usage")
removed the skeleton.dtsi usage since we want to get rid of it.

But this can cause issues when booting a kernel with a boot-loader
that doesn't create a chosen node if this isn't present in the DTB
since the decompressor relies on a pre-existing chosen node to be
available to insert the command line and merge other ATAGS info.

Fixes: da6269e7e3dd ("ARM: dts: omap4: Remove skeleton.dtsi usage")
Reported-by: Pali Rohar <pali.rohar@gmail.com>
Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com>
---

 arch/arm/boot/dts/omap4.dtsi | 1 +
 1 file changed, 1 insertion(+)

diff --git a/arch/arm/boot/dts/omap4.dtsi b/arch/arm/boot/dts/omap4.dtsi
index 8087456b5fbe..578c53f08309 100644
--- a/arch/arm/boot/dts/omap4.dtsi
+++ b/arch/arm/boot/dts/omap4.dtsi
@@ -15,6 +15,7 @@
 	interrupt-parent = <&wakeupgen>;
 	#address-cells = <1>;
 	#size-cells = <1>;
+	chosen { };
 
 	aliases {
 		i2c0 = &i2c1;
-- 
2.7.4

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

* [PATCH 4/9] ARM: dts: omap5: Add an empty chosen node to top level DTSI
  2016-12-19 14:44 [PATCH 0/9] ARM: omap: Add empty chosen node in SoCs top level DTSI Javier Martinez Canillas
                   ` (2 preceding siblings ...)
  2016-12-19 14:44 ` [PATCH 3/9] ARM: dts: omap4: " Javier Martinez Canillas
@ 2016-12-19 14:44 ` Javier Martinez Canillas
  2016-12-19 14:44 ` [PATCH 5/9] ARM: dts: am33xx: " Javier Martinez Canillas
                   ` (5 subsequent siblings)
  9 siblings, 0 replies; 12+ messages in thread
From: Javier Martinez Canillas @ 2016-12-19 14:44 UTC (permalink / raw)
  To: linux-kernel
  Cc: Pali Rohar, Javier Martinez Canillas, Benoît Cousson,
	devicetree, Rob Herring, Tony Lindgren, Mark Rutland, linux-omap,
	Russell King, linux-arm-kernel

Commit 76a8548ea987 ("ARM: dts: omap5: Remove skeleton.dtsi usage")
removed the skeleton.dtsi usage since we want to get rid of it.

But this can cause issues when booting a kernel with a boot-loader
that doesn't create a chosen node if this isn't present in the DTB
since the decompressor relies on a pre-existing chosen node to be
available to insert the command line and merge other ATAGS info.

Fixes: 76a8548ea987 ("ARM: dts: omap5: Remove skeleton.dtsi usage")
Reported-by: Pali Rohar <pali.rohar@gmail.com>
Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com>
---

 arch/arm/boot/dts/omap5.dtsi | 1 +
 1 file changed, 1 insertion(+)

diff --git a/arch/arm/boot/dts/omap5.dtsi b/arch/arm/boot/dts/omap5.dtsi
index 968c67a49dbd..7cd92babc41a 100644
--- a/arch/arm/boot/dts/omap5.dtsi
+++ b/arch/arm/boot/dts/omap5.dtsi
@@ -17,6 +17,7 @@
 
 	compatible = "ti,omap5";
 	interrupt-parent = <&wakeupgen>;
+	chosen { };
 
 	aliases {
 		i2c0 = &i2c1;
-- 
2.7.4

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

* [PATCH 5/9] ARM: dts: am33xx: Add an empty chosen node to top level DTSI
  2016-12-19 14:44 [PATCH 0/9] ARM: omap: Add empty chosen node in SoCs top level DTSI Javier Martinez Canillas
                   ` (3 preceding siblings ...)
  2016-12-19 14:44 ` [PATCH 4/9] ARM: dts: omap5: " Javier Martinez Canillas
@ 2016-12-19 14:44 ` Javier Martinez Canillas
  2016-12-19 14:44 ` [PATCH 6/9] ARM: dts: am4372: " Javier Martinez Canillas
                   ` (4 subsequent siblings)
  9 siblings, 0 replies; 12+ messages in thread
From: Javier Martinez Canillas @ 2016-12-19 14:44 UTC (permalink / raw)
  To: linux-kernel
  Cc: Pali Rohar, Javier Martinez Canillas, Benoît Cousson,
	devicetree, Rob Herring, Tony Lindgren, Mark Rutland, linux-omap,
	Russell King, linux-arm-kernel

Commit f8bf01611c99 ("ARM: dts: am33xx: Remove skeleton.dtsi usage")
removed the skeleton.dtsi usage since we want to get rid of it.

But this can cause issues when booting a kernel with a boot-loader
that doesn't create a chosen node if this isn't present in the DTB
since the decompressor relies on a pre-existing chosen node to be
available to insert the command line and merge other ATAGS info.

Fixes: f8bf01611c99 ("ARM: dts: am33xx: Remove skeleton.dtsi usage")
Reported-by: Pali Rohar <pali.rohar@gmail.com>
Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com>
---

 arch/arm/boot/dts/am33xx.dtsi | 1 +
 1 file changed, 1 insertion(+)

diff --git a/arch/arm/boot/dts/am33xx.dtsi b/arch/arm/boot/dts/am33xx.dtsi
index 64c8aa9057a3..18d72a245e88 100644
--- a/arch/arm/boot/dts/am33xx.dtsi
+++ b/arch/arm/boot/dts/am33xx.dtsi
@@ -16,6 +16,7 @@
 	interrupt-parent = <&intc>;
 	#address-cells = <1>;
 	#size-cells = <1>;
+	chosen { };
 
 	aliases {
 		i2c0 = &i2c0;
-- 
2.7.4

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

* [PATCH 6/9] ARM: dts: am4372: Add an empty chosen node to top level DTSI
  2016-12-19 14:44 [PATCH 0/9] ARM: omap: Add empty chosen node in SoCs top level DTSI Javier Martinez Canillas
                   ` (4 preceding siblings ...)
  2016-12-19 14:44 ` [PATCH 5/9] ARM: dts: am33xx: " Javier Martinez Canillas
@ 2016-12-19 14:44 ` Javier Martinez Canillas
  2016-12-19 14:44 ` [PATCH 7/9] ARM: dts: dm814x: " Javier Martinez Canillas
                   ` (3 subsequent siblings)
  9 siblings, 0 replies; 12+ messages in thread
From: Javier Martinez Canillas @ 2016-12-19 14:44 UTC (permalink / raw)
  To: linux-kernel
  Cc: Pali Rohar, Javier Martinez Canillas, Benoît Cousson,
	devicetree, Rob Herring, Tony Lindgren, Mark Rutland, linux-omap,
	Russell King, linux-arm-kernel

Commit 75813028bbd7 ("ARM: dts: am4372: Remove skeleton.dtsi usage")
removed the skeleton.dtsi usage since we want to get rid of it.

But this can cause issues when booting a kernel with a boot-loader
that doesn't create a chosen node if this isn't present in the DTB
since the decompressor relies on a pre-existing chosen node to be
available to insert the command line and merge other ATAGS info.

Fixes: 75813028bbd7 ("ARM: dts: am4372: Remove skeleton.dtsi usage")
Reported-by: Pali Rohar <pali.rohar@gmail.com>
Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com>
---

 arch/arm/boot/dts/am4372.dtsi | 1 +
 1 file changed, 1 insertion(+)

diff --git a/arch/arm/boot/dts/am4372.dtsi b/arch/arm/boot/dts/am4372.dtsi
index ac55f93fc91e..2df9e6050c2f 100644
--- a/arch/arm/boot/dts/am4372.dtsi
+++ b/arch/arm/boot/dts/am4372.dtsi
@@ -16,6 +16,7 @@
 	interrupt-parent = <&wakeupgen>;
 	#address-cells = <1>;
 	#size-cells = <1>;
+	chosen { };
 
 	memory@0 {
 		device_type = "memory";
-- 
2.7.4

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

* [PATCH 7/9] ARM: dts: dm814x: Add an empty chosen node to top level DTSI
  2016-12-19 14:44 [PATCH 0/9] ARM: omap: Add empty chosen node in SoCs top level DTSI Javier Martinez Canillas
                   ` (5 preceding siblings ...)
  2016-12-19 14:44 ` [PATCH 6/9] ARM: dts: am4372: " Javier Martinez Canillas
@ 2016-12-19 14:44 ` Javier Martinez Canillas
  2016-12-19 14:44 ` [PATCH 8/9] ARM: dts: dm816x: " Javier Martinez Canillas
                   ` (2 subsequent siblings)
  9 siblings, 0 replies; 12+ messages in thread
From: Javier Martinez Canillas @ 2016-12-19 14:44 UTC (permalink / raw)
  To: linux-kernel
  Cc: Pali Rohar, Javier Martinez Canillas, devicetree, Rob Herring,
	Mark Rutland, Russell King, linux-arm-kernel

Commit 76155b378c59 ("ARM: dts: dm814x: Remove skeleton.dtsi usage")
removed the skeleton.dtsi usage since we want to get rid of it.

But this can cause issues when booting a kernel with a boot-loader
that doesn't create a chosen node if this isn't present in the DTB
since the decompressor relies on a pre-existing chosen node to be
available to insert the command line and merge other ATAGS info.

Fixes: 76155b378c59 ("ARM: dts: dm814x: Remove skeleton.dtsi usage")
Reported-by: Pali Rohar <pali.rohar@gmail.com>
Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com>
---

 arch/arm/boot/dts/dm814x.dtsi | 1 +
 1 file changed, 1 insertion(+)

diff --git a/arch/arm/boot/dts/dm814x.dtsi b/arch/arm/boot/dts/dm814x.dtsi
index 1facc5f12cef..81b8cecb5820 100644
--- a/arch/arm/boot/dts/dm814x.dtsi
+++ b/arch/arm/boot/dts/dm814x.dtsi
@@ -12,6 +12,7 @@
 	interrupt-parent = <&intc>;
 	#address-cells = <1>;
 	#size-cells = <1>;
+	chosen { };
 
 	aliases {
 		i2c0 = &i2c1;
-- 
2.7.4

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

* [PATCH 8/9] ARM: dts: dm816x: Add an empty chosen node to top level DTSI
  2016-12-19 14:44 [PATCH 0/9] ARM: omap: Add empty chosen node in SoCs top level DTSI Javier Martinez Canillas
                   ` (6 preceding siblings ...)
  2016-12-19 14:44 ` [PATCH 7/9] ARM: dts: dm814x: " Javier Martinez Canillas
@ 2016-12-19 14:44 ` Javier Martinez Canillas
  2016-12-19 14:44 ` [PATCH 9/9] ARM: dts: dra7: " Javier Martinez Canillas
  2016-12-27 17:30 ` [PATCH 0/9] ARM: omap: Add empty chosen node in SoCs " Tony Lindgren
  9 siblings, 0 replies; 12+ messages in thread
From: Javier Martinez Canillas @ 2016-12-19 14:44 UTC (permalink / raw)
  To: linux-kernel
  Cc: Pali Rohar, Javier Martinez Canillas, devicetree, Rob Herring,
	Mark Rutland, Russell King, linux-arm-kernel

Commit 06bfb9c19957 ("ARM: dts: dm816x: Remove skeleton.dtsi usage")
removed the skeleton.dtsi usage since we want to get rid of it.

But this can cause issues when booting a kernel with a boot-loader
that doesn't create a chosen node if this isn't present in the DTB
since the decompressor relies on a pre-existing chosen node to be
available to insert the command line and merge other ATAGS info.

Fixes: 06bfb9c19957 ("ARM: dts: dm816x: Remove skeleton.dtsi usage")
Reported-by: Pali Rohar <pali.rohar@gmail.com>
Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com>
---

 arch/arm/boot/dts/dm816x.dtsi | 1 +
 1 file changed, 1 insertion(+)

diff --git a/arch/arm/boot/dts/dm816x.dtsi b/arch/arm/boot/dts/dm816x.dtsi
index 61dd2f6b02bc..6db652ae9bd5 100644
--- a/arch/arm/boot/dts/dm816x.dtsi
+++ b/arch/arm/boot/dts/dm816x.dtsi
@@ -12,6 +12,7 @@
 	interrupt-parent = <&intc>;
 	#address-cells = <1>;
 	#size-cells = <1>;
+	chosen { };
 
 	aliases {
 		i2c0 = &i2c1;
-- 
2.7.4

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

* [PATCH 9/9] ARM: dts: dra7: Add an empty chosen node to top level DTSI
  2016-12-19 14:44 [PATCH 0/9] ARM: omap: Add empty chosen node in SoCs top level DTSI Javier Martinez Canillas
                   ` (7 preceding siblings ...)
  2016-12-19 14:44 ` [PATCH 8/9] ARM: dts: dm816x: " Javier Martinez Canillas
@ 2016-12-19 14:44 ` Javier Martinez Canillas
  2016-12-27 17:30 ` [PATCH 0/9] ARM: omap: Add empty chosen node in SoCs " Tony Lindgren
  9 siblings, 0 replies; 12+ messages in thread
From: Javier Martinez Canillas @ 2016-12-19 14:44 UTC (permalink / raw)
  To: linux-kernel
  Cc: Pali Rohar, Javier Martinez Canillas, Benoît Cousson,
	devicetree, Rob Herring, Tony Lindgren, Mark Rutland, linux-omap,
	Russell King, linux-arm-kernel

Commit 55871eb6e2cc ("ARM: dts: dra7: Remove skeleton.dtsi usage")
removed the skeleton.dtsi usage since we want to get rid of it.

But this can cause issues when booting a kernel with a boot-loader
that doesn't create a chosen node if this isn't present in the DTB
since the decompressor relies on a pre-existing chosen node to be
available to insert the command line and merge other ATAGS info.

Fixes: 55871eb6e2cc ("ARM: dts: dra7: Remove skeleton.dtsi usage")
Reported-by: Pali Rohar <pali.rohar@gmail.com>
Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com>

---

 arch/arm/boot/dts/dra7.dtsi | 1 +
 1 file changed, 1 insertion(+)

diff --git a/arch/arm/boot/dts/dra7.dtsi b/arch/arm/boot/dts/dra7.dtsi
index addb7530cfbe..1faf24acd521 100644
--- a/arch/arm/boot/dts/dra7.dtsi
+++ b/arch/arm/boot/dts/dra7.dtsi
@@ -18,6 +18,7 @@
 
 	compatible = "ti,dra7xx";
 	interrupt-parent = <&crossbar_mpu>;
+	chosen { };
 
 	aliases {
 		i2c0 = &i2c1;
-- 
2.7.4

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

* Re: [PATCH 0/9] ARM: omap: Add empty chosen node in SoCs top level DTSI
  2016-12-19 14:44 [PATCH 0/9] ARM: omap: Add empty chosen node in SoCs top level DTSI Javier Martinez Canillas
                   ` (8 preceding siblings ...)
  2016-12-19 14:44 ` [PATCH 9/9] ARM: dts: dra7: " Javier Martinez Canillas
@ 2016-12-27 17:30 ` Tony Lindgren
  9 siblings, 0 replies; 12+ messages in thread
From: Tony Lindgren @ 2016-12-27 17:30 UTC (permalink / raw)
  To: Javier Martinez Canillas
  Cc: linux-kernel, Pali Rohar, Benoît Cousson, devicetree,
	Rob Herring, Mark Rutland, linux-omap, Russell King,
	linux-arm-kernel

* Javier Martinez Canillas <javier@osg.samsung.com> [161219 06:45]:
> Hello Tony,
> 
> As discussed in [0], there's a regression when booting a kernel with a DTB
> that doesn't have a pre-existing chosen node. This is usually not an issue
> for most boards since u-boot creates an empty chosen node if isn't present
> in the DTB.
> 
> But it can be an issue for others bootloaders as Pali pointed out with the
> N9/900/950 phones and the Nokia Loader (NoLo).
> 
> This patch series add chosen nodes in the top level DTSI for all OMAP SoCs.
> 
> [0]: http://www.mail-archive.com/linux-kernel@vger.kernel.org/msg1294379.html

Applying these into omap-for-v4.10/fixes thanks.

Tony

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

* Re: [PATCH 2/9] ARM: dts: omap3: Add an empty chosen node to top level DTSI
  2016-12-19 14:44 ` [PATCH 2/9] ARM: dts: omap3: " Javier Martinez Canillas
@ 2017-01-02 18:01   ` Pali Rohár
  0 siblings, 0 replies; 12+ messages in thread
From: Pali Rohár @ 2017-01-02 18:01 UTC (permalink / raw)
  To: Javier Martinez Canillas
  Cc: linux-kernel, Benoît Cousson, devicetree, Rob Herring,
	Tony Lindgren, Mark Rutland, linux-omap, Russell King,
	linux-arm-kernel

[-- Attachment #1: Type: Text/Plain, Size: 825 bytes --]

On Monday 19 December 2016 15:44:34 Javier Martinez Canillas wrote:
> Commit 008a2ebcd677 ("ARM: dts: omap3: Remove skeleton.dtsi usage")
> removed the skeleton.dtsi usage since we want to get rid of it.
> 
> But this can cause issues when booting a kernel with a boot-loader
> that doesn't create a chosen node if this isn't present in the DTB
> since the decompressor relies on a pre-existing chosen node to be
> available to insert the command line and merge other ATAGS info.
> 
> Fixes: 008a2ebcd677 ("ARM: dts: omap3: Remove skeleton.dtsi usage")
> Reported-by: Pali Rohar <pali.rohar@gmail.com>
> Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com>

Including empty chosen node fixes (or rather workaround) this problem. 
You can add my Tested-By.

-- 
Pali Rohár
pali.rohar@gmail.com

[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 198 bytes --]

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

end of thread, other threads:[~2017-01-02 18:02 UTC | newest]

Thread overview: 12+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-12-19 14:44 [PATCH 0/9] ARM: omap: Add empty chosen node in SoCs top level DTSI Javier Martinez Canillas
2016-12-19 14:44 ` [PATCH 1/9] ARM: dts: omap2: Add an empty chosen node to " Javier Martinez Canillas
2016-12-19 14:44 ` [PATCH 2/9] ARM: dts: omap3: " Javier Martinez Canillas
2017-01-02 18:01   ` Pali Rohár
2016-12-19 14:44 ` [PATCH 3/9] ARM: dts: omap4: " Javier Martinez Canillas
2016-12-19 14:44 ` [PATCH 4/9] ARM: dts: omap5: " Javier Martinez Canillas
2016-12-19 14:44 ` [PATCH 5/9] ARM: dts: am33xx: " Javier Martinez Canillas
2016-12-19 14:44 ` [PATCH 6/9] ARM: dts: am4372: " Javier Martinez Canillas
2016-12-19 14:44 ` [PATCH 7/9] ARM: dts: dm814x: " Javier Martinez Canillas
2016-12-19 14:44 ` [PATCH 8/9] ARM: dts: dm816x: " Javier Martinez Canillas
2016-12-19 14:44 ` [PATCH 9/9] ARM: dts: dra7: " Javier Martinez Canillas
2016-12-27 17:30 ` [PATCH 0/9] ARM: omap: Add empty chosen node in SoCs " Tony Lindgren

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).