From mboxrd@z Thu Jan 1 00:00:00 1970 From: yingjoe.chen@mediatek.com (Yingjoe Chen) Date: Wed, 22 Oct 2014 21:31:36 +0800 Subject: [PATCH v4 8/8] ARM: mediatek: Add config option for mediatek SoCs. In-Reply-To: References: <1413973797-17619-1-git-send-email-yingjoe.chen@mediatek.com> <1413973797-17619-9-git-send-email-yingjoe.chen@mediatek.com> Message-ID: <1413984696.19686.72.camel@mtksdaap41> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Hi Matthias, On Wed, 2014-10-22 at 15:18 +0200, Matthias Brugger wrote: > 2014-10-22 12:29 GMT+02:00 Joe.C : > > + > > +if ARCH_MEDIATEK > > + > > +config MACH_MT6589 > > + bool "MediaTek MT6589 SoCs support" > > + default ARCH_MEDIATEK > > + > > +config MACH_MT8127 > > + bool "MediaTek MT8127 SoCs support" > > + default ARCH_MEDIATEK > > + > > +config MACH_MT8135 > > + bool "MediaTek MT8135 SoCs support" > > + default ARCH_MEDIATEK > > + > > +endif > > MACH_MTxxxx are not used nowhere, please remove them. Yes, they are not used in this series yet. MACH_MT8135 was used in mt8135 pinctrl driver [1] and will be used in mt8135 clk driver we are about to send. I don't want to create false dependency between these 2 series so I put it here. Do you prefer add them when they are first used? Joe.C [1] http://lists.infradead.org/pipermail/linux-arm-kernel/2014-September/289288.html