Hi Simon,
On Monday 10 Jul 2017 11:07:50 Simon Horman wrote:
> 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?
Good point. I'll resubmit patches 4/7 and 5/7 with a fix.
> > + };
> > + };
> > + port@1 {
> > + reg = <1>;
> > + };
> > + };
> > + };
> > +
> > du: display@feb00000 {
> > /* placeholder */
--
Regards,
Laurent Pinchart
