From mboxrd@z Thu Jan  1 00:00:00 1970
From: rick at andestech.com <rick@andestech.com>
Date: Mon, 7 May 2018 02:33:07 +0000
Subject: [U-Boot] [PATCH v3 7/8] riscv: nx25: Enable distro boot
In-Reply-To: <20180423055950.78818-8-agraf@suse.de>
References: <20180423055950.78818-1-agraf@suse.de>
 <20180423055950.78818-8-agraf@suse.de>
Message-ID: <752D002CFF5D0F4FA35C0100F1D73F3F6B78FFDA@ATCPCS16.andestech.com>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 8bit
To: u-boot@lists.denx.de



> -----Original Message-----
> From: Alexander Graf [mailto:agraf at suse.de]
> Sent: Monday, April 23, 2018 2:00 PM
> To: u-boot at lists.denx.de
> Cc: Rick Jian-Zhi Chen(陳建志); Greentime Hu; Philipp Tomsich; Heinrich
> Schuchardt; schwab at suse.de
> Subject: [PATCH v3 7/8] riscv: nx25: Enable distro boot
>
> Distro boot allows for a common boot path on systems that allow distributions to
> easily boot from a default configuration.
>
> This patch enables distro boot for the nx25-ae250. Hopefully this can serve as a
> good example for new boards, so they enable it as well.
>
> Signed-off-by: Alexander Graf <agraf@suse.de>
> ---
>  configs/nx25-ae250_defconfig |  1 +
>  include/configs/nx25-ae250.h | 17 +++++++++++++++++
>  2 files changed, 18 insertions(+)
>
> diff --git a/configs/nx25-ae250_defconfig b/configs/nx25-ae250_defconfig index
> 4f9bd58f75..437083231b 100644
> --- a/configs/nx25-ae250_defconfig
> +++ b/configs/nx25-ae250_defconfig
> @@ -37,3 +37,4 @@ CONFIG_DM_SPI=y
>  CONFIG_ATCSPI200_SPI=y
>  CONFIG_TIMER=y
>  CONFIG_ATCPIT100_TIMER=y
> +CONFIG_DISTRO_DEFAULTS=y
> diff --git a/include/configs/nx25-ae250.h b/include/configs/nx25-ae250.h index
> 0e4c431cab..a90c75abc4 100644
> --- a/include/configs/nx25-ae250.h
> +++ b/include/configs/nx25-ae250.h
> @@ -105,4 +105,21 @@
>  /* Increase max gunzip size */
>  #define CONFIG_SYS_BOOTM_LEN (64 << 20)
>
> +/* When we use RAM as ENV */
> +#define CONFIG_ENV_SIZE 0x2000
> +
> +/* Enable distro boot */
> +#define BOOT_TARGET_DEVICES(func) \
> +     func(MMC, mmc, 0) \
> +     func(DHCP, dhcp, na)
> +#include <config_distro_bootcmd.h>
> +
> +#define CONFIG_EXTRA_ENV_SETTINGS    \
> +                             "kernel_addr_r=0x00080000\0" \
> +                             "pxefile_addr_r=0x01f00000\0" \
> +                             "scriptaddr=0x01f00000\0" \
> +                             "fdt_addr_r=0x02000000\0" \
> +                             "ramdisk_addr_r=0x02800000\0" \
> +                             BOOTENV
> +

Maybe Add #ifdef CONFIG_DISTRO_DEFAULTS to encapsulate the CONFIG_EXTRA_ENV_SETTINGS can disable auto boot easily.

Reviewed-by: Rick Chen <rick@andestech.com>

>  #endif /* __CONFIG_H */
> --
> 2.12.3

CONFIDENTIALITY NOTICE:

This e-mail (and its attachments) may contain confidential and legally privileged information or information protected from disclosure. If you are not the intended recipient, you are hereby notified that any disclosure, copying, distribution, or use of the information contained herein is strictly prohibited. In this case, please immediately notify the sender by return e-mail, delete the message (and any accompanying documents) and destroy all printed hard copies. Thank you for your cooperation.

Copyright ANDES TECHNOLOGY CORPORATION - All Rights Reserved.