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. > + > +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. Arnd -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/