Messages in this thread | | | Subject | Re: [PATCH v8 04/11] ARM: dts: r7s72100: Add Capture Engine Unit (CEU) | From | Sergei Shtylyov <> | Date | Wed, 31 Jan 2018 11:07:44 +0300 |
| |
Hello!
On 1/30/2018 12:58 PM, Jacopo Mondi wrote:
> Add Capture Engine Unit (CEU) node to device tree. > > Signed-off-by: Jacopo Mondi <jacopo+renesas@jmondi.org> > Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be> > Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> > Acked-by: Hans Verkuil <hans.verkuil@cisco.com> > --- > 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..5fe62f9 100644 > --- a/arch/arm/boot/dts/r7s72100.dtsi > +++ b/arch/arm/boot/dts/r7s72100.dtsi [...] > @@ -667,4 +667,13 @@ > power-domains = <&cpg_clocks>; > status = "disabled"; > }; > + > + ceu: ceu@e8210000 {
The DT spec dictates the generic node names should be used. For the R-Car VIN we use "video@...", hence I suggest that use the same here.
> + reg = <0xe8210000 0x3000>; > + compatible = "renesas,r7s72100-ceu"; > + interrupts = <GIC_SPI 332 IRQ_TYPE_LEVEL_HIGH>; > + clocks = <&mstp6_clks R7S72100_CLK_CEU>; > + power-domains = <&cpg_clocks>; > + status = "disabled"; > + }; > };
MBR, Sergei
| |