From mboxrd@z Thu Jan  1 00:00:00 1970
From: Bo Shen <voice.shen@atmel.com>
Date: Thu, 6 Nov 2014 14:21:26 +0800
Subject: [U-Boot] [PATCH v3 3/3] ARM: atmel: add sama5d4ek board support
In-Reply-To: <20141106151303.7120.AA925319@jp.panasonic.com>
References: <1414559526-24602-1-git-send-email-voice.shen@atmel.com>
	<1414559526-24602-4-git-send-email-voice.shen@atmel.com>
	<20141106151303.7120.AA925319@jp.panasonic.com>
Message-ID: <545B1366.1080807@atmel.com>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

Hi Masahiro,

On 11/06/2014 02:13 PM, Masahiro Yamada wrote:
>
> On Wed, 29 Oct 2014 13:12:06 +0800
> Bo Shen <voice.shen@atmel.com> wrote:
>
>> The code for this board supports following features:
>>    - Boot media support: NAND flash/SD card/SPI flash
>>    - Support LCD display
>>    - Support ethernet
>>    - Support USB mass storage
>>
>> Signed-off-by: Bo Shen <voice.shen@atmel.com>
>> ---
>
>> diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig
>> index 8f910f3..8234eb6 100644
>> --- a/arch/arm/Kconfig
>> +++ b/arch/arm/Kconfig
>> @@ -330,6 +330,9 @@ config TARGET_SAMA5D3_XPLAINED
>>   config TARGET_SAMA5D3XEK
>>   	bool "Support sama5d3xek"
>>
>> +config TARGET_SAMA5D4EK
>> +	bool "Support sama5d4ek"
>> +
>>   config TARGET_BCM28155_AP
>>   	bool "Support bcm28155_ap"
>>
>
> When you rebase this patch,
> please make sure to add "select CPU_V7"

OK, thanks.

>
>> diff --git a/arch/arm/include/asm/arch-at91/sama5d4.h b/arch/arm/include/asm/arch-at91/sama5d4.h
>> new file mode 100644
>> index 0000000..6ebd38a
>> --- /dev/null
>> +++ b/arch/arm/include/asm/arch-at91/sama5d4.h
>> @@ -0,0 +1,207 @@
>> +/*
>> + * Chip-specific header file for the SAMA5D4 SoC
>> + *
>> + * Copyright (C) 2014 Atmel
>> + *		      Bo Shen <voice.shen@atmel.com>
>> + *
>> + * SPDX-License-Identifier:     GPL-2.0+
>> + */
>> +
>> +#ifndef __SAMA5D4_H
>> +#define __SAMA5D4_H
>> +
>> +/*
>> + * defines to be used in other places
>> + */
>> +#define CONFIG_ARMV7		/* ARM Cortex-A5 Core */
>
> If you have a chance to send the next version,
> I recommend to drop the CONFIG_ARMV7 define.

Thanks for remind this.

> I am removing this macro.
> http://patchwork.ozlabs.org/patch/407305/
>
>
> Best Regards
> Masahiro Yamada
>

Best Regards,
Bo Shen