* [U-Boot] [PATCH] rockchip: configs: use rockchip-common.h for rk3368
@ 2017-10-06 8:12 Klaus Goger
2017-10-06 8:15 ` [U-Boot] " Philipp Tomsich
` (2 more replies)
0 siblings, 3 replies; 4+ messages in thread
From: Klaus Goger @ 2017-10-06 8:12 UTC (permalink / raw)
To: u-boot
rockchip-common.h already defines values that are missing from
rk3368_common.h
For example BOOT_TARGET_DEVICES was defined empty and therefore
distroboot had no boot targets.
Signed-off-by: Klaus Goger <klaus.goger@theobroma-systems.com>
---
include/configs/rk3368_common.h | 6 ++----
1 file changed, 2 insertions(+), 4 deletions(-)
diff --git a/include/configs/rk3368_common.h b/include/configs/rk3368_common.h
index 8b4155f37c..b643cc2464 100644
--- a/include/configs/rk3368_common.h
+++ b/include/configs/rk3368_common.h
@@ -7,6 +7,8 @@
#ifndef __CONFIG_RK3368_COMMON_H
#define __CONFIG_RK3368_COMMON_H
+#include "rockchip-common.h"
+
#define CONFIG_SYS_CACHELINE_SIZE 64
#include <asm/arch/hardware.h>
@@ -44,10 +46,6 @@
"kernel_addr_r=0x280000\0" \
"ramdisk_addr_r=0x5bf0000\0"
-#include <config_distro_defaults.h>
-
-#define BOOT_TARGET_DEVICES(func)
-
#include <config_distro_bootcmd.h>
#define CONFIG_EXTRA_ENV_SETTINGS \
--
2.11.0
^ permalink raw reply related [flat|nested] 4+ messages in thread
* [U-Boot] rockchip: configs: use rockchip-common.h for rk3368
2017-10-06 8:12 [U-Boot] [PATCH] rockchip: configs: use rockchip-common.h for rk3368 Klaus Goger
@ 2017-10-06 8:15 ` Philipp Tomsich
2017-10-06 8:16 ` Philipp Tomsich
2017-10-06 8:46 ` [U-Boot] [PATCH] " Dr. Philipp Tomsich
2 siblings, 0 replies; 4+ messages in thread
From: Philipp Tomsich @ 2017-10-06 8:15 UTC (permalink / raw)
To: u-boot
> rockchip-common.h already defines values that are missing from
> rk3368_common.h
>
> For example BOOT_TARGET_DEVICES was defined empty and therefore
> distroboot had no boot targets.
>
> Signed-off-by: Klaus Goger <klaus.goger@theobroma-systems.com>
> ---
>
> include/configs/rk3368_common.h | 6 ++----
> 1 file changed, 2 insertions(+), 4 deletions(-)
>
Acked-by: Philipp Tomsich <philipp.tomsich@theobroma-systems.com>
^ permalink raw reply [flat|nested] 4+ messages in thread
* [U-Boot] rockchip: configs: use rockchip-common.h for rk3368
2017-10-06 8:12 [U-Boot] [PATCH] rockchip: configs: use rockchip-common.h for rk3368 Klaus Goger
2017-10-06 8:15 ` [U-Boot] " Philipp Tomsich
@ 2017-10-06 8:16 ` Philipp Tomsich
2017-10-06 8:46 ` [U-Boot] [PATCH] " Dr. Philipp Tomsich
2 siblings, 0 replies; 4+ messages in thread
From: Philipp Tomsich @ 2017-10-06 8:16 UTC (permalink / raw)
To: u-boot
> rockchip-common.h already defines values that are missing from
> rk3368_common.h
>
> For example BOOT_TARGET_DEVICES was defined empty and therefore
> distroboot had no boot targets.
>
> Signed-off-by: Klaus Goger <klaus.goger@theobroma-systems.com>
> ---
>
> include/configs/rk3368_common.h | 6 ++----
> 1 file changed, 2 insertions(+), 4 deletions(-)
>
Reviewed-by: Philipp Tomsich <philipp.tomsich@theobroma-systems.com>
^ permalink raw reply [flat|nested] 4+ messages in thread
* [U-Boot] [PATCH] rockchip: configs: use rockchip-common.h for rk3368
2017-10-06 8:12 [U-Boot] [PATCH] rockchip: configs: use rockchip-common.h for rk3368 Klaus Goger
2017-10-06 8:15 ` [U-Boot] " Philipp Tomsich
2017-10-06 8:16 ` Philipp Tomsich
@ 2017-10-06 8:46 ` Dr. Philipp Tomsich
2 siblings, 0 replies; 4+ messages in thread
From: Dr. Philipp Tomsich @ 2017-10-06 8:46 UTC (permalink / raw)
To: u-boot
Klaus,
> On 6 Oct 2017, at 10:12, Klaus Goger <klaus.goger@theobroma-systems.com> wrote:
>
> rockchip-common.h already defines values that are missing from
> rk3368_common.h
>
> For example BOOT_TARGET_DEVICES was defined empty and therefore
> distroboot had no boot targets.
>
> Signed-off-by: Klaus Goger <klaus.goger@theobroma-systems.com>
>
> ---
>
> include/configs/rk3368_common.h | 6 ++----
> 1 file changed, 2 insertions(+), 4 deletions(-)
>
> diff --git a/include/configs/rk3368_common.h b/include/configs/rk3368_common.h
> index 8b4155f37c..b643cc2464 100644
> --- a/include/configs/rk3368_common.h
> +++ b/include/configs/rk3368_common.h
> @@ -7,6 +7,8 @@
> #ifndef __CONFIG_RK3368_COMMON_H
> #define __CONFIG_RK3368_COMMON_H
>
> +#include "rockchip-common.h"
> +
> #define CONFIG_SYS_CACHELINE_SIZE 64
>
> #include <asm/arch/hardware.h>
> @@ -44,10 +46,6 @@
> "kernel_addr_r=0x280000\0" \
> "ramdisk_addr_r=0x5bf0000\0"
>
> -#include <config_distro_defaults.h>
> -
> -#define BOOT_TARGET_DEVICES(func)
> -
> #include <config_distro_bootcmd.h>
>
> #define CONFIG_EXTRA_ENV_SETTINGS \
> --
> 2.11.0
>
This causes build-failures in buildman; please fix & resubmit:
Building current source for 25 boards (4 threads, 1 job per thread)
aarch64: + geekbox
+In file included from include/configs/rk3368_common.h:49:0,
+ from include/configs/geekbox.h:10,
+ from include/config.h:5,
+ from include/common.h:21,
+ from env/common.c:11:
+include/config_distro_bootcmd.h:68:2: error: expected '}' before 'BOOT_TARGET_DEVICES_references_MMC_without_CONFIG_CMD_MMC'
+ BOOT_TARGET_DEVICES_references_MMC_without_CONFIG_CMD_MMC
+ ^
+include/config_distro_bootcmd.h:68:2: note: in definition of macro 'BOOTENV_DEV_NAME_MMC'
+ ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
+include/configs/rockchip-common.h:24:2: note: in expansion of macro 'BOOTENV_DEV_NAME'
+ func(MMC, mmc, 0) \
+ ^~~~
+include/config_distro_bootcmd.h:316:18: note: in expansion of macro 'BOOT_TARGET_DEVICES'
+ "boot_targets=" BOOT_TARGET_DEVICES(BOOTENV_DEV_NAME) "\0"
+ ^~~~~~~~~~~~~~~~~~~
+include/config_distro_bootcmd.h:333:2: note: in expansion of macro 'BOOTENV_BOOT_TARGETS'
+ BOOTENV_BOOT_TARGETS \
+ ^~~~~~~~~~~~~~~~~~~~
+include/configs/rk3368_common.h:53:2: note: in expansion of macro 'BOOTENV'
+ BOOTENV
+ ^~~~~~~
+include/env_default.h:108:2: note: in expansion of macro 'CONFIG_EXTRA_ENV_SETTINGS'
+ CONFIG_EXTRA_ENV_SETTINGS
+ ^~~~~~~~~~~~~~~~~~~~~~~~~
+make[2]: *** [env/common.o] Error 1
+make[1]: *** [env] Error 2
+make: *** [sub-make] Error 2
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2017-10-06 8:46 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-10-06 8:12 [U-Boot] [PATCH] rockchip: configs: use rockchip-common.h for rk3368 Klaus Goger
2017-10-06 8:15 ` [U-Boot] " Philipp Tomsich
2017-10-06 8:16 ` Philipp Tomsich
2017-10-06 8:46 ` [U-Boot] [PATCH] " Dr. Philipp Tomsich
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.