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+rene...@jmondi.org>
> ---
>  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.pinch...@ideasonboard.com>

> +             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

Reply via email to