[PATCH] ARM: dts: pxa3xx: add gcu node

[PATCH] ARM: dts: pxa3xx: add gcu node

[Daniel Mack]

Add a device node for hardware graphic acceleration.

Signed-off-by: Daniel Mack <daniel@zonque.org>
---
 arch/arm/boot/dts/pxa3xx.dtsi | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/arch/arm/boot/dts/pxa3xx.dtsi b/arch/arm/boot/dts/pxa3xx.dtsi
index 3a8f0edc3af9..264f681bebd9 100644
--- a/arch/arm/boot/dts/pxa3xx.dtsi
+++ b/arch/arm/boot/dts/pxa3xx.dtsi
@@ -300,4 +300,12 @@
 		clocks = <&clks CLK_OSTIMER>;
 		status = "okay";
 	};
+
+	gcu: display-controller@54000000 {
+		compatible = "marvell,pxa300-gcu";
+		reg = <0x54000000 0x1000>;
+		interrupts = <39>;
+		clocks = <&clks CLK_PXA300_GCU>;
+		status = "disabled";
+	};
 };
-- 
2.17.2

[PATCH] ARM: dts: pxa3xx: add gcu node

[Daniel Mack]

Add a device node for hardware graphic acceleration.

Signed-off-by: Daniel Mack <daniel@zonque.org>
---
 arch/arm/boot/dts/pxa3xx.dtsi | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/arch/arm/boot/dts/pxa3xx.dtsi b/arch/arm/boot/dts/pxa3xx.dtsi
index 3a8f0edc3af9..264f681bebd9 100644
--- a/arch/arm/boot/dts/pxa3xx.dtsi
+++ b/arch/arm/boot/dts/pxa3xx.dtsi
@@ -300,4 +300,12 @@
 		clocks = <&clks CLK_OSTIMER>;
 		status = "okay";
 	};
+
+	gcu: display-controller at 54000000 {
+		compatible = "marvell,pxa300-gcu";
+		reg = <0x54000000 0x1000>;
+		interrupts = <39>;
+		clocks = <&clks CLK_PXA300_GCU>;
+		status = "disabled";
+	};
 };
-- 
2.17.2

Re: [PATCH] ARM: dts: pxa3xx: add gcu node

[Robert Jarzmik]

Daniel Mack <daniel@zonque.org> writes:

> Add a device node for hardware graphic acceleration.
>
> Signed-off-by: Daniel Mack <daniel@zonque.org>
Hi Daniel,

> +	gcu: display-controller@54000000 {
> +		compatible = "marvell,pxa300-gcu";
Isn't every IP devicetree entry supposed to have a documentation in
Documentation/devicetree going along a definition in the .dtsi files ?

Cheers.

-- 
Robert

[PATCH] ARM: dts: pxa3xx: add gcu node

[Robert Jarzmik]

Daniel Mack <daniel@zonque.org> writes:

> Add a device node for hardware graphic acceleration.
>
> Signed-off-by: Daniel Mack <daniel@zonque.org>
Hi Daniel,

> +	gcu: display-controller at 54000000 {
> +		compatible = "marvell,pxa300-gcu";
Isn't every IP devicetree entry supposed to have a documentation in
Documentation/devicetree going along a definition in the .dtsi files ?

Cheers.

-- 
Robert

Re: [PATCH] ARM: dts: pxa3xx: add gcu node

[Daniel Mack]

Hi Robert,

On 10/11/2018 9:43 PM, Robert Jarzmik wrote:
> Daniel Mack <daniel@zonque.org> writes:
> 
>> Add a device node for hardware graphic acceleration.
>>
>> Signed-off-by: Daniel Mack <daniel@zonque.org>
> Hi Daniel,
> 
>> +	gcu: display-controller@54000000 {
>> +		compatible = "marvell,pxa300-gcu";
> Isn't every IP devicetree entry supposed to have a documentation in
> Documentation/devicetree going along a definition in the .dtsi files ?

Ah, that's already there, see cf925575d292b.

I did that when I taught the driver DT probing.


Thanks,
Daniel

[PATCH] ARM: dts: pxa3xx: add gcu node

[Daniel Mack]

Hi Robert,

On 10/11/2018 9:43 PM, Robert Jarzmik wrote:
> Daniel Mack <daniel@zonque.org> writes:
> 
>> Add a device node for hardware graphic acceleration.
>>
>> Signed-off-by: Daniel Mack <daniel@zonque.org>
> Hi Daniel,
> 
>> +	gcu: display-controller at 54000000 {
>> +		compatible = "marvell,pxa300-gcu";
> Isn't every IP devicetree entry supposed to have a documentation in
> Documentation/devicetree going along a definition in the .dtsi files ?

Ah, that's already there, see cf925575d292b.

I did that when I taught the driver DT probing.


Thanks,
Daniel

Re: [PATCH] ARM: dts: pxa3xx: add gcu node

[Robert Jarzmik]

Daniel Mack <daniel@zonque.org> writes:

> Ah, that's already there, see cf925575d292b.
Applied to pxa/dt, thanks.

Cheers.

-- 
Robert

[PATCH] ARM: dts: pxa3xx: add gcu node

[Robert Jarzmik]

Daniel Mack <daniel@zonque.org> writes:

> Ah, that's already there, see cf925575d292b.
Applied to pxa/dt, thanks.

Cheers.

-- 
Robert