Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1283576

Re: [PATCH v2 3/9] ARM: hisi: enable Hi3519 soc

Show all headers | View raw

Hi Arnd,

On 2015/12/3 17:40, Arnd Bergmann wrote:
> On Thursday 03 December 2015 10:42:45 Jiancheng Xue wrote:
>> --- a/arch/arm/mach-hisi/Kconfig
>> +++ b/arch/arm/mach-hisi/Kconfig
>> @@ -12,6 +12,14 @@ if ARCH_HISI
>>  
>>  menu "Hisilicon platform type"
>>  
>> +config ARCH_HI3519
>> +       bool "Hisilicon Hi3519 Soc" if ARCH_MULTI_V7
>> +       select HAVE_ARM_ARCH_TIMER
>> +       select ARCH_HAS_RESET_CONTROLLER
>> +
>> +       help
>> +         Support for Hisilicon Hi3519 Soc
>> +
>>  config ARCH_HI3xxx
>>         bool "Hisilicon Hi36xx family" if ARCH_MULTI_V7
>>         select CACHE_L2X0
> 
> Do those need to be separate? I would just extend the Hi36xx
> to cover all Hi3xxx, if nothing in the platform code really
> depends on this.

For HI3519, there is really no special platform code. But HI35xx and HI36xx soc families
belong to different product lines in hisilicon. HI35xx family also composes of various
architectures socs(single core, smp and big-little). So I think it may be clear to have
separate arch definitions.

Could you give me more suggestions about this?  Thank you!

>> +
>> +static const char *const hi3519_compat[] __initconst = {
>> +       "hisilicon,hi3519",
>> +       NULL,
>> +};
>> +
>> +DT_MACHINE_START(HI3519_DT, "Hisilicon Hi3519 (Flattened Device Tree)")
>> +       .dt_compat      = hi3519_compat,
>> +MACHINE_END
> 
> Also drop the "(Flattened Device Tree)" part of the name, we don't
> support any other kind anyway.

OK.

> 
> 	Arnd
> 
> .
> 

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH v2 3/9] ARM: hisi: enable Hi3519 soc Jiancheng Xue <xuejiancheng@huawei.com> - 2015-12-03 04:10 +0100
  Re: [PATCH v2 3/9] ARM: hisi: enable Hi3519 soc Arnd Bergmann <arnd@arndb.de> - 2015-12-03 10:50 +0100
    Re: [PATCH v2 3/9] ARM: hisi: enable Hi3519 soc xuejiancheng <xuejiancheng@huawei.com> - 2015-12-04 05:20 +0100
      Re: [PATCH v2 3/9] ARM: hisi: enable Hi3519 soc Arnd Bergmann <arnd@arndb.de> - 2015-12-04 23:00 +0100
        Re: [PATCH v2 3/9] ARM: hisi: enable Hi3519 soc xuejiancheng <xuejiancheng@huawei.com> - 2015-12-07 08:10 +0100
          Re: [PATCH v2 3/9] ARM: hisi: enable Hi3519 soc Arnd Bergmann <arnd@arndb.de> - 2015-12-07 10:50 +0100
            Re: [PATCH v2 3/9] ARM: hisi: enable Hi3519 soc xuejiancheng <xuejiancheng@huawei.com> - 2015-12-08 03:00 +0100
              Re: [PATCH v2 3/9] ARM: hisi: enable Hi3519 soc xuejiancheng <xuejiancheng@huawei.com> - 2015-12-08 04:10 +0100
                Re: [PATCH v2 3/9] ARM: hisi: enable Hi3519 soc Arnd Bergmann <arnd@arndb.de> - 2015-12-09 16:40 +0100
                Re: [PATCH v2 3/9] ARM: hisi: enable Hi3519 soc xuejiancheng <xuejiancheng@huawei.com> - 2015-12-10 07:20 +0100
                Re: [PATCH v2 3/9] ARM: hisi: enable Hi3519 soc Arnd Bergmann <arnd@arndb.de> - 2015-12-10 09:30 +0100

csiph-web