Skip to content

Commit

Permalink
arm: Remove unnecessary of_platform_populate with default match table
Browse files Browse the repository at this point in the history
After patch "of/platform: Add common method to populate default bus",
it is possible for arch code to remove unnecessary callers of
of_platform_populate with default match table.

Acked-by: Viresh Kumar <[email protected]>
Cc: Nicolas Ferre <[email protected]>
Cc: Ray Jui <[email protected]>
Cc: Lee Jones <[email protected]>
Cc: Krzysztof Halasa <[email protected]>
Cc: Kukjin Kim <[email protected]>
Cc: Rob Herring <[email protected]>
Cc: Shawn Guo <[email protected]>
Cc: Santosh Shilimkar <[email protected]>
Cc: Roland Stigge <[email protected]>
Cc: Jason Cooper <[email protected]>
Cc: Haojian Zhuang <[email protected]>
Cc: Heiko Stuebner <[email protected]>
Cc: Viresh Kumar <[email protected]>
Cc: Shiraz Hashim <[email protected]>
Cc: Tony Prisk <[email protected]>
Cc: Arnd Bergmann <[email protected]>
Cc: Russell King <[email protected]>
Signed-off-by: Kefeng Wang <[email protected]>
Signed-off-by: Rob Herring <[email protected]>
  • Loading branch information
Kefeng Wang authored and robherring committed Jun 23, 2016
1 parent 61c7864 commit 850bea2
Show file tree
Hide file tree
Showing 23 changed files with 1 addition and 59 deletions.
6 changes: 1 addition & 5 deletions arch/arm/kernel/setup.c
Original file line number Diff line number Diff line change
Expand Up @@ -904,11 +904,7 @@ static int __init customize_machine(void)
*/
if (machine_desc->init_machine)
machine_desc->init_machine();
#ifdef CONFIG_OF
else
of_platform_populate(NULL, of_default_bus_match_table,
NULL, NULL);
#endif

return 0;
}
arch_initcall(customize_machine);
Expand Down
3 changes: 0 additions & 3 deletions arch/arm/mach-artpec/board-artpec6.c
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,6 @@
#include <linux/irqchip.h>
#include <linux/irqchip/arm-gic.h>
#include <linux/mfd/syscon.h>
#include <linux/of_platform.h>
#include <linux/of.h>
#include <linux/of_address.h>
#include <linux/clk-provider.h>
Expand Down Expand Up @@ -44,8 +43,6 @@ static void __init artpec6_init_machine(void)
regmap_write(regmap, ARTPEC6_DMACFG_REGNUM,
ARTPEC6_DMACFG_UARTS_BURST);
};

of_platform_populate(NULL, of_default_bus_match_table, NULL, NULL);
}

static void artpec6_l2c310_write_sec(unsigned long val, unsigned reg)
Expand Down
2 changes: 0 additions & 2 deletions arch/arm/mach-bcm/board_bcm21664.c
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,6 @@
*/

#include <linux/of_address.h>
#include <linux/of_platform.h>
#include <linux/io.h>

#include <asm/mach/arch.h>
Expand Down Expand Up @@ -60,7 +59,6 @@ static void bcm21664_restart(enum reboot_mode mode, const char *cmd)

static void __init bcm21664_init(void)
{
of_platform_populate(NULL, of_default_bus_match_table, NULL, NULL);
kona_l2_cache_init();
}

Expand Down
2 changes: 0 additions & 2 deletions arch/arm/mach-bcm/board_bcm281xx.c
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,6 @@

#include <linux/clocksource.h>
#include <linux/of_address.h>
#include <linux/of_platform.h>

#include <asm/mach/arch.h>

Expand Down Expand Up @@ -58,7 +57,6 @@ static void bcm281xx_restart(enum reboot_mode mode, const char *cmd)

static void __init bcm281xx_init(void)
{
of_platform_populate(NULL, of_default_bus_match_table, NULL, NULL);
kona_l2_cache_init();
}

Expand Down
10 changes: 0 additions & 10 deletions arch/arm/mach-bcm/board_bcm2835.c
Original file line number Diff line number Diff line change
Expand Up @@ -15,24 +15,14 @@
#include <linux/init.h>
#include <linux/irqchip.h>
#include <linux/of_address.h>
#include <linux/of_platform.h>
#include <linux/clk/bcm2835.h>

#include <asm/mach/arch.h>
#include <asm/mach/map.h>

static void __init bcm2835_init(void)
{
int ret;

bcm2835_init_clocks();

ret = of_platform_populate(NULL, of_default_bus_match_table, NULL,
NULL);
if (ret) {
pr_err("of_platform_populate failed: %d\n", ret);
BUG();
}
}

static const char * const bcm2835_compat[] = {
Expand Down
3 changes: 0 additions & 3 deletions arch/arm/mach-exynos/exynos.c
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,6 @@
#include <linux/of.h>
#include <linux/of_address.h>
#include <linux/of_fdt.h>
#include <linux/of_platform.h>
#include <linux/platform_device.h>
#include <linux/irqchip.h>
#include <linux/soc/samsung/exynos-regs-pmu.h>
Expand Down Expand Up @@ -217,8 +216,6 @@ static void __init exynos_dt_machine_init(void)
of_machine_is_compatible("samsung,exynos3250") ||
of_machine_is_compatible("samsung,exynos5250"))
platform_device_register(&exynos_cpuidle);

of_platform_populate(NULL, of_default_bus_match_table, NULL, NULL);
}

static char const *const exynos_dt_compat[] __initconst = {
Expand Down
3 changes: 0 additions & 3 deletions arch/arm/mach-highbank/highbank.c
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,6 @@
#include <linux/pl320-ipc.h>
#include <linux/of.h>
#include <linux/of_irq.h>
#include <linux/of_platform.h>
#include <linux/of_address.h>
#include <linux/reboot.h>
#include <linux/amba/bus.h>
Expand Down Expand Up @@ -163,8 +162,6 @@ static void __init highbank_init(void)

pl320_ipc_register_notifier(&hb_keys_nb);

of_platform_populate(NULL, of_default_bus_match_table, NULL, NULL);

if (psci_ops.cpu_suspend)
platform_device_register(&highbank_cpuidle_device);
}
Expand Down
2 changes: 0 additions & 2 deletions arch/arm/mach-imx/mach-imx51.c
Original file line number Diff line number Diff line change
Expand Up @@ -52,8 +52,6 @@ static void __init imx51_dt_init(void)
{
imx51_ipu_mipi_setup();
imx_src_init();

of_platform_populate(NULL, of_default_bus_match_table, NULL, NULL);
}

static void __init imx51_init_late(void)
Expand Down
2 changes: 0 additions & 2 deletions arch/arm/mach-imx/mach-imx53.c
Original file line number Diff line number Diff line change
Expand Up @@ -32,8 +32,6 @@ static void __init imx53_dt_init(void)
{
imx_src_init();

of_platform_populate(NULL, of_default_bus_match_table, NULL, NULL);

imx_aips_allow_unprivileged_access("fsl,imx53-aipstz");
}

Expand Down
1 change: 0 additions & 1 deletion arch/arm/mach-imx/mach-imx6ul.c
Original file line number Diff line number Diff line change
Expand Up @@ -64,7 +64,6 @@ static void __init imx6ul_init_machine(void)
if (parent == NULL)
pr_warn("failed to initialize soc device\n");

of_platform_populate(NULL, of_default_bus_match_table, NULL, NULL);
imx6ul_enet_init();
imx_anatop_init();
imx6ul_pm_init();
Expand Down
1 change: 0 additions & 1 deletion arch/arm/mach-imx/mach-imx7d.c
Original file line number Diff line number Diff line change
Expand Up @@ -93,7 +93,6 @@ static void __init imx7d_init_machine(void)
if (parent == NULL)
pr_warn("failed to initialize soc device\n");

of_platform_populate(NULL, of_default_bus_match_table, NULL, NULL);
imx_anatop_init();
imx7d_enet_init();
}
Expand Down
1 change: 0 additions & 1 deletion arch/arm/mach-keystone/keystone.c
Original file line number Diff line number Diff line change
Expand Up @@ -60,7 +60,6 @@ static void __init keystone_init(void)
bus_register_notifier(&platform_bus_type, &platform_nb);
}
keystone_pm_runtime_init();
of_platform_populate(NULL, of_default_bus_match_table, NULL, NULL);
}

static long long __init keystone_pv_fixup(void)
Expand Down
3 changes: 0 additions & 3 deletions arch/arm/mach-mvebu/board-v7.c
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,6 @@
#include <linux/init.h>
#include <linux/of_address.h>
#include <linux/of_fdt.h>
#include <linux/of_platform.h>
#include <linux/io.h>
#include <linux/clocksource.h>
#include <linux/dma-mapping.h>
Expand Down Expand Up @@ -144,8 +143,6 @@ static void __init mvebu_dt_init(void)
{
if (of_machine_is_compatible("marvell,armadaxp"))
i2c_quirk();

of_platform_populate(NULL, of_default_bus_match_table, NULL, NULL);
}

static const char * const armada_370_xp_dt_compat[] __initconst = {
Expand Down
2 changes: 0 additions & 2 deletions arch/arm/mach-mvebu/dove.c
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,6 @@
#include <linux/init.h>
#include <linux/mbus.h>
#include <linux/of.h>
#include <linux/of_platform.h>
#include <linux/soc/dove/pmu.h>
#include <asm/hardware/cache-tauros2.h>
#include <asm/mach/arch.h>
Expand All @@ -26,7 +25,6 @@ static void __init dove_init(void)
#endif
BUG_ON(mvebu_mbus_dt_init(false));
dove_init_pmu();
of_platform_populate(NULL, of_default_bus_match_table, NULL, NULL);
}

static const char * const dove_dt_compat[] __initconst = {
Expand Down
2 changes: 0 additions & 2 deletions arch/arm/mach-picoxcell/common.c
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,6 @@
#include <linux/delay.h>
#include <linux/of.h>
#include <linux/of_address.h>
#include <linux/of_platform.h>
#include <linux/reboot.h>

#include <asm/mach/arch.h>
Expand Down Expand Up @@ -54,7 +53,6 @@ static void __init picoxcell_map_io(void)

static void __init picoxcell_init_machine(void)
{
of_platform_populate(NULL, of_default_bus_match_table, NULL, NULL);
picoxcell_setup_restart();
}

Expand Down
1 change: 0 additions & 1 deletion arch/arm/mach-rockchip/rockchip.c
Original file line number Diff line number Diff line change
Expand Up @@ -73,7 +73,6 @@ static void __init rockchip_timer_init(void)
static void __init rockchip_dt_init(void)
{
rockchip_suspend_init();
of_platform_populate(NULL, of_default_bus_match_table, NULL, NULL);
}

static const char * const rockchip_board_dt_compat[] = {
Expand Down
2 changes: 0 additions & 2 deletions arch/arm/mach-s3c24xx/mach-s3c2416-dt.c
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,6 @@

#include <linux/clocksource.h>
#include <linux/irqchip.h>
#include <linux/of_platform.h>
#include <linux/serial_s3c.h>

#include <asm/mach/arch.h>
Expand All @@ -35,7 +34,6 @@ static void __init s3c2416_dt_map_io(void)

static void __init s3c2416_dt_machine_init(void)
{
of_platform_populate(NULL, of_default_bus_match_table, NULL, NULL);
s3c_pm_init();
}

Expand Down
3 changes: 0 additions & 3 deletions arch/arm/mach-s3c64xx/mach-s3c64xx-dt.c
Original file line number Diff line number Diff line change
Expand Up @@ -8,8 +8,6 @@
* published by the Free Software Foundation.
*/

#include <linux/of_platform.h>

#include <asm/mach/arch.h>
#include <asm/mach/map.h>
#include <asm/system_misc.h>
Expand Down Expand Up @@ -48,7 +46,6 @@ static void __init s3c64xx_dt_map_io(void)
static void __init s3c64xx_dt_init_machine(void)
{
samsung_wdt_reset_of_init();
of_platform_populate(NULL, of_default_bus_match_table, NULL, NULL);
}

static void s3c64xx_dt_restart(enum reboot_mode mode, const char *cmd)
Expand Down
3 changes: 0 additions & 3 deletions arch/arm/mach-shmobile/setup-r8a7740.c
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,6 @@
#include <linux/io.h>
#include <linux/irqchip.h>
#include <linux/irqchip/arm-gic.h>
#include <linux/of_platform.h>

#include <asm/mach/map.h>
#include <asm/mach/arch.h>
Expand Down Expand Up @@ -77,8 +76,6 @@ static void __init r8a7740_init_irq_of(void)
static void __init r8a7740_generic_init(void)
{
r8a7740_meram_workaround();

of_platform_populate(NULL, of_default_bus_match_table, NULL, NULL);
}

static const char *const r8a7740_boards_compat_dt[] __initconst = {
Expand Down
2 changes: 0 additions & 2 deletions arch/arm/mach-shmobile/setup-sh73a0.c
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,6 @@
#include <linux/init.h>
#include <linux/interrupt.h>
#include <linux/irq.h>
#include <linux/of_platform.h>
#include <linux/delay.h>
#include <linux/input.h>
#include <linux/io.h>
Expand Down Expand Up @@ -55,7 +54,6 @@ static void __init sh73a0_generic_init(void)
/* Shared attribute override enable, 64K*8way */
l2x0_init(IOMEM(0xf0100000), 0x00400000, 0xc20f0fff);
#endif
of_platform_populate(NULL, of_default_bus_match_table, NULL, NULL);
}

static const char *const sh73a0_boards_compat_dt[] __initconst = {
Expand Down
2 changes: 0 additions & 2 deletions arch/arm/mach-spear/spear1310.c
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,6 @@
#define pr_fmt(fmt) "SPEAr1310: " fmt

#include <linux/amba/pl022.h>
#include <linux/of_platform.h>
#include <linux/pata_arasan_cf_data.h>
#include <asm/mach/arch.h>
#include <asm/mach/map.h>
Expand All @@ -27,7 +26,6 @@

static void __init spear1310_dt_init(void)
{
of_platform_populate(NULL, of_default_bus_match_table, NULL, NULL);
platform_device_register_simple("spear-cpufreq", -1, NULL, 0);
}

Expand Down
1 change: 0 additions & 1 deletion arch/arm/mach-spear/spear1340.c
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,6 @@

static void __init spear1340_dt_init(void)
{
of_platform_populate(NULL, of_default_bus_match_table, NULL, NULL);
platform_device_register_simple("spear-cpufreq", -1, NULL, 0);
}

Expand Down
3 changes: 0 additions & 3 deletions arch/arm/mach-vt8500/vt8500.c
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,6 @@
#include <linux/of.h>
#include <linux/of_address.h>
#include <linux/of_irq.h>
#include <linux/of_platform.h>

#define LEGACY_GPIO_BASE 0xD8110000
#define LEGACY_PMC_BASE 0xD8130000
Expand Down Expand Up @@ -158,8 +157,6 @@ static void __init vt8500_init(void)
pm_power_off = &vt8500_power_off;
else
pr_err("%s: PMC Hibernation register could not be remapped, not enabling power off!\n", __func__);

of_platform_populate(NULL, of_default_bus_match_table, NULL, NULL);
}

static const char * const vt8500_dt_compat[] = {
Expand Down

0 comments on commit 850bea2

Please sign in to comment.