[U-Boot] [PATCH v2 3/3] arm: mediatek: remove arch_misc_init

[U-Boot] [PATCH v2 3/3] arm: mediatek: remove arch_misc_init

[Weijie Gao]

The watchdog of mediatek chips is enabled by bootrom before u-boot is
running. Previously we choose to enable the wdt driver only to disable the
watchdog hardware.

Now wdt service is enabled by default. The function arch_misc_init which is
only used to disable wdt is no longer needed.

Reviewed-by: Stefan Roese <sr@denx.de>
Reviewed-by: Ryder Lee <ryder.lee@mediatek.com>
Signed-off-by: Weijie Gao <weijie.gao@mediatek.com>
---
Changes since v1: none
---
 arch/arm/mach-mediatek/Kconfig |  3 ---
 arch/arm/mach-mediatek/cpu.c   | 12 ------------
 2 files changed, 15 deletions(-)

diff --git a/arch/arm/mach-mediatek/Kconfig b/arch/arm/mach-mediatek/Kconfig
index b5e91d4a7d..60aef15f15 100644
--- a/arch/arm/mach-mediatek/Kconfig
+++ b/arch/arm/mach-mediatek/Kconfig
@@ -12,7 +12,6 @@ choice
 config TARGET_MT7623
 	bool "MediaTek MT7623 SoC"
 	select CPU_V7A
-	select ARCH_MISC_INIT
 	help
 	  The MediaTek MT7623 is a ARM-based SoC with a quad-core Cortex-A7
 	  including NEON and GPU, Mali-450 graphics, several DDR3 options,
@@ -25,7 +24,6 @@ config TARGET_MT7629
 	bool "MediaTek MT7629 SoC"
 	select CPU_V7A
 	select SPL
-	select ARCH_MISC_INIT
 	help
 	  The MediaTek MT7629 is a ARM-based SoC with a dual-core Cortex-A7
 	  including DDR3, crypto engine, 3x3 11n/ac Wi-Fi, Gigabit Ethernet,
@@ -34,7 +32,6 @@ config TARGET_MT7629
 config TARGET_MT8516
 	bool "MediaTek MT8516 SoC"
 	select ARM64
-	select ARCH_MISC_INIT
 	help
 	  The MediaTek MT8516 is a ARM64-based SoC with a quad-core Cortex-A35.
 	  including UART, SPI, USB2.0 and OTG, SD and MMC cards, NAND, PWM,
diff --git a/arch/arm/mach-mediatek/cpu.c b/arch/arm/mach-mediatek/cpu.c
index b37e299b74..1923c9e527 100644
--- a/arch/arm/mach-mediatek/cpu.c
+++ b/arch/arm/mach-mediatek/cpu.c
@@ -8,18 +8,6 @@
 #include <wdt.h>
 #include <dm/uclass-internal.h>
 
-int arch_misc_init(void)
-{
-	struct udevice *wdt;
-	int ret;
-
-	ret = uclass_first_device_err(UCLASS_WDT, &wdt);
-	if (!ret)
-		wdt_stop(wdt);
-
-	return 0;
-}
-
 int arch_cpu_init(void)
 {
 	icache_enable();
-- 
2.18.0

[U-Boot] [PATCH v2 3/3] arm: mediatek: remove arch_misc_init

[Tom Rini]

On Thu, May 16, 2019 at 05:19:50PM +0800, Weijie Gao wrote:

> The watchdog of mediatek chips is enabled by bootrom before u-boot is
> running. Previously we choose to enable the wdt driver only to disable the
> watchdog hardware.
> 
> Now wdt service is enabled by default. The function arch_misc_init which is
> only used to disable wdt is no longer needed.
> 
> Reviewed-by: Stefan Roese <sr@denx.de>
> Reviewed-by: Ryder Lee <ryder.lee@mediatek.com>
> Signed-off-by: Weijie Gao <weijie.gao@mediatek.com>

Applied to u-boot/master, thanks!

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20190707/701fc898/attachment.sig>