public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Tom Rini <trini@konsulko.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH] arc: make AXS101 default platform
Date: Mon, 24 Aug 2015 12:09:03 -0400	[thread overview]
Message-ID: <20150824160903.GE25532@bill-the-cat> (raw)
In-Reply-To: <1440432012-14835-1-git-send-email-abrodkin@synopsys.com>

On Mon, Aug 24, 2015 at 07:00:12PM +0300, Alexey Brodkin wrote:
> This fixes building in automated flow that doesn't use defconfigs.
> 
> See discussion on that topic here:
>  http://patchwork.ozlabs.org/patch/502558/
> 
> See similar patches for other architectures/platforms  here:
>  [1] http://git.denx.de/?p=u-boot.git;a=commit;h=ff560a13056a565a4e9ce1761bd04276a3cace88
>  [2] http://git.denx.de/?p=u-boot.git;a=commit;h=589907e2c187ec69b351c38ccda36730d25ab5d6
> 
> And while at it add missing shell prompt to axs103.
> 
> Cc: Tom Rini <trini@konsulko.com>
> Cc: Joe Hershberger <joe.hershberger@gmail.com>
> Cc: Masahiro Yamada <yamada.masahiro@socionext.com>
> Signed-off-by: Alexey Brodkin <abrodkin@synopsys.com>
> ---
>  arch/arc/Kconfig         | 2 +-
>  configs/axs101_defconfig | 3 +--
>  configs/axs103_defconfig | 2 +-
>  3 files changed, 3 insertions(+), 4 deletions(-)
> 
> diff --git a/arch/arc/Kconfig b/arch/arc/Kconfig
> index 925e312..264c83d 100644
> --- a/arch/arc/Kconfig
> +++ b/arch/arc/Kconfig
> @@ -129,7 +129,7 @@ config ARC_CACHE_LINE_SHIFT
>  
>  choice
>  	prompt "Target select"
> -	optional
> +	default TARGET_AXS101
>  
>  config TARGET_TB100
>  	bool "Support tb100"
> diff --git a/configs/axs101_defconfig b/configs/axs101_defconfig
> index 9a6cef3..fdb3149 100644
> --- a/configs/axs101_defconfig
> +++ b/configs/axs101_defconfig
> @@ -1,13 +1,12 @@
>  CONFIG_ARC=y
>  CONFIG_SYS_DCACHE_OFF=y
>  CONFIG_ARC_CACHE_LINE_SHIFT=5
> -CONFIG_TARGET_AXS101=y
>  CONFIG_SYS_CLK_FREQ=750000000
>  CONFIG_SYS_TEXT_BASE=0x81000000
> +CONFIG_SYS_PROMPT="AXS# "
>  # CONFIG_CMD_IMLS is not set
>  # CONFIG_CMD_FLASH is not set
>  # CONFIG_CMD_SETEXPR is not set
>  CONFIG_NETDEVICES=y
>  CONFIG_ETH_DESIGNWARE=y
>  CONFIG_USE_PRIVATE_LIBGCC=y
> -CONFIG_SYS_PROMPT="AXS# "
> diff --git a/configs/axs103_defconfig b/configs/axs103_defconfig
> index 8315b61..b3dbb8d 100644
> --- a/configs/axs103_defconfig
> +++ b/configs/axs103_defconfig
> @@ -1,8 +1,8 @@
>  CONFIG_ARC=y
>  CONFIG_ISA_ARCV2=y
> -CONFIG_TARGET_AXS101=y
>  CONFIG_SYS_CLK_FREQ=50000000
>  CONFIG_SYS_TEXT_BASE=0x81000000
> +CONFIG_SYS_PROMPT="AXS# "
>  # CONFIG_CMD_IMLS is not set
>  # CONFIG_CMD_FLASH is not set
>  # CONFIG_CMD_SETEXPR is not set

It all makes sense.  But.. should it be TARGET_ASX10X ?  Or is there (or
might be..) future 10x devices that wouldn't be supported by the same
code / config option?

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20150824/3d30064b/attachment.sig>

  reply	other threads:[~2015-08-24 16:09 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-08-24 16:00 [U-Boot] [PATCH] arc: make AXS101 default platform Alexey Brodkin
2015-08-24 16:09 ` Tom Rini [this message]
2015-08-24 16:12   ` Alexey Brodkin
2015-08-24 20:00     ` Tom Rini
2015-09-07 10:47 ` Alexey Brodkin

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=20150824160903.GE25532@bill-the-cat \
    --to=trini@konsulko.com \
    --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