]> git.dujemihanovic.xyz Git - u-boot.git/commitdiff
event: Convert arch_cpu_init_dm() to use events
authorSimon Glass <sjg@chromium.org>
Fri, 4 Mar 2022 15:43:05 +0000 (08:43 -0700)
committerTom Rini <trini@konsulko.com>
Thu, 10 Mar 2022 13:28:36 +0000 (08:28 -0500)
Instead of a special function, send an event after driver model is inited
and adjust the boards which use this function.

Signed-off-by: Simon Glass <sjg@chromium.org>
26 files changed:
arch/Kconfig
arch/arm/Kconfig
arch/arm/mach-imx/imx8/cpu.c
arch/arm/mach-imx/imx8m/soc.c
arch/arm/mach-imx/imx8ulp/soc.c
arch/arm/mach-omap2/am33xx/board.c
arch/arm/mach-omap2/hwinit-common.c
arch/mips/Kconfig
arch/mips/mach-pic32/cpu.c
arch/nios2/cpu/cpu.c
arch/riscv/cpu/cpu.c
arch/riscv/include/asm/system.h
arch/riscv/lib/spl.c
arch/x86/cpu/baytrail/cpu.c
arch/x86/cpu/broadwell/cpu.c
arch/x86/cpu/ivybridge/cpu.c
arch/x86/cpu/quark/quark.c
arch/x86/include/asm/fsp2/fsp_api.h
arch/x86/lib/fsp2/fsp_init.c
arch/x86/lib/spl.c
arch/x86/lib/tpl.c
common/board_f.c
common/event.c
drivers/core/root.c
include/event.h
include/init.h

index e6191446a357c3250ee06406c01755cae406ad9a..1b35fda64cc81c293d0af06a3361a7a89f8bf497 100644 (file)
@@ -94,6 +94,7 @@ config NIOS2
        bool "Nios II architecture"
        select CPU
        select DM
+       imply DM_EVENT
        select OF_CONTROL
        select SUPPORT_OF_CONTROL
        imply CMD_DM
@@ -113,6 +114,7 @@ config RISCV
        select DM
        imply DM_SERIAL
        imply DM_ETH
+       imply DM_EVENT
        imply DM_MMC
        imply DM_SPI
        imply DM_SPI_FLASH
@@ -238,6 +240,7 @@ config X86
        imply CMD_SF_TEST
        imply CMD_ZBOOT
        imply DM_ETH
+       imply DM_EVENT
        imply DM_GPIO
        imply DM_KEYBOARD
        imply DM_MMC
index 8c7f31769708b33ba5b1b14d84a671750ba9b519..a6f2e7a1008ac6962df4a34b33a40584812032d5 100644 (file)
@@ -774,6 +774,7 @@ config ARCH_OMAP2PLUS
        select SUPPORT_SPL
        imply TI_SYSC if DM && OF_CONTROL
        imply FIT
+       imply DM_EVENT
 
 config ARCH_MESON
        bool "Amlogic Meson"
@@ -818,6 +819,7 @@ config ARCH_IMX8
        select MACH_IMX
        select OF_CONTROL
        select ENABLE_ARM_SOC_BOOT0_HOOK
+       imply DM_EVENT
 
 config ARCH_IMX8M
        bool "NXP i.MX8M platform"
@@ -831,6 +833,7 @@ config ARCH_IMX8M
        select DM
        select SUPPORT_SPL
        imply CMD_DM
+       imply DM_EVENT
 
 config ARCH_IMX8ULP
        bool "NXP i.MX8ULP platform"
@@ -841,6 +844,7 @@ config ARCH_IMX8ULP
        select SUPPORT_SPL
        select GPIO_EXTRA_HEADER
        imply CMD_DM
+       imply DM_EVENT
 
 config ARCH_IMXRT
        bool "NXP i.MXRT platform"
index ee5cc479039bc523bc292cb871c9bedd09d476cf..359f8c796eb35b41e920af43effe6a1cb79554b8 100644 (file)
@@ -8,6 +8,7 @@
 #include <cpu.h>
 #include <cpu_func.h>
 #include <dm.h>
+#include <event.h>
 #include <init.h>
 #include <log.h>
 #include <asm/cache.h>
@@ -66,7 +67,7 @@ int arch_cpu_init(void)
        return 0;
 }
 
-int arch_cpu_init_dm(void)
+static int imx8_init_mu(void *ctx, struct event *event)
 {
        struct udevice *devp;
        int node, ret;
@@ -88,6 +89,7 @@ int arch_cpu_init_dm(void)
 
        return 0;
 }
+EVENT_SPY(EVT_DM_POST_INIT, imx8_init_mu);
 
 int print_bootinfo(void)
 {
index 1a5a391443dea5cc0c1053846de5ef8e25d1c2d0..838f0a37496829f54a4de51cfa508dec6725c3ce 100644 (file)
@@ -7,6 +7,7 @@
 
 #include <common.h>
 #include <cpu_func.h>
+#include <event.h>
 #include <init.h>
 #include <log.h>
 #include <asm/arch/imx-regs.h>
@@ -494,7 +495,7 @@ static void imx_set_wdog_powerdown(bool enable)
        writew(enable, &wdog3->wmcr);
 }
 
-int arch_cpu_init_dm(void)
+static int imx8m_check_clock(void *ctx, struct event *event)
 {
        struct udevice *dev;
        int ret;
@@ -511,6 +512,7 @@ int arch_cpu_init_dm(void)
 
        return 0;
 }
+EVENT_SPY(EVT_DM_POST_INIT, imx8m_check_clock);
 
 int arch_cpu_init(void)
 {
index 934b0ef038ce83dc2830d5458c975cc50869668e..e6d417ed48b218f54d4707087b195f045262f4c7 100644 (file)
@@ -11,6 +11,7 @@
 #include <asm/mach-imx/boot_mode.h>
 #include <asm/global_data.h>
 #include <efi_loader.h>
+#include <event.h>
 #include <spl.h>
 #include <asm/arch/rdc.h>
 #include <asm/arch/s400_api.h>
@@ -569,7 +570,7 @@ int arch_cpu_init(void)
        return 0;
 }
 
-int arch_cpu_init_dm(void)
+static int imx8ulp_check_mu(void *ctx, struct event *event)
 {
        struct udevice *devp;
        int node, ret;
@@ -584,6 +585,7 @@ int arch_cpu_init_dm(void)
 
        return 0;
 }
+EVENT_SPY(EVT_DM_POST_INIT, imx8ulp_check_mu);
 
 #if defined(CONFIG_SPL_BUILD)
 __weak void __noreturn jump_to_image_no_args(struct spl_image_info *spl_image)
index c44667668e9de5eca7672148be20bd67b5f54f47..bcc907ce36263601b7cf2cdee40ed7d5ecfe5755 100644 (file)
@@ -11,6 +11,7 @@
 #include <dm.h>
 #include <debug_uart.h>
 #include <errno.h>
+#include <event.h>
 #include <init.h>
 #include <net.h>
 #include <ns16550.h>
@@ -596,7 +597,7 @@ void board_init_f(ulong dummy)
 
 #endif
 
-int arch_cpu_init_dm(void)
+static int am33xx_dm_post_init(void *ctx, struct event *event)
 {
        hw_data_init();
 #if !CONFIG_IS_ENABLED(SKIP_LOWLEVEL_INIT)
@@ -604,3 +605,4 @@ int arch_cpu_init_dm(void)
 #endif
        return 0;
 }
+EVENT_SPY(EVT_DM_POST_INIT, am33xx_dm_post_init);
index 3da50f974dc0c0daa4a2f2c6ee998502080d95d5..c4a8eabc3ebe1e7e8645380e0848a628498fd13f 100644 (file)
@@ -12,6 +12,7 @@
  */
 #include <common.h>
 #include <debug_uart.h>
+#include <event.h>
 #include <fdtdec.h>
 #include <init.h>
 #include <spl.h>
@@ -239,11 +240,13 @@ void board_init_f(ulong dummy)
 }
 #endif
 
-int arch_cpu_init_dm(void)
+static int omap2_system_init(void *ctx, struct event *event)
 {
        early_system_init();
+
        return 0;
 }
+EVENT_SPY(EVT_DM_POST_INIT, omap2_system_init);
 
 /*
  * Routine: wait_for_command_complete
index 28234aa0bb6c4f02a22e5fe3695080e94f19d8de..06cae68ee57668de90f5469861463af646288c3a 100644 (file)
@@ -130,6 +130,7 @@ config MACH_PIC32
 config TARGET_BOSTON
        bool "Support Boston"
        select DM
+       imply DM_EVENT
        select DM_SERIAL
        select MIPS_CM
        select SYS_CACHE_SHIFT_6
index eac2fe5f8c949165f528c36249857480e5cbb392..de449e3c6a20f382b108548495817bf6bc09dd26 100644 (file)
@@ -7,6 +7,7 @@
 #include <common.h>
 #include <clk.h>
 #include <dm.h>
+#include <event.h>
 #include <init.h>
 #include <malloc.h>
 #include <asm/global_data.h>
@@ -95,12 +96,13 @@ static void prefetch_init(void)
 }
 
 /* arch specific CPU init after DM */
-int arch_cpu_init_dm(void)
+static int pic32_flash_prefetch(void *ctx, struct event *event)
 {
        /* flash prefetch */
        prefetch_init();
        return 0;
 }
+EVENT_SPY(EVT_DM_POST_INIT, pic32_flash_prefetch);
 
 /* Un-gate DDR2 modules (gated by default) */
 static void ddr2_pmd_ungate(void)
index b55c8fbc5841715680c18825540d8839bfc1f5c5..4dd9c10faa5ae5835e5c83c4afc340e567b3add7 100644 (file)
@@ -10,6 +10,7 @@
 #include <cpu_func.h>
 #include <dm.h>
 #include <errno.h>
+#include <event.h>
 #include <init.h>
 #include <irq_func.h>
 #include <asm/cache.h>
@@ -63,7 +64,7 @@ static void copy_exception_trampoline(void)
 }
 #endif
 
-int arch_cpu_init_dm(void)
+static int nios_cpu_setup(void *ctx, struct event *event)
 {
        struct udevice *dev;
        int ret;
@@ -79,6 +80,7 @@ int arch_cpu_init_dm(void)
 
        return 0;
 }
+EVENT_SPY(EVT_DM_POST_INIT, nios_cpu_setup);
 
 static int altera_nios2_get_desc(const struct udevice *dev, char *buf,
                                 int size)
index 8d90c5e6b8aa5ba8d5bd716727ef82705e0eca6f..3ffcbbd23fa62f4b198ccfd2531fbed0139fe5fa 100644 (file)
@@ -7,9 +7,11 @@
 #include <cpu.h>
 #include <dm.h>
 #include <dm/lists.h>
+#include <event.h>
 #include <init.h>
 #include <log.h>
 #include <asm/encoding.h>
+#include <asm/system.h>
 #include <dm/uclass-internal.h>
 #include <linux/bitops.h>
 
@@ -81,7 +83,7 @@ static void dummy_pending_ipi_clear(ulong hart, ulong arg0, ulong arg1)
 }
 #endif
 
-int arch_cpu_init_dm(void)
+int riscv_cpu_setup(void *ctx, struct event *event)
 {
        int ret;
 
@@ -133,6 +135,7 @@ int arch_cpu_init_dm(void)
 
        return 0;
 }
+EVENT_SPY(EVT_DM_POST_INIT, riscv_cpu_setup);
 
 int arch_early_init_r(void)
 {
index a3404758235a30d9aa7bff542413943c6c4f7e48..9d8e43e394269883bec2883bb1e0e94334473419 100644 (file)
@@ -7,6 +7,8 @@
 #ifndef __ASM_RISCV_SYSTEM_H
 #define __ASM_RISCV_SYSTEM_H
 
+struct event;
+
 /*
  * Interrupt configuring macros.
  *
@@ -14,4 +16,7 @@
  *
  */
 
+/* Hook to set up the CPU (called from SPL too) */
+int riscv_cpu_setup(void *ctx, struct event *event);
+
 #endif /* __ASM_RISCV_SYSTEM_H */
index 8baee07beac1eceb33c45b9021a796fc7113ce2f..f4d3b67e5dd08c7e4d6a95edeae7bf5770bc6098 100644 (file)
@@ -11,6 +11,7 @@
 #include <spl.h>
 #include <asm/global_data.h>
 #include <asm/smp.h>
+#include <asm/system.h>
 
 DECLARE_GLOBAL_DATA_PTR;
 
@@ -27,7 +28,7 @@ __weak void board_init_f(ulong dummy)
        if (ret)
                panic("spl_early_init() failed: %d\n", ret);
 
-       arch_cpu_init_dm();
+       riscv_cpu_setup(NULL, NULL);
 
        preloader_console_init();
 
index 309a50a11611d716b4ab866210f0ae285d0704c6..68bf40ba8e818d7240bece418c1136c7fe570301 100644 (file)
@@ -8,6 +8,7 @@
 #include <common.h>
 #include <cpu.h>
 #include <dm.h>
+#include <event.h>
 #include <init.h>
 #include <log.h>
 #include <pci.h>
@@ -44,7 +45,7 @@ static void hsuart_clock_set(void *base)
  * Configure the internal clock of both SIO HS-UARTs, if they are enabled
  * via FSP
  */
-int arch_cpu_init_dm(void)
+static int baytrail_uart_init(void *ctx, struct event *event)
 {
        struct udevice *dev;
        void *base;
@@ -63,6 +64,7 @@ int arch_cpu_init_dm(void)
 
        return 0;
 }
+EVENT_SPY(EVT_DM_POST_INIT, baytrail_uart_init);
 
 static void set_max_freq(void)
 {
index 3832a97f2c7311181b8a25ce116dae8909a376bc..2adcf4b242c5a37e94b298566cc3512dd8cac95a 100644 (file)
@@ -8,6 +8,7 @@
 #include <common.h>
 #include <dm.h>
 #include <cpu.h>
+#include <event.h>
 #include <init.h>
 #include <log.h>
 #include <asm/cpu.h>
@@ -24,7 +25,7 @@
 #include <asm/arch/pch.h>
 #include <asm/arch/rcb.h>
 
-int arch_cpu_init_dm(void)
+static int broadwell_init_cpu(void *ctx, struct event *event)
 {
        struct udevice *dev;
        int ret;
@@ -41,6 +42,7 @@ int arch_cpu_init_dm(void)
 
        return 0;
 }
+EVENT_SPY(EVT_DM_POST_INIT, broadwell_init_cpu);
 
 void set_max_freq(void)
 {
index a02f4f9600eff28b7ea8c597bae0edf066de9c6c..cffc5d5b1d861c14c79d1b0fe0557fe3abfa5cca 100644 (file)
@@ -14,6 +14,7 @@
 #include <cpu_func.h>
 #include <dm.h>
 #include <errno.h>
+#include <event.h>
 #include <fdtdec.h>
 #include <init.h>
 #include <log.h>
@@ -53,7 +54,7 @@ int arch_cpu_init(void)
        return x86_cpu_init_f();
 }
 
-int arch_cpu_init_dm(void)
+static int ivybridge_cpu_init(void *ctx, struct event *ev)
 {
        struct pci_controller *hose;
        struct udevice *bus, *dev;
@@ -85,6 +86,7 @@ int arch_cpu_init_dm(void)
 
        return 0;
 }
+EVENT_SPY(EVT_DM_POST_INIT, ivybridge_cpu_init);
 
 #define PCH_EHCI0_TEMP_BAR0 0xe8000000
 #define PCH_EHCI1_TEMP_BAR0 0xe8000400
index 30b4711b9a59569d73f68c50d03607978d1e2117..e016fae04f9668b79168874e4e880b7a123aa307 100644 (file)
@@ -5,6 +5,7 @@
 
 #include <common.h>
 #include <cpu_func.h>
+#include <event.h>
 #include <init.h>
 #include <mmc.h>
 #include <asm/cache.h>
@@ -247,7 +248,7 @@ int arch_cpu_init(void)
        return 0;
 }
 
-int arch_cpu_init_dm(void)
+static int quark_init_pcie(void *ctx, struct event *event)
 {
        /*
         * Initialize PCIe controller
@@ -262,6 +263,7 @@ int arch_cpu_init_dm(void)
 
        return 0;
 }
+EVENT_SPY(EVT_DM_POST_INIT, quark_init_pcie);
 
 int checkcpu(void)
 {
index dccbfa45a1b043a9a9be888800a6fe66faa97c29..ca3f6848b61b6653633aacb0e867e515a55efab5 100644 (file)
@@ -60,4 +60,12 @@ int fsp_silicon_init(bool s3wake, bool use_spi_flash);
 
 typedef asmlinkage int (*fsp_silicon_init_func)(struct fsps_upd *params);
 
+/**
+ *  fsp_setup_pinctrl() - Set up the pinctrl for FSP
+ *
+ * @ctx: Event context (not used)
+ * @event: Event information (not used)
+ */
+int fsp_setup_pinctrl(void *ctx, struct event *event);
+
 #endif
index 5afdce1e0d468c48e527ccc372ac652b207ded53..b15926e8247b85c9cdc956a80a0ccbb7f7b74d54 100644 (file)
@@ -9,6 +9,7 @@
 #include <bootstage.h>
 #include <cbfs.h>
 #include <dm.h>
+#include <event.h>
 #include <init.h>
 #include <log.h>
 #include <spi.h>
@@ -18,7 +19,7 @@
 #include <dm/uclass-internal.h>
 #include <asm/fsp2/fsp_internal.h>
 
-int arch_cpu_init_dm(void)
+int fsp_setup_pinctrl(void *ctx, struct event *event)
 {
        struct udevice *dev;
        ofnode node;
@@ -41,6 +42,7 @@ int arch_cpu_init_dm(void)
 
        return ret;
 }
+EVENT_SPY(EVT_DM_POST_INIT, fsp_setup_pinctrl);
 
 #if !defined(CONFIG_TPL_BUILD)
 binman_sym_declare(ulong, intel_fsp_m, image_pos);
index b18c1cd6092126eb2f4ee6b61a8a5f4cff72d63d..2d50c62964cf2828f706b0b21166a853da51884a 100644 (file)
@@ -17,6 +17,7 @@
 #include <syscon.h>
 #include <asm/cpu.h>
 #include <asm/cpu_common.h>
+#include <asm/fsp2/fsp_api.h>
 #include <asm/global_data.h>
 #include <asm/mrccache.h>
 #include <asm/mtrr.h>
@@ -27,7 +28,7 @@
 
 DECLARE_GLOBAL_DATA_PTR;
 
-__weak int arch_cpu_init_dm(void)
+__weak int fsp_setup_pinctrl(void *ctx, struct event *event)
 {
        return 0;
 }
@@ -89,7 +90,7 @@ static int x86_spl_init(void)
                return ret;
        }
 #ifndef CONFIG_TPL
-       ret = arch_cpu_init_dm();
+       ret = fsp_setup_pinctrl(NULL, NULL);
        if (ret) {
                debug("%s: arch_cpu_init_dm() failed\n", __func__);
                return ret;
index 5b57e53c2ddf1b1db37e41b25f8f7519777eb5af..18b05b2f67208de0e2561cdd94036619c5ed8257 100644 (file)
 
 DECLARE_GLOBAL_DATA_PTR;
 
-__weak int arch_cpu_init_dm(void)
-{
-       return 0;
-}
-
 static int x86_tpl_init(void)
 {
        int ret;
@@ -44,11 +39,6 @@ static int x86_tpl_init(void)
                debug("%s: arch_cpu_init() failed\n", __func__);
                return ret;
        }
-       ret = arch_cpu_init_dm();
-       if (ret) {
-               debug("%s: arch_cpu_init_dm() failed\n", __func__);
-               return ret;
-       }
        preloader_console_init();
 
        return 0;
index 0ef34c7575937c0a4a162984eb47f41002680f03..5b655ad6efe4ce36a895c17918e9aa5f51915795 100644 (file)
@@ -803,11 +803,6 @@ __weak int reserve_arch(void)
        return 0;
 }
 
-__weak int arch_cpu_init_dm(void)
-{
-       return 0;
-}
-
 __weak int checkcpu(void)
 {
        return 0;
@@ -848,7 +843,6 @@ static const init_fnc_t init_sequence_f[] = {
        arch_cpu_init,          /* basic arch cpu dependent setup */
        mach_cpu_init,          /* SoC/machine dependent CPU setup */
        initf_dm,
-       arch_cpu_init_dm,
 #if defined(CONFIG_BOARD_EARLY_INIT_F)
        board_early_init_f,
 #endif
index 4270809d49636bce17b317aae022505878adecbd..9d67a060a021f73763a3dbc75b9c44602446ba75 100644 (file)
@@ -26,6 +26,7 @@ const char *const type_name[] = {
        "test",
 
        /* Events related to driver model */
+       "dm_post_init",
        "dm_pre_probe",
        "dm_post_probe",
        "dm_pre_remove",
index e3f87956d866e1debb0e496843b92669e092a0dd..8efb4256b27e8c83944777531de877f801c5f6ee 100644 (file)
@@ -404,6 +404,11 @@ int dm_init_and_scan(bool pre_reloc_only)
                        return ret;
                }
        }
+       if (CONFIG_IS_ENABLED(DM_EVENT)) {
+               ret = event_notify_null(EVT_DM_POST_INIT);
+               if (ret)
+                       return log_msg_ret("ev", ret);
+       }
 
        return 0;
 }
index 6b347e92f0874b1ebc9e06dc5bbe6eb127bafb0e..78a42374acbfbb598c59467301638efe1c40bc9f 100644 (file)
@@ -20,6 +20,7 @@ enum event_t {
        EVT_TEST,
 
        /* Events related to driver model */
+       EVT_DM_POST_INIT,
        EVT_DM_PRE_PROBE,
        EVT_DM_POST_PROBE,
        EVT_DM_PRE_REMOVE,
index c03b29bb0dbda1947c4b48c92e97fcb6e4c63859..74496500d290338fded55c99e485134d847ef345 100644 (file)
@@ -45,17 +45,6 @@ void board_init_f(ulong dummy);
  */
 int arch_cpu_init(void);
 
-/**
- * arch_cpu_init_dm() - init CPU after driver model is available
- *
- * This is called immediately after driver model is available before
- * relocation. This is similar to arch_cpu_init() but is able to reference
- * devices
- *
- * Return: 0 if OK, -ve on error
- */
-int arch_cpu_init_dm(void);
-
 /**
  * mach_cpu_init() - SoC/machine dependent CPU setup
  *