[RFC,PATCH 0/3] Common struct clk implementation, v11

From: Jeremy Kerr
Date: Mon Feb 07 2011 - 01:08:45 EST


Hi all,

These patches are an attempt to allow platforms to share clock code. At
present, the definitions of 'struct clk' are local to platform code,
which makes allocating and initialising cross-platform clock sources
difficult, and makes it impossible to compile a single image containing
support for two ARM platforms with different struct clks.

The three patches are for the architecture-independent kernel code,
introducing the common clk infrastructure. The changelog for the first
patch includes details about the new clock definitions.

Ben Herrenschmidt is looking at using common struct clk code for powerpc
too, hence the kernel-wide approach.

Many thanks to the following for their input:
* Ben Dooks <ben-linux@xxxxxxxxx>
* Baruch Siach <baruch@xxxxxxxxxx>
* Russell King <linux@xxxxxxxxxxxxxxxx>
* Uwe Kleine-KÃnig <u.kleine-koenig@xxxxxxxxxxxxxx>
* Lorenzo Pieralisi <Lorenzo.Pieralisi@xxxxxxx>
* Vincent Guittot <vincent.guittot@xxxxxxxxxx>
* Sascha Hauer <s.hauer@xxxxxxxxxxxxxx>

Cheers,


Jeremy

--
v11:
* add prepare/unprepare for non-atomic switching, document atomicity
* move to drivers/clk/

v10:
* comment fixups, from Uwe's review
* added DEFINE_CLK_FIXED

v9:
* comment improvements
* kerneldoc fixups
* add WARN_ON to clk_disable

v8:
* add atomic clocks, and locking wrappers
* expand comments on clk and clk_ops

v7:
* change CLK_INIT to initialise clk->mutex statically

v6:
* fixed up references to 'clk_operations' in the changelog

v5:
* uninline main API, and share definitions with !USE_COMMON_STRUCT_CLK
* add __clk_get
* delay mutex init
* kerneldoc for struct clk

v4:
* use mutex for enable/disable locking
* DEFINE_CLK -> INIT_CLK, and pass the clk name for mutex init
* struct clk_operations -> struct clk_ops

v3:
* do clock usage refcounting in common code
* provide sample port

v2:
* no longer ARM-specific
* use clk_operations

---
Jeremy Kerr (3):
Add a common struct clk
clk: Generic support for fixed-rate clocks
clk: add warnings for incorrect enable/prepare semantics

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/