Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit cf30fb4a authored by Russell King's avatar Russell King Committed by Russell King
Browse files

[ARM] realview: convert to clkdev and lookup clocks by device name 



People often point to the Integrator/Versatile/Realview
implementations to justify using the consumer name as the sole
selector for clocks.

Eliminate this excuse by changing the Realview implementation, so
it provides a better example of how it should be done.

Signed-off-by: default avatarRussell King <rmk+kernel@arm.linux.org.uk>
parent 0318e693

arch/arm/Kconfig

+1 −0
Original line number Diff line number Diff line
@@ -218,6 +218,7 @@ config ARCH_REALVIEW 
	bool "ARM Ltd. RealView family"

	select ARM_AMBA

	select HAVE_CLK

	select COMMON_CLKDEV

	select ICST307

	select GENERIC_TIME

	select GENERIC_CLOCKEVENTS

arch/arm/mach-realview/clock.c

+5 −75
Original line number Diff line number Diff line
@@ -10,9 +10,11 @@ 
 */

#include <linux/module.h>

#include <linux/kernel.h>

#include <linux/device.h>

#include <linux/list.h>

#include <linux/errno.h>

#include <linux/err.h>

#include <linux/string.h>

#include <linux/clk.h>

#include <linux/mutex.h>
@@ -20,32 +22,6 @@ 


#include "clock.h"



static LIST_HEAD(clocks);

static DEFINE_MUTEX(clocks_mutex);



struct clk *clk_get(struct device *dev, const char *id)

{

	struct clk *p, *clk = ERR_PTR(-ENOENT);



	mutex_lock(&clocks_mutex);

	list_for_each_entry(p, &clocks, node) {

		if (strcmp(id, p->name) == 0 && try_module_get(p->owner)) {

			clk = p;

			break;

		}

	}

	mutex_unlock(&clocks_mutex);



	return clk;

}

EXPORT_SYMBOL(clk_get);



void clk_put(struct clk *clk)

{

	module_put(clk->owner);

}

EXPORT_SYMBOL(clk_put);



int clk_enable(struct clk *clk)

{

	return 0;
@@ -65,7 +41,9 @@ EXPORT_SYMBOL(clk_get_rate); 


long clk_round_rate(struct clk *clk, unsigned long rate)

{

	return rate;

	struct icst307_vco vco;

	vco = icst307_khz_to_vco(clk->params, rate / 1000);

	return icst307_khz(clk->params, vco) * 1000;

}

EXPORT_SYMBOL(clk_round_rate);
@@ -78,57 +56,9 @@ int clk_set_rate(struct clk *clk, unsigned long rate) 


		vco = icst307_khz_to_vco(clk->params, rate / 1000);

		clk->rate = icst307_khz(clk->params, vco) * 1000;



		printk("Clock %s: setting VCO reg params: S=%d R=%d V=%d\n",

			clk->name, vco.s, vco.r, vco.v);



		clk->setvco(clk, vco);

		ret = 0;

	}

	return ret;

}

EXPORT_SYMBOL(clk_set_rate);
 


/*

 * These are fixed clocks.

 */

static struct clk kmi_clk = {

	.name	= "KMIREFCLK",

	.rate	= 24000000,

};



static struct clk uart_clk = {

	.name	= "UARTCLK",

	.rate	= 24000000,

};



static struct clk mmci_clk = {

	.name	= "MCLK",

	.rate	= 24000000,

};



int clk_register(struct clk *clk)

{

	mutex_lock(&clocks_mutex);

	list_add(&clk->node, &clocks);

	mutex_unlock(&clocks_mutex);

	return 0;

}

EXPORT_SYMBOL(clk_register);



void clk_unregister(struct clk *clk)

{

	mutex_lock(&clocks_mutex);

	list_del(&clk->node);

	mutex_unlock(&clocks_mutex);

}

EXPORT_SYMBOL(clk_unregister);



static int __init clk_init(void)

{

	clk_register(&kmi_clk);

	clk_register(&uart_clk);

	clk_register(&mmci_clk);

	return 0;

}

arch_initcall(clk_init);

arch/arm/mach-realview/clock.h

+0 −6
Original line number Diff line number Diff line
@@ -12,14 +12,8 @@ struct module; 
struct icst307_params;



struct clk {

	struct list_head	node;

	unsigned long		rate;

	struct module		*owner;

	const char		*name;

	const struct icst307_params *params;

	void			*data;

	void			(*setvco)(struct clk *, struct icst307_vco vco);

};
 


int clk_register(struct clk *clk);

void clk_unregister(struct clk *clk);

arch/arm/mach-realview/core.c

+50 −2
Original line number Diff line number Diff line
@@ -29,6 +29,7 @@ 
#include <linux/clockchips.h>

#include <linux/io.h>



#include <asm/clkdev.h>

#include <asm/system.h>

#include <mach/hardware.h>

#include <asm/irq.h>
@@ -188,12 +189,59 @@ static void realview_oscvco_set(struct clk *clk, struct icst307_vco vco) 
	writel(0, sys_lock);

}



struct clk realview_clcd_clk = {

	.name	= "CLCDCLK",

static struct clk oscvco_clk = {

	.params	= &realview_oscvco_params,

	.setvco = realview_oscvco_set,

};



/*

 * These are fixed clocks.

 */

static struct clk ref24_clk = {

	.rate	= 24000000,

};



static struct clk_lookup lookups[] = {

	{	/* UART0 */

		.dev_id		= "dev:f1",

		.clk		= &ref24_clk,

	}, {	/* UART1 */

		.dev_id		= "dev:f2",

		.clk		= &ref24_clk,

	}, {	/* UART2 */

		.dev_id		= "dev:f3",

		.clk		= &ref24_clk,

	}, {	/* UART3 */

		.dev_id		= "fpga:09",

		.clk		= &ref24_clk,

	}, {	/* KMI0 */

		.dev_id		= "fpga:06",

		.clk		= &ref24_clk,

	}, {	/* KMI1 */

		.dev_id		= "fpga:07",

		.clk		= &ref24_clk,

	}, {	/* MMC0 */

		.dev_id		= "fpga:05",

		.clk		= &ref24_clk,

	}, {	/* EB:CLCD */

		.dev_id		= "dev:20",

		.clk		= &oscvco_clk,

	}, {	/* PB:CLCD */

		.dev_id		= "issp:20",

		.clk		= &oscvco_clk,

	}

};



static int __init clk_init(void)

{

	int i;



	for (i = 0; i < ARRAY_SIZE(lookups); i++)

		clkdev_add(&lookups[i]);

	return 0;

}

arch_initcall(clk_init);



/*

 * CLCD support.

 */

arch/arm/mach-realview/core.h

+0 −1
Original line number Diff line number Diff line
@@ -48,7 +48,6 @@ extern struct platform_device realview_flash_device; 
extern struct platform_device realview_i2c_device;

extern struct mmc_platform_data realview_mmc0_plat_data;

extern struct mmc_platform_data realview_mmc1_plat_data;

extern struct clk realview_clcd_clk;

extern struct clcd_board clcd_plat_data;

extern void __iomem *gic_cpu_base_addr;

#ifdef CONFIG_LOCAL_TIMERS