From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from ozlabs.org (ozlabs.org [IPv6:2401:3900:2:1::2]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by lists.ozlabs.org (Postfix) with ESMTPS id A11571A02B6 for ; Wed, 25 Mar 2015 11:45:45 +1100 (AEDT) Received: from na01-by2-obe.outbound.protection.outlook.com (mail-by2on0146.outbound.protection.outlook.com [207.46.100.146]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-SHA384 (256/256 bits)) (No client certificate requested) by ozlabs.org (Postfix) with ESMTPS id 506841400F1 for ; Wed, 25 Mar 2015 11:45:43 +1100 (AEDT) Message-ID: <1427244330.22867.76.camel@freescale.com> Subject: Re: [PATCH 0/8] powerpc/8xx: Getting rid of CONFIG_8xx From: Scott Wood To: Michael Ellerman Date: Tue, 24 Mar 2015 19:45:30 -0500 In-Reply-To: <1426203271.2772.3.camel@ellerman.id.au> References: <20150312152420.11C821A2425@localhost.localdomain> <1426203271.2772.3.camel@ellerman.id.au> Content-Type: text/plain; charset="UTF-8" MIME-Version: 1.0 Cc: linuxppc-dev list , Paul Mackerras List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On Fri, 2015-03-13 at 10:34 +1100, Michael Ellerman wrote: > On Thu, 2015-03-12 at 16:24 +0100, Christophe Leroy wrote: > > Two config options exist to define powerpc MPC8xx: > > * CONFIG_PPC_8xx > > * CONFIG_8xx > > In addition, CONFIG_PPC_8xx also defines CONFIG_CPM1 as > > communication co-processor > > > > arch/powerpc/platforms/Kconfig.cputype has contained the following > > comment about CONFIG_8xx item for some years: > > "# this is temp to handle compat with arch=ppc" > > > > It looks like not many places still have that old CONFIG_8xx used, > > so it is likely to be a good time to get rid of it completely ? > > > > Patchset is composed of the following patches: > > [1/8] powerpc: replace CONFIG_8xx by CONFIG_PPC_8xx > > [2/8] um: replace CONFIG_8xx by CONFIG_PPC_8xx > > [3/8] video: replace CONFIG_8xx by CONFIG_PPC_8xx > > [4/8] net: freescale: replace CONFIG_8xx by CONFIG_PPC_8xx > > [5/8] tty: cpm_uart: replace CONFIG_8xx by CONFIG_CPM1 > > [6/8] mtd: replace CONFIG_8xx by CONFIG_PPC_8xx > > [7/8] isdn: replace CONFIG_8xx by CONFIG_PPC_8xx > > [8/8] powerpc: get rid of CONFIG_8xx > > > > All but the last one are independant and can be applied in any > > order. Only the 8th one requires the first 7 patches to be applied. > > Sounds good. > > I only see 0, 4 and 7 on linuxppc-dev for some reason. I see them all in linuxppc-dev patchwork. > You'll need to collect ACKs, or get the individual patches merged, and then we > can take patch 8 through the powerpc tree once those are all in - probably for > 4.2. It looks like CONFIG_8xx is used a lot more than CONFIG_PPC_8xx, so it would be less churn to get rid of the latter (plus, we also have CONFIG_4xx, CONFIG_6xx, etc). The only use of PPC_8xx I see outside arch/powerpc is in drivers/watchdog/Kconfig. -Scott