Re: [U-Boot] [PATCH] ARM: exynos5: Try to boot on mmc2 before mmc0/1

2019-09-05 Thread Minkyu Kang
Hi,

On 31/08/2019 00:51, Oleksandr Suvorov wrote:
> On Fri, Aug 30, 2019 at 4:20 PM Guillaume Gardet
>  wrote:
>>
>> Hi,
>>
>> I got no feedback on this patch. Is there anything blocking it?
>>
>> Guillaume
>>
>>
>>> -Original Message-
>>> From: Guillaume GARDET 
>>> Sent: 24 July 2019 09:10
>>> To: u-boot@lists.denx.de
>>> Cc: Guillaume Gardet ; Lukasz Majewski
>>> ; Anand Moon ; Simon Glass
>>> ; Minkyu Kang 
>>> Subject: [U-Boot] [PATCH] ARM: exynos5: Try to boot on mmc2 before
>>> mmc0/1
>>>
>>> As stated in commit a61a4a1db009e3e600258551a01b54c4f50ec103 with
>>> DM_MMC, exynos boards now enumarates external SD/MMC slot as mmc2,
>>> instead of mmc1 with legacy mode. Moving mmc2 before mmc1/0 restore
>>> the previous behavior of trying external SD/MMC before internal slot.
>>>
>>> Signed-off-by: Guillaume GARDET 
> 
> Reviewed-by: Oleksandr Suvorov 
> 
>>>
>>> Cc: Lukasz Majewski 
>>> Cc: Anand Moon 
>>> Cc: Simon Glass 
>>> Cc: Minkyu Kang 
>>>
>>> ---
>>>  include/configs/exynos5-common.h | 2 +-
>>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>>
>>> diff --git a/include/configs/exynos5-common.h b/include/configs/exynos5-
>>> common.h
>>> index 397bbf6d9d..e0a4d76d53 100644
>>> --- a/include/configs/exynos5-common.h
>>> +++ b/include/configs/exynos5-common.h
>>> @@ -128,9 +128,9 @@
>>>  #define EXYNOS_IRAM_SECONDARY_BASE   0x02020018
>>>
>>>  #define BOOT_TARGET_DEVICES(func) \
>>> + func(MMC, mmc, 2) \
>>>   func(MMC, mmc, 1) \
>>>   func(MMC, mmc, 0) \
>>> - func(MMC, mmc, 2) \
>>>   func(PXE, pxe, na) \
>>>   func(DHCP, dhcp, na)
>>>
>>> --
>>> 2.22.0
>>
>> ___
>> U-Boot mailing list
>> U-Boot@lists.denx.de
>> https://lists.denx.de/listinfo/u-boot
> 
> 
> 

applied to u-boot-samsung.

Thanks,
Minkyu Kang.
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot


Re: [U-Boot] [PATCH] ARM: exynos5: Try to boot on mmc2 before mmc0/1

2019-08-30 Thread Oleksandr Suvorov
On Fri, Aug 30, 2019 at 4:20 PM Guillaume Gardet
 wrote:
>
> Hi,
>
> I got no feedback on this patch. Is there anything blocking it?
>
> Guillaume
>
>
> > -Original Message-
> > From: Guillaume GARDET 
> > Sent: 24 July 2019 09:10
> > To: u-boot@lists.denx.de
> > Cc: Guillaume Gardet ; Lukasz Majewski
> > ; Anand Moon ; Simon Glass
> > ; Minkyu Kang 
> > Subject: [U-Boot] [PATCH] ARM: exynos5: Try to boot on mmc2 before
> > mmc0/1
> >
> > As stated in commit a61a4a1db009e3e600258551a01b54c4f50ec103 with
> > DM_MMC, exynos boards now enumarates external SD/MMC slot as mmc2,
> > instead of mmc1 with legacy mode. Moving mmc2 before mmc1/0 restore
> > the previous behavior of trying external SD/MMC before internal slot.
> >
> > Signed-off-by: Guillaume GARDET 

Reviewed-by: Oleksandr Suvorov 

> >
> > Cc: Lukasz Majewski 
> > Cc: Anand Moon 
> > Cc: Simon Glass 
> > Cc: Minkyu Kang 
> >
> > ---
> >  include/configs/exynos5-common.h | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/include/configs/exynos5-common.h b/include/configs/exynos5-
> > common.h
> > index 397bbf6d9d..e0a4d76d53 100644
> > --- a/include/configs/exynos5-common.h
> > +++ b/include/configs/exynos5-common.h
> > @@ -128,9 +128,9 @@
> >  #define EXYNOS_IRAM_SECONDARY_BASE   0x02020018
> >
> >  #define BOOT_TARGET_DEVICES(func) \
> > + func(MMC, mmc, 2) \
> >   func(MMC, mmc, 1) \
> >   func(MMC, mmc, 0) \
> > - func(MMC, mmc, 2) \
> >   func(PXE, pxe, na) \
> >   func(DHCP, dhcp, na)
> >
> > --
> > 2.22.0
>
> ___
> U-Boot mailing list
> U-Boot@lists.denx.de
> https://lists.denx.de/listinfo/u-boot



-- 
Best regards

Oleksandr Suvorov
cryo...@gmail.com
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot


Re: [U-Boot] [PATCH] ARM: exynos5: Try to boot on mmc2 before mmc0/1

2019-08-30 Thread Guillaume Gardet
Hi,

I got no feedback on this patch. Is there anything blocking it?

Guillaume


> -Original Message-
> From: Guillaume GARDET 
> Sent: 24 July 2019 09:10
> To: u-boot@lists.denx.de
> Cc: Guillaume Gardet ; Lukasz Majewski
> ; Anand Moon ; Simon Glass
> ; Minkyu Kang 
> Subject: [U-Boot] [PATCH] ARM: exynos5: Try to boot on mmc2 before
> mmc0/1
> 
> As stated in commit a61a4a1db009e3e600258551a01b54c4f50ec103 with
> DM_MMC, exynos boards now enumarates external SD/MMC slot as mmc2,
> instead of mmc1 with legacy mode. Moving mmc2 before mmc1/0 restore
> the previous behavior of trying external SD/MMC before internal slot.
> 
> Signed-off-by: Guillaume GARDET 
> 
> Cc: Lukasz Majewski 
> Cc: Anand Moon 
> Cc: Simon Glass 
> Cc: Minkyu Kang 
> 
> ---
>  include/configs/exynos5-common.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/include/configs/exynos5-common.h b/include/configs/exynos5-
> common.h
> index 397bbf6d9d..e0a4d76d53 100644
> --- a/include/configs/exynos5-common.h
> +++ b/include/configs/exynos5-common.h
> @@ -128,9 +128,9 @@
>  #define EXYNOS_IRAM_SECONDARY_BASE   0x02020018
> 
>  #define BOOT_TARGET_DEVICES(func) \
> + func(MMC, mmc, 2) \
>   func(MMC, mmc, 1) \
>   func(MMC, mmc, 0) \
> - func(MMC, mmc, 2) \
>   func(PXE, pxe, na) \
>   func(DHCP, dhcp, na)
> 
> --
> 2.22.0

___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot