[PATCH v2 1/1] sandbox: fix building with CONFIG_SPL_TIMER=y

Building sandbox_defconfig with CONFIG_SPL_TIMER=y results in an error
include/dm/platdata.h:63:33: error: static assertion failed: "Cannot use U_BOOT_DRVINFO with of-platdata. Please use devicetree instead"
Add a missing condition in the sandbox driver.
Signed-off-by: Heinrich Schuchardt heinrich.schuchardt@canonical.com --- v2: don't create U_BOOT_DRVINFO for DT_PLAT_C (as requested in review comment by Simon) --- drivers/timer/sandbox_timer.c | 2 ++ 1 file changed, 2 insertions(+)
diff --git a/drivers/timer/sandbox_timer.c b/drivers/timer/sandbox_timer.c index c846bfb9f1..1da7e0c3a7 100644 --- a/drivers/timer/sandbox_timer.c +++ b/drivers/timer/sandbox_timer.c @@ -66,6 +66,8 @@ U_BOOT_DRIVER(sandbox_timer) = { };
/* This is here in case we don't have a device tree */ +#if !CONFIG_IS_ENABLED(OF_PLATDATA) U_BOOT_DRVINFO(sandbox_timer_non_fdt) = { .name = "sandbox_timer", }; +#endif

On Tue, 21 Feb 2023 at 17:39, Heinrich Schuchardt heinrich.schuchardt@canonical.com wrote:
Building sandbox_defconfig with CONFIG_SPL_TIMER=y results in an error
include/dm/platdata.h:63:33: error: static assertion failed: "Cannot use U_BOOT_DRVINFO with of-platdata. Please use devicetree instead"
Add a missing condition in the sandbox driver.
Signed-off-by: Heinrich Schuchardt heinrich.schuchardt@canonical.com
v2: don't create U_BOOT_DRVINFO for DT_PLAT_C (as requested in review comment by Simon)
drivers/timer/sandbox_timer.c | 2 ++ 1 file changed, 2 insertions(+)
Reviewed-by: Simon Glass sjg@chromium.org

On Tue, 21 Feb 2023 at 17:39, Heinrich Schuchardt heinrich.schuchardt@canonical.com wrote:
Building sandbox_defconfig with CONFIG_SPL_TIMER=y results in an error
include/dm/platdata.h:63:33: error: static assertion failed: "Cannot use U_BOOT_DRVINFO with of-platdata. Please use devicetree instead"
Add a missing condition in the sandbox driver.
Signed-off-by: Heinrich Schuchardt heinrich.schuchardt@canonical.com
v2: don't create U_BOOT_DRVINFO for DT_PLAT_C (as requested in review comment by Simon)
drivers/timer/sandbox_timer.c | 2 ++ 1 file changed, 2 insertions(+)
Reviewed-by: Simon Glass sjg@chromium.org
Applied to u-boot-dm, thanks!
participants (2)
-
Heinrich Schuchardt
-
Simon Glass