linux-clk.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] clk: renesas: r8a7796: add PWM clock
@ 2017-04-19 17:46 Yoshihiro Kaneko
  2017-04-20 17:53 ` Geert Uytterhoeven
  0 siblings, 1 reply; 6+ messages in thread
From: Yoshihiro Kaneko @ 2017-04-19 17:46 UTC (permalink / raw)
  To: linux-clk
  Cc: Michael Turquette, Stephen Boyd, Geert Uytterhoeven, Simon Horman,
	Magnus Damm, linux-renesas-soc

From: Ryo Kodama <ryo.kodama.vz@renesas.com>

This patch adds PWM clock for PWM.

Signed-off-by: Ryo Kodama <ryo.kodama.vz@renesas.com>
Signed-off-by: Takeshi Kihara <takeshi.kihara.df@renesas.com>
Signed-off-by: Yoshihiro Kaneko <ykaneko0929@gmail.com>
---
This patch is based on the clk-next branch of linux-clk tree.

 drivers/clk/renesas/r8a7796-cpg-mssr.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/clk/renesas/r8a7796-cpg-mssr.c b/drivers/clk/renesas/r8a7796-cpg-mssr.c
index c5523aa..64714ca 100644
--- a/drivers/clk/renesas/r8a7796-cpg-mssr.c
+++ b/drivers/clk/renesas/r8a7796-cpg-mssr.c
@@ -156,6 +156,7 @@ enum clk_ids {
 	DEF_MOD("hscif1",		 519,	R8A7796_CLK_S3D1),
 	DEF_MOD("hscif0",		 520,	R8A7796_CLK_S3D1),
 	DEF_MOD("thermal",		 522,	R8A7796_CLK_CP),
+	DEF_MOD("pwm",			 523,	R8A7796_CLK_S3D4),
 	DEF_MOD("fcpvd2",		 601,	R8A7796_CLK_S0D2),
 	DEF_MOD("fcpvd1",		 602,	R8A7796_CLK_S0D2),
 	DEF_MOD("fcpvd0",		 603,	R8A7796_CLK_S0D2),
-- 
1.9.1

^ permalink raw reply related	[flat|nested] 6+ messages in thread

* Re: [PATCH] clk: renesas: r8a7796: add PWM clock
  2017-04-19 17:46 [PATCH] clk: renesas: r8a7796: add " Yoshihiro Kaneko
@ 2017-04-20 17:53 ` Geert Uytterhoeven
  2017-04-24  8:09   ` Geert Uytterhoeven
  0 siblings, 1 reply; 6+ messages in thread
From: Geert Uytterhoeven @ 2017-04-20 17:53 UTC (permalink / raw)
  To: Yoshihiro Kaneko
  Cc: linux-clk, Michael Turquette, Stephen Boyd, Geert Uytterhoeven,
	Simon Horman, Magnus Damm, Linux-Renesas

On Wed, Apr 19, 2017 at 7:46 PM, Yoshihiro Kaneko <ykaneko0929@gmail.com> wrote:
> From: Ryo Kodama <ryo.kodama.vz@renesas.com>
>
> This patch adds PWM clock for PWM.
>
> Signed-off-by: Ryo Kodama <ryo.kodama.vz@renesas.com>
> Signed-off-by: Takeshi Kihara <takeshi.kihara.df@renesas.com>
> Signed-off-by: Yoshihiro Kaneko <ykaneko0929@gmail.com>
> ---
> This patch is based on the clk-next branch of linux-clk tree.
>
>  drivers/clk/renesas/r8a7796-cpg-mssr.c | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/drivers/clk/renesas/r8a7796-cpg-mssr.c b/drivers/clk/renesas/r8a7796-cpg-mssr.c
> index c5523aa..64714ca 100644
> --- a/drivers/clk/renesas/r8a7796-cpg-mssr.c
> +++ b/drivers/clk/renesas/r8a7796-cpg-mssr.c
> @@ -156,6 +156,7 @@ enum clk_ids {
>         DEF_MOD("hscif1",                519,   R8A7796_CLK_S3D1),
>         DEF_MOD("hscif0",                520,   R8A7796_CLK_S3D1),
>         DEF_MOD("thermal",               522,   R8A7796_CLK_CP),
> +       DEF_MOD("pwm",                   523,   R8A7796_CLK_S3D4),

Repeating myself, cfr. my response to Ulrich:

     My copy of the R-Car Gen3 user manual rev. 0.53 says S0D12.

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@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

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] clk: renesas: r8a7796: add PWM clock
  2017-04-20 17:53 ` Geert Uytterhoeven
@ 2017-04-24  8:09   ` Geert Uytterhoeven
  0 siblings, 0 replies; 6+ messages in thread
From: Geert Uytterhoeven @ 2017-04-24  8:09 UTC (permalink / raw)
  To: Yoshihiro Kaneko
  Cc: linux-clk, Michael Turquette, Stephen Boyd, Geert Uytterhoeven,
	Simon Horman, Magnus Damm, Linux-Renesas

On Thu, Apr 20, 2017 at 7:53 PM, Geert Uytterhoeven
<geert@linux-m68k.org> wrote:
> On Wed, Apr 19, 2017 at 7:46 PM, Yoshihiro Kaneko <ykaneko0929@gmail.com> wrote:
>> From: Ryo Kodama <ryo.kodama.vz@renesas.com>
>>
>> This patch adds PWM clock for PWM.
>>
>> Signed-off-by: Ryo Kodama <ryo.kodama.vz@renesas.com>
>> Signed-off-by: Takeshi Kihara <takeshi.kihara.df@renesas.com>
>> Signed-off-by: Yoshihiro Kaneko <ykaneko0929@gmail.com>
>> ---
>> This patch is based on the clk-next branch of linux-clk tree.
>>
>>  drivers/clk/renesas/r8a7796-cpg-mssr.c | 1 +
>>  1 file changed, 1 insertion(+)
>>
>> diff --git a/drivers/clk/renesas/r8a7796-cpg-mssr.c b/drivers/clk/renesas/r8a7796-cpg-mssr.c
>> index c5523aa..64714ca 100644
>> --- a/drivers/clk/renesas/r8a7796-cpg-mssr.c
>> +++ b/drivers/clk/renesas/r8a7796-cpg-mssr.c
>> @@ -156,6 +156,7 @@ enum clk_ids {
>>         DEF_MOD("hscif1",                519,   R8A7796_CLK_S3D1),
>>         DEF_MOD("hscif0",                520,   R8A7796_CLK_S3D1),
>>         DEF_MOD("thermal",               522,   R8A7796_CLK_CP),
>> +       DEF_MOD("pwm",                   523,   R8A7796_CLK_S3D4),
>
> Repeating myself, cfr. my response to Ulrich:
>
>      My copy of the R-Car Gen3 user manual rev. 0.53 says S0D12.

Fixing that up while applying.

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@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

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [PATCH] clk: renesas: r8a7796: Add PWM clock
@ 2017-06-26 17:26 Laurent Pinchart
  2017-06-26 17:38 ` Geert Uytterhoeven
  0 siblings, 1 reply; 6+ messages in thread
From: Laurent Pinchart @ 2017-06-26 17:26 UTC (permalink / raw)
  To: linux-renesas-soc; +Cc: linux-clk

Signed-off-by: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
---
 drivers/clk/renesas/r8a7796-cpg-mssr.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/clk/renesas/r8a7796-cpg-mssr.c b/drivers/clk/renesas/r8a7796-cpg-mssr.c
index 1d8c5c2b6174..30583cdc1350 100644
--- a/drivers/clk/renesas/r8a7796-cpg-mssr.c
+++ b/drivers/clk/renesas/r8a7796-cpg-mssr.c
@@ -152,6 +152,7 @@ static const struct mssr_mod_clk r8a7796_mod_clks[] __initconst = {
 	DEF_MOD("hscif1",		 519,	R8A7796_CLK_S3D1),
 	DEF_MOD("hscif0",		 520,	R8A7796_CLK_S3D1),
 	DEF_MOD("thermal",		 522,	R8A7796_CLK_CP),
+	DEF_MOD("pwm",			 523,	R8A7796_CLK_S0D12),
 	DEF_MOD("fcpvd2",		 601,	R8A7796_CLK_S0D2),
 	DEF_MOD("fcpvd1",		 602,	R8A7796_CLK_S0D2),
 	DEF_MOD("fcpvd0",		 603,	R8A7796_CLK_S0D2),
-- 
Regards,

Laurent Pinchart


^ permalink raw reply related	[flat|nested] 6+ messages in thread

* Re: [PATCH] clk: renesas: r8a7796: Add PWM clock
  2017-06-26 17:26 [PATCH] clk: renesas: r8a7796: Add PWM clock Laurent Pinchart
@ 2017-06-26 17:38 ` Geert Uytterhoeven
  2017-06-26 18:26   ` Laurent Pinchart
  0 siblings, 1 reply; 6+ messages in thread
From: Geert Uytterhoeven @ 2017-06-26 17:38 UTC (permalink / raw)
  To: Laurent Pinchart; +Cc: Linux-Renesas, linux-clk

On Mon, Jun 26, 2017 at 7:26 PM, Laurent Pinchart
<laurent.pinchart+renesas@ideasonboard.com> wrote:
> --- a/drivers/clk/renesas/r8a7796-cpg-mssr.c
> +++ b/drivers/clk/renesas/r8a7796-cpg-mssr.c
> @@ -152,6 +152,7 @@ static const struct mssr_mod_clk r8a7796_mod_clks[] __initconst = {
>         DEF_MOD("hscif1",                519,   R8A7796_CLK_S3D1),
>         DEF_MOD("hscif0",                520,   R8A7796_CLK_S3D1),
>         DEF_MOD("thermal",               522,   R8A7796_CLK_CP),
> +       DEF_MOD("pwm",                   523,   R8A7796_CLK_S0D12),

Cfr. commit a0b381fafffaf07c ("clk: renesas: r8a7796: Add PWM clock")
in clk-next.

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@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

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] clk: renesas: r8a7796: Add PWM clock
  2017-06-26 17:38 ` Geert Uytterhoeven
@ 2017-06-26 18:26   ` Laurent Pinchart
  0 siblings, 0 replies; 6+ messages in thread
From: Laurent Pinchart @ 2017-06-26 18:26 UTC (permalink / raw)
  To: Geert Uytterhoeven; +Cc: Laurent Pinchart, Linux-Renesas, linux-clk

Hi Geert,

On Monday 26 Jun 2017 19:38:58 Geert Uytterhoeven wrote:
> On Mon, Jun 26, 2017 at 7:26 PM, Laurent Pinchart wrote:
> > --- a/drivers/clk/renesas/r8a7796-cpg-mssr.c
> > +++ b/drivers/clk/renesas/r8a7796-cpg-mssr.c
> > @@ -152,6 +152,7 @@ static const struct mssr_mod_clk r8a7796_mod_clks[]
> > __initconst = {> 
> >         DEF_MOD("hscif1",                519,   R8A7796_CLK_S3D1),
> >         DEF_MOD("hscif0",                520,   R8A7796_CLK_S3D1),
> >         DEF_MOD("thermal",               522,   R8A7796_CLK_CP),
> > 
> > +       DEF_MOD("pwm",                   523,   R8A7796_CLK_S0D12),
> 
> Cfr. commit a0b381fafffaf07c ("clk: renesas: r8a7796: Add PWM clock")
> in clk-next.

Same mistake twice... I wonder why I failed to find that one :-/ Sorry about 
the noise.

-- 
Regards,

Laurent Pinchart


^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2017-06-26 18:26 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-06-26 17:26 [PATCH] clk: renesas: r8a7796: Add PWM clock Laurent Pinchart
2017-06-26 17:38 ` Geert Uytterhoeven
2017-06-26 18:26   ` Laurent Pinchart
  -- strict thread matches above, loose matches on Subject: below --
2017-04-19 17:46 [PATCH] clk: renesas: r8a7796: add " Yoshihiro Kaneko
2017-04-20 17:53 ` Geert Uytterhoeven
2017-04-24  8:09   ` Geert Uytterhoeven

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).