Skip to content

Commit

Permalink
ARM: arch-shmobile: only run HDMI init on respective boards
Browse files Browse the repository at this point in the history
If several boards are enabled in the kernel configuration,
hdmi_init_pm_clock() functions from board-ap4evb.c and board-mackerel.c
will run on any of them. Prevent this by calling these functions from the
.init_machine() callback instead of using device_initcall().

Signed-off-by: Guennadi Liakhovetski <g.liakhovetski@gmx.de>
Cc: Magnus Damm <damm@opensource.se>
Tested-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
Signed-off-by: Paul Mundt <lethal@linux-sh.org>
  • Loading branch information
Guennadi Liakhovetski authored and Paul Mundt committed Apr 6, 2011
1 parent 4a0f081 commit 2ce51f8
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 13 deletions.
11 changes: 4 additions & 7 deletions arch/arm/mach-shmobile/board-ap4evb.c
Original file line number Diff line number Diff line change
Expand Up @@ -947,7 +947,7 @@ static struct platform_device *ap4evb_devices[] __initdata = {
&ap4evb_camera,
};

static int __init hdmi_init_pm_clock(void)
static void __init hdmi_init_pm_clock(void)
{
struct clk *hdmi_ick = clk_get(&hdmi_device.dev, "ick");
int ret;
Expand Down Expand Up @@ -988,19 +988,14 @@ static int __init hdmi_init_pm_clock(void)
pr_debug("PLLC2 set frequency %lu\n", rate);

ret = clk_set_parent(hdmi_ick, &sh7372_pllc2_clk);
if (ret < 0) {
if (ret < 0)
pr_err("Cannot set HDMI parent: %d\n", ret);
goto out;
}

out:
if (!IS_ERR(hdmi_ick))
clk_put(hdmi_ick);
return ret;
}

device_initcall(hdmi_init_pm_clock);

static int __init fsi_init_pm_clock(void)
{
struct clk *fsia_ick;
Expand Down Expand Up @@ -1348,6 +1343,8 @@ static void __init ap4evb_init(void)
__raw_writel(srcr4 & ~(1 << 13), SRCR4);

platform_add_devices(ap4evb_devices, ARRAY_SIZE(ap4evb_devices));

hdmi_init_pm_clock();
}

static void __init ap4evb_timer_init(void)
Expand Down
10 changes: 4 additions & 6 deletions arch/arm/mach-shmobile/board-mackerel.c
Original file line number Diff line number Diff line change
Expand Up @@ -423,7 +423,7 @@ static struct platform_device fsi_hdmi_device = {
.name = "sh_fsi2_b_hdmi",
};

static int __init hdmi_init_pm_clock(void)
static void __init hdmi_init_pm_clock(void)
{
struct clk *hdmi_ick = clk_get(&hdmi_device.dev, "ick");
int ret;
Expand Down Expand Up @@ -467,17 +467,13 @@ static int __init hdmi_init_pm_clock(void)
pr_debug("PLLC2 set frequency %lu\n", rate);

ret = clk_set_parent(hdmi_ick, &sh7372_pllc2_clk);
if (ret < 0) {
if (ret < 0)
pr_err("Cannot set HDMI parent: %d\n", ret);
goto out;
}

out:
if (!IS_ERR(hdmi_ick))
clk_put(hdmi_ick);
return ret;
}
device_initcall(hdmi_init_pm_clock);

/* USB1 (Host) */
static void usb1_host_port_power(int port, int power)
Expand Down Expand Up @@ -1218,6 +1214,8 @@ static void __init mackerel_init(void)
sh7372_add_standard_devices();

platform_add_devices(mackerel_devices, ARRAY_SIZE(mackerel_devices));

hdmi_init_pm_clock();
}

static void __init mackerel_timer_init(void)
Expand Down

0 comments on commit 2ce51f8

Please sign in to comment.