On Wed, Jun 21, 2017 at 03:17:30PM +0300, Laurent Pinchart wrote:
> Add the HDMI encoder to the R8A7796 DT in disabled state.
> 
> Signed-off-by: Laurent Pinchart <[email protected]>
> ---
>  arch/arm64/boot/dts/renesas/r8a7796.dtsi | 25 +++++++++++++++++++++++++
>  1 file changed, 25 insertions(+)
> 
> Changes since v2:
> 
> - Rename node to hdmi@fead0000
> 
> diff --git a/arch/arm64/boot/dts/renesas/r8a7796.dtsi 
> b/arch/arm64/boot/dts/renesas/r8a7796.dtsi
> index ad9cd1c3199f..8c1a770c41cd 100644
> --- a/arch/arm64/boot/dts/renesas/r8a7796.dtsi
> +++ b/arch/arm64/boot/dts/renesas/r8a7796.dtsi
> @@ -1532,6 +1532,31 @@
>                       power-domains = <&sysc R8A7796_PD_ALWAYS_ON>;
>               };
>  
> +             hdmi0: hdmi@fead0000 {
> +                     compatible = "renesas,r8a7796-hdmi", 
> "renesas,rcar-gen3-hdmi";
> +                     reg = <0 0xfead0000 0 0x10000>;
> +                     interrupts = <GIC_SPI 389 IRQ_TYPE_LEVEL_HIGH>;
> +                     clocks = <&cpg CPG_MOD 729>, <&cpg CPG_CORE 
> R8A7796_CLK_HDMI>;
> +                     clock-names = "iahb", "isfr";
> +                     power-domains = <&sysc R8A7796_PD_ALWAYS_ON>;
> +                     resets = <&cpg 729>;
> +                     status = "disabled";
> +
> +                     ports {
> +                             #address-cells = <1>;
> +                             #size-cells = <0>;
> +                             port@0 {
> +                                     reg = <0>;
> +                                     dw_hdmi0_in: endpoint {
> +                                             remote-endpoint = 
> <&du_out_hdmi0>;

du_out_hdmi0 is used here but does not exist until the following patch is
applied. What to do?

> +                                     };
> +                             };
> +                             port@1 {
> +                                     reg = <1>;
> +                             };
> +                     };
> +             };
> +
>               du: display@feb00000 {
>                       /* placeholder */
>  
> -- 
> Regards,
> 
> Laurent Pinchart
> 

Reply via email to