mbox series

[0/3] cpufreq: merge arm_big_little and vexpress-spc

Message ID 20191016110344.15259-1-sudeep.holla@arm.com
Headers show
Series cpufreq: merge arm_big_little and vexpress-spc | expand

Message

Sudeep Holla Oct. 16, 2019, 11:03 a.m. UTC
Hi,

Since vexpress-spc is the sole user of arm_big_little cpufreq driver,
there's no point in keeping it separate anymore. I wanted to post these
patches for ages but kept postponing for no reason.

Regards,
Sudeep

Sudeep Holla (3):
  cpufreq: scpi: remove stale/outdated comment about the driver
  cpufreq: merge arm_big_little and vexpress-spc
  cpufreq: simplify and remove lots of debug messages

 MAINTAINERS                            |   5 +-
 drivers/cpufreq/Kconfig.arm            |  12 +-
 drivers/cpufreq/Makefile               |   2 -
 drivers/cpufreq/arm_big_little.c       | 658 -------------------------
 drivers/cpufreq/arm_big_little.h       |  43 --
 drivers/cpufreq/scpi-cpufreq.c         |   2 -
 drivers/cpufreq/vexpress-spc-cpufreq.c | 559 ++++++++++++++++++++-
 7 files changed, 539 insertions(+), 742 deletions(-)
 delete mode 100644 drivers/cpufreq/arm_big_little.c
 delete mode 100644 drivers/cpufreq/arm_big_little.h

--
2.17.1

Comments

Viresh Kumar Oct. 17, 2019, 2:43 a.m. UTC | #1
+Nico in case he has any issues with this series.

On 16-10-19, 12:03, Sudeep Holla wrote:
> Hi,

> 

> Since vexpress-spc is the sole user of arm_big_little cpufreq driver,

> there's no point in keeping it separate anymore. I wanted to post these

> patches for ages but kept postponing for no reason.

> 

> Regards,

> Sudeep

> 

> Sudeep Holla (3):

>   cpufreq: scpi: remove stale/outdated comment about the driver

>   cpufreq: merge arm_big_little and vexpress-spc

>   cpufreq: simplify and remove lots of debug messages

> 

>  MAINTAINERS                            |   5 +-

>  drivers/cpufreq/Kconfig.arm            |  12 +-

>  drivers/cpufreq/Makefile               |   2 -

>  drivers/cpufreq/arm_big_little.c       | 658 -------------------------

>  drivers/cpufreq/arm_big_little.h       |  43 --

>  drivers/cpufreq/scpi-cpufreq.c         |   2 -

>  drivers/cpufreq/vexpress-spc-cpufreq.c | 559 ++++++++++++++++++++-

>  7 files changed, 539 insertions(+), 742 deletions(-)

>  delete mode 100644 drivers/cpufreq/arm_big_little.c

>  delete mode 100644 drivers/cpufreq/arm_big_little.h

> 

> --

> 2.17.1


-- 
viresh
Sudeep Holla Oct. 17, 2019, 9:14 a.m. UTC | #2
On Thu, Oct 17, 2019 at 08:12:00AM +0530, Viresh Kumar wrote:
> On 16-10-19, 12:03, Sudeep Holla wrote:

> > cpufreq_arm_bL_ops is no longer needed after merging the generic

> > arm_big_little and vexpress-spc driver. Remove cpufreq_arm_bL_ops

> > and rename all the bL_* function names to ve_spc_*.

> >

> > This driver have been used for year now and the extensive debug

> > messages in the driver are not really required anymore.

>

> This does lots of cleanup in this patch and not strictly what the commit log

> says. Can you please create separate patches for remove ops, debug messages and

> other formatting things ?

>


Yes I did notice just after posting. These patches were sitting in my tree
for long time and didn't look at them in detail before posting. I will
split the patch accordingly.

--
Regards,
Sudeep