[PATCH v9 0/3] Per-user clock constraints

* [PATCH v9 0/3] Per-user clock constraints
@ 2015-01-19 17:20 Tomeu Vizoso
  2015-01-19 17:20 ` [PATCH v9 1/3] clk: Remove unneeded NULL checks Tomeu Vizoso
                   ` (2 more replies)
  0 siblings, 3 replies; 18+ messages in thread
From: Tomeu Vizoso @ 2015-01-19 17:20 UTC (permalink / raw)
  To: linux-kernel, Mike Turquette, Stephen Boyd
  Cc: Javier Martinez Canillas, Tomeu Vizoso

Hi,

I have addressed here the last comments from Stephen, most notably:

* Removed some unneeded NULL checks that were already there

* Propagate our rate range when querying our parent for the rate

* Take constraints into account in clk_round_rate

* Add __clk_determine_rate() for clk providers to ask their parents for a rate
  within their range

* Make sure that what ops->round_rate returns when changing rates is within the
  range

The second patch actually moves the per-clock data that was stored in struct clk
to a new struct clk_core and adds references to it from both struct clk and
struct clk_hw. struct clk is now ready to contain information that is specific
to a given clk consumer.

The third patch adds API for setting floor and ceiling constraints and stores
that information on the per-user struct clk, which is iterable from struct
clk_core. The constraints are made available to clock implementations in the
determine_rate callback.

A rough test module was used to test this:

http://cgit.collabora.com/git/user/tomeu/linux.git/commit/?h=per-user-clk-constraints-v9&id=5306f3eb7d4df9f29ddcf9ba35298c7c1a422916

http://cgit.collabora.com/git/user/tomeu/linux.git/log/?h=per-user-clk-constraints-v9

Thanks,

Tomeu

Tomeu Vizoso (3):
  clk: Remove unneeded NULL checks
  clk: Make clk API return per-user struct clk instances
  clk: Add floor and ceiling constraints to clock rates

 Documentation/clk.txt                   |   2 +
 arch/arm/mach-omap2/cclock3xxx_data.c   | 108 ++--
 arch/arm/mach-omap2/clock.h             |  11 +-
 arch/arm/mach-omap2/clock_common_data.c |   5 +-
 arch/arm/mach-omap2/dpll3xxx.c          |   2 +
 arch/arm/mach-omap2/dpll44xx.c          |   2 +
 arch/mips/alchemy/common/clock.c        |   8 +
 drivers/clk/at91/clk-programmable.c     |   2 +
 drivers/clk/bcm/clk-kona.c              |   2 +
 drivers/clk/clk-composite.c             |   9 +-
 drivers/clk/clk.c                       | 840 ++++++++++++++++++++++----------
 drivers/clk/clk.h                       |   5 +
 drivers/clk/clkdev.c                    |  80 ++-
 drivers/clk/hisilicon/clk-hi3620.c      |   2 +
 drivers/clk/mmp/clk-mix.c               |   2 +
 drivers/clk/qcom/clk-pll.c              |   1 +
 drivers/clk/qcom/clk-rcg.c              |  10 +-
 drivers/clk/qcom/clk-rcg2.c             |   6 +
 drivers/clk/sunxi/clk-factors.c         |   2 +
 drivers/clk/sunxi/clk-sun6i-ar100.c     |   2 +
 include/linux/clk-private.h             |  41 +-
 include/linux/clk-provider.h            |  26 +-
 include/linux/clk.h                     |  28 ++
 include/linux/clk/ti.h                  |   4 +
 24 files changed, 855 insertions(+), 345 deletions(-)

-- 
1.9.3

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