From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jeff Garzik Subject: Re: [PATCH 1/1] ata/sata_sil24: MSI support, disabled by default Date: Tue, 17 Nov 2009 02:42:07 -0500 Message-ID: <4B0253CF.3090506@garzik.org> References: <1258352362-11231-1-git-send-email-vivek.mahajan@freescale.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Return-path: Received: from srv5.dvmed.net ([207.36.208.214]:60773 "EHLO mail.dvmed.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755823AbZKQHmH (ORCPT ); Tue, 17 Nov 2009 02:42:07 -0500 In-Reply-To: Sender: linux-ide-owner@vger.kernel.org List-Id: linux-ide@vger.kernel.org To: Mahajan Vivek-B08308 Cc: Grant Grundler , linux-ide@vger.kernel.org, linuxppc-dev@ozlabs.org On 11/17/2009 01:59 AM, Mahajan Vivek-B08308 wrote: >> From: Grant Grundler [mailto:grundler@google.com] >> Sent: Monday, November 16, 2009 11:08 PM >>> +static int sata_sil24_msi; /* Disable MSI */ >>> +module_param_named(msi, sata_sil24_msi, bool, S_IRUGO); >>> +MODULE_PARM_DESC(msi, "Enable MSI (Default: false)"); >> >> Vivek, >> Do we even still need the parameter? I'm thinking either MSI >> works with a chipset or it doesn't. The kernel has globals to >> "know" which state is true. > > Sometimes even in a platform, some PCIe endpoints do very > well with MSI while others may have to resort to legacy ints. > Should we let the endpoints make the final call. > >> >> If the parameter is needed, when this driver is compiled into >> the kernel, how is "msi" parameter specified? >> I think the parameter needs to be documented and fit in with >> other "msi" parameters. >> See "nomsi" in Documentation/kernel-parameters.txt. > > In this case "msi" is supposed to be passed via insmod and > not via kernel cmdline. If the driver is built-in the kernel, > then force sata_sil24_msi = 1 in the driver to enable it. First, the original patch was just fine, and it was applied. You should have received email confirmation of this already. Second, all module options are available on the kernel command line, when a module is built into the kernel. You supply a module name prefix to each module option, on the kernel command line. Jeff