Re: [U-Boot] [PATCH v2 06/13] pico-imx7d: Update the README file

[Stefano Babic]

On 06/09/2018 14:17, Otavio Salvador wrote:
> 
> 
> On Thu, Sep 6, 2018 at 6:54 AM Stefano Babic  > wrote:
> 
> On 05/09/2018 21:13, Otavio Salvador wrote:
> > Hello Tom and Stefano,
> >
> > This should be applied for release as it puts the README aligned with
> > current use. Please consider it.
> >
> 
> You patches look good and I can confirm they are just related to the
> pico board. I have no comments at all, and I do not see any reason you
> need to change them.  I have set them to go into my (local) next branch.
> 
> 
> This README update is trivial and Tom can apply it directly.


Sorry, I thought the whole patchset - of course, README can be applied
directly.

Regards,
Stefano


-- 
=
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 06/13] pico-imx7d: Update the README file

[Otavio Salvador]

On Thu, Sep 6, 2018 at 6:54 AM Stefano Babic  wrote:

> On 05/09/2018 21:13, Otavio Salvador wrote:
> > Hello Tom and Stefano,
> >
> > This should be applied for release as it puts the README aligned with
> > current use. Please consider it.
> >
>
> You patches look good and I can confirm they are just related to the
> pico board. I have no comments at all, and I do not see any reason you
> need to change them.  I have set them to go into my (local) next branch.
>

This README update is trivial and Tom can apply it directly.

-- 
Otavio Salvador O.S. Systems
http://www.ossystems.com.brhttp://code.ossystems.com.br
Mobile: +55 (53) 9 9981-7854  Mobile: +1 (347) 903-9750
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH v2 06/13] pico-imx7d: Update the README file

[Stefano Babic]

Hi Otavio,

On 05/09/2018 21:13, Otavio Salvador wrote:
> Hello Tom and Stefano,
> 
> This should be applied for release as it puts the README aligned with
> current use. Please consider it.
> 

You patches look good and I can confirm they are just related to the
pico board. I have no comments at all, and I do not see any reason you
need to change them.  I have set them to go into my (local) next branch.

But patches (V1) were sent after Window was closed (last week) and we
are a quite more restrictive as in the past, so I have not considered to
get them in before release. My last PR was already too late and raises
some new issues and I guess Tom is "quite" releasing. I want to avoid to
push him other stuff just some days before release if they are not
strict important fixes.

Best regards,
Stefano


> On Tue, Sep 4, 2018 at 4:13 PM Otavio Salvador  > wrote:
> 
> On Tue, Sep 4, 2018 at 11:06 AM Michael Nazzareno Trimarchi
> mailto:mich...@amarulasolutions.com>>
> wrote:
> ...
> > Those change are about imx6ul and you update the readme of imx7d
> 
> Those put them on sync and it is part of our work. I see no reason for
> another PR just for a single commit.
> --
> Otavio Salvador                             O.S. Systems
> http://www.ossystems.com.br        http://code.ossystems.com.br
> Mobile: +55 (53) 9 9981-7854          Mobile: +1 (347) 903-9750
> 
> 
> 
> -- 
> Otavio Salvador                             O.S. Systems
> http://www.ossystems.com.br        http://code.ossystems.com.br
> Mobile: +55 (53) 9 9981-7854          Mobile: +1 (347) 903-9750


-- 
=
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 06/13] pico-imx7d: Update the README file

[Otavio Salvador]

Hello Tom and Stefano,

This should be applied for release as it puts the README aligned with
current use. Please consider it.

On Tue, Sep 4, 2018 at 4:13 PM Otavio Salvador 
wrote:

> On Tue, Sep 4, 2018 at 11:06 AM Michael Nazzareno Trimarchi
>  wrote:
> ...
> > Those change are about imx6ul and you update the readme of imx7d
>
> Those put them on sync and it is part of our work. I see no reason for
> another PR just for a single commit.
> --
> Otavio Salvador O.S. Systems
> http://www.ossystems.com.brhttp://code.ossystems.com.br
> Mobile: +55 (53) 9 9981-7854  Mobile: +1 (347) 903-9750
>


-- 
Otavio Salvador O.S. Systems
http://www.ossystems.com.brhttp://code.ossystems.com.br
Mobile: +55 (53) 9 9981-7854  Mobile: +1 (347) 903-9750
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH v2 06/13] pico-imx7d: Update the README file

[Otavio Salvador]

On Tue, Sep 4, 2018 at 11:06 AM Michael Nazzareno Trimarchi
 wrote:
...
> Those change are about imx6ul and you update the readme of imx7d

Those put them on sync and it is part of our work. I see no reason for
another PR just for a single commit.
--
Otavio Salvador O.S. Systems
http://www.ossystems.com.brhttp://code.ossystems.com.br
Mobile: +55 (53) 9 9981-7854  Mobile: +1 (347) 903-9750
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH v2 06/13] pico-imx7d: Update the README file

[Michael Nazzareno Trimarchi]

Hi

On Tue, Sep 4, 2018 at 3:28 PM Otavio Salvador  wrote:
>
> From: Fabio Estevam 
>
> Update the README file to take into accound the switch to SPL.
>
> Signed-off-by: Fabio Estevam 
> Signed-off-by: Fabio Berton 
> Signed-off-by: Otavio Salvador 
> ---
>
> Changes in v2: None
>
>  board/technexion/pico-imx7d/README | 24 +---
>  1 file changed, 17 insertions(+), 7 deletions(-)
>

Those change are about imx6ul and you update the readme of imx7d

Michael

> diff --git a/board/technexion/pico-imx7d/README 
> b/board/technexion/pico-imx7d/README
> index aa9d72c5d1..24eb97e82c 100644
> --- a/board/technexion/pico-imx7d/README
> +++ b/board/technexion/pico-imx7d/README
> @@ -11,7 +11,13 @@ $ make mrproper
>  $ make pico-imx7d_defconfig
>  $ make
>
> -This generates the U-Boot binary called u-boot.imx.
> +This generates the SPL and u-boot.img binaries.
> +
> +1. Loading U-Boot via USB Serial Download Protocol
> +
> +Note: This method is convenient for development purposes.
> +If the eMMC has already a U-Boot flashed with DFU support then
> +the user can go to step 2 below in order to update U-Boot.
>
>  Put pico board in USB download mode (refer to the PICO-iMX7D Quick Start 
> Guide
>  page 3)
> @@ -22,11 +28,15 @@ Connect a USB cable between the OTG pico port and the 
> host PC.
>
>  Open a terminal program such as minicom.
>
> -Copy u-boot.imx to the imx_usb_loader folder.
> +Copy SPL and u-boot.img to the imx_usb_loader folder.
> +
> +Load the SPL binary via USB:
>
> -Load u-boot.imx via USB:
> +$ sudo ./imx_usb SPL
>
> -$ sudo ./imx_usb u-boot.imx
> +Load the u-boot.img binary via USB:
> +
> +$ sudo ./imx_usb u-boot.img
>
>  Then U-Boot starts and its messages appear in the console program.
>
> @@ -35,16 +45,16 @@ Use the default environment variables:
>  => env default -f -a
>  => saveenv
>
> +2. Flashing U-Boot into the eMMC
> +
>  Run the DFU agent so we can flash the new images using dfu-util tool:
>
>  => dfu 0 mmc 0
>
> -Flash SPL into the eMMC:
> +Flash SPL and u-boot.img into the eMMC running the following commands on a 
> PC:
>
>  $ sudo dfu-util -D SPL -a spl
>
> -Flash u-boot.img into the eMMC:
> -
>  $ sudo dfu-util -D u-boot.img -a u-boot
>
>  Remove power from the pico board.
> --
> 2.18.0
>
> ___
> U-Boot mailing list
> U-Boot@lists.denx.de
> https://lists.denx.de/listinfo/u-boot



-- 
| Michael Nazzareno Trimarchi Amarula Solutions BV |
| COO  -  Founder  Cruquiuskade 47 |
| +31(0)851119172 Amsterdam 1018 AM NL |
|  [`as] http://www.amarulasolutions.com   |
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

[U-Boot] [PATCH v2 06/13] pico-imx7d: Update the README file

[Otavio Salvador]

From: Fabio Estevam 

Update the README file to take into accound the switch to SPL.

Signed-off-by: Fabio Estevam 
Signed-off-by: Fabio Berton 
Signed-off-by: Otavio Salvador 
---

Changes in v2: None

 board/technexion/pico-imx7d/README | 24 +---
 1 file changed, 17 insertions(+), 7 deletions(-)

diff --git a/board/technexion/pico-imx7d/README 
b/board/technexion/pico-imx7d/README
index aa9d72c5d1..24eb97e82c 100644
--- a/board/technexion/pico-imx7d/README
+++ b/board/technexion/pico-imx7d/README
@@ -11,7 +11,13 @@ $ make mrproper
 $ make pico-imx7d_defconfig
 $ make
 
-This generates the U-Boot binary called u-boot.imx.
+This generates the SPL and u-boot.img binaries.
+
+1. Loading U-Boot via USB Serial Download Protocol
+
+Note: This method is convenient for development purposes.
+If the eMMC has already a U-Boot flashed with DFU support then
+the user can go to step 2 below in order to update U-Boot.
 
 Put pico board in USB download mode (refer to the PICO-iMX7D Quick Start Guide
 page 3)
@@ -22,11 +28,15 @@ Connect a USB cable between the OTG pico port and the host 
PC.
 
 Open a terminal program such as minicom.
 
-Copy u-boot.imx to the imx_usb_loader folder.
+Copy SPL and u-boot.img to the imx_usb_loader folder.
+
+Load the SPL binary via USB:
 
-Load u-boot.imx via USB:
+$ sudo ./imx_usb SPL
 
-$ sudo ./imx_usb u-boot.imx
+Load the u-boot.img binary via USB:
+
+$ sudo ./imx_usb u-boot.img
 
 Then U-Boot starts and its messages appear in the console program.
 
@@ -35,16 +45,16 @@ Use the default environment variables:
 => env default -f -a
 => saveenv
 
+2. Flashing U-Boot into the eMMC
+
 Run the DFU agent so we can flash the new images using dfu-util tool:
 
 => dfu 0 mmc 0
 
-Flash SPL into the eMMC:
+Flash SPL and u-boot.img into the eMMC running the following commands on a PC:
 
 $ sudo dfu-util -D SPL -a spl
 
-Flash u-boot.img into the eMMC:
-
 $ sudo dfu-util -D u-boot.img -a u-boot
 
 Remove power from the pico board.
-- 
2.18.0

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