Re: [PATCH v2 2/2] ARM: dts: r8a7740: Add CEU0
On Wed, May 16, 2018 at 09:40:09AM +0200, Geert Uytterhoeven wrote: > Hi Jacopo, > > On Thu, Apr 26, 2018 at 8:24 PM, Jacopo Mondi> wrote: > > Describe CEU0 peripheral for Renesas R-Mobile A1 R8A7740 Soc. > > > > Reported-by: Geert Uytterhoeven > > Signed-off-by: Jacopo Mondi > > Thanks for your patch! > > Reviewed-by: Geert Uytterhoeven > > Minor question below. > > > --- a/arch/arm/boot/dts/r8a7740.dtsi > > +++ b/arch/arm/boot/dts/r8a7740.dtsi > > @@ -67,6 +67,16 @@ > > power-domains = <_d4>; > > }; > > > > + ceu0: ceu@fe91 { > > + reg = <0xfe91 0x3000>; > > + compatible = "renesas,r8a7740-ceu"; > > + interrupts = ; > > + clocks = <_clks R8A7740_CLK_CEU20>; > > + clock-names = "ceu20"; > > Why the "clock-names" property? It's not mentioned in the DT bindings, and > may cause issues if the bindings are ever amended. I have dropped that property for now. > > > + power-domains = <_a4r>; > > + status = "disabled"; > > + }; > > + > > Gr{oetje,eeting}s, > > Geert > > -- > Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- > ge...@linux-m68k.org > > In personal conversations with technical people, I call myself a hacker. But > when I'm talking to journalists I just say "programmer" or something like > that. > -- Linus Torvalds >
Re: [PATCH v2 2/2] ARM: dts: r8a7740: Add CEU0
On Wed, May 16, 2018 at 09:40:09AM +0200, Geert Uytterhoeven wrote: > Hi Jacopo, > > On Thu, Apr 26, 2018 at 8:24 PM, Jacopo Mondi > wrote: > > Describe CEU0 peripheral for Renesas R-Mobile A1 R8A7740 Soc. > > > > Reported-by: Geert Uytterhoeven > > Signed-off-by: Jacopo Mondi > > Thanks for your patch! > > Reviewed-by: Geert Uytterhoeven > > Minor question below. > > > --- a/arch/arm/boot/dts/r8a7740.dtsi > > +++ b/arch/arm/boot/dts/r8a7740.dtsi > > @@ -67,6 +67,16 @@ > > power-domains = <_d4>; > > }; > > > > + ceu0: ceu@fe91 { > > + reg = <0xfe91 0x3000>; > > + compatible = "renesas,r8a7740-ceu"; > > + interrupts = ; > > + clocks = <_clks R8A7740_CLK_CEU20>; > > + clock-names = "ceu20"; > > Why the "clock-names" property? It's not mentioned in the DT bindings, and > may cause issues if the bindings are ever amended. I have dropped that property for now. > > > + power-domains = <_a4r>; > > + status = "disabled"; > > + }; > > + > > Gr{oetje,eeting}s, > > Geert > > -- > Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- > ge...@linux-m68k.org > > In personal conversations with technical people, I call myself a hacker. But > when I'm talking to journalists I just say "programmer" or something like > that. > -- Linus Torvalds >
Re: [PATCH v2 2/2] ARM: dts: r8a7740: Add CEU0
Hi Jacopo, On Thu, Apr 26, 2018 at 8:24 PM, Jacopo Mondiwrote: > Describe CEU0 peripheral for Renesas R-Mobile A1 R8A7740 Soc. > > Reported-by: Geert Uytterhoeven > Signed-off-by: Jacopo Mondi Thanks for your patch! Reviewed-by: Geert Uytterhoeven Minor question below. > --- a/arch/arm/boot/dts/r8a7740.dtsi > +++ b/arch/arm/boot/dts/r8a7740.dtsi > @@ -67,6 +67,16 @@ > power-domains = <_d4>; > }; > > + ceu0: ceu@fe91 { > + reg = <0xfe91 0x3000>; > + compatible = "renesas,r8a7740-ceu"; > + interrupts = ; > + clocks = <_clks R8A7740_CLK_CEU20>; > + clock-names = "ceu20"; Why the "clock-names" property? It's not mentioned in the DT bindings, and may cause issues if the bindings are ever amended. > + power-domains = <_a4r>; > + status = "disabled"; > + }; > + Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds
Re: [PATCH v2 2/2] ARM: dts: r8a7740: Add CEU0
Hi Jacopo, On Thu, Apr 26, 2018 at 8:24 PM, Jacopo Mondi wrote: > Describe CEU0 peripheral for Renesas R-Mobile A1 R8A7740 Soc. > > Reported-by: Geert Uytterhoeven > Signed-off-by: Jacopo Mondi Thanks for your patch! Reviewed-by: Geert Uytterhoeven Minor question below. > --- a/arch/arm/boot/dts/r8a7740.dtsi > +++ b/arch/arm/boot/dts/r8a7740.dtsi > @@ -67,6 +67,16 @@ > power-domains = <_d4>; > }; > > + ceu0: ceu@fe91 { > + reg = <0xfe91 0x3000>; > + compatible = "renesas,r8a7740-ceu"; > + interrupts = ; > + clocks = <_clks R8A7740_CLK_CEU20>; > + clock-names = "ceu20"; Why the "clock-names" property? It's not mentioned in the DT bindings, and may cause issues if the bindings are ever amended. > + power-domains = <_a4r>; > + status = "disabled"; > + }; > + Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds
Re: [PATCH v2 2/2] ARM: dts: r8a7740: Add CEU0
On Thu, Apr 26, 2018 at 08:24:43PM +0200, Jacopo Mondi wrote: > Describe CEU0 peripheral for Renesas R-Mobile A1 R8A7740 Soc. > > Reported-by: Geert Uytterhoeven> Signed-off-by: Jacopo Mondi Thanks, applied.
Re: [PATCH v2 2/2] ARM: dts: r8a7740: Add CEU0
On Thu, Apr 26, 2018 at 08:24:43PM +0200, Jacopo Mondi wrote: > Describe CEU0 peripheral for Renesas R-Mobile A1 R8A7740 Soc. > > Reported-by: Geert Uytterhoeven > Signed-off-by: Jacopo Mondi Thanks, applied.