Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932144AbWIYOKY (ORCPT ); Mon, 25 Sep 2006 10:10:24 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S932142AbWIYOKX (ORCPT ); Mon, 25 Sep 2006 10:10:23 -0400 Received: from mtagate2.de.ibm.com ([195.212.29.151]:7206 "EHLO mtagate2.de.ibm.com") by vger.kernel.org with ESMTP id S932134AbWIYOKW (ORCPT ); Mon, 25 Sep 2006 10:10:22 -0400 Date: Mon, 25 Sep 2006 17:10:19 +0300 From: Muli Ben-Yehuda To: James Bottomley Cc: Alan Cox , linux-scsi , Linux-Kernel Subject: Re: mainline aic94xx firmware woes Message-ID: <20060925141019.GP6374@rhun.haifa.ibm.com> References: <20060925101124.GH6374@rhun.haifa.ibm.com> <1159183984.11049.59.camel@localhost.localdomain> <1159184336.3463.3.camel@mulgrave.il.steeleye.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1159184336.3463.3.camel@mulgrave.il.steeleye.com> User-Agent: Mutt/1.5.11 Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1403 Lines: 39 On Mon, Sep 25, 2006 at 06:38:56AM -0500, James Bottomley wrote: > On Mon, 2006-09-25 at 12:33 +0100, Alan Cox wrote: > > We should not be including non-free firmware in the kernel, we should be > > continuing to drive it out into things like initramfs. > > Right, which is why this was done as one of the conditions for accepting > the driver Fair enough. > > > Also, aic94xx does not compile unless FW_LOADER is set in .config due > > > to missing 'request_firmware'. What's the right thing to do here - > > > aic94xx selecting it, depending on it > > > > Either select or depend > > select, I think. aic94xx relies on external firmware and thus requires FW_LOADER. Signed-off-by: Muli Ben-Yehuda diff -r 2e01eba444f0 drivers/scsi/aic94xx/Kconfig --- a/drivers/scsi/aic94xx/Kconfig Mon Sep 25 10:07:49 2006 +0700 +++ b/drivers/scsi/aic94xx/Kconfig Mon Sep 25 17:08:09 2006 +0300 @@ -28,6 +28,7 @@ config SCSI_AIC94XX tristate "Adaptec AIC94xx SAS/SATA support" depends on PCI select SCSI_SAS_LIBSAS + select FW_LOADER help This driver supports Adaptec's SAS/SATA 3Gb/s 64 bit PCI-X AIC94xx chip based host adapters. - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/