diff mbox

[v4,5/5] ARM: config: append arch hi3xxx into multi defconfig

Message ID 1363245584-24488-6-git-send-email-haojian.zhuang@linaro.org
State Superseded
Headers show

Commit Message

Haojian Zhuang March 14, 2013, 7:19 a.m. UTC 
Append ARCH_HI3xxx (Hisilicon SoC) into multi_v7_defconfig.

Signed-off-by: Haojian Zhuang <haojian.zhuang@linaro.org>
---
 arch/arm/configs/multi_v7_defconfig |    1 +
 1 file changed, 1 insertion(+)

Comments

Arnd Bergmann March 14, 2013, 9:10 a.m. UTC | #1 
On Thursday 14 March 2013, Haojian Zhuang wrote:
> --- a/arch/arm/configs/multi_v7_defconfig
> +++ b/arch/arm/configs/multi_v7_defconfig
> @@ -5,6 +5,7 @@ CONFIG_ARCH_MVEBU=y
>  CONFIG_MACH_ARMADA_370=y
>  CONFIG_MACH_ARMADA_XP=y
>  CONFIG_ARCH_HIGHBANK=y
> +CONFIG_ARCH_HI3xxx=y
>  CONFIG_ARCH_SOCFPGA=y
>  CONFIG_ARCH_SUNXI=y
>  # CONFIG_ARCH_VEXPRESS_CORTEX_A5_A9_ERRATA is not set

Are you sure this is enough? Normally we also have to enable
some device drivers to get a bootable system.

	Arnd
Haojian Zhuang March 14, 2013, 9:47 a.m. UTC | #2 
On 14 March 2013 17:10, Arnd Bergmann <arnd@arndb.de> wrote:
> On Thursday 14 March 2013, Haojian Zhuang wrote:
>> --- a/arch/arm/configs/multi_v7_defconfig
>> +++ b/arch/arm/configs/multi_v7_defconfig
>> @@ -5,6 +5,7 @@ CONFIG_ARCH_MVEBU=y
>>  CONFIG_MACH_ARMADA_370=y
>>  CONFIG_MACH_ARMADA_XP=y
>>  CONFIG_ARCH_HIGHBANK=y
>> +CONFIG_ARCH_HI3xxx=y
>>  CONFIG_ARCH_SOCFPGA=y
>>  CONFIG_ARCH_SUNXI=y
>>  # CONFIG_ARCH_VEXPRESS_CORTEX_A5_A9_ERRATA is not set
>
> Are you sure this is enough? Normally we also have to enable
> some device drivers to get a bootable system.
>
>         Arnd

Yes, it could boot.  Those components are already enabled in multi_v7_defconfig.
Since hi3xxx used a lot of AMBA IPs.

Regards
Haojian
Arnd Bergmann March 14, 2013, 1:32 p.m. UTC | #3 
On Thursday 14 March 2013, Haojian Zhuang wrote:
> On 14 March 2013 17:10, Arnd Bergmann <arnd@arndb.de> wrote:
> >
> > Are you sure this is enough? Normally we also have to enable
> > some device drivers to get a bootable system.
> >
> Yes, it could boot.  Those components are already enabled in multi_v7_defconfig.
> Since hi3xxx used a lot of AMBA IPs.

Ok, very good then.

	Arnd
diff mbox

Patch

diff --git a/arch/arm/configs/multi_v7_defconfig b/arch/arm/configs/multi_v7_defconfig
index e31d442..1891611 100644
--- a/arch/arm/configs/multi_v7_defconfig
+++ b/arch/arm/configs/multi_v7_defconfig
@@ -5,6 +5,7 @@  CONFIG_ARCH_MVEBU=y
 CONFIG_MACH_ARMADA_370=y
 CONFIG_MACH_ARMADA_XP=y
 CONFIG_ARCH_HIGHBANK=y
+CONFIG_ARCH_HI3xxx=y
 CONFIG_ARCH_SOCFPGA=y
 CONFIG_ARCH_SUNXI=y
 # CONFIG_ARCH_VEXPRESS_CORTEX_A5_A9_ERRATA is not set