[PATCH 0/7] ARM: shmobile: Enable CPU FREQ support

[PATCH 0/7] ARM: shmobile: Enable CPU FREQ support

[Gaku Inami]

This patch series is initial support for CPU FREQ.

Based on : renesas-devel-v3.15-rc7-20140528

Support :
 [CPU]
 - Coretex A15( Z clock) : Supported
 - Coretex A7 (Z2 clock) : Not supported
 [HW]
 - R-CAR H2 : Supported
 - R-CAR M2 : Supported
 - Other HW : Not supported
 [DVFS feature]
 - DFS : Supported
 - DVS : Not supported

Benoit Cousson (2):
  ARM: dts: r8a7790-lager: Add DVFS parameters into cpu0 node for
    r8a7790
  ARM: shmobile: Make r8a77900 CPUFreq capable

Gaku Inami (5):
  ARM: shmobile: Disable DVS support for r8a7790
  ARM: shmobile: add cpufreq-cpu0 driver for common SH-Mobile
  ARM: dts: r8a7791-koelsch: Add DVFS parameters into cpu0 node for
    r8a7791
  ARM: shmobile: Make r8a7791 CPUFreq capable
  ARM: shmobile: Enable CPUFREQ configuration

 arch/arm/boot/dts/r8a7790.dtsi               |   11 +++++++++++
 arch/arm/boot/dts/r8a7791.dtsi               |   11 +++++++++++
 arch/arm/configs/shmobile_defconfig          |   12 ++++++++++++
 arch/arm/mach-shmobile/Kconfig               |    4 ++++
 arch/arm/mach-shmobile/Makefile              |    1 +
 arch/arm/mach-shmobile/cpufreq.c             |   21 +++++++++++++++++++++
 arch/arm/mach-shmobile/include/mach/common.h |    7 +++++++
 7 files changed, 67 insertions(+)
 create mode 100644 arch/arm/mach-shmobile/cpufreq.c

-- 
1.7.9.5

Re: [PATCH 0/7] ARM: shmobile: Enable CPU FREQ support

[Kuninori Morimoto]

Hi Inami-san

> Benoit Cousson (2):
>   ARM: dts: r8a7790-lager: Add DVFS parameters into cpu0 node for
>     r8a7790
>   ARM: shmobile: Make r8a77900 CPUFreq capable

According to this, [1/7] [2/7] patches were
created by Benoit ?

Maybe you want to add extra "From" to these patches,
and I guess you want to add your Signed-off-by also ?
Otherwise, patch author will be you instead of Benoit

like this ?

-----------------------
    From: Gaku Inami <gaku.inami.xw@bp.renesas.com>
    Subject: [PATCH 2/7] ARM: shmobile: Make r8a77900 CPUFreq capable

=>  From: Benoit Cousson <bcousson@baylibre.com>

    Mark r8a77900 platform as CPUFreq capable
    on multiplatform build only.

=>  Signed-off-by: Gaku Inami <gaku.inami.xw@bp.renesas.com>
    Signed-off-by: Benoit Cousson <bcousson@baylibre.com>
    ---
    ...

Re: [PATCH 0/7] ARM: shmobile: Enable CPU FREQ support

[Gaku Inami]

Hi Morimoto-san,

Thank you for your comment.

> According to this, [1/7] [2/7] patches were
> created by Benoit ?

Yes, two patches is based on the following post.

http://marc.info/?l=linux-sh&m\x139345628817803&w=2
http://marc.info/?l=linux-sh&m\x139345629117809&w=2

> Maybe you want to add extra "From" to these patches,
> and I guess you want to add your Signed-off-by also ?
> Otherwise, patch author will be you instead of Benoit
> 
> like this ?
> 
> -----------------------
>     From: Gaku Inami <gaku.inami.xw@bp.renesas.com>
>     Subject: [PATCH 2/7] ARM: shmobile: Make r8a77900 CPUFreq capable
> 
> =>  From: Benoit Cousson <bcousson@baylibre.com>
> 
>     Mark r8a77900 platform as CPUFreq capable
>     on multiplatform build only.
> 
> =>  Signed-off-by: Gaku Inami <gaku.inami.xw@bp.renesas.com>
>     Signed-off-by: Benoit Cousson <bcousson@baylibre.com>
>     ---
>     ...

Thank you, I will fix it.

Re: [PATCH 0/7] ARM: shmobile: Enable CPU FREQ support

[Simon Horman]

On Thu, May 29, 2014 at 01:09:17AM -0700, Kuninori Morimoto wrote:
> 
> Hi Inami-san
> 
> > Benoit Cousson (2):
> >   ARM: dts: r8a7790-lager: Add DVFS parameters into cpu0 node for
> >     r8a7790
> >   ARM: shmobile: Make r8a77900 CPUFreq capable
> 
> According to this, [1/7] [2/7] patches were
> created by Benoit ?
> 
> Maybe you want to add extra "From" to these patches,
> and I guess you want to add your Signed-off-by also ?
> Otherwise, patch author will be you instead of Benoit
> 
> like this ?
> 
> -----------------------
>     From: Gaku Inami <gaku.inami.xw@bp.renesas.com>
>     Subject: [PATCH 2/7] ARM: shmobile: Make r8a77900 CPUFreq capable
> 
> =>  From: Benoit Cousson <bcousson@baylibre.com>
> 
>     Mark r8a77900 platform as CPUFreq capable
>     on multiplatform build only.
> 
> =>  Signed-off-by: Gaku Inami <gaku.inami.xw@bp.renesas.com>
>     Signed-off-by: Benoit Cousson <bcousson@baylibre.com>

If Benoit Cousson wrote the patch and Inami-san is passing it on
(with or without modifications) then I think the Signed-off-by lines
should be the other way around.

    Signed-off-by: Benoit Cousson <bcousson@baylibre.com>
=>  Signed-off-by: Gaku Inami <gaku.inami.xw@bp.renesas.com>


Also, I think it would be sane tough not essential to CC Benoit Cousson
when reposting the patches.

>     ---
>     ...
>