From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jeff Garzik Subject: Re: [PATCH] pata_pcmcia/ide-cs: move IDs table to Date: Fri, 14 Oct 2011 12:55:36 -0400 Message-ID: <4E986988.6080106@pobox.com> References: <201110131312.14498.bzolnier@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Return-path: Received: from mail-qw0-f46.google.com ([209.85.216.46]:42485 "EHLO mail-qw0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755236Ab1JNQzk (ORCPT ); Fri, 14 Oct 2011 12:55:40 -0400 In-Reply-To: <201110131312.14498.bzolnier@gmail.com> Sender: linux-ide-owner@vger.kernel.org List-Id: linux-ide@vger.kernel.org To: Bartlomiej Zolnierkiewicz , David Miller Cc: linux-ide@vger.kernel.org, linux-kernel@vger.kernel.org, Alan Cox On 10/13/2011 07:12 AM, Bartlomiej Zolnierkiewicz wrote: > From: Bartlomiej Zolnierkiewicz > Subject: [PATCH] pata_pcmcia/ide-cs: move IDs table to > > Signed-off-by: Bartlomiej Zolnierkiewicz > --- > drivers/ata/pata_pcmcia.c | 74 +--------------------------------------- > drivers/ide/ide-cs.c | 74 +--------------------------------------- > include/linux/pata_pcmcia_ids.h | 70 +++++++++++++++++++++++++++++++++++++ > 3 files changed, 75 insertions(+), 143 deletions(-) I'm torn... it is an obvious and simple consolidation, but I dislike dropping such a driver-specific turd in include/linux. I also wonder if we want to continue adding new IDs to ide-cs? David, any opinions? I'm basically ok with any option: (a) take Bart's patch as-is (with your acked-by), (b) ignore the patch and set a no-new-ids policy for ide-cs, (c) other? Jeff