public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Lukasz Majewski <lukma@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 5/5] davinci: omapl138_lcdk: switch to using common mmc args
Date: Thu, 6 Apr 2017 12:06:56 +0200	[thread overview]
Message-ID: <20170406120656.08ca60c9@jawa> (raw)
In-Reply-To: <20170406092257.15541-6-nsekhar@ti.com>

On Thu, 6 Apr 2017 14:52:57 +0530
Sekhar Nori <nsekhar@ti.com> wrote:

> Now that we have common MMC/SD boot environment
> variables that can be used across TI platforms,
> switch OMAP-L138 LCDK to use them.
> 
> As a nice side-effect, we get support for using
> uEnv.txt on this platform.
> 
> Signed-off-by: Sekhar Nori <nsekhar@ti.com>
> ---
>  include/configs/omapl138_lcdk.h | 23 ++++++++++-------------
>  1 file changed, 10 insertions(+), 13 deletions(-)
> 
> diff --git a/include/configs/omapl138_lcdk.h
> b/include/configs/omapl138_lcdk.h index 599fd5b7a874..e70760402b43
> 100644 --- a/include/configs/omapl138_lcdk.h
> +++ b/include/configs/omapl138_lcdk.h
> @@ -261,30 +261,27 @@
>  #define CONFIG_CMDLINE_TAG
>  #define CONFIG_REVISION_TAG
>  #define CONFIG_SETUP_MEMORY_TAGS
> -#define CONFIG_BOOTARGS		"console=ttyS2,115200n8
> root=/dev/mmcblk0p2 rw rootwait ip=off" #define CONFIG_BOOTCOMMAND \
> -	"if mmc rescan; then " \
> +		"run envboot; " \
>  		"run mmcboot; "
> -	"fi"
>  
>  #define DEFAULT_LINUX_BOOT_ENV \
>  	"loadaddr=0xc0700000\0" \
>  	"fdtaddr=0xc0600000\0" \
>  	"scriptaddr=0xc0600000\0"
>  
> +#include <environment/ti/mmc.h>
> +
>  #define CONFIG_EXTRA_ENV_SETTINGS \
>  	DEFAULT_LINUX_BOOT_ENV \
> +	DEFAULT_MMC_TI_ARGS \
> +	"bootpart=0:2\0" \
> +	"bootdir=/boot\0" \
> +	"bootfile=zImage\0" \
>  	"fdtfile=da850-lcdk.dtb\0" \
> -	"fdtboot=bootz ${loadaddr} - ${fdtaddr};\0" \
> -	"mmcboot=" \
> -		"if fatload mmc 0 ${scriptaddr} boot.scr; then " \
> -			"source ${scriptaddr}; " \
> -		"else " \
> -			"fatload mmc 0 ${loadaddr} " \
> -				__stringify(CONFIG_BOOTFILE) "; " \
> -			"fatload mmc 0 ${fdtaddr} ${fdtfile}; " \
> -			"run fdtboot; " \
> -		"fi;\0"
> +	"boot_fdt=yes\0" \
> +	"boot_fit=0\0" \
> +	"console=ttyS2,115200n8\0"
>  
>  /*
>   * U-Boot commands

Reviewed-by: Lukasz Majewski <lukma@denx.de>


Best regards,

Lukasz Majewski

--

DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de

  reply	other threads:[~2017-04-06 10:06 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-04-06  9:22 [U-Boot] [PATCH 0/5] davinci: omapl138_lcdk: align mmcboot with rest of TI boards Sekhar Nori
2017-04-06  9:22 ` [U-Boot] [PATCH 1/5] davinci: omapl138_lcdk: remove spiboot Sekhar Nori
2017-04-06  9:57   ` Lukasz Majewski
2017-04-06 16:23   ` Tom Rini
2017-04-10 18:24   ` [U-Boot] [U-Boot,1/5] " Tom Rini
2017-04-06  9:22 ` [U-Boot] [PATCH 2/5] davinci: omapl138_lcdk: enable some filesystem related commands Sekhar Nori
2017-04-06  9:58   ` Lukasz Majewski
2017-04-07  5:20     ` Sekhar Nori
2017-04-06 16:23   ` Tom Rini
2017-04-10 18:24   ` [U-Boot] [U-Boot, " Tom Rini
2017-04-06  9:22 ` [U-Boot] [PATCH 3/5] davinci: omapl138_lcdk: use environment variables for memory addresses Sekhar Nori
2017-04-06  9:59   ` Lukasz Majewski
2017-04-06 16:23   ` Tom Rini
2017-04-10 18:24   ` [U-Boot] [U-Boot, " Tom Rini
2017-04-06  9:22 ` [U-Boot] [PATCH 4/5] ARM: ti: consolidate mmc environment variables Sekhar Nori
2017-04-06 10:05   ` Lukasz Majewski
2017-04-06 16:23   ` Tom Rini
2017-04-10 18:24   ` [U-Boot] [U-Boot, " Tom Rini
2017-04-06  9:22 ` [U-Boot] [PATCH 5/5] davinci: omapl138_lcdk: switch to using common mmc args Sekhar Nori
2017-04-06 10:06   ` Lukasz Majewski [this message]
2017-04-06 16:23   ` Tom Rini
2017-04-10 18:24   ` [U-Boot] [U-Boot, " Tom Rini

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20170406120656.08ca60c9@jawa \
    --to=lukma@denx.de \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox