From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S934187AbYBNVKz (ORCPT ); Thu, 14 Feb 2008 16:10:55 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S932957AbYBNVKd (ORCPT ); Thu, 14 Feb 2008 16:10:33 -0500 Received: from hu-out-0506.google.com ([72.14.214.238]:21632 "EHLO hu-out-0506.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1765245AbYBNVKa (ORCPT ); Thu, 14 Feb 2008 16:10:30 -0500 DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=from:to:subject:date:user-agent:cc:references:in-reply-to:mime-version:content-disposition:message-id:content-type:content-transfer-encoding; b=Jx3o1JDxjGKXQjMo1zNyxak16SAfUdYjKBJydu/vhiCk1i6N0fsxzVVc360F9/RBQtRkavfTHJ2ygl7sQKLYGQK024fXeUAx4nYD+FbB3Jpn8D727b8IyRx0YhGILZP15PSb2pRl3MtdH4uQ4bKa10KL2w8Cx1tqAGh//I65eFA= From: Bartlomiej Zolnierkiewicz To: Sam Ravnborg Subject: Re: [PATCH 1/2] ide: add CONFIG_IDE_ARCH_OBSOLETE_DEFAULTS Date: Thu, 14 Feb 2008 22:24:22 +0100 User-Agent: KMail/1.9.6 (enterprise 0.20071204.744707) Cc: linux-ide@vger.kernel.org, linux-kernel@vger.kernel.org References: <200802142155.50893.bzolnier@gmail.com> <20080214205500.GE21894@uranus.ravnborg.org> In-Reply-To: <20080214205500.GE21894@uranus.ravnborg.org> MIME-Version: 1.0 Content-Disposition: inline Message-Id: <200802142224.22621.bzolnier@gmail.com> Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi, On Thursday 14 February 2008, Sam Ravnborg wrote: > Hi Bart. > > On Thu, Feb 14, 2008 at 09:55:50PM +0100, Bartlomiej Zolnierkiewicz wrote: > > Add CONFIG_IDE_ARCH_OBSOLETE_DEFAULTS to drivers/ide/Kconfig and use > > it instead of defining IDE_ARCH_OBSOLETE_DEFAULTS in . > > > > Signed-off-by: Bartlomiej Zolnierkiewicz > > --- > > drivers/ide/Kconfig | 3 +++ > > drivers/ide/ide.c | 5 +++++ > > include/asm-alpha/ide.h | 3 --- > > include/asm-ia64/ide.h | 2 -- > > include/asm-m32r/ide.h | 2 -- > > include/asm-mips/mach-generic/ide.h | 2 -- > > include/asm-powerpc/ide.h | 2 -- > > include/asm-x86/ide.h | 2 -- > > include/linux/ide.h | 7 ------- > > 9 files changed, 8 insertions(+), 20 deletions(-) > > > > Index: b/drivers/ide/Kconfig > > =================================================================== > > --- a/drivers/ide/Kconfig > > +++ b/drivers/ide/Kconfig > > @@ -1099,6 +1099,9 @@ config BLK_DEV_IDEDMA > > config IDE_ARCH_OBSOLETE_INIT > > def_bool ALPHA || (ARM && !ARCH_L7200) || BLACKFIN || X86 || IA64 || M32R || MIPS || PARISC || PPC || (SUPERH64 && BLK_DEV_IDEPCI) || SPARC > > > > +config IDE_ARCH_OBSOLETE_DEFAULTS > > + def_bool ALPHA || X86 || IA64 || M32R || MIPS || PPC32 > > + > > endif > > > This time it is maybe borderline as it makes sense to see a list of > archs that uses this. > But still I would prefer you did like this: > > drivers/ide/Kconfig: > > config HAVE_IDE_ARCH_OBSOLETE_DEFAULTS > bool > > config IDE_ARCH_OBSOLETE_DEFAULTS > depends on HAVE_IDE_ARCH_OBSOLETE_DEFAULTS > > And then in each arch Kconfig include: > > arch/alpha/Kconfig: > config ALPHA > select HAVE_ARCH_OBSOLETE_DEFAULTS > > And the same for the others. I agree but this option is meant to be only temporary to ease the cleanup process. We can think about the proper solution after default IDE ports initialization gets moved to ide_generic host driver. Thanks, Bart