All of lore.kernel.org
 help / color / mirror / Atom feed
* [U-Boot] [PATCH] snow: set fdtfile
@ 2018-06-14 20:34 Guillaume GARDET
  2018-07-13 16:33 ` Guillaume Gardet
  2018-07-13 17:20 ` Peter Robinson
  0 siblings, 2 replies; 6+ messages in thread
From: Guillaume GARDET @ 2018-06-14 20:34 UTC (permalink / raw)
  To: u-boot

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>
-- 
2.17.1

^ permalink raw reply related	[flat|nested] 6+ messages in thread
* [U-Boot] [PATCH] snow: set fdtfile
  2018-06-14 20:34 [U-Boot] [PATCH] snow: set fdtfile Guillaume GARDET
@ 2018-07-13 16:33 ` Guillaume Gardet
  2018-07-13 17:20 ` Peter Robinson
  1 sibling, 0 replies; 6+ messages in thread
From: Guillaume Gardet @ 2018-07-13 16:33 UTC (permalink / raw)
  To: u-boot

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>

^ permalink raw reply	[flat|nested] 6+ messages in thread
* [U-Boot] [PATCH] snow: set fdtfile
  2018-06-14 20:34 [U-Boot] [PATCH] snow: set fdtfile Guillaume GARDET
  2018-07-13 16:33 ` Guillaume Gardet
@ 2018-07-13 17:20 ` Peter Robinson
  2018-07-16  8:46   ` Guillaume Gardet
  1 sibling, 1 reply; 6+ messages in thread
From: Peter Robinson @ 2018-07-13 17:20 UTC (permalink / raw)
  To: u-boot

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 at lists.denx.de
> https://lists.denx.de/listinfo/u-boot

^ permalink raw reply	[flat|nested] 6+ messages in thread
* [U-Boot] [PATCH] snow: set fdtfile
  2018-07-13 17:20 ` Peter Robinson
@ 2018-07-16  8:46   ` Guillaume Gardet
  2018-07-16  9:04     ` Minkyu Kang
  0 siblings, 1 reply; 6+ messages in thread
From: Guillaume Gardet @ 2018-07-16  8:46 UTC (permalink / raw)
  To: 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 at lists.denx.de
>> https://lists.denx.de/listinfo/u-boot

^ permalink raw reply	[flat|nested] 6+ messages in thread
* [U-Boot] [PATCH] snow: set fdtfile
  2018-07-16  8:46   ` Guillaume Gardet
@ 2018-07-16  9:04     ` Minkyu Kang
  2018-07-27  6:54       ` Minkyu Kang
  0 siblings, 1 reply; 6+ messages in thread
From: Minkyu Kang @ 2018-07-16  9:04 UTC (permalink / raw)
  To: 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 at lists.denx.de
> >> https://lists.denx.de/listinfo/u-boot
>
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> https://lists.denx.de/listinfo/u-boot


I'll pick this patch up soon.

Thanks,
Minkyu Kang.

^ permalink raw reply	[flat|nested] 6+ messages in thread
* [U-Boot] [PATCH] snow: set fdtfile
  2018-07-16  9:04     ` Minkyu Kang
@ 2018-07-27  6:54       ` Minkyu Kang
  0 siblings, 0 replies; 6+ messages in thread
From: Minkyu Kang @ 2018-07-27  6:54 UTC (permalink / raw)
  To: u-boot

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 at lists.denx.de
>> >> https://lists.denx.de/listinfo/u-boot
>>
>> _______________________________________________
>> U-Boot mailing list
>> U-Boot at 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.

>

^ permalink raw reply	[flat|nested] 6+ messages in thread
end of thread, other threads:[~2018-07-27  6:54 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-06-14 20:34 [U-Boot] [PATCH] snow: set fdtfile Guillaume GARDET
2018-07-13 16:33 ` Guillaume Gardet
2018-07-13 17:20 ` Peter Robinson
2018-07-16  8:46   ` Guillaume Gardet
2018-07-16  9:04     ` Minkyu Kang
2018-07-27  6:54       ` Minkyu Kang

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.