[U-Boot] [PATCH] snow: set fdtfile

Needed to boot with EFI distro boot.
Signed-off-by: Guillaume GARDET guillaume.gardet@free.fr
Cc: Akshay Saraswat akshay.s@samsung.com Cc: Tom Rini trini@konsulko.com --- include/configs/snow.h | 3 +++ 1 file changed, 3 insertions(+)
diff --git a/include/configs/snow.h b/include/configs/snow.h index 3b0db32ece..c546a5a6d0 100644 --- a/include/configs/snow.h +++ b/include/configs/snow.h @@ -8,6 +8,9 @@ #ifndef __CONFIG_SNOW_H #define __CONFIG_SNOW_H
+#define EXYNOS_FDTFILE_SETTING \ + "fdtfile=exynos5250-snow.dtb\0" + #include <configs/exynos5250-common.h> #include <configs/exynos5-dt-common.h> #include <configs/exynos5-common.h>

Hi,
This is a reminder for this patch.
Guillaume
Le 14/06/2018 à 22:34, Guillaume GARDET a écrit :
Needed to boot with EFI distro boot.
Signed-off-by: Guillaume GARDET guillaume.gardet@free.fr
Cc: Akshay Saraswat akshay.s@samsung.com Cc: Tom Rini trini@konsulko.com
include/configs/snow.h | 3 +++ 1 file changed, 3 insertions(+)
diff --git a/include/configs/snow.h b/include/configs/snow.h index 3b0db32ece..c546a5a6d0 100644 --- a/include/configs/snow.h +++ b/include/configs/snow.h @@ -8,6 +8,9 @@ #ifndef __CONFIG_SNOW_H #define __CONFIG_SNOW_H
+#define EXYNOS_FDTFILE_SETTING \
- "fdtfile=exynos5250-snow.dtb\0"
- #include <configs/exynos5250-common.h> #include <configs/exynos5-dt-common.h> #include <configs/exynos5-common.h>

On Thu, Jun 14, 2018 at 9:34 PM, Guillaume GARDET guillaume.gardet@free.fr wrote:
Needed to boot with EFI distro boot.
Signed-off-by: Guillaume GARDET guillaume.gardet@free.fr
Cc: Akshay Saraswat akshay.s@samsung.com Cc: Tom Rini trini@konsulko.com
include/configs/snow.h | 3 +++ 1 file changed, 3 insertions(+)
diff --git a/include/configs/snow.h b/include/configs/snow.h index 3b0db32ece..c546a5a6d0 100644 --- a/include/configs/snow.h +++ b/include/configs/snow.h @@ -8,6 +8,9 @@ #ifndef __CONFIG_SNOW_H #define __CONFIG_SNOW_H
+#define EXYNOS_FDTFILE_SETTING \
"fdtfile=exynos5250-snow.dtb\0"
Isn't it set in configs/snow_defconfig with CONFIG_DEFAULT_DEVICE_TREE="exynos5250-snow"
#include <configs/exynos5250-common.h> #include <configs/exynos5-dt-common.h>
#include <configs/exynos5-common.h>
2.17.1
U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot

Hi,
Le 13/07/2018 à 19:20, Peter Robinson a écrit :
On Thu, Jun 14, 2018 at 9:34 PM, Guillaume GARDET guillaume.gardet@free.fr wrote:
Needed to boot with EFI distro boot.
Signed-off-by: Guillaume GARDET guillaume.gardet@free.fr
Cc: Akshay Saraswat akshay.s@samsung.com Cc: Tom Rini trini@konsulko.com
include/configs/snow.h | 3 +++ 1 file changed, 3 insertions(+)
diff --git a/include/configs/snow.h b/include/configs/snow.h index 3b0db32ece..c546a5a6d0 100644 --- a/include/configs/snow.h +++ b/include/configs/snow.h @@ -8,6 +8,9 @@ #ifndef __CONFIG_SNOW_H #define __CONFIG_SNOW_H
+#define EXYNOS_FDTFILE_SETTING \
"fdtfile=exynos5250-snow.dtb\0"
Isn't it set in configs/snow_defconfig with CONFIG_DEFAULT_DEVICE_TREE="exynos5250-snow"
No, CONFIG_DEFAULT_DEVICE_TREE is only used for U-Boot DT, not to set fdtfile var.
Guillaume
#include <configs/exynos5250-common.h> #include <configs/exynos5-dt-common.h>
#include <configs/exynos5-common.h>
2.17.1
U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot

Hi,
2018년 7월 16일 (월) 17:49, Guillaume Gardet guillaume.gardet@free.fr님이 작성:
Hi,
Le 13/07/2018 à 19:20, Peter Robinson a écrit :
On Thu, Jun 14, 2018 at 9:34 PM, Guillaume GARDET guillaume.gardet@free.fr wrote:
Needed to boot with EFI distro boot.
Signed-off-by: Guillaume GARDET guillaume.gardet@free.fr
Cc: Akshay Saraswat akshay.s@samsung.com Cc: Tom Rini trini@konsulko.com
include/configs/snow.h | 3 +++ 1 file changed, 3 insertions(+)
diff --git a/include/configs/snow.h b/include/configs/snow.h index 3b0db32ece..c546a5a6d0 100644 --- a/include/configs/snow.h +++ b/include/configs/snow.h @@ -8,6 +8,9 @@ #ifndef __CONFIG_SNOW_H #define __CONFIG_SNOW_H
+#define EXYNOS_FDTFILE_SETTING \
"fdtfile=exynos5250-snow.dtb\0"
Isn't it set in configs/snow_defconfig with CONFIG_DEFAULT_DEVICE_TREE="exynos5250-snow"
No, CONFIG_DEFAULT_DEVICE_TREE is only used for U-Boot DT, not to set fdtfile var.
Guillaume
#include <configs/exynos5250-common.h> #include <configs/exynos5-dt-common.h>
#include <configs/exynos5-common.h>
2.17.1
U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
I'll pick this patch up soon.
Thanks, Minkyu Kang.

Hi,
2018년 7월 16일 (월) 18:04, Minkyu Kang promsoft@gmail.com님이 작성:
Hi,
2018년 7월 16일 (월) 17:49, Guillaume Gardet guillaume.gardet@free.fr님이 작성:
Hi,
Le 13/07/2018 à 19:20, Peter Robinson a écrit :
On Thu, Jun 14, 2018 at 9:34 PM, Guillaume GARDET guillaume.gardet@free.fr wrote:
Needed to boot with EFI distro boot.
Signed-off-by: Guillaume GARDET guillaume.gardet@free.fr
Cc: Akshay Saraswat akshay.s@samsung.com Cc: Tom Rini trini@konsulko.com
include/configs/snow.h | 3 +++ 1 file changed, 3 insertions(+)
diff --git a/include/configs/snow.h b/include/configs/snow.h index 3b0db32ece..c546a5a6d0 100644 --- a/include/configs/snow.h +++ b/include/configs/snow.h @@ -8,6 +8,9 @@ #ifndef __CONFIG_SNOW_H #define __CONFIG_SNOW_H
+#define EXYNOS_FDTFILE_SETTING \
"fdtfile=exynos5250-snow.dtb\0"
Isn't it set in configs/snow_defconfig with CONFIG_DEFAULT_DEVICE_TREE="exynos5250-snow"
No, CONFIG_DEFAULT_DEVICE_TREE is only used for U-Boot DT, not to set fdtfile var.
Guillaume
#include <configs/exynos5250-common.h> #include <configs/exynos5-dt-common.h>
#include <configs/exynos5-common.h>
2.17.1
U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
I'll pick this patch up soon.
Thanks, Minkyu Kang.
applied to u-boot-samsung.
Thanks, Minkyu Kang.
participants (4)
-
Guillaume GARDET
-
Guillaume Gardet
-
Minkyu Kang
-
Peter Robinson