Hi Jacopo, Thank you for the patch. On Thursday, 28 December 2017 16:01:16 EET Jacopo Mondi wrote: > Add Capture Engine Unit (CEU) node to device tree. > > Signed-off-by: Jacopo Mondi <jacopo+renesas@xxxxxxxxxx> > --- > arch/arm/boot/dts/r7s72100.dtsi | 15 ++++++++++++--- > 1 file changed, 12 insertions(+), 3 deletions(-) > > diff --git a/arch/arm/boot/dts/r7s72100.dtsi > b/arch/arm/boot/dts/r7s72100.dtsi index ab9645a..b09b032 100644 > --- a/arch/arm/boot/dts/r7s72100.dtsi > +++ b/arch/arm/boot/dts/r7s72100.dtsi > @@ -135,9 +135,9 @@ > #clock-cells = <1>; > compatible = "renesas,r7s72100-mstp-clocks", "renesas,cpg-mstp- clocks"; > reg = <0xfcfe042c 4>; > - clocks = <&p0_clk>; > - clock-indices = <R7S72100_CLK_RTC>; > - clock-output-names = "rtc"; > + clocks = <&b_clk>, <&p0_clk>; > + clock-indices = <R7S72100_CLK_CEU R7S72100_CLK_RTC>; > + clock-output-names = "ceu", "rtc"; > }; > > mstp7_clks: mstp7_clks@fcfe0430 { > @@ -667,4 +667,13 @@ > power-domains = <&cpg_clocks>; > status = "disabled"; > }; > + > + ceu: ceu@e8210000 { > + reg = <0xe8210000 0x209c>; With the reg issue pointed out by Geert fixed, Reviewed-by: Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx> > + compatible = "renesas,r7s72100-ceu"; > + interrupts = <GIC_SPI 332 IRQ_TYPE_LEVEL_HIGH>; > + clocks = <&mstp6_clks R7S72100_CLK_CEU>; > + power-domains = <&cpg_clocks>; > + status = "disabled"; > + }; > }; -- Regards, Laurent Pinchart