diff options
author | Paul Walmsley <paul@pwsan.com> | 2013-02-11 22:58:35 -0500 |
---|---|---|
committer | Tony Lindgren <tony@atomide.com> | 2013-02-12 12:00:33 -0500 |
commit | 6efc3fe0f4058d6764e5c58c7377037f9da24d22 (patch) | |
tree | a186ddbd974f003488bab6f7bfaef144a2524c84 | |
parent | c1d1cd597fc77af3086470f8627d77f52f7f8b6c (diff) |
ARM: OMAP2+: fix some omap_device_build() calls that aren't compiled by default
Commit c1d1cd597fc77af3086470f8627d77f52f7f8b6c ("ARM: OMAP2+:
omap_device: remove obsolete pm_lats and early_device code") missed a
few omap_device_build() calls that aren't included as part of the default
OMAP2+ Kconfig, omap2plus_defconfig.
Ideally, all devices that are present on the SoC should be created by
default, and only the corresponding device driver should be configured
or deconfigured in Kconfig. This allows drivers to be built as
modules and loaded later, even if they weren't part of the original
kernel build. Unfortunately, we're not quite there yet.
Thanks to Tony Lindgren for reporting this, found during his
randconfig tests.
Signed-off-by: Paul Walmsley <paul@pwsan.com>
Signed-off-by: Tony Lindgren <tony@atomide.com>
-rw-r--r-- | arch/arm/mach-omap2/am35xx-emac.c | 3 | ||||
-rw-r--r-- | arch/arm/mach-omap2/devices.c | 2 | ||||
-rw-r--r-- | arch/arm/mach-omap2/sr_device.c | 2 |
3 files changed, 3 insertions, 4 deletions
diff --git a/arch/arm/mach-omap2/am35xx-emac.c b/arch/arm/mach-omap2/am35xx-emac.c index a00d39107a21..25b79a297365 100644 --- a/arch/arm/mach-omap2/am35xx-emac.c +++ b/arch/arm/mach-omap2/am35xx-emac.c | |||
@@ -62,8 +62,7 @@ static int __init omap_davinci_emac_dev_init(struct omap_hwmod *oh, | |||
62 | { | 62 | { |
63 | struct platform_device *pdev; | 63 | struct platform_device *pdev; |
64 | 64 | ||
65 | pdev = omap_device_build(oh->class->name, 0, oh, pdata, pdata_len, | 65 | pdev = omap_device_build(oh->class->name, 0, oh, pdata, pdata_len); |
66 | false); | ||
67 | if (IS_ERR(pdev)) { | 66 | if (IS_ERR(pdev)) { |
68 | WARN(1, "Can't build omap_device for %s:%s.\n", | 67 | WARN(1, "Can't build omap_device for %s:%s.\n", |
69 | oh->class->name, oh->name); | 68 | oh->class->name, oh->name); |
diff --git a/arch/arm/mach-omap2/devices.c b/arch/arm/mach-omap2/devices.c index d8a0cc3b9d2c..6ecc89adda87 100644 --- a/arch/arm/mach-omap2/devices.c +++ b/arch/arm/mach-omap2/devices.c | |||
@@ -382,7 +382,7 @@ static void __init omap_init_hdmi_audio(void) | |||
382 | return; | 382 | return; |
383 | } | 383 | } |
384 | 384 | ||
385 | pdev = omap_device_build("omap-hdmi-audio-dai", -1, oh, NULL, 0, 0); | 385 | pdev = omap_device_build("omap-hdmi-audio-dai", -1, oh, NULL, 0); |
386 | WARN(IS_ERR(pdev), | 386 | WARN(IS_ERR(pdev), |
387 | "Can't build omap_device for omap-hdmi-audio-dai.\n"); | 387 | "Can't build omap_device for omap-hdmi-audio-dai.\n"); |
388 | 388 | ||
diff --git a/arch/arm/mach-omap2/sr_device.c b/arch/arm/mach-omap2/sr_device.c index bb829e065400..d7bc33f15344 100644 --- a/arch/arm/mach-omap2/sr_device.c +++ b/arch/arm/mach-omap2/sr_device.c | |||
@@ -152,7 +152,7 @@ static int __init sr_dev_init(struct omap_hwmod *oh, void *user) | |||
152 | 152 | ||
153 | sr_data->enable_on_init = sr_enable_on_init; | 153 | sr_data->enable_on_init = sr_enable_on_init; |
154 | 154 | ||
155 | pdev = omap_device_build(name, i, oh, sr_data, sizeof(*sr_data), 0); | 155 | pdev = omap_device_build(name, i, oh, sr_data, sizeof(*sr_data)); |
156 | if (IS_ERR(pdev)) | 156 | if (IS_ERR(pdev)) |
157 | pr_warning("%s: Could not build omap_device for %s: %s.\n\n", | 157 | pr_warning("%s: Could not build omap_device for %s: %s.\n\n", |
158 | __func__, name, oh->name); | 158 | __func__, name, oh->name); |