diff mbox series

[10/10] cpufreq: dt: Add support for r8a7744

Message ID 1536660771-32278-11-git-send-email-biju.das@bp.renesas.com (mailing list archive)
State Accepted
Delegated to: Geert Uytterhoeven
Headers show
Series Add RZ/G1N support. | expand

Commit Message

Biju Das Sept. 11, 2018, 10:12 a.m. UTC 
Add the compatible strings for supporting the generic cpufreq driver on
the Renesas RZ/G1N (R8A7744) SoC.

Signed-off-by: Biju Das <biju.das@bp.renesas.com>
Reviewed-by: Fabrizio Castro <fabrizio.castro@bp.renesas.com>
---
 drivers/cpufreq/cpufreq-dt-platdev.c | 1 +
 1 file changed, 1 insertion(+)

Comments

Rafael J. Wysocki Sept. 11, 2018, 10:59 a.m. UTC | #1 
On Tue, Sep 11, 2018 at 12:20 PM Biju Das <biju.das@bp.renesas.com> wrote:
>
> Add the compatible strings for supporting the generic cpufreq driver on
> the Renesas RZ/G1N (R8A7744) SoC.
>
> Signed-off-by: Biju Das <biju.das@bp.renesas.com>
> Reviewed-by: Fabrizio Castro <fabrizio.castro@bp.renesas.com>
> ---
>  drivers/cpufreq/cpufreq-dt-platdev.c | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/drivers/cpufreq/cpufreq-dt-platdev.c b/drivers/cpufreq/cpufreq-dt-platdev.c
> index fe14c57..805f8a0 100644
> --- a/drivers/cpufreq/cpufreq-dt-platdev.c
> +++ b/drivers/cpufreq/cpufreq-dt-platdev.c
> @@ -58,6 +58,7 @@ static const struct of_device_id whitelist[] __initconst = {
>         { .compatible = "renesas,r8a73a4", },
>         { .compatible = "renesas,r8a7740", },
>         { .compatible = "renesas,r8a7743", },
> +       { .compatible = "renesas,r8a7744", },
>         { .compatible = "renesas,r8a7745", },
>         { .compatible = "renesas,r8a7778", },
>         { .compatible = "renesas,r8a7779", },
> --

Since this is the last patch in a series of 10, do you want me to
apply it or should it go in along with the rest?

Thanks,
Rafael
Biju Das Sept. 11, 2018, 11:10 a.m. UTC | #2 
Hi Rafael,

Thanks for the feedback.

> -----Original Message-----
> From: Rafael J. Wysocki <rafael@kernel.org>
> Sent: 11 September 2018 11:59
> To: Biju Das <biju.das@bp.renesas.com>
> Cc: Rafael J. Wysocki <rjw@rjwysocki.net>; Viresh Kumar
> <viresh.kumar@linaro.org>; Linux PM <linux-pm@vger.kernel.org>; Simon
> Horman <horms@verge.net.au>; Geert Uytterhoeven
> <geert+renesas@glider.be>; Chris Paterson
> <Chris.Paterson2@renesas.com>; Fabrizio Castro
> <fabrizio.castro@bp.renesas.com>; Linux-Renesas <linux-renesas-
> soc@vger.kernel.org>
> Subject: Re: [PATCH 10/10] cpufreq: dt: Add support for r8a7744
>
> On Tue, Sep 11, 2018 at 12:20 PM Biju Das <biju.das@bp.renesas.com>
> wrote:
> >
> > Add the compatible strings for supporting the generic cpufreq driver
> > on the Renesas RZ/G1N (R8A7744) SoC.
> >
> > Signed-off-by: Biju Das <biju.das@bp.renesas.com>
> > Reviewed-by: Fabrizio Castro <fabrizio.castro@bp.renesas.com>
> > ---
> >  drivers/cpufreq/cpufreq-dt-platdev.c | 1 +
> >  1 file changed, 1 insertion(+)
> >
> > diff --git a/drivers/cpufreq/cpufreq-dt-platdev.c
> > b/drivers/cpufreq/cpufreq-dt-platdev.c
> > index fe14c57..805f8a0 100644
> > --- a/drivers/cpufreq/cpufreq-dt-platdev.c
> > +++ b/drivers/cpufreq/cpufreq-dt-platdev.c
> > @@ -58,6 +58,7 @@ static const struct of_device_id whitelist[] __initconst
> = {
> >         { .compatible = "renesas,r8a73a4", },
> >         { .compatible = "renesas,r8a7740", },
> >         { .compatible = "renesas,r8a7743", },
> > +       { .compatible = "renesas,r8a7744", },
> >         { .compatible = "renesas,r8a7745", },
> >         { .compatible = "renesas,r8a7778", },
> >         { .compatible = "renesas,r8a7779", },
> > --
>
> Since this is the last patch in a series of 10, do you want me to apply it or
> should it go in along with the rest?

Please apply the patch. Already binding patch is available with cfb9f93437eb ("ARM: shmobile: Document RZ/G1N SoC DT binding").

Regards,
Biju





Renesas Electronics Europe Ltd, Dukes Meadow, Millboard Road, Bourne End, Buckinghamshire, SL8 5FH, UK. Registered in England & Wales under Registered No. 04586709.
Viresh Kumar Sept. 12, 2018, 4:57 a.m. UTC | #3 
On 11-09-18, 11:12, Biju Das wrote:
> Add the compatible strings for supporting the generic cpufreq driver on
> the Renesas RZ/G1N (R8A7744) SoC.
> 
> Signed-off-by: Biju Das <biju.das@bp.renesas.com>
> Reviewed-by: Fabrizio Castro <fabrizio.castro@bp.renesas.com>
> ---
>  drivers/cpufreq/cpufreq-dt-platdev.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/drivers/cpufreq/cpufreq-dt-platdev.c b/drivers/cpufreq/cpufreq-dt-platdev.c
> index fe14c57..805f8a0 100644
> --- a/drivers/cpufreq/cpufreq-dt-platdev.c
> +++ b/drivers/cpufreq/cpufreq-dt-platdev.c
> @@ -58,6 +58,7 @@ static const struct of_device_id whitelist[] __initconst = {
>  	{ .compatible = "renesas,r8a73a4", },
>  	{ .compatible = "renesas,r8a7740", },
>  	{ .compatible = "renesas,r8a7743", },
> +	{ .compatible = "renesas,r8a7744", },
>  	{ .compatible = "renesas,r8a7745", },
>  	{ .compatible = "renesas,r8a7778", },
>  	{ .compatible = "renesas,r8a7779", },

Acked-by: Viresh Kumar <viresh.kumar@linaro.org>
Simon Horman Sept. 12, 2018, 10:11 a.m. UTC | #4 
On Tue, Sep 11, 2018 at 11:12:51AM +0100, Biju Das wrote:
> Add the compatible strings for supporting the generic cpufreq driver on
> the Renesas RZ/G1N (R8A7744) SoC.
> 
> Signed-off-by: Biju Das <biju.das@bp.renesas.com>
> Reviewed-by: Fabrizio Castro <fabrizio.castro@bp.renesas.com>

Reviewed-by: Simon Horman <horms+renesas@verge.net.au>
Geert Uytterhoeven Sept. 14, 2018, 12:48 p.m. UTC | #5 
On Tue, Sep 11, 2018 at 12:20 PM Biju Das <biju.das@bp.renesas.com> wrote:
> Add the compatible strings for supporting the generic cpufreq driver on
> the Renesas RZ/G1N (R8A7744) SoC.
>
> Signed-off-by: Biju Das <biju.das@bp.renesas.com>
> Reviewed-by: Fabrizio Castro <fabrizio.castro@bp.renesas.com>

Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>

Gr{oetje,eeting}s,

                        Geert
Biju Das Sept. 25, 2018, 8:16 a.m. UTC | #6 
Hi Rafeal,

Sorry to bother you.  Are you happy to merge the below patch with 4.20

https://patchwork.kernel.org/patch/10595451/

Regards,
Biju

> -----Original Message-----
> From: Biju Das
> Sent: 11 September 2018 12:11
> To: Rafael J. Wysocki <rafael@kernel.org>
> Cc: Rafael J. Wysocki <rjw@rjwysocki.net>; Viresh Kumar
> <viresh.kumar@linaro.org>; Linux PM <linux-pm@vger.kernel.org>; Simon
> Horman <horms@verge.net.au>; Geert Uytterhoeven
> <geert+renesas@glider.be>; Chris Paterson
> <Chris.Paterson2@renesas.com>; Fabrizio Castro
> <fabrizio.castro@bp.renesas.com>; Linux-Renesas <linux-renesas-
> soc@vger.kernel.org>
> Subject: RE: [PATCH 10/10] cpufreq: dt: Add support for r8a7744
>
> Hi Rafael,
>
> Thanks for the feedback.
>
> > -----Original Message-----
> > From: Rafael J. Wysocki <rafael@kernel.org>
> > Sent: 11 September 2018 11:59
> > To: Biju Das <biju.das@bp.renesas.com>
> > Cc: Rafael J. Wysocki <rjw@rjwysocki.net>; Viresh Kumar
> > <viresh.kumar@linaro.org>; Linux PM <linux-pm@vger.kernel.org>; Simon
> > Horman <horms@verge.net.au>; Geert Uytterhoeven
> > <geert+renesas@glider.be>; Chris Paterson
> > <Chris.Paterson2@renesas.com>; Fabrizio Castro
> > <fabrizio.castro@bp.renesas.com>; Linux-Renesas <linux-renesas-
> > soc@vger.kernel.org>
> > Subject: Re: [PATCH 10/10] cpufreq: dt: Add support for r8a7744
> >
> > On Tue, Sep 11, 2018 at 12:20 PM Biju Das <biju.das@bp.renesas.com>
> > wrote:
> > >
> > > Add the compatible strings for supporting the generic cpufreq driver
> > > on the Renesas RZ/G1N (R8A7744) SoC.
> > >
> > > Signed-off-by: Biju Das <biju.das@bp.renesas.com>
> > > Reviewed-by: Fabrizio Castro <fabrizio.castro@bp.renesas.com>
> > > ---
> > >  drivers/cpufreq/cpufreq-dt-platdev.c | 1 +
> > >  1 file changed, 1 insertion(+)
> > >
> > > diff --git a/drivers/cpufreq/cpufreq-dt-platdev.c
> > > b/drivers/cpufreq/cpufreq-dt-platdev.c
> > > index fe14c57..805f8a0 100644
> > > --- a/drivers/cpufreq/cpufreq-dt-platdev.c
> > > +++ b/drivers/cpufreq/cpufreq-dt-platdev.c
> > > @@ -58,6 +58,7 @@ static const struct of_device_id whitelist[]
> > > __initconst
> > = {
> > >         { .compatible = "renesas,r8a73a4", },
> > >         { .compatible = "renesas,r8a7740", },
> > >         { .compatible = "renesas,r8a7743", },
> > > +       { .compatible = "renesas,r8a7744", },
> > >         { .compatible = "renesas,r8a7745", },
> > >         { .compatible = "renesas,r8a7778", },
> > >         { .compatible = "renesas,r8a7779", },
> > > --
> >
> > Since this is the last patch in a series of 10, do you want me to
> > apply it or should it go in along with the rest?
>
> Please apply the patch. Already binding patch is available with cfb9f93437eb
> ("ARM: shmobile: Document RZ/G1N SoC DT binding").
>
> Regards,
> Biju
>




Renesas Electronics Europe Ltd, Dukes Meadow, Millboard Road, Bourne End, Buckinghamshire, SL8 5FH, UK. Registered in England & Wales under Registered No. 04586709.
Rafael J. Wysocki Sept. 25, 2018, 8:22 p.m. UTC | #7 
On Tuesday, September 25, 2018 10:16:12 AM CEST Biju Das wrote:
> Hi Rafeal,
> 
> Sorry to bother you.  Are you happy to merge the below patch with 4.20
> 
> https://patchwork.kernel.org/patch/10595451/

Yes.

You had told me that you wanted me to apply it, so I queued it up.

Thanks,
Rafael
diff mbox series

Patch

diff --git a/drivers/cpufreq/cpufreq-dt-platdev.c b/drivers/cpufreq/cpufreq-dt-platdev.c
index fe14c57..805f8a0 100644
--- a/drivers/cpufreq/cpufreq-dt-platdev.c
+++ b/drivers/cpufreq/cpufreq-dt-platdev.c
@@ -58,6 +58,7 @@  static const struct of_device_id whitelist[] __initconst = {
 	{ .compatible = "renesas,r8a73a4", },
 	{ .compatible = "renesas,r8a7740", },
 	{ .compatible = "renesas,r8a7743", },
+	{ .compatible = "renesas,r8a7744", },
 	{ .compatible = "renesas,r8a7745", },
 	{ .compatible = "renesas,r8a7778", },
 	{ .compatible = "renesas,r8a7779", },