Dear Russell King,
On Mon, 02 Mar 2015 17:06:42 +, Russell King wrote:
clkdev_create() is a shorter way to write clkdev_alloc() followed by
clkdev_add(). Use this instead.
Signed-off-by: Russell King rmk+ker...@arm.linux.org.uk
---
arch/arm/plat-orion/common.c | 6 +-
1 file changed, 1 insertion(+), 5 deletions(-)
diff --git a/arch/arm/plat-orion/common.c b/arch/arm/plat-orion/common.c
index f5b00f41c4f6..2235081a04ee 100644
--- a/arch/arm/plat-orion/common.c
+++ b/arch/arm/plat-orion/common.c
@@ -28,11 +28,7 @@
void __init orion_clkdev_add(const char *con_id, const char *dev_id,
struct clk *clk)
{
- struct clk_lookup *cl;
-
- cl = clkdev_alloc(clk, con_id, dev_id);
- if (cl)
- clkdev_add(cl);
+ clkdev_create(clk, con_id, %s, dev_id);
}
/* Create clkdev entries for all orion platforms except kirkwood.
Looks good, but instead of having orion_clkdev_add() being just an
alias for clkdev_create(), what about going ahead and simply reoving
orion_clkdev_add() entirely? Something like the below patch (not even
compile tested) :
diff --git a/arch/arm/mach-dove/common.c b/arch/arm/mach-dove/common.c
index 0d1a892..ec00183 100644
--- a/arch/arm/mach-dove/common.c
+++ b/arch/arm/mach-dove/common.c
@@ -109,28 +109,28 @@ static void __init dove_clk_init(void)
gephy = dove_register_gate(gephy, tclk, CLOCK_GATING_BIT_GIGA_PHY);
ge = dove_register_gate(ge, gephy, CLOCK_GATING_BIT_GBE);
- orion_clkdev_add(NULL, orion_spi.0, tclk);
- orion_clkdev_add(NULL, orion_spi.1, tclk);
- orion_clkdev_add(NULL, orion_wdt, tclk);
- orion_clkdev_add(NULL, mv64xxx_i2c.0, tclk);
-
- orion_clkdev_add(NULL, orion-ehci.0, usb0);
- orion_clkdev_add(NULL, orion-ehci.1, usb1);
- orion_clkdev_add(NULL, mv643xx_eth_port.0, ge);
- orion_clkdev_add(NULL, sata_mv.0, sata);
- orion_clkdev_add(0, pcie, pex0);
- orion_clkdev_add(1, pcie, pex1);
- orion_clkdev_add(NULL, sdhci-dove.0, sdio0);
- orion_clkdev_add(NULL, sdhci-dove.1, sdio1);
- orion_clkdev_add(NULL, orion_nand, nand);
- orion_clkdev_add(NULL, cafe1000-ccic.0, camera);
- orion_clkdev_add(NULL, mvebu-audio.0, i2s0);
- orion_clkdev_add(NULL, mvebu-audio.1, i2s1);
- orion_clkdev_add(NULL, mv_crypto, crypto);
- orion_clkdev_add(NULL, dove-ac97, ac97);
- orion_clkdev_add(NULL, dove-pdma, pdma);
- orion_clkdev_add(NULL, MV_XOR_NAME .0, xor0);
- orion_clkdev_add(NULL, MV_XOR_NAME .1, xor1);
+ clkdev_create(tclk, NULL, %s, orion_spi.0);
+ clkdev_create(tclk, NULL, %s, orion_spi.1);
+ clkdev_create(tclk, NULL, %s, orion_wdt);
+ clkdev_create(tclk, NULL, %s, mv64xxx_i2c.0);
+
+ clkdev_create(usb0, NULL, %s, orion-ehci.0);
+ clkdev_create(usb1, NULL, %s, orion-ehci.1);
+ clkdev_create(ge, NULL, %s, mv643xx_eth_port.0);
+ clkdev_create(sata, NULL, %s, sata_mv.0);
+ clkdev_create(pex0, 0, %s, pcie);
+ clkdev_create(pex1, 1, %s, pcie);
+ clkdev_create(sdio0, NULL, %s, sdhci-dove.0);
+ clkdev_create(sdio1, NULL, %s, sdhci-dove.1);
+ clkdev_create(nand, NULL, %s, orion_nand);
+ clkdev_create(camera, NULL, %s, cafe1000-ccic.0);
+ clkdev_create(i2s0, NULL, %s, mvebu-audio.0);
+ clkdev_create(i2s1, NULL, %s, mvebu-audio.1);
+ clkdev_create(crypto, NULL, %s, mv_crypto);
+ clkdev_create(ac97, NULL, %s, dove-ac97);
+ clkdev_create(pdma, NULL, %s, dove-pdma);
+ clkdev_create(xor0, NULL, %s, MV_XOR_NAME .0);
+ clkdev_create(xor1, NULL, %s, MV_XOR_NAME .1);
}
/*
diff --git a/arch/arm/plat-orion/common.c b/arch/arm/plat-orion/common.c
index f5b00f4..6ac3549 100644
--- a/arch/arm/plat-orion/common.c
+++ b/arch/arm/plat-orion/common.c
@@ -24,31 +24,20 @@
#include mach/bridge-regs.h
#include plat/common.h
-/* Create a clkdev entry for a given device/clk */
-void __init orion_clkdev_add(const char *con_id, const char *dev_id,
-struct clk *clk)
-{
- struct clk_lookup *cl;
-
- cl = clkdev_alloc(clk, con_id, dev_id);
- if (cl)
- clkdev_add(cl);
-}
-
/* Create clkdev entries for all orion platforms except kirkwood.
Kirkwood has gated clocks for some of its peripherals, so creates
its own clkdev entries. For all the other orion devices, create
clkdev entries to the tclk. */
void __init orion_clkdev_init(struct clk *tclk)
{
- orion_clkdev_add(NULL, orion_spi.0, tclk);
- orion_clkdev_add(NULL, orion_spi.1, tclk);
- orion_clkdev_add(NULL, MV643XX_ETH_NAME .0, tclk);
- orion_clkdev_add(NULL, MV643XX_ETH_NAME .1, tclk);
- orion_clkdev_add(NULL, MV643XX_ETH_NAME .2, tclk);
- orion_clkdev_add(NULL, MV643XX_ETH_NAME .3, tclk);
- orion_clkdev_add(NULL, orion_wdt, tclk);
-
