* [PATCH v1 0/3] Add Vivante GPU support on STM32MP157c
@ 2019-02-15 15:58 ` Pierre-Yves MORDRET
0 siblings, 0 replies; 22+ messages in thread
From: Pierre-Yves MORDRET @ 2019-02-15 15:58 UTC (permalink / raw)
To: Rob Herring, Mark Rutland, Maxime Coquelin, Alexandre Torgue
Cc: devicetree, Philippe Cornu, Pierre-Yves MORDRET, linux-kernel,
linux-stm32, linux-arm-kernel
Add and enable Vivante GPU on stm32mp157c for ED1, DK1 and DK2 boards.
Pierre-Yves MORDRET (3):
ARM: dts: stm32: Add Vivante GPU support on STM32MP157c
ARM: dts: stm32: enable Vivante GPU support on stm32mp157c-ed1 board
ARM: dts: stm32: enable Vivante GPU support on stm32mp157c-dk1 board
arch/arm/boot/dts/stm32mp157a-dk1.dts | 16 ++++++++++++++++
arch/arm/boot/dts/stm32mp157c-ed1.dts | 16 ++++++++++++++++
arch/arm/boot/dts/stm32mp157c.dtsi | 10 ++++++++++
3 files changed, 42 insertions(+)
--
2.7.4
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 22+ messages in thread
* [PATCH v1 0/3] Add Vivante GPU support on STM32MP157c
@ 2019-02-15 15:58 ` Pierre-Yves MORDRET
0 siblings, 0 replies; 22+ messages in thread
From: Pierre-Yves MORDRET @ 2019-02-15 15:58 UTC (permalink / raw)
To: Rob Herring, Mark Rutland, Maxime Coquelin, Alexandre Torgue
Cc: devicetree, Philippe Cornu, Pierre-Yves MORDRET, linux-kernel,
linux-stm32, linux-arm-kernel
Add and enable Vivante GPU on stm32mp157c for ED1, DK1 and DK2 boards.
Pierre-Yves MORDRET (3):
ARM: dts: stm32: Add Vivante GPU support on STM32MP157c
ARM: dts: stm32: enable Vivante GPU support on stm32mp157c-ed1 board
ARM: dts: stm32: enable Vivante GPU support on stm32mp157c-dk1 board
arch/arm/boot/dts/stm32mp157a-dk1.dts | 16 ++++++++++++++++
arch/arm/boot/dts/stm32mp157c-ed1.dts | 16 ++++++++++++++++
arch/arm/boot/dts/stm32mp157c.dtsi | 10 ++++++++++
3 files changed, 42 insertions(+)
--
2.7.4
^ permalink raw reply [flat|nested] 22+ messages in thread
* [PATCH v1 1/3] ARM: dts: stm32: Add Vivante GPU support on STM32MP157c
2019-02-15 15:58 ` Pierre-Yves MORDRET
(?)
@ 2019-02-15 15:58 ` Pierre-Yves MORDRET
-1 siblings, 0 replies; 22+ messages in thread
From: Pierre-Yves MORDRET @ 2019-02-15 15:58 UTC (permalink / raw)
To: Rob Herring, Mark Rutland, Maxime Coquelin, Alexandre Torgue
Cc: linux-stm32, devicetree, linux-arm-kernel, linux-kernel,
Pierre-Yves MORDRET, Philippe Cornu
Append Vivante GPU DT configuration.
Signed-off-by: Pierre-Yves MORDRET <pierre-yves.mordret@st.com>
---
arch/arm/boot/dts/stm32mp157c.dtsi | 10 ++++++++++
1 file changed, 10 insertions(+)
diff --git a/arch/arm/boot/dts/stm32mp157c.dtsi b/arch/arm/boot/dts/stm32mp157c.dtsi
index f8bbfff..29540bc 100644
--- a/arch/arm/boot/dts/stm32mp157c.dtsi
+++ b/arch/arm/boot/dts/stm32mp157c.dtsi
@@ -1106,6 +1106,16 @@
status = "disabled";
};
+ gpu: gpu@59000000 {
+ compatible = "vivante,gc";
+ reg = <0x59000000 0x800>;
+ interrupts = <GIC_SPI 109 IRQ_TYPE_LEVEL_HIGH>;
+ clocks = <&rcc GPU>, <&rcc GPU_K>;
+ clock-names = "bus" ,"core";
+ resets = <&rcc GPU_R>;
+ status = "disabled";
+ };
+
dsi: dsi@5a000000 {
compatible = "st,stm32-dsi";
reg = <0x5a000000 0x800>;
--
2.7.4
^ permalink raw reply related [flat|nested] 22+ messages in thread* [PATCH v1 1/3] ARM: dts: stm32: Add Vivante GPU support on STM32MP157c
@ 2019-02-15 15:58 ` Pierre-Yves MORDRET
0 siblings, 0 replies; 22+ messages in thread
From: Pierre-Yves MORDRET @ 2019-02-15 15:58 UTC (permalink / raw)
To: Rob Herring, Mark Rutland, Maxime Coquelin, Alexandre Torgue
Cc: devicetree, Philippe Cornu, Pierre-Yves MORDRET, linux-kernel,
linux-stm32, linux-arm-kernel
Append Vivante GPU DT configuration.
Signed-off-by: Pierre-Yves MORDRET <pierre-yves.mordret@st.com>
---
arch/arm/boot/dts/stm32mp157c.dtsi | 10 ++++++++++
1 file changed, 10 insertions(+)
diff --git a/arch/arm/boot/dts/stm32mp157c.dtsi b/arch/arm/boot/dts/stm32mp157c.dtsi
index f8bbfff..29540bc 100644
--- a/arch/arm/boot/dts/stm32mp157c.dtsi
+++ b/arch/arm/boot/dts/stm32mp157c.dtsi
@@ -1106,6 +1106,16 @@
status = "disabled";
};
+ gpu: gpu@59000000 {
+ compatible = "vivante,gc";
+ reg = <0x59000000 0x800>;
+ interrupts = <GIC_SPI 109 IRQ_TYPE_LEVEL_HIGH>;
+ clocks = <&rcc GPU>, <&rcc GPU_K>;
+ clock-names = "bus" ,"core";
+ resets = <&rcc GPU_R>;
+ status = "disabled";
+ };
+
dsi: dsi@5a000000 {
compatible = "st,stm32-dsi";
reg = <0x5a000000 0x800>;
--
2.7.4
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 22+ messages in thread* [PATCH v1 1/3] ARM: dts: stm32: Add Vivante GPU support on STM32MP157c
@ 2019-02-15 15:58 ` Pierre-Yves MORDRET
0 siblings, 0 replies; 22+ messages in thread
From: Pierre-Yves MORDRET @ 2019-02-15 15:58 UTC (permalink / raw)
To: Rob Herring, Mark Rutland, Maxime Coquelin, Alexandre Torgue
Cc: linux-stm32, devicetree, linux-arm-kernel, linux-kernel,
Pierre-Yves MORDRET, Philippe Cornu
Append Vivante GPU DT configuration.
Signed-off-by: Pierre-Yves MORDRET <pierre-yves.mordret@st.com>
---
arch/arm/boot/dts/stm32mp157c.dtsi | 10 ++++++++++
1 file changed, 10 insertions(+)
diff --git a/arch/arm/boot/dts/stm32mp157c.dtsi b/arch/arm/boot/dts/stm32mp157c.dtsi
index f8bbfff..29540bc 100644
--- a/arch/arm/boot/dts/stm32mp157c.dtsi
+++ b/arch/arm/boot/dts/stm32mp157c.dtsi
@@ -1106,6 +1106,16 @@
status = "disabled";
};
+ gpu: gpu@59000000 {
+ compatible = "vivante,gc";
+ reg = <0x59000000 0x800>;
+ interrupts = <GIC_SPI 109 IRQ_TYPE_LEVEL_HIGH>;
+ clocks = <&rcc GPU>, <&rcc GPU_K>;
+ clock-names = "bus" ,"core";
+ resets = <&rcc GPU_R>;
+ status = "disabled";
+ };
+
dsi: dsi@5a000000 {
compatible = "st,stm32-dsi";
reg = <0x5a000000 0x800>;
--
2.7.4
^ permalink raw reply related [flat|nested] 22+ messages in thread* Re: [PATCH v1 1/3] ARM: dts: stm32: Add Vivante GPU support on STM32MP157c
2019-02-15 15:58 ` Pierre-Yves MORDRET
@ 2019-02-15 16:11 ` Lucas Stach
-1 siblings, 0 replies; 22+ messages in thread
From: Lucas Stach @ 2019-02-15 16:11 UTC (permalink / raw)
To: Pierre-Yves MORDRET, Rob Herring, Mark Rutland, Maxime Coquelin,
Alexandre Torgue
Cc: devicetree, Philippe Cornu, linux-kernel, linux-stm32, linux-arm-kernel
Hi Pierre-Yves,
Am Freitag, den 15.02.2019, 16:58 +0100 schrieb Pierre-Yves MORDRET:
> Append Vivante GPU DT configuration.
>
> Signed-off-by: Pierre-Yves MORDRET <pierre-yves.mordret@st.com>
> ---
> arch/arm/boot/dts/stm32mp157c.dtsi | 10 ++++++++++
> 1 file changed, 10 insertions(+)
>
> diff --git a/arch/arm/boot/dts/stm32mp157c.dtsi b/arch/arm/boot/dts/stm32mp157c.dtsi
> index f8bbfff..29540bc 100644
> --- a/arch/arm/boot/dts/stm32mp157c.dtsi
> +++ b/arch/arm/boot/dts/stm32mp157c.dtsi
> @@ -1106,6 +1106,16 @@
> > status = "disabled";
> > };
>
> + gpu: gpu@59000000 {
> + compatible = "vivante,gc";
> + reg = <0x59000000 0x800>;
> + interrupts = <GIC_SPI 109 IRQ_TYPE_LEVEL_HIGH>;
> + clocks = <&rcc GPU>, <&rcc GPU_K>;
> + clock-names = "bus" ,"core";
Is this a 2D only core? If it is also 3D capable the binding says you
need to specify the shader clock (may be the same clock feeding the
core).
Regards,
Lucas
> + resets = <&rcc GPU_R>;
> + status = "disabled";
> + };
> +
> dsi: dsi@5a000000 {
> compatible = "st,stm32-dsi";
> reg = <0x5a000000 0x800>;
^ permalink raw reply [flat|nested] 22+ messages in thread* Re: [PATCH v1 1/3] ARM: dts: stm32: Add Vivante GPU support on STM32MP157c
@ 2019-02-15 16:11 ` Lucas Stach
0 siblings, 0 replies; 22+ messages in thread
From: Lucas Stach @ 2019-02-15 16:11 UTC (permalink / raw)
To: Pierre-Yves MORDRET, Rob Herring, Mark Rutland, Maxime Coquelin,
Alexandre Torgue
Cc: devicetree, linux-stm32, Philippe Cornu, linux-arm-kernel, linux-kernel
Hi Pierre-Yves,
Am Freitag, den 15.02.2019, 16:58 +0100 schrieb Pierre-Yves MORDRET:
> Append Vivante GPU DT configuration.
>
> Signed-off-by: Pierre-Yves MORDRET <pierre-yves.mordret@st.com>
> ---
> arch/arm/boot/dts/stm32mp157c.dtsi | 10 ++++++++++
> 1 file changed, 10 insertions(+)
>
> diff --git a/arch/arm/boot/dts/stm32mp157c.dtsi b/arch/arm/boot/dts/stm32mp157c.dtsi
> index f8bbfff..29540bc 100644
> --- a/arch/arm/boot/dts/stm32mp157c.dtsi
> +++ b/arch/arm/boot/dts/stm32mp157c.dtsi
> @@ -1106,6 +1106,16 @@
> > status = "disabled";
> > };
>
> + gpu: gpu@59000000 {
> + compatible = "vivante,gc";
> + reg = <0x59000000 0x800>;
> + interrupts = <GIC_SPI 109 IRQ_TYPE_LEVEL_HIGH>;
> + clocks = <&rcc GPU>, <&rcc GPU_K>;
> + clock-names = "bus" ,"core";
Is this a 2D only core? If it is also 3D capable the binding says you
need to specify the shader clock (may be the same clock feeding the
core).
Regards,
Lucas
> + resets = <&rcc GPU_R>;
> + status = "disabled";
> + };
> +
> dsi: dsi@5a000000 {
> compatible = "st,stm32-dsi";
> reg = <0x5a000000 0x800>;
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 22+ messages in thread* Re: [PATCH v1 1/3] ARM: dts: stm32: Add Vivante GPU support on STM32MP157c
2019-02-15 16:11 ` Lucas Stach
@ 2019-02-25 15:48 ` Pierre Yves MORDRET
-1 siblings, 0 replies; 22+ messages in thread
From: Pierre Yves MORDRET @ 2019-02-25 15:48 UTC (permalink / raw)
To: Lucas Stach, Rob Herring, Mark Rutland, Maxime Coquelin,
Alexandre Torgue
Cc: devicetree, Philippe Cornu, linux-kernel, linux-stm32, linux-arm-kernel
Hi Lucas
sorry for the delay : winter season :)
On 2/15/19 5:11 PM, Lucas Stach wrote:
> Hi Pierre-Yves,
>
> Am Freitag, den 15.02.2019, 16:58 +0100 schrieb Pierre-Yves MORDRET:
>> Append Vivante GPU DT configuration.
>>
>> Signed-off-by: Pierre-Yves MORDRET <pierre-yves.mordret@st.com>
>> ---
>> arch/arm/boot/dts/stm32mp157c.dtsi | 10 ++++++++++
>> 1 file changed, 10 insertions(+)
>>
>> diff --git a/arch/arm/boot/dts/stm32mp157c.dtsi b/arch/arm/boot/dts/stm32mp157c.dtsi
>> index f8bbfff..29540bc 100644
>> --- a/arch/arm/boot/dts/stm32mp157c.dtsi
>> +++ b/arch/arm/boot/dts/stm32mp157c.dtsi
>> @@ -1106,6 +1106,16 @@
>>> status = "disabled";
>>> };
>>
>> + gpu: gpu@59000000 {
>> + compatible = "vivante,gc";
>> + reg = <0x59000000 0x800>;
>> + interrupts = <GIC_SPI 109 IRQ_TYPE_LEVEL_HIGH>;
>> + clocks = <&rcc GPU>, <&rcc GPU_K>;
>> + clock-names = "bus" ,"core";
>
> Is this a 2D only core? If it is also 3D capable the binding says you
> need to specify the shader clock (may be the same clock feeding the
> core).
Yes, this is a 3D core.
As for "shader" clock both "core" and "shader" are connected internally. This is
why no "shader" input clock is seen.
Nonetheless we may add it for bindings compliance but from STM32MP1 pov this
won't make any changes.
Hope it clarifies.
Regards
>
> Regards,
> Lucas
>
>> + resets = <&rcc GPU_R>;
>> + status = "disabled";
>> + };
>> +
>> dsi: dsi@5a000000 {
>> compatible = "st,stm32-dsi";
>> reg = <0x5a000000 0x800>;
^ permalink raw reply [flat|nested] 22+ messages in thread* Re: [PATCH v1 1/3] ARM: dts: stm32: Add Vivante GPU support on STM32MP157c
@ 2019-02-25 15:48 ` Pierre Yves MORDRET
0 siblings, 0 replies; 22+ messages in thread
From: Pierre Yves MORDRET @ 2019-02-25 15:48 UTC (permalink / raw)
To: Lucas Stach, Rob Herring, Mark Rutland, Maxime Coquelin,
Alexandre Torgue
Cc: devicetree, linux-stm32, Philippe Cornu, linux-arm-kernel, linux-kernel
Hi Lucas
sorry for the delay : winter season :)
On 2/15/19 5:11 PM, Lucas Stach wrote:
> Hi Pierre-Yves,
>
> Am Freitag, den 15.02.2019, 16:58 +0100 schrieb Pierre-Yves MORDRET:
>> Append Vivante GPU DT configuration.
>>
>> Signed-off-by: Pierre-Yves MORDRET <pierre-yves.mordret@st.com>
>> ---
>> arch/arm/boot/dts/stm32mp157c.dtsi | 10 ++++++++++
>> 1 file changed, 10 insertions(+)
>>
>> diff --git a/arch/arm/boot/dts/stm32mp157c.dtsi b/arch/arm/boot/dts/stm32mp157c.dtsi
>> index f8bbfff..29540bc 100644
>> --- a/arch/arm/boot/dts/stm32mp157c.dtsi
>> +++ b/arch/arm/boot/dts/stm32mp157c.dtsi
>> @@ -1106,6 +1106,16 @@
>>> status = "disabled";
>>> };
>>
>> + gpu: gpu@59000000 {
>> + compatible = "vivante,gc";
>> + reg = <0x59000000 0x800>;
>> + interrupts = <GIC_SPI 109 IRQ_TYPE_LEVEL_HIGH>;
>> + clocks = <&rcc GPU>, <&rcc GPU_K>;
>> + clock-names = "bus" ,"core";
>
> Is this a 2D only core? If it is also 3D capable the binding says you
> need to specify the shader clock (may be the same clock feeding the
> core).
Yes, this is a 3D core.
As for "shader" clock both "core" and "shader" are connected internally. This is
why no "shader" input clock is seen.
Nonetheless we may add it for bindings compliance but from STM32MP1 pov this
won't make any changes.
Hope it clarifies.
Regards
>
> Regards,
> Lucas
>
>> + resets = <&rcc GPU_R>;
>> + status = "disabled";
>> + };
>> +
>> dsi: dsi@5a000000 {
>> compatible = "st,stm32-dsi";
>> reg = <0x5a000000 0x800>;
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 22+ messages in thread
* [PATCH v1 2/3] ARM: dts: stm32: enable Vivante GPU support on stm32mp157c-ed1 board
2019-02-15 15:58 ` Pierre-Yves MORDRET
(?)
@ 2019-02-15 15:58 ` Pierre-Yves MORDRET
-1 siblings, 0 replies; 22+ messages in thread
From: Pierre-Yves MORDRET @ 2019-02-15 15:58 UTC (permalink / raw)
To: Rob Herring, Mark Rutland, Maxime Coquelin, Alexandre Torgue
Cc: linux-stm32, devicetree, linux-arm-kernel, linux-kernel,
Pierre-Yves MORDRET, Philippe Cornu
Enable Vivante GPU driver for stm32mp157c-ed1 board.
Signed-off-by: Pierre-Yves MORDRET <pierre-yves.mordret@st.com>
---
arch/arm/boot/dts/stm32mp157c-ed1.dts | 16 ++++++++++++++++
1 file changed, 16 insertions(+)
diff --git a/arch/arm/boot/dts/stm32mp157c-ed1.dts b/arch/arm/boot/dts/stm32mp157c-ed1.dts
index 98ef7a0..792f402 100644
--- a/arch/arm/boot/dts/stm32mp157c-ed1.dts
+++ b/arch/arm/boot/dts/stm32mp157c-ed1.dts
@@ -20,6 +20,17 @@
reg = <0xC0000000 0x40000000>;
};
+ reserved-memory {
+ #address-cells = <1>;
+ #size-cells = <1>;
+ ranges;
+
+ gpu_reserved: gpu@f8000000 {
+ reg = <0xf8000000 0x8000000>;
+ no-map;
+ };
+ };
+
aliases {
serial0 = &uart4;
};
@@ -53,6 +64,11 @@
status = "okay";
};
+&gpu {
+ contiguous-area = <&gpu_reserved>;
+ status = "okay";
+};
+
&i2c4 {
pinctrl-names = "default";
pinctrl-0 = <&i2c4_pins_a>;
--
2.7.4
^ permalink raw reply related [flat|nested] 22+ messages in thread* [PATCH v1 2/3] ARM: dts: stm32: enable Vivante GPU support on stm32mp157c-ed1 board
@ 2019-02-15 15:58 ` Pierre-Yves MORDRET
0 siblings, 0 replies; 22+ messages in thread
From: Pierre-Yves MORDRET @ 2019-02-15 15:58 UTC (permalink / raw)
To: Rob Herring, Mark Rutland, Maxime Coquelin, Alexandre Torgue
Cc: devicetree, Philippe Cornu, Pierre-Yves MORDRET, linux-kernel,
linux-stm32, linux-arm-kernel
Enable Vivante GPU driver for stm32mp157c-ed1 board.
Signed-off-by: Pierre-Yves MORDRET <pierre-yves.mordret@st.com>
---
arch/arm/boot/dts/stm32mp157c-ed1.dts | 16 ++++++++++++++++
1 file changed, 16 insertions(+)
diff --git a/arch/arm/boot/dts/stm32mp157c-ed1.dts b/arch/arm/boot/dts/stm32mp157c-ed1.dts
index 98ef7a0..792f402 100644
--- a/arch/arm/boot/dts/stm32mp157c-ed1.dts
+++ b/arch/arm/boot/dts/stm32mp157c-ed1.dts
@@ -20,6 +20,17 @@
reg = <0xC0000000 0x40000000>;
};
+ reserved-memory {
+ #address-cells = <1>;
+ #size-cells = <1>;
+ ranges;
+
+ gpu_reserved: gpu@f8000000 {
+ reg = <0xf8000000 0x8000000>;
+ no-map;
+ };
+ };
+
aliases {
serial0 = &uart4;
};
@@ -53,6 +64,11 @@
status = "okay";
};
+&gpu {
+ contiguous-area = <&gpu_reserved>;
+ status = "okay";
+};
+
&i2c4 {
pinctrl-names = "default";
pinctrl-0 = <&i2c4_pins_a>;
--
2.7.4
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 22+ messages in thread* [PATCH v1 2/3] ARM: dts: stm32: enable Vivante GPU support on stm32mp157c-ed1 board
@ 2019-02-15 15:58 ` Pierre-Yves MORDRET
0 siblings, 0 replies; 22+ messages in thread
From: Pierre-Yves MORDRET @ 2019-02-15 15:58 UTC (permalink / raw)
To: Rob Herring, Mark Rutland, Maxime Coquelin, Alexandre Torgue
Cc: linux-stm32, devicetree, linux-arm-kernel, linux-kernel,
Pierre-Yves MORDRET, Philippe Cornu
Enable Vivante GPU driver for stm32mp157c-ed1 board.
Signed-off-by: Pierre-Yves MORDRET <pierre-yves.mordret@st.com>
---
arch/arm/boot/dts/stm32mp157c-ed1.dts | 16 ++++++++++++++++
1 file changed, 16 insertions(+)
diff --git a/arch/arm/boot/dts/stm32mp157c-ed1.dts b/arch/arm/boot/dts/stm32mp157c-ed1.dts
index 98ef7a0..792f402 100644
--- a/arch/arm/boot/dts/stm32mp157c-ed1.dts
+++ b/arch/arm/boot/dts/stm32mp157c-ed1.dts
@@ -20,6 +20,17 @@
reg = <0xC0000000 0x40000000>;
};
+ reserved-memory {
+ #address-cells = <1>;
+ #size-cells = <1>;
+ ranges;
+
+ gpu_reserved: gpu@f8000000 {
+ reg = <0xf8000000 0x8000000>;
+ no-map;
+ };
+ };
+
aliases {
serial0 = &uart4;
};
@@ -53,6 +64,11 @@
status = "okay";
};
+&gpu {
+ contiguous-area = <&gpu_reserved>;
+ status = "okay";
+};
+
&i2c4 {
pinctrl-names = "default";
pinctrl-0 = <&i2c4_pins_a>;
--
2.7.4
^ permalink raw reply related [flat|nested] 22+ messages in thread* Re: [PATCH v1 2/3] ARM: dts: stm32: enable Vivante GPU support on stm32mp157c-ed1 board
2019-02-15 15:58 ` Pierre-Yves MORDRET
@ 2019-02-15 16:14 ` Lucas Stach
-1 siblings, 0 replies; 22+ messages in thread
From: Lucas Stach @ 2019-02-15 16:14 UTC (permalink / raw)
To: Pierre-Yves MORDRET, Rob Herring, Mark Rutland, Maxime Coquelin,
Alexandre Torgue
Cc: devicetree, Philippe Cornu, linux-kernel, linux-stm32, linux-arm-kernel
Am Freitag, den 15.02.2019, 16:58 +0100 schrieb Pierre-Yves MORDRET:
> Enable Vivante GPU driver for stm32mp157c-ed1 board.
>
> Signed-off-by: Pierre-Yves MORDRET <pierre-yves.mordret@st.com>
> ---
> arch/arm/boot/dts/stm32mp157c-ed1.dts | 16 ++++++++++++++++
> 1 file changed, 16 insertions(+)
>
> diff --git a/arch/arm/boot/dts/stm32mp157c-ed1.dts
> b/arch/arm/boot/dts/stm32mp157c-ed1.dts
> index 98ef7a0..792f402 100644
> --- a/arch/arm/boot/dts/stm32mp157c-ed1.dts
> +++ b/arch/arm/boot/dts/stm32mp157c-ed1.dts
> @@ -20,6 +20,17 @@
> reg = <0xC0000000 0x40000000>;
> };
>
> + reserved-memory {
> + #address-cells = <1>;
> + #size-cells = <1>;
> + ranges;
> +
> + gpu_reserved: gpu@f8000000 {
> + reg = <0xf8000000 0x8000000>;
> + no-map;
> + };
> + };
> +
I don't think you need this. The GPU driver uses very little contig dma
memory. Most of the GPU memory is ordinary paged memory, so setting
aside a private memory region of this size is a waste of memory. Most
likely you can just allow the GPU driver to use the system global CMA
region.
Regards,
Lucas
> aliases {
> serial0 = &uart4;
> };
> @@ -53,6 +64,11 @@
> status = "okay";
> };
>
> +&gpu {
> + contiguous-area = <&gpu_reserved>;
> + status = "okay";
> +};
> +
> &i2c4 {
> pinctrl-names = "default";
> pinctrl-0 = <&i2c4_pins_a>;
^ permalink raw reply [flat|nested] 22+ messages in thread* Re: [PATCH v1 2/3] ARM: dts: stm32: enable Vivante GPU support on stm32mp157c-ed1 board
@ 2019-02-15 16:14 ` Lucas Stach
0 siblings, 0 replies; 22+ messages in thread
From: Lucas Stach @ 2019-02-15 16:14 UTC (permalink / raw)
To: Pierre-Yves MORDRET, Rob Herring, Mark Rutland, Maxime Coquelin,
Alexandre Torgue
Cc: devicetree, linux-stm32, Philippe Cornu, linux-arm-kernel, linux-kernel
Am Freitag, den 15.02.2019, 16:58 +0100 schrieb Pierre-Yves MORDRET:
> Enable Vivante GPU driver for stm32mp157c-ed1 board.
>
> Signed-off-by: Pierre-Yves MORDRET <pierre-yves.mordret@st.com>
> ---
> arch/arm/boot/dts/stm32mp157c-ed1.dts | 16 ++++++++++++++++
> 1 file changed, 16 insertions(+)
>
> diff --git a/arch/arm/boot/dts/stm32mp157c-ed1.dts
> b/arch/arm/boot/dts/stm32mp157c-ed1.dts
> index 98ef7a0..792f402 100644
> --- a/arch/arm/boot/dts/stm32mp157c-ed1.dts
> +++ b/arch/arm/boot/dts/stm32mp157c-ed1.dts
> @@ -20,6 +20,17 @@
> reg = <0xC0000000 0x40000000>;
> };
>
> + reserved-memory {
> + #address-cells = <1>;
> + #size-cells = <1>;
> + ranges;
> +
> + gpu_reserved: gpu@f8000000 {
> + reg = <0xf8000000 0x8000000>;
> + no-map;
> + };
> + };
> +
I don't think you need this. The GPU driver uses very little contig dma
memory. Most of the GPU memory is ordinary paged memory, so setting
aside a private memory region of this size is a waste of memory. Most
likely you can just allow the GPU driver to use the system global CMA
region.
Regards,
Lucas
> aliases {
> serial0 = &uart4;
> };
> @@ -53,6 +64,11 @@
> status = "okay";
> };
>
> +&gpu {
> + contiguous-area = <&gpu_reserved>;
> + status = "okay";
> +};
> +
> &i2c4 {
> pinctrl-names = "default";
> pinctrl-0 = <&i2c4_pins_a>;
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 22+ messages in thread* Re: [PATCH v1 2/3] ARM: dts: stm32: enable Vivante GPU support on stm32mp157c-ed1 board
2019-02-15 16:14 ` Lucas Stach
@ 2019-02-25 15:57 ` Pierre Yves MORDRET
-1 siblings, 0 replies; 22+ messages in thread
From: Pierre Yves MORDRET @ 2019-02-25 15:57 UTC (permalink / raw)
To: Lucas Stach, Rob Herring, Mark Rutland, Maxime Coquelin,
Alexandre Torgue
Cc: devicetree, Philippe Cornu, linux-kernel, linux-stm32, linux-arm-kernel
Hi again,
On 2/15/19 5:14 PM, Lucas Stach wrote:
> Am Freitag, den 15.02.2019, 16:58 +0100 schrieb Pierre-Yves MORDRET:
>> Enable Vivante GPU driver for stm32mp157c-ed1 board.
>>
>> Signed-off-by: Pierre-Yves MORDRET <pierre-yves.mordret@st.com>
>> ---
>> arch/arm/boot/dts/stm32mp157c-ed1.dts | 16 ++++++++++++++++
>> 1 file changed, 16 insertions(+)
>>
>> diff --git a/arch/arm/boot/dts/stm32mp157c-ed1.dts
>> b/arch/arm/boot/dts/stm32mp157c-ed1.dts
>> index 98ef7a0..792f402 100644
>> --- a/arch/arm/boot/dts/stm32mp157c-ed1.dts
>> +++ b/arch/arm/boot/dts/stm32mp157c-ed1.dts
>> @@ -20,6 +20,17 @@
>> reg = <0xC0000000 0x40000000>;
>> };
>>
>> + reserved-memory {
>> + #address-cells = <1>;
>> + #size-cells = <1>;
>> + ranges;
>> +
>> + gpu_reserved: gpu@f8000000 {
>> + reg = <0xf8000000 0x8000000>;
>> + no-map;
>> + };
>> + };
>> +
>
> I don't think you need this. The GPU driver uses very little contig dma
> memory. Most of the GPU memory is ordinary paged memory, so setting
> aside a private memory region of this size is a waste of memory. Most
> likely you can just allow the GPU driver to use the system global CMA
> region.
That's correct, but this memory area is also used for U-BOOT splash screen
rendering.
Moreover, even if we lack of proof as for today, it turns out performances are
better with this region as reduces MMU accesses (with Vivante driver)
Eventually using Android framework application is started more gently whatever
previous application and CMA status(fragmentation).
Provided this memory, CMA region is decreased accordingly.
Hope it clarifies.
Regards
>
> Regards,
> Lucas
>
>
>> aliases {
>> serial0 = &uart4;
>> };
>> @@ -53,6 +64,11 @@
>> status = "okay";
>> };
>>
>> +&gpu {
>> + contiguous-area = <&gpu_reserved>;
>> + status = "okay";
>> +};
>> +
>> &i2c4 {
>> pinctrl-names = "default";
>> pinctrl-0 = <&i2c4_pins_a>;
^ permalink raw reply [flat|nested] 22+ messages in thread* Re: [PATCH v1 2/3] ARM: dts: stm32: enable Vivante GPU support on stm32mp157c-ed1 board
@ 2019-02-25 15:57 ` Pierre Yves MORDRET
0 siblings, 0 replies; 22+ messages in thread
From: Pierre Yves MORDRET @ 2019-02-25 15:57 UTC (permalink / raw)
To: Lucas Stach, Rob Herring, Mark Rutland, Maxime Coquelin,
Alexandre Torgue
Cc: devicetree, linux-stm32, Philippe Cornu, linux-arm-kernel, linux-kernel
Hi again,
On 2/15/19 5:14 PM, Lucas Stach wrote:
> Am Freitag, den 15.02.2019, 16:58 +0100 schrieb Pierre-Yves MORDRET:
>> Enable Vivante GPU driver for stm32mp157c-ed1 board.
>>
>> Signed-off-by: Pierre-Yves MORDRET <pierre-yves.mordret@st.com>
>> ---
>> arch/arm/boot/dts/stm32mp157c-ed1.dts | 16 ++++++++++++++++
>> 1 file changed, 16 insertions(+)
>>
>> diff --git a/arch/arm/boot/dts/stm32mp157c-ed1.dts
>> b/arch/arm/boot/dts/stm32mp157c-ed1.dts
>> index 98ef7a0..792f402 100644
>> --- a/arch/arm/boot/dts/stm32mp157c-ed1.dts
>> +++ b/arch/arm/boot/dts/stm32mp157c-ed1.dts
>> @@ -20,6 +20,17 @@
>> reg = <0xC0000000 0x40000000>;
>> };
>>
>> + reserved-memory {
>> + #address-cells = <1>;
>> + #size-cells = <1>;
>> + ranges;
>> +
>> + gpu_reserved: gpu@f8000000 {
>> + reg = <0xf8000000 0x8000000>;
>> + no-map;
>> + };
>> + };
>> +
>
> I don't think you need this. The GPU driver uses very little contig dma
> memory. Most of the GPU memory is ordinary paged memory, so setting
> aside a private memory region of this size is a waste of memory. Most
> likely you can just allow the GPU driver to use the system global CMA
> region.
That's correct, but this memory area is also used for U-BOOT splash screen
rendering.
Moreover, even if we lack of proof as for today, it turns out performances are
better with this region as reduces MMU accesses (with Vivante driver)
Eventually using Android framework application is started more gently whatever
previous application and CMA status(fragmentation).
Provided this memory, CMA region is decreased accordingly.
Hope it clarifies.
Regards
>
> Regards,
> Lucas
>
>
>> aliases {
>> serial0 = &uart4;
>> };
>> @@ -53,6 +64,11 @@
>> status = "okay";
>> };
>>
>> +&gpu {
>> + contiguous-area = <&gpu_reserved>;
>> + status = "okay";
>> +};
>> +
>> &i2c4 {
>> pinctrl-names = "default";
>> pinctrl-0 = <&i2c4_pins_a>;
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 22+ messages in thread* Re: [PATCH v1 2/3] ARM: dts: stm32: enable Vivante GPU support on stm32mp157c-ed1 board
2019-02-25 15:57 ` Pierre Yves MORDRET
@ 2019-03-18 9:21 ` Pierre Yves MORDRET
-1 siblings, 0 replies; 22+ messages in thread
From: Pierre Yves MORDRET @ 2019-03-18 9:21 UTC (permalink / raw)
To: Lucas Stach, Rob Herring, Mark Rutland, Maxime Coquelin,
Alexandre Torgue
Cc: devicetree, linux-stm32, Philippe Cornu, linux-arm-kernel, linux-kernel
Hi Lucas,
I'm going to resend a new version on this DT. I have to move this region elsewhere.
However I didn't get your feedback about statement I did.
Would it be possible to have your feelings ?
Thanks :)
On 2/25/19 4:57 PM, Pierre Yves MORDRET wrote:
> Hi again,
>
> On 2/15/19 5:14 PM, Lucas Stach wrote:
>> Am Freitag, den 15.02.2019, 16:58 +0100 schrieb Pierre-Yves MORDRET:
>>> Enable Vivante GPU driver for stm32mp157c-ed1 board.
>>>
>>> Signed-off-by: Pierre-Yves MORDRET <pierre-yves.mordret@st.com>
>>> ---
>>> arch/arm/boot/dts/stm32mp157c-ed1.dts | 16 ++++++++++++++++
>>> 1 file changed, 16 insertions(+)
>>>
>>> diff --git a/arch/arm/boot/dts/stm32mp157c-ed1.dts
>>> b/arch/arm/boot/dts/stm32mp157c-ed1.dts
>>> index 98ef7a0..792f402 100644
>>> --- a/arch/arm/boot/dts/stm32mp157c-ed1.dts
>>> +++ b/arch/arm/boot/dts/stm32mp157c-ed1.dts
>>> @@ -20,6 +20,17 @@
>>> reg = <0xC0000000 0x40000000>;
>>> };
>>>
>>> + reserved-memory {
>>> + #address-cells = <1>;
>>> + #size-cells = <1>;
>>> + ranges;
>>> +
>>> + gpu_reserved: gpu@f8000000 {
>>> + reg = <0xf8000000 0x8000000>;
>>> + no-map;
>>> + };
>>> + };
>>> +
>>
>> I don't think you need this. The GPU driver uses very little contig dma
>> memory. Most of the GPU memory is ordinary paged memory, so setting
>> aside a private memory region of this size is a waste of memory. Most
>> likely you can just allow the GPU driver to use the system global CMA
>> region.
>
> That's correct, but this memory area is also used for U-BOOT splash screen
> rendering.
> Moreover, even if we lack of proof as for today, it turns out performances are
> better with this region as reduces MMU accesses (with Vivante driver)
> Eventually using Android framework application is started more gently whatever
> previous application and CMA status(fragmentation).
> Provided this memory, CMA region is decreased accordingly.
>
> Hope it clarifies.
>
> Regards
>>
>> Regards,
>> Lucas
>>
>>
>>> aliases {
>>> serial0 = &uart4;
>>> };
>>> @@ -53,6 +64,11 @@
>>> status = "okay";
>>> };
>>>
>>> +&gpu {
>>> + contiguous-area = <&gpu_reserved>;
>>> + status = "okay";
>>> +};
>>> +
>>> &i2c4 {
>>> pinctrl-names = "default";
>>> pinctrl-0 = <&i2c4_pins_a>;
>
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
>
^ permalink raw reply [flat|nested] 22+ messages in thread* Re: [PATCH v1 2/3] ARM: dts: stm32: enable Vivante GPU support on stm32mp157c-ed1 board
@ 2019-03-18 9:21 ` Pierre Yves MORDRET
0 siblings, 0 replies; 22+ messages in thread
From: Pierre Yves MORDRET @ 2019-03-18 9:21 UTC (permalink / raw)
To: Lucas Stach, Rob Herring, Mark Rutland, Maxime Coquelin,
Alexandre Torgue
Cc: devicetree, linux-kernel, linux-stm32, linux-arm-kernel, Philippe Cornu
Hi Lucas,
I'm going to resend a new version on this DT. I have to move this region elsewhere.
However I didn't get your feedback about statement I did.
Would it be possible to have your feelings ?
Thanks :)
On 2/25/19 4:57 PM, Pierre Yves MORDRET wrote:
> Hi again,
>
> On 2/15/19 5:14 PM, Lucas Stach wrote:
>> Am Freitag, den 15.02.2019, 16:58 +0100 schrieb Pierre-Yves MORDRET:
>>> Enable Vivante GPU driver for stm32mp157c-ed1 board.
>>>
>>> Signed-off-by: Pierre-Yves MORDRET <pierre-yves.mordret@st.com>
>>> ---
>>> arch/arm/boot/dts/stm32mp157c-ed1.dts | 16 ++++++++++++++++
>>> 1 file changed, 16 insertions(+)
>>>
>>> diff --git a/arch/arm/boot/dts/stm32mp157c-ed1.dts
>>> b/arch/arm/boot/dts/stm32mp157c-ed1.dts
>>> index 98ef7a0..792f402 100644
>>> --- a/arch/arm/boot/dts/stm32mp157c-ed1.dts
>>> +++ b/arch/arm/boot/dts/stm32mp157c-ed1.dts
>>> @@ -20,6 +20,17 @@
>>> reg = <0xC0000000 0x40000000>;
>>> };
>>>
>>> + reserved-memory {
>>> + #address-cells = <1>;
>>> + #size-cells = <1>;
>>> + ranges;
>>> +
>>> + gpu_reserved: gpu@f8000000 {
>>> + reg = <0xf8000000 0x8000000>;
>>> + no-map;
>>> + };
>>> + };
>>> +
>>
>> I don't think you need this. The GPU driver uses very little contig dma
>> memory. Most of the GPU memory is ordinary paged memory, so setting
>> aside a private memory region of this size is a waste of memory. Most
>> likely you can just allow the GPU driver to use the system global CMA
>> region.
>
> That's correct, but this memory area is also used for U-BOOT splash screen
> rendering.
> Moreover, even if we lack of proof as for today, it turns out performances are
> better with this region as reduces MMU accesses (with Vivante driver)
> Eventually using Android framework application is started more gently whatever
> previous application and CMA status(fragmentation).
> Provided this memory, CMA region is decreased accordingly.
>
> Hope it clarifies.
>
> Regards
>>
>> Regards,
>> Lucas
>>
>>
>>> aliases {
>>> serial0 = &uart4;
>>> };
>>> @@ -53,6 +64,11 @@
>>> status = "okay";
>>> };
>>>
>>> +&gpu {
>>> + contiguous-area = <&gpu_reserved>;
>>> + status = "okay";
>>> +};
>>> +
>>> &i2c4 {
>>> pinctrl-names = "default";
>>> pinctrl-0 = <&i2c4_pins_a>;
>
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
>
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 22+ messages in thread
* [PATCH v1 3/3] ARM: dts: stm32: enable Vivante GPU support on stm32mp157c-dk1 board
2019-02-15 15:58 ` Pierre-Yves MORDRET
(?)
@ 2019-02-15 15:58 ` Pierre-Yves MORDRET
-1 siblings, 0 replies; 22+ messages in thread
From: Pierre-Yves MORDRET @ 2019-02-15 15:58 UTC (permalink / raw)
To: Rob Herring, Mark Rutland, Maxime Coquelin, Alexandre Torgue
Cc: linux-stm32, devicetree, linux-arm-kernel, linux-kernel,
Pierre-Yves MORDRET, Philippe Cornu
Enable Vivante GPU driver for stm32mp157c-dk1 and dk2 boards.
Signed-off-by: Pierre-Yves MORDRET <pierre-yves.mordret@st.com>
---
arch/arm/boot/dts/stm32mp157a-dk1.dts | 16 ++++++++++++++++
1 file changed, 16 insertions(+)
diff --git a/arch/arm/boot/dts/stm32mp157a-dk1.dts b/arch/arm/boot/dts/stm32mp157a-dk1.dts
index 873a80e..fb2ee06 100644
--- a/arch/arm/boot/dts/stm32mp157a-dk1.dts
+++ b/arch/arm/boot/dts/stm32mp157a-dk1.dts
@@ -27,6 +27,17 @@
reg = <0xc0000000 0x20000000>;
};
+ reserved-memory {
+ #address-cells = <1>;
+ #size-cells = <1>;
+ ranges;
+
+ gpu_reserved: gpu@dc000000 {
+ reg = <0xdc000000 0x4000000>;
+ no-map;
+ };
+ };
+
led {
compatible = "gpio-leds";
blue {
@@ -57,6 +68,11 @@
};
};
+&gpu {
+ contiguous-area = <&gpu_reserved>;
+ status = "okay";
+};
+
&iwdg2 {
timeout-sec = <32>;
status = "okay";
--
2.7.4
^ permalink raw reply related [flat|nested] 22+ messages in thread* [PATCH v1 3/3] ARM: dts: stm32: enable Vivante GPU support on stm32mp157c-dk1 board
@ 2019-02-15 15:58 ` Pierre-Yves MORDRET
0 siblings, 0 replies; 22+ messages in thread
From: Pierre-Yves MORDRET @ 2019-02-15 15:58 UTC (permalink / raw)
To: Rob Herring, Mark Rutland, Maxime Coquelin, Alexandre Torgue
Cc: devicetree, Philippe Cornu, Pierre-Yves MORDRET, linux-kernel,
linux-stm32, linux-arm-kernel
Enable Vivante GPU driver for stm32mp157c-dk1 and dk2 boards.
Signed-off-by: Pierre-Yves MORDRET <pierre-yves.mordret@st.com>
---
arch/arm/boot/dts/stm32mp157a-dk1.dts | 16 ++++++++++++++++
1 file changed, 16 insertions(+)
diff --git a/arch/arm/boot/dts/stm32mp157a-dk1.dts b/arch/arm/boot/dts/stm32mp157a-dk1.dts
index 873a80e..fb2ee06 100644
--- a/arch/arm/boot/dts/stm32mp157a-dk1.dts
+++ b/arch/arm/boot/dts/stm32mp157a-dk1.dts
@@ -27,6 +27,17 @@
reg = <0xc0000000 0x20000000>;
};
+ reserved-memory {
+ #address-cells = <1>;
+ #size-cells = <1>;
+ ranges;
+
+ gpu_reserved: gpu@dc000000 {
+ reg = <0xdc000000 0x4000000>;
+ no-map;
+ };
+ };
+
led {
compatible = "gpio-leds";
blue {
@@ -57,6 +68,11 @@
};
};
+&gpu {
+ contiguous-area = <&gpu_reserved>;
+ status = "okay";
+};
+
&iwdg2 {
timeout-sec = <32>;
status = "okay";
--
2.7.4
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 22+ messages in thread* [PATCH v1 3/3] ARM: dts: stm32: enable Vivante GPU support on stm32mp157c-dk1 board
@ 2019-02-15 15:58 ` Pierre-Yves MORDRET
0 siblings, 0 replies; 22+ messages in thread
From: Pierre-Yves MORDRET @ 2019-02-15 15:58 UTC (permalink / raw)
To: Rob Herring, Mark Rutland, Maxime Coquelin, Alexandre Torgue
Cc: linux-stm32, devicetree, linux-arm-kernel, linux-kernel,
Pierre-Yves MORDRET, Philippe Cornu
Enable Vivante GPU driver for stm32mp157c-dk1 and dk2 boards.
Signed-off-by: Pierre-Yves MORDRET <pierre-yves.mordret@st.com>
---
arch/arm/boot/dts/stm32mp157a-dk1.dts | 16 ++++++++++++++++
1 file changed, 16 insertions(+)
diff --git a/arch/arm/boot/dts/stm32mp157a-dk1.dts b/arch/arm/boot/dts/stm32mp157a-dk1.dts
index 873a80e..fb2ee06 100644
--- a/arch/arm/boot/dts/stm32mp157a-dk1.dts
+++ b/arch/arm/boot/dts/stm32mp157a-dk1.dts
@@ -27,6 +27,17 @@
reg = <0xc0000000 0x20000000>;
};
+ reserved-memory {
+ #address-cells = <1>;
+ #size-cells = <1>;
+ ranges;
+
+ gpu_reserved: gpu@dc000000 {
+ reg = <0xdc000000 0x4000000>;
+ no-map;
+ };
+ };
+
led {
compatible = "gpio-leds";
blue {
@@ -57,6 +68,11 @@
};
};
+&gpu {
+ contiguous-area = <&gpu_reserved>;
+ status = "okay";
+};
+
&iwdg2 {
timeout-sec = <32>;
status = "okay";
--
2.7.4
^ permalink raw reply related [flat|nested] 22+ messages in thread