diff mbox series

[linux,dev-4.13,v2] ARM: dts: romulus: Enable the GFX IP

Message ID 20180426021026.12548-1-joel@jms.id.au
State Accepted, archived
Headers show
Series [linux,dev-4.13,v2] ARM: dts: romulus: Enable the GFX IP | expand

Commit Message

Joel Stanley April 26, 2018, 2:10 a.m. UTC
The GFX controller is the internal graphics device used by the SoC
(opposed to the one connected via the PCIe device and used by the host).

This configures it with a framebuffer region and enables the GFX node.

Signed-off-by: Joel Stanley <joel@jms.id.au>
---
v2: Fix the commit message, as we do not output bmc kenrel messages to
the VGA device with this change

 arch/arm/boot/dts/aspeed-bmc-opp-romulus.dts | 12 ++++++++++++
 1 file changed, 12 insertions(+)

Comments

Andrew Jeffery April 30, 2018, 11:04 p.m. UTC | #1
On Thu, 26 Apr 2018, at 11:40, Joel Stanley wrote:
> The GFX controller is the internal graphics device used by the SoC
> (opposed to the one connected via the PCIe device and used by the host).
> 
> This configures it with a framebuffer region and enables the GFX node.
> 
> Signed-off-by: Joel Stanley <joel@jms.id.au>

Acked-by: Andrew Jeffery <andrew@aj.id.au>

> ---
> v2: Fix the commit message, as we do not output bmc kenrel messages to
> the VGA device with this change
> 
>  arch/arm/boot/dts/aspeed-bmc-opp-romulus.dts | 12 ++++++++++++
>  1 file changed, 12 insertions(+)
> 
> diff --git a/arch/arm/boot/dts/aspeed-bmc-opp-romulus.dts b/arch/arm/
> boot/dts/aspeed-bmc-opp-romulus.dts
> index 00812a2fecf0..d746e562e8fd 100644
> --- a/arch/arm/boot/dts/aspeed-bmc-opp-romulus.dts
> +++ b/arch/arm/boot/dts/aspeed-bmc-opp-romulus.dts
> @@ -30,6 +30,13 @@
>  			no-map;
>  			reg = <0x98000000 0x04000000>; /* 64M */
>  		};
> +
> +		gfx_memory: framebuffer {
> +			size = <0x01000000>;
> +			alignment = <0x01000000>;
> +			compatible = "shared-dma-pool";
> +			reusable;
> +		};
>  	};
>  
>  	leds {
> @@ -280,3 +287,8 @@
>  &ibt {
>  	status = "okay";
>  };
> +
> +&gfx {
> +     status = "okay";
> +     memory-region = <&gfx_memory>;
> +};
> -- 
> 2.17.0
>
Joel Stanley May 1, 2018, 3:18 a.m. UTC | #2
On 1 May 2018 at 08:34, Andrew Jeffery <andrew@aj.id.au> wrote:
> On Thu, 26 Apr 2018, at 11:40, Joel Stanley wrote:
>> The GFX controller is the internal graphics device used by the SoC
>> (opposed to the one connected via the PCIe device and used by the host).
>>
>> This configures it with a framebuffer region and enables the GFX node.
>>
>> Signed-off-by: Joel Stanley <joel@jms.id.au>
>
> Acked-by: Andrew Jeffery <andrew@aj.id.au>

Merged to dev-4.13.

Cheers,

Joel
diff mbox series

Patch

diff --git a/arch/arm/boot/dts/aspeed-bmc-opp-romulus.dts b/arch/arm/boot/dts/aspeed-bmc-opp-romulus.dts
index 00812a2fecf0..d746e562e8fd 100644
--- a/arch/arm/boot/dts/aspeed-bmc-opp-romulus.dts
+++ b/arch/arm/boot/dts/aspeed-bmc-opp-romulus.dts
@@ -30,6 +30,13 @@ 
 			no-map;
 			reg = <0x98000000 0x04000000>; /* 64M */
 		};
+
+		gfx_memory: framebuffer {
+			size = <0x01000000>;
+			alignment = <0x01000000>;
+			compatible = "shared-dma-pool";
+			reusable;
+		};
 	};
 
 	leds {
@@ -280,3 +287,8 @@ 
 &ibt {
 	status = "okay";
 };
+
+&gfx {
+     status = "okay";
+     memory-region = <&gfx_memory>;
+};