[PATCH] Add a vga alias node for P1022

[PATCH] Add a vga alias node for P1022

[Jason Jin]

From: Jason Jin <Jason.jin@freescale.com>

When the vga was set as the stdout and stderr in u-boot, the stdout fixup
code in u-boot need to find out the vga alias node in dtb.

Signed-off-by: Jason Jin <Jason.jin@freescale.com>
---
 arch/powerpc/boot/dts/fsl/p1022si-post.dtsi | 2 +-
 arch/powerpc/boot/dts/fsl/p1022si-pre.dtsi  | 1 +
 2 files changed, 2 insertions(+), 1 deletion(-)

diff --git a/arch/powerpc/boot/dts/fsl/p1022si-post.dtsi b/arch/powerpc/boot/dts/fsl/p1022si-post.dtsi
index 1f189a9..c907c8e 100644
--- a/arch/powerpc/boot/dts/fsl/p1022si-post.dtsi
+++ b/arch/powerpc/boot/dts/fsl/p1022si-post.dtsi
@@ -175,7 +175,7 @@
 
 /include/ "pq3-gpio-0.dtsi"
 
-	display@10000 {
+	display: display@10000 {
 		compatible = "fsl,diu", "fsl,p1022-diu";
 		reg = <0x10000 1000>;
 		interrupts = <64 2 0 0>;
diff --git a/arch/powerpc/boot/dts/fsl/p1022si-pre.dtsi b/arch/powerpc/boot/dts/fsl/p1022si-pre.dtsi
index 1956dea..cb8d816 100644
--- a/arch/powerpc/boot/dts/fsl/p1022si-pre.dtsi
+++ b/arch/powerpc/boot/dts/fsl/p1022si-pre.dtsi
@@ -50,6 +50,7 @@
 		pci0 = &pci0;
 		pci1 = &pci1;
 		pci2 = &pci2;
+		vga = &display;
 	};
 
 	cpus {
-- 
1.8.0

RE: [PATCH] Add a vga alias node for P1022

[Zhengxiong Jin]

> -----Original Message-----
> From: Jin Zhengxiong-R64188
> Sent: Friday, September 06, 2013 5:33 PM
> To: Wood Scott-B07421
> Cc: linuxppc-dev@lists.ozlabs.org; galak@kernel.crashing.org; Jin
> Zhengxiong-R64188
> Subject: [PATCH] Add a vga alias node for P1022
>=20
> From: Jason Jin <Jason.jin@freescale.com>
>=20
> When the vga was set as the stdout and stderr in u-boot, the stdout fixup
> code in u-boot need to find out the vga alias node in dtb.
>=20
> Signed-off-by: Jason Jin <Jason.jin@freescale.com>
> ---
>  arch/powerpc/boot/dts/fsl/p1022si-post.dtsi | 2 +-
> arch/powerpc/boot/dts/fsl/p1022si-pre.dtsi  | 1 +
>  2 files changed, 2 insertions(+), 1 deletion(-)

[Jason Jin-R64188] Any comments about this patch? Thanks.

Re: [PATCH] Add a vga alias node for P1022

[Scott Wood]

On Wed, 2013-11-13 at 00:28 -0600, Jin Zhengxiong-R64188 wrote:
> > -----Original Message-----
> > From: Jin Zhengxiong-R64188
> > Sent: Friday, September 06, 2013 5:33 PM
> > To: Wood Scott-B07421
> > Cc: linuxppc-dev@lists.ozlabs.org; galak@kernel.crashing.org; Jin
> > Zhengxiong-R64188
> > Subject: [PATCH] Add a vga alias node for P1022
> > 
> > From: Jason Jin <Jason.jin@freescale.com>
> > 
> > When the vga was set as the stdout and stderr in u-boot, the stdout fixup
> > code in u-boot need to find out the vga alias node in dtb.
> > 
> > Signed-off-by: Jason Jin <Jason.jin@freescale.com>
> > ---
> >  arch/powerpc/boot/dts/fsl/p1022si-post.dtsi | 2 +-
> > arch/powerpc/boot/dts/fsl/p1022si-pre.dtsi  | 1 +
> >  2 files changed, 2 insertions(+), 1 deletion(-)
> 
> [Jason Jin-R64188] Any comments about this patch? Thanks.

This isn't a VGA-compatible device.  If this is needed by U-Boot, please
provide a comment explaining the compatibility issue, and ideally also
provide a display alias so that no other components grow dependencies on
the vga alias.

-Scott

RE: [PATCH] Add a vga alias node for P1022

[Zhengxiong Jin]

DQo+ID4gPg0KPiA+ID4gV2hlbiB0aGUgdmdhIHdhcyBzZXQgYXMgdGhlIHN0ZG91dCBhbmQgc3Rk
ZXJyIGluIHUtYm9vdCwgdGhlIHN0ZG91dA0KPiA+ID4gZml4dXAgY29kZSBpbiB1LWJvb3QgbmVl
ZCB0byBmaW5kIG91dCB0aGUgdmdhIGFsaWFzIG5vZGUgaW4gZHRiLg0KPiA+ID4NCj4gPiA+IFNp
Z25lZC1vZmYtYnk6IEphc29uIEppbiA8SmFzb24uamluQGZyZWVzY2FsZS5jb20+DQo+ID4gPiAt
LS0NCj4gPiA+ICBhcmNoL3Bvd2VycGMvYm9vdC9kdHMvZnNsL3AxMDIyc2ktcG9zdC5kdHNpIHwg
MiArLQ0KPiA+ID4gYXJjaC9wb3dlcnBjL2Jvb3QvZHRzL2ZzbC9wMTAyMnNpLXByZS5kdHNpICB8
IDEgKw0KPiA+ID4gIDIgZmlsZXMgY2hhbmdlZCwgMiBpbnNlcnRpb25zKCspLCAxIGRlbGV0aW9u
KC0pDQo+ID4NCj4gPiBbSmFzb24gSmluLVI2NDE4OF0gQW55IGNvbW1lbnRzIGFib3V0IHRoaXMg
cGF0Y2g/IFRoYW5rcy4NCj4gDQo+IFRoaXMgaXNuJ3QgYSBWR0EtY29tcGF0aWJsZSBkZXZpY2Uu
ICBJZiB0aGlzIGlzIG5lZWRlZCBieSBVLUJvb3QsIHBsZWFzZQ0KPiBwcm92aWRlIGEgY29tbWVu
dCBleHBsYWluaW5nIHRoZSBjb21wYXRpYmlsaXR5IGlzc3VlLCBhbmQgaWRlYWxseSBhbHNvDQo+
IHByb3ZpZGUgYSBkaXNwbGF5IGFsaWFzIHNvIHRoYXQgbm8gb3RoZXIgY29tcG9uZW50cyBncm93
IGRlcGVuZGVuY2llcyBvbg0KPiB0aGUgdmdhIGFsaWFzLg0KPiANCj4gLVNjb3R0DQo+IA0KW0ph
c29uIEppbi1SNjQxODhdIFRoYW5rcywgd2lsbCBhZGQgZGVzY3JpcHRpb24gZm9yIHdoeSB0aGUg
dmdhIGFsaWFzIG5vZGUgaXMgbmVlZGVkIGJ5IHUtYm9vdCwgYW5kIGFkZCBhbm90aGVyIGRpc3Bs
YXkgYWxpYXMgbm9kZS4NCg==