Re: [U-Boot] [PATCH v2 2/2] imx6ul: opos6ul: in Serial Downloader boot mode use ymodem

2018-04-27 Thread Stefano Babic
On 17/04/2018 17:29, Sébastien Szymanski wrote:
> When booting in Serial Downloader mode load the U-Boot image using
> ymodem.
> 
> Signed-off-by: Sébastien Szymanski 
> ---
> 
> Changes for v2:
>  - Add commit log.
> 
>  arch/arm/mach-imx/mx6/opos6ul.c | 10 ++
>  1 file changed, 10 insertions(+)
> 
> diff --git a/arch/arm/mach-imx/mx6/opos6ul.c b/arch/arm/mach-imx/mx6/opos6ul.c
> index ef70a7d323..5d39c0bc1b 100644
> --- a/arch/arm/mach-imx/mx6/opos6ul.c
> +++ b/arch/arm/mach-imx/mx6/opos6ul.c
> @@ -208,6 +208,16 @@ static struct mx6_ddr3_cfg mem_ddr = {
>   .trasmin = 3750,
>  };
>  
> +void board_boot_order(u32 *spl_boot_list)
> +{
> + unsigned int bmode = readl(_base->sbmr2);
> +
> + if (((bmode >> 24) & 0x03) == 0x01) /* Serial Downloader */
> + spl_boot_list[0] = BOOT_DEVICE_UART;
> + else
> + spl_boot_list[0] = spl_boot_device();
> +}
> +
>  static void ccgr_init(void)
>  {
>   struct mxc_ccm_reg *ccm = (struct mxc_ccm_reg *)CCM_BASE_ADDR;
> 

Applied to u-boot-imx, thanks !

Best regards,
Stefano Babic

-- 
=
DENX Software Engineering GmbH,  Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sba...@denx.de
=
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot


Re: [U-Boot] [PATCH v2 2/2] imx6ul: opos6ul: in Serial Downloader boot mode use ymodem

2018-04-18 Thread Sébastien Szymanski
Hi Fabio,

On 04/17/2018 11:52 PM, Fabio Estevam wrote:
> Hi Sébastien,
> 
> On Tue, Apr 17, 2018 at 12:29 PM, Sébastien Szymanski
>  wrote:
>> When booting in Serial Downloader mode load the U-Boot image using
>> ymodem.
> 
> Not sure if you are aware, but it is possible to use imx_usb_loader to
> also load SPL + u-boot.img as described at:
> doc/README.sdp
> 
> Your patch looks good, but just wanted to point out this alternative
> in case it fits your needs.

I am aware about the SDP support in U-Boot but I prefer to keep using
ymodem at the moment.

Thanks!

Regards,

> 


-- 
Sébastien Szymanski
Software engineer, Armadeus Systems
Tel: +33 (0)9 72 29 41 44
Fax: +33 (0)9 72 28 79 26
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot


Re: [U-Boot] [PATCH v2 2/2] imx6ul: opos6ul: in Serial Downloader boot mode use ymodem

2018-04-17 Thread Fabio Estevam
Hi Sébastien,

On Tue, Apr 17, 2018 at 12:29 PM, Sébastien Szymanski
 wrote:
> When booting in Serial Downloader mode load the U-Boot image using
> ymodem.

Not sure if you are aware, but it is possible to use imx_usb_loader to
also load SPL + u-boot.img as described at:
doc/README.sdp

Your patch looks good, but just wanted to point out this alternative
in case it fits your needs.
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot


[U-Boot] [PATCH v2 2/2] imx6ul: opos6ul: in Serial Downloader boot mode use ymodem

2018-04-17 Thread Sébastien Szymanski
When booting in Serial Downloader mode load the U-Boot image using
ymodem.

Signed-off-by: Sébastien Szymanski 
---

Changes for v2:
 - Add commit log.

 arch/arm/mach-imx/mx6/opos6ul.c | 10 ++
 1 file changed, 10 insertions(+)

diff --git a/arch/arm/mach-imx/mx6/opos6ul.c b/arch/arm/mach-imx/mx6/opos6ul.c
index ef70a7d323..5d39c0bc1b 100644
--- a/arch/arm/mach-imx/mx6/opos6ul.c
+++ b/arch/arm/mach-imx/mx6/opos6ul.c
@@ -208,6 +208,16 @@ static struct mx6_ddr3_cfg mem_ddr = {
.trasmin = 3750,
 };
 
+void board_boot_order(u32 *spl_boot_list)
+{
+   unsigned int bmode = readl(_base->sbmr2);
+
+   if (((bmode >> 24) & 0x03) == 0x01) /* Serial Downloader */
+   spl_boot_list[0] = BOOT_DEVICE_UART;
+   else
+   spl_boot_list[0] = spl_boot_device();
+}
+
 static void ccgr_init(void)
 {
struct mxc_ccm_reg *ccm = (struct mxc_ccm_reg *)CCM_BASE_ADDR;
-- 
2.16.1

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