[U-Boot] [PATCH v3] tegra: nyan-big: Update CONFIG_SYS_TEXT to the default in README.chromium

[U-Boot] [PATCH v3] tegra: nyan-big: Update CONFIG_SYS_TEXT to the default in README.chromium

[Peter Robinson]

To build U-Boot on a Nyan Big Chromebook the docs outline adjusting the Tegra124
defined CONFIG_SYS_TEXT_BASE but this has since been moved to individual config
files. We should have the default required for U-Boot chain loading on the
chromebook as the default CONFIG_SYS_TEXT_BASE and update the docs to remove
this now non required step.

Signed-off-by: Peter Robinson <pbrobinson@gmail.com>
---

v3: Update commit message
v2: Update to 2018.07 RC1

 configs/nyan-big_defconfig |  2 +-
 doc/README.chromium        | 27 +++++++--------------------
 2 files changed, 8 insertions(+), 21 deletions(-)

diff --git a/configs/nyan-big_defconfig b/configs/nyan-big_defconfig
index 8868d1f9e10..3b64c643716 100644
--- a/configs/nyan-big_defconfig
+++ b/configs/nyan-big_defconfig
@@ -1,6 +1,6 @@
 CONFIG_ARM=y
 CONFIG_TEGRA=y
-CONFIG_SYS_TEXT_BASE=0x80110000
+CONFIG_SYS_TEXT_BASE=0x81000100
 CONFIG_DEBUG_UART_BASE=0x70006000
 CONFIG_DEBUG_UART_CLOCK=408000000
 CONFIG_TEGRA124=y
diff --git a/doc/README.chromium b/doc/README.chromium
index 7bf4d874474..45eaeced2da 100644
--- a/doc/README.chromium
+++ b/doc/README.chromium
@@ -43,26 +43,13 @@ https://git.collabora.com/cgit/user/tomeu/u-boot.git/commit/?h=nyan-big
 https://lists.denx.de/pipermail/u-boot/2017-May/289491.html
 https://github.com/chromeos-nvidia-androidtv/gnu-linux-on-acer-chromebook-13#copy-data-to-the-sd-card
 
-1. Patch U-Boot
-
-Open include/configs/tegra124-common.h
-
-Change:
-
-#define CONFIG_SYS_TEXT_BASE	0x80110000
-
-to:
-
-#define CONFIG_SYS_TEXT_BASE	0x81000100
-
-
-2. Build U-Boot
+1. Build U-Boot
 
    mkdir b
    make -j8 O=b/nyan-big CROSS_COMPILE=arm-linux-gnueabi- nyan-big_defconfig all
 
 
-3. Select a .its file
+2. Select a .its file
 
 Select something from doc/chromium which matches your board, or create your
 own.
@@ -72,7 +59,7 @@ used by U-Boot. This is because the Chromebook expects to pass it to the
 kernel, and crashes if it is not present.
 
 
-4. Build and sign an image
+3. Build and sign an image
 
    ./b/nyan-big/tools/mkimage -f doc/chromium/nyan-big.its u-boot-chromium.fit
    echo test >dummy.txt
@@ -82,7 +69,7 @@ kernel, and crashes if it is not present.
 	--bootloader dummy.txt --pack u-boot.kpart
 
 
-5. Prepare an SD card
+4. Prepare an SD card
 
    DISK=/dev/sdc   # Replace with your actual SD card device
    sudo cgpt create $DISK
@@ -91,12 +78,12 @@ kernel, and crashes if it is not present.
    sudo gdisk $DISK   # Enter command 'w' to write a protective MBR to the disk
 
 
-6. Write U-Boot to the SD card
+5. Write U-Boot to the SD card
 
    sudo dd if=u-boot.kpart of=/dev/sdc1; sync
 
 
-7. Start it up
+6. Start it up
 
 Reboot the device in dev mode. Make sure that you have USB booting enabled. To
 do this, login as root (via Ctrl-Alt-forward_arrow) and type
@@ -115,7 +102,7 @@ mode screen. It should show something like the following on the display:
    Tegra124 (Nyan-big) #
 
 
-8. Known problems
+7. Known problems
 
 On the serial console the word MMC is chopped at the start of the line:
 
-- 
2.17.1

[U-Boot] [PATCH v3] tegra: nyan-big: Update CONFIG_SYS_TEXT to the default in README.chromium

[Simon Glass]

On 9 June 2018 at 21:17, Peter Robinson <pbrobinson@gmail.com> wrote:
> To build U-Boot on a Nyan Big Chromebook the docs outline adjusting the Tegra124
> defined CONFIG_SYS_TEXT_BASE but this has since been moved to individual config
> files. We should have the default required for U-Boot chain loading on the
> chromebook as the default CONFIG_SYS_TEXT_BASE and update the docs to remove
> this now non required step.
>
> Signed-off-by: Peter Robinson <pbrobinson@gmail.com>
> ---
>
> v3: Update commit message
> v2: Update to 2018.07 RC1
>
>  configs/nyan-big_defconfig |  2 +-
>  doc/README.chromium        | 27 +++++++--------------------
>  2 files changed, 8 insertions(+), 21 deletions(-)

Reviewed-by: Simon Glass <sjg@chromium.org>

[U-Boot] [PATCH v3] tegra: nyan-big: Update CONFIG_SYS_TEXT to the default in README.chromium

[Peter Robinson]

Tom,

Ping, could we get this applied please?

Peter

On Sun, Jun 10, 2018 at 6:17 AM, Peter Robinson <pbrobinson@gmail.com> wrote:
> To build U-Boot on a Nyan Big Chromebook the docs outline adjusting the Tegra124
> defined CONFIG_SYS_TEXT_BASE but this has since been moved to individual config
> files. We should have the default required for U-Boot chain loading on the
> chromebook as the default CONFIG_SYS_TEXT_BASE and update the docs to remove
> this now non required step.
>
> Signed-off-by: Peter Robinson <pbrobinson@gmail.com>
> ---
>
> v3: Update commit message
> v2: Update to 2018.07 RC1
>
>  configs/nyan-big_defconfig |  2 +-
>  doc/README.chromium        | 27 +++++++--------------------
>  2 files changed, 8 insertions(+), 21 deletions(-)
>
> diff --git a/configs/nyan-big_defconfig b/configs/nyan-big_defconfig
> index 8868d1f9e10..3b64c643716 100644
> --- a/configs/nyan-big_defconfig
> +++ b/configs/nyan-big_defconfig
> @@ -1,6 +1,6 @@
>  CONFIG_ARM=y
>  CONFIG_TEGRA=y
> -CONFIG_SYS_TEXT_BASE=0x80110000
> +CONFIG_SYS_TEXT_BASE=0x81000100
>  CONFIG_DEBUG_UART_BASE=0x70006000
>  CONFIG_DEBUG_UART_CLOCK=408000000
>  CONFIG_TEGRA124=y
> diff --git a/doc/README.chromium b/doc/README.chromium
> index 7bf4d874474..45eaeced2da 100644
> --- a/doc/README.chromium
> +++ b/doc/README.chromium
> @@ -43,26 +43,13 @@ https://git.collabora.com/cgit/user/tomeu/u-boot.git/commit/?h=nyan-big
>  https://lists.denx.de/pipermail/u-boot/2017-May/289491.html
>  https://github.com/chromeos-nvidia-androidtv/gnu-linux-on-acer-chromebook-13#copy-data-to-the-sd-card
>
> -1. Patch U-Boot
> -
> -Open include/configs/tegra124-common.h
> -
> -Change:
> -
> -#define CONFIG_SYS_TEXT_BASE   0x80110000
> -
> -to:
> -
> -#define CONFIG_SYS_TEXT_BASE   0x81000100
> -
> -
> -2. Build U-Boot
> +1. Build U-Boot
>
>     mkdir b
>     make -j8 O=b/nyan-big CROSS_COMPILE=arm-linux-gnueabi- nyan-big_defconfig all
>
>
> -3. Select a .its file
> +2. Select a .its file
>
>  Select something from doc/chromium which matches your board, or create your
>  own.
> @@ -72,7 +59,7 @@ used by U-Boot. This is because the Chromebook expects to pass it to the
>  kernel, and crashes if it is not present.
>
>
> -4. Build and sign an image
> +3. Build and sign an image
>
>     ./b/nyan-big/tools/mkimage -f doc/chromium/nyan-big.its u-boot-chromium.fit
>     echo test >dummy.txt
> @@ -82,7 +69,7 @@ kernel, and crashes if it is not present.
>         --bootloader dummy.txt --pack u-boot.kpart
>
>
> -5. Prepare an SD card
> +4. Prepare an SD card
>
>     DISK=/dev/sdc   # Replace with your actual SD card device
>     sudo cgpt create $DISK
> @@ -91,12 +78,12 @@ kernel, and crashes if it is not present.
>     sudo gdisk $DISK   # Enter command 'w' to write a protective MBR to the disk
>
>
> -6. Write U-Boot to the SD card
> +5. Write U-Boot to the SD card
>
>     sudo dd if=u-boot.kpart of=/dev/sdc1; sync
>
>
> -7. Start it up
> +6. Start it up
>
>  Reboot the device in dev mode. Make sure that you have USB booting enabled. To
>  do this, login as root (via Ctrl-Alt-forward_arrow) and type
> @@ -115,7 +102,7 @@ mode screen. It should show something like the following on the display:
>     Tegra124 (Nyan-big) #
>
>
> -8. Known problems
> +7. Known problems
>
>  On the serial console the word MMC is chopped at the start of the line:
>
> --
> 2.17.1
>

[U-Boot] [PATCH v3] tegra: nyan-big: Update CONFIG_SYS_TEXT to the default in README.chromium

[Tom Warren]

Applied to tegra-master, just waiting on our automated build/test framework to give it a green light, then I'll issue a PR to TomR.  Sorry for the delay.

Tom

-----Original Message-----
From: Peter Robinson <pbrobinson@gmail.com> 
Sent: Thursday, July 5, 2018 11:38 PM
To: Allen Martin <AMartin@nvidia.com>; Simon Glass <sjg@chromium.org>; Tom Warren <TWarren@nvidia.com>; u-boot at lists.denx.de
Cc: Peter Robinson <pbrobinson@gmail.com>
Subject: Re: [PATCH v3] tegra: nyan-big: Update CONFIG_SYS_TEXT to the default in README.chromium

Tom,

Ping, could we get this applied please?

Peter

On Sun, Jun 10, 2018 at 6:17 AM, Peter Robinson <pbrobinson@gmail.com> wrote:
> To build U-Boot on a Nyan Big Chromebook the docs outline adjusting 
> the Tegra124 defined CONFIG_SYS_TEXT_BASE but this has since been 
> moved to individual config files. We should have the default required 
> for U-Boot chain loading on the chromebook as the default 
> CONFIG_SYS_TEXT_BASE and update the docs to remove this now non required step.
>
> Signed-off-by: Peter Robinson <pbrobinson@gmail.com>
> ---
>
> v3: Update commit message
> v2: Update to 2018.07 RC1
>
>  configs/nyan-big_defconfig |  2 +-
>  doc/README.chromium        | 27 +++++++--------------------
>  2 files changed, 8 insertions(+), 21 deletions(-)
>
> diff --git a/configs/nyan-big_defconfig b/configs/nyan-big_defconfig 
> index 8868d1f9e10..3b64c643716 100644
> --- a/configs/nyan-big_defconfig
> +++ b/configs/nyan-big_defconfig
> @@ -1,6 +1,6 @@
>  CONFIG_ARM=y
>  CONFIG_TEGRA=y
> -CONFIG_SYS_TEXT_BASE=0x80110000
> +CONFIG_SYS_TEXT_BASE=0x81000100
>  CONFIG_DEBUG_UART_BASE=0x70006000
>  CONFIG_DEBUG_UART_CLOCK=408000000
>  CONFIG_TEGRA124=y
> diff --git a/doc/README.chromium b/doc/README.chromium index 
> 7bf4d874474..45eaeced2da 100644
> --- a/doc/README.chromium
> +++ b/doc/README.chromium
> @@ -43,26 +43,13 @@ 
> https://git.collabora.com/cgit/user/tomeu/u-boot.git/commit/?h=nyan-bi
> g  https://lists.denx.de/pipermail/u-boot/2017-May/289491.html
>  
> https://github.com/chromeos-nvidia-androidtv/gnu-linux-on-acer-chromeb
> ook-13#copy-data-to-the-sd-card
>
> -1. Patch U-Boot
> -
> -Open include/configs/tegra124-common.h
> -
> -Change:
> -
> -#define CONFIG_SYS_TEXT_BASE   0x80110000
> -
> -to:
> -
> -#define CONFIG_SYS_TEXT_BASE   0x81000100
> -
> -
> -2. Build U-Boot
> +1. Build U-Boot
>
>     mkdir b
>     make -j8 O=b/nyan-big CROSS_COMPILE=arm-linux-gnueabi- 
> nyan-big_defconfig all
>
>
> -3. Select a .its file
> +2. Select a .its file
>
>  Select something from doc/chromium which matches your board, or 
> create your  own.
> @@ -72,7 +59,7 @@ used by U-Boot. This is because the Chromebook 
> expects to pass it to the  kernel, and crashes if it is not present.
>
>
> -4. Build and sign an image
> +3. Build and sign an image
>
>     ./b/nyan-big/tools/mkimage -f doc/chromium/nyan-big.its u-boot-chromium.fit
>     echo test >dummy.txt
> @@ -82,7 +69,7 @@ kernel, and crashes if it is not present.
>         --bootloader dummy.txt --pack u-boot.kpart
>
>
> -5. Prepare an SD card
> +4. Prepare an SD card
>
>     DISK=/dev/sdc   # Replace with your actual SD card device
>     sudo cgpt create $DISK
> @@ -91,12 +78,12 @@ kernel, and crashes if it is not present.
>     sudo gdisk $DISK   # Enter command 'w' to write a protective MBR to the disk
>
>
> -6. Write U-Boot to the SD card
> +5. Write U-Boot to the SD card
>
>     sudo dd if=u-boot.kpart of=/dev/sdc1; sync
>
>
> -7. Start it up
> +6. Start it up
>
>  Reboot the device in dev mode. Make sure that you have USB booting 
> enabled. To  do this, login as root (via Ctrl-Alt-forward_arrow) and 
> type @@ -115,7 +102,7 @@ mode screen. It should show something like the following on the display:
>     Tegra124 (Nyan-big) #
>
>
> -8. Known problems
> +7. Known problems
>
>  On the serial console the word MMC is chopped at the start of the line:
>
> --
> 2.17.1
>
--
nvpublic

[U-Boot] [PATCH v3] tegra: nyan-big: Update CONFIG_SYS_TEXT to the default in README.chromium

[Tom Rini]

On Fri, Jul 06, 2018 at 05:31:00PM +0000, Tom Warren wrote:

> Applied to tegra-master, just waiting on our automated build/test framework to give it a green light, then I'll issue a PR to TomR.  Sorry for the delay.

Please note that the release is scheduled for Monday, so if you have
stuff that should be in the release, that's OK, but otherwise it'll be
taken after.  Thanks!

> 
> Tom
> 
> -----Original Message-----
> From: Peter Robinson <pbrobinson@gmail.com> 
> Sent: Thursday, July 5, 2018 11:38 PM
> To: Allen Martin <AMartin@nvidia.com>; Simon Glass <sjg@chromium.org>; Tom Warren <TWarren@nvidia.com>; u-boot at lists.denx.de
> Cc: Peter Robinson <pbrobinson@gmail.com>
> Subject: Re: [PATCH v3] tegra: nyan-big: Update CONFIG_SYS_TEXT to the default in README.chromium
> 
> Tom,
> 
> Ping, could we get this applied please?
> 
> Peter
> 
> On Sun, Jun 10, 2018 at 6:17 AM, Peter Robinson <pbrobinson@gmail.com> wrote:
> > To build U-Boot on a Nyan Big Chromebook the docs outline adjusting 
> > the Tegra124 defined CONFIG_SYS_TEXT_BASE but this has since been 
> > moved to individual config files. We should have the default required 
> > for U-Boot chain loading on the chromebook as the default 
> > CONFIG_SYS_TEXT_BASE and update the docs to remove this now non required step.
> >
> > Signed-off-by: Peter Robinson <pbrobinson@gmail.com>
> > ---
> >
> > v3: Update commit message
> > v2: Update to 2018.07 RC1
> >
> >  configs/nyan-big_defconfig |  2 +-
> >  doc/README.chromium        | 27 +++++++--------------------
> >  2 files changed, 8 insertions(+), 21 deletions(-)
> >
> > diff --git a/configs/nyan-big_defconfig b/configs/nyan-big_defconfig 
> > index 8868d1f9e10..3b64c643716 100644
> > --- a/configs/nyan-big_defconfig
> > +++ b/configs/nyan-big_defconfig
> > @@ -1,6 +1,6 @@
> >  CONFIG_ARM=y
> >  CONFIG_TEGRA=y
> > -CONFIG_SYS_TEXT_BASE=0x80110000
> > +CONFIG_SYS_TEXT_BASE=0x81000100
> >  CONFIG_DEBUG_UART_BASE=0x70006000
> >  CONFIG_DEBUG_UART_CLOCK=408000000
> >  CONFIG_TEGRA124=y
> > diff --git a/doc/README.chromium b/doc/README.chromium index 
> > 7bf4d874474..45eaeced2da 100644
> > --- a/doc/README.chromium
> > +++ b/doc/README.chromium
> > @@ -43,26 +43,13 @@ 
> > https://git.collabora.com/cgit/user/tomeu/u-boot.git/commit/?h=nyan-bi
> > g  https://lists.denx.de/pipermail/u-boot/2017-May/289491.html
> >  
> > https://github.com/chromeos-nvidia-androidtv/gnu-linux-on-acer-chromeb
> > ook-13#copy-data-to-the-sd-card
> >
> > -1. Patch U-Boot
> > -
> > -Open include/configs/tegra124-common.h
> > -
> > -Change:
> > -
> > -#define CONFIG_SYS_TEXT_BASE   0x80110000
> > -
> > -to:
> > -
> > -#define CONFIG_SYS_TEXT_BASE   0x81000100
> > -
> > -
> > -2. Build U-Boot
> > +1. Build U-Boot
> >
> >     mkdir b
> >     make -j8 O=b/nyan-big CROSS_COMPILE=arm-linux-gnueabi- 
> > nyan-big_defconfig all
> >
> >
> > -3. Select a .its file
> > +2. Select a .its file
> >
> >  Select something from doc/chromium which matches your board, or 
> > create your  own.
> > @@ -72,7 +59,7 @@ used by U-Boot. This is because the Chromebook 
> > expects to pass it to the  kernel, and crashes if it is not present.
> >
> >
> > -4. Build and sign an image
> > +3. Build and sign an image
> >
> >     ./b/nyan-big/tools/mkimage -f doc/chromium/nyan-big.its u-boot-chromium.fit
> >     echo test >dummy.txt
> > @@ -82,7 +69,7 @@ kernel, and crashes if it is not present.
> >         --bootloader dummy.txt --pack u-boot.kpart
> >
> >
> > -5. Prepare an SD card
> > +4. Prepare an SD card
> >
> >     DISK=/dev/sdc   # Replace with your actual SD card device
> >     sudo cgpt create $DISK
> > @@ -91,12 +78,12 @@ kernel, and crashes if it is not present.
> >     sudo gdisk $DISK   # Enter command 'w' to write a protective MBR to the disk
> >
> >
> > -6. Write U-Boot to the SD card
> > +5. Write U-Boot to the SD card
> >
> >     sudo dd if=u-boot.kpart of=/dev/sdc1; sync
> >
> >
> > -7. Start it up
> > +6. Start it up
> >
> >  Reboot the device in dev mode. Make sure that you have USB booting 
> > enabled. To  do this, login as root (via Ctrl-Alt-forward_arrow) and 
> > type @@ -115,7 +102,7 @@ mode screen. It should show something like the following on the display:
> >     Tegra124 (Nyan-big) #
> >
> >
> > -8. Known problems
> > +7. Known problems
> >
> >  On the serial console the word MMC is chopped at the start of the line:
> >
> > --
> > 2.17.1
> >
> --
> nvpublic
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> https://lists.denx.de/listinfo/u-boot

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20180706/506b0a53/attachment.sig>