Re: [PATCH] ARM: dts: r8a7794: link DU to VSPD

[Laurent Pinchart]

Hi Sergei,

On Thursday 29 Dec 2016 00:35:07 Sergei Shtylyov wrote:
> Add the "vsps" property to the DU device node in order to link this node to
> the (single) VSPD node.
> 
> Signed-off-by: Sergei Shtylyov 

I'd like to first address the issues I've pointed out on the driver side of 
this change.

> ---
> This patch is against the 'renesas-devel-20161220-v4.9' of Simon Horman's
> 'renesas.git' repo.  It's  only meaningful if the "Enable R8A7794 DU VSPD
> compositor" DU driver patches are applied...
> 
>  arch/arm/boot/dts/r8a7794.dtsi |3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> Index: renesas/arch/arm/boot/dts/r8a7794.dtsi
> ===
> --- renesas.orig/arch/arm/boot/dts/r8a7794.dtsi
> +++ renesas/arch/arm/boot/dts/r8a7794.dtsi
> @@ -908,7 +908,7 @@
>   power-domains = < R8A7794_PD_ALWAYS_ON>;
>   };
> 
> - vsp1@fe93 {
> + vspd0: vsp1@fe93 {
>   compatible = "renesas,vsp1";
>   reg = <0 0xfe93 0 0x8000>;
>   interrupts = ;
> @@ -925,6 +925,7 @@
>   clocks = <_clks R8A7794_CLK_DU0>,
><_clks R8A7794_CLK_DU0>;
>   clock-names = "du.0", "du.1";
> + vsps = <>;
>   status = "disabled";
> 
>   ports {

-- 
Regards,

Laurent Pinchart

[PATCH] ARM: dts: r8a7794: link DU to VSPD

[Sergei Shtylyov]

Add the "vsps" property to the DU device node in order to link this node to
the (single) VSPD node.

Signed-off-by: Sergei Shtylyov 

---
This patch is against the 'renesas-devel-20161220-v4.9' of Simon Horman's
'renesas.git' repo.  It's  only meaningful if the "Enable R8A7794 DU VSPD
compositor" DU driver patches are applied...

 arch/arm/boot/dts/r8a7794.dtsi |3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

Index: renesas/arch/arm/boot/dts/r8a7794.dtsi
===
--- renesas.orig/arch/arm/boot/dts/r8a7794.dtsi
+++ renesas/arch/arm/boot/dts/r8a7794.dtsi
@@ -908,7 +908,7 @@
power-domains = < R8A7794_PD_ALWAYS_ON>;
};
 
-   vsp1@fe93 {
+   vspd0: vsp1@fe93 {
compatible = "renesas,vsp1";
reg = <0 0xfe93 0 0x8000>;
interrupts = ;
@@ -925,6 +925,7 @@
clocks = <_clks R8A7794_CLK_DU0>,
 <_clks R8A7794_CLK_DU0>;
clock-names = "du.0", "du.1";
+   vsps = <>;
status = "disabled";
 
ports {