From: Kefeng Wang <wangkefeng.wang@huawei.com> To: <robh+dt@kernel.org>, <linux-kernel@vger.kernel.org>, <linux-arm-kernel@lists.infradead.org> Cc: <arnd@arndb.de>, <shawnguo@kernel.org>, <catalin.marinas@arm.com>, <grant.likely@linaro.org>, <olof@lixom.net>, <linux@arm.linux.org.uk>, <yamada.masahiro@socionext.com>, <guohanjun@huawei.com>, <wangkefeng.wang@huawei.com>, Chris Zankel <chris@zankel.net>, Max Filippov <jcmvbkbc@gmail.com> Subject: [PATCH v5 11/18] xtensa: Remove unnecessary of_platform_populate with default match table Date: Wed, 1 Jun 2016 14:53:04 +0800 [thread overview] Message-ID: <1464763991-26634-12-git-send-email-wangkefeng.wang@huawei.com> (raw) In-Reply-To: <1464763991-26634-1-git-send-email-wangkefeng.wang@huawei.com> 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. Move of_clk_init() into time_init(), then drop xtensa_device_probe() fully. Cc: Chris Zankel <chris@zankel.net> Cc: Max Filippov <jcmvbkbc@gmail.com> Signed-off-by: Kefeng Wang <wangkefeng.wang@huawei.com> --- arch/xtensa/kernel/setup.c | 11 ----------- arch/xtensa/kernel/time.c | 2 ++ 2 files changed, 2 insertions(+), 11 deletions(-) diff --git a/arch/xtensa/kernel/setup.c b/arch/xtensa/kernel/setup.c index 9735691..18af563 100644 --- a/arch/xtensa/kernel/setup.c +++ b/arch/xtensa/kernel/setup.c @@ -22,10 +22,8 @@ #include <linux/bootmem.h> #include <linux/kernel.h> #include <linux/percpu.h> -#include <linux/clk-provider.h> #include <linux/cpu.h> #include <linux/of_fdt.h> -#include <linux/of_platform.h> #if defined(CONFIG_VGA_CONSOLE) || defined(CONFIG_DUMMY_CONSOLE) # include <linux/console.h> @@ -252,15 +250,6 @@ void __init early_init_devtree(void *params) strlcpy(command_line, boot_command_line, COMMAND_LINE_SIZE); } -static int __init xtensa_device_probe(void) -{ - of_clk_init(NULL); - of_platform_populate(NULL, of_default_bus_match_table, NULL, NULL); - return 0; -} - -device_initcall(xtensa_device_probe); - #endif /* CONFIG_OF */ /* diff --git a/arch/xtensa/kernel/time.c b/arch/xtensa/kernel/time.c index b9ad9fe..6ec73c9 100644 --- a/arch/xtensa/kernel/time.c +++ b/arch/xtensa/kernel/time.c @@ -15,6 +15,7 @@ #include <linux/errno.h> #include <linux/sched.h> #include <linux/time.h> +#include <linux/clk-provider.h> #include <linux/clocksource.h> #include <linux/clockchips.h> #include <linux/interrupt.h> @@ -148,6 +149,7 @@ void __init time_init(void) local_timer_setup(0); setup_irq(this_cpu_ptr(&ccount_timer)->evt.irq, &timer_irqaction); sched_clock_register(ccount_sched_clock_read, 32, ccount_freq); + of_clk_init(NULL); clocksource_probe(); } -- 1.7.12.4
WARNING: multiple messages have this Message-ID (diff)
From: wangkefeng.wang@huawei.com (Kefeng Wang) To: linux-arm-kernel@lists.infradead.org Subject: [PATCH v5 11/18] xtensa: Remove unnecessary of_platform_populate with default match table Date: Wed, 1 Jun 2016 14:53:04 +0800 [thread overview] Message-ID: <1464763991-26634-12-git-send-email-wangkefeng.wang@huawei.com> (raw) In-Reply-To: <1464763991-26634-1-git-send-email-wangkefeng.wang@huawei.com> 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. Move of_clk_init() into time_init(), then drop xtensa_device_probe() fully. Cc: Chris Zankel <chris@zankel.net> Cc: Max Filippov <jcmvbkbc@gmail.com> Signed-off-by: Kefeng Wang <wangkefeng.wang@huawei.com> --- arch/xtensa/kernel/setup.c | 11 ----------- arch/xtensa/kernel/time.c | 2 ++ 2 files changed, 2 insertions(+), 11 deletions(-) diff --git a/arch/xtensa/kernel/setup.c b/arch/xtensa/kernel/setup.c index 9735691..18af563 100644 --- a/arch/xtensa/kernel/setup.c +++ b/arch/xtensa/kernel/setup.c @@ -22,10 +22,8 @@ #include <linux/bootmem.h> #include <linux/kernel.h> #include <linux/percpu.h> -#include <linux/clk-provider.h> #include <linux/cpu.h> #include <linux/of_fdt.h> -#include <linux/of_platform.h> #if defined(CONFIG_VGA_CONSOLE) || defined(CONFIG_DUMMY_CONSOLE) # include <linux/console.h> @@ -252,15 +250,6 @@ void __init early_init_devtree(void *params) strlcpy(command_line, boot_command_line, COMMAND_LINE_SIZE); } -static int __init xtensa_device_probe(void) -{ - of_clk_init(NULL); - of_platform_populate(NULL, of_default_bus_match_table, NULL, NULL); - return 0; -} - -device_initcall(xtensa_device_probe); - #endif /* CONFIG_OF */ /* diff --git a/arch/xtensa/kernel/time.c b/arch/xtensa/kernel/time.c index b9ad9fe..6ec73c9 100644 --- a/arch/xtensa/kernel/time.c +++ b/arch/xtensa/kernel/time.c @@ -15,6 +15,7 @@ #include <linux/errno.h> #include <linux/sched.h> #include <linux/time.h> +#include <linux/clk-provider.h> #include <linux/clocksource.h> #include <linux/clockchips.h> #include <linux/interrupt.h> @@ -148,6 +149,7 @@ void __init time_init(void) local_timer_setup(0); setup_irq(this_cpu_ptr(&ccount_timer)->evt.irq, &timer_irqaction); sched_clock_register(ccount_sched_clock_read, 32, ccount_freq); + of_clk_init(NULL); clocksource_probe(); } -- 1.7.12.4
next prev parent reply other threads:[~2016-06-01 6:50 UTC|newest] Thread overview: 46+ messages / expand[flat|nested] mbox.gz Atom feed top 2016-06-01 6:52 [PATCH v5 00/18] Cleanup for populate default bus Kefeng Wang 2016-06-01 6:52 ` Kefeng Wang 2016-06-01 6:52 ` [PATCH v5 01/18] of/platform: Add common method to " Kefeng Wang 2016-06-01 6:52 ` Kefeng Wang 2016-06-01 6:52 ` [PATCH v5 02/18] arc: Remove unnecessary of_platform_populate with default match table Kefeng Wang 2016-06-01 6:52 ` Kefeng Wang 2016-06-01 6:52 ` [PATCH v5 03/18] arm: " Kefeng Wang 2016-06-01 6:52 ` Kefeng Wang 2016-06-01 6:52 ` [PATCH v5 04/18] arm64: " Kefeng Wang 2016-06-01 6:52 ` Kefeng Wang 2016-06-01 8:16 ` Catalin Marinas 2016-06-01 8:16 ` Catalin Marinas 2016-06-01 6:52 ` [PATCH v5 05/18] c6x: " Kefeng Wang 2016-06-01 6:52 ` Kefeng Wang 2016-06-01 6:52 ` [PATCH v5 06/18] cris: " Kefeng Wang 2016-06-01 6:52 ` Kefeng Wang 2016-06-01 6:53 ` [PATCH v5 07/18] metag: " Kefeng Wang 2016-06-01 6:53 ` Kefeng Wang 2016-06-01 6:53 ` [PATCH v5 08/18] mips: " Kefeng Wang 2016-06-01 6:53 ` Kefeng Wang 2016-06-01 6:53 ` [PATCH v5 09/18] nios2: " Kefeng Wang 2016-06-01 6:53 ` Kefeng Wang 2016-06-01 6:53 ` [PATCH v5 10/18] sh: " Kefeng Wang 2016-06-01 6:53 ` Kefeng Wang 2016-06-01 6:53 ` Kefeng Wang [this message] 2016-06-01 6:53 ` [PATCH v5 11/18] xtensa: " Kefeng Wang 2016-06-01 6:53 ` [PATCH v5 12/18] arm: use of_platform_default_populate() to populate Kefeng Wang 2016-06-01 6:53 ` Kefeng Wang 2016-06-01 6:53 ` [PATCH v5 13/18] mips: use of_platform_default_populate() to populate default bus Kefeng Wang 2016-06-01 6:53 ` Kefeng Wang 2016-06-01 6:53 ` [PATCH v5 14/18] bus: imx-weim: " Kefeng Wang 2016-06-01 6:53 ` Kefeng Wang 2016-06-01 6:53 ` [PATCH v5 15/18] bus: uniphier-system-bus: " Kefeng Wang 2016-06-01 6:53 ` Kefeng Wang 2016-06-01 6:53 ` [PATCH v5 16/18] memory: omap-gpmc: " Kefeng Wang 2016-06-01 6:53 ` Kefeng Wang 2016-06-01 6:53 ` [PATCH v5 17/18] of: unittest: " Kefeng Wang 2016-06-01 6:53 ` Kefeng Wang 2016-06-01 6:53 ` [PATCH v5 18/18] Revert "of/platform: export of_default_bus_match_table" Kefeng Wang 2016-06-01 6:53 ` Kefeng Wang 2016-06-08 15:42 ` [PATCH v5 00/18] Cleanup for populate default bus Rob Herring 2016-06-08 15:42 ` Rob Herring 2016-06-15 14:02 ` Joerg Roedel 2016-06-15 14:02 ` Joerg Roedel 2016-06-24 20:36 ` Rob Herring 2016-06-24 20:36 ` Rob Herring
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=1464763991-26634-12-git-send-email-wangkefeng.wang@huawei.com \ --to=wangkefeng.wang@huawei.com \ --cc=arnd@arndb.de \ --cc=catalin.marinas@arm.com \ --cc=chris@zankel.net \ --cc=grant.likely@linaro.org \ --cc=guohanjun@huawei.com \ --cc=jcmvbkbc@gmail.com \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux@arm.linux.org.uk \ --cc=olof@lixom.net \ --cc=robh+dt@kernel.org \ --cc=shawnguo@kernel.org \ --cc=yamada.masahiro@socionext.com \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.