Re: [PATCH/RFC 6/9] ARM64: renesas: r8a7795: Add FCPV nodes

[Laurent Pinchart <laurent.pinchart@ideasonboard.com>]

Hi Geert,

Thank you for the review.

On Monday 15 February 2016 10:45:39 Geert Uytterhoeven wrote:
> On Fri, Feb 12, 2016 at 3:00 AM, Laurent Pinchart wrote:
> > The FCPs handle the interface between various IP cores and memory. Add
> > the instances related to the VSP2s.
> > 
> > Signed-off-by: Laurent Pinchart
> > <laurent.pinchart+renesas@ideasonboard.com>
> > ---
> >  arch/arm64/boot/dts/renesas/r8a7795.dtsi | 63 ++++++++++++++++++++++++++
> >  1 file changed, 63 insertions(+)
> > 
> > diff --git a/arch/arm64/boot/dts/renesas/r8a7795.dtsi
> > b/arch/arm64/boot/dts/renesas/r8a7795.dtsi index
> > b5e46e4ff72a..f62d6fa28acc 100644
> > --- a/arch/arm64/boot/dts/renesas/r8a7795.dtsi
> > +++ b/arch/arm64/boot/dts/renesas/r8a7795.dtsi
> > @@ -960,5 +960,68 @@
> >                         #dma-cells = <1>;
> >                         dma-channels = <2>;
> >                 };
> > 
> > +
> > +               fcpvb1: fcp@fe92f000 {
> > +                       compatible = "renesas,fcpv";
> > +                       reg = <0 0xfe92f000 0 0x200>;
> > +                       clocks = <&cpg CPG_MOD 606>;
> > +                       power-domains = <&cpg>;
> > +               };
> 
> The FCP_V modules are located in the A3VP Power Area. But adding this
> information to DT depends on the SYSC PM Domain driver.
> 
> I'll try to post my WIP PM Domain patchset for R-Car ASAP...

As soon as you add the A3VP power domain to DT I'll make sure to make use of 
it :-)

-- 
Regards,

Laurent Pinchart