On Mon, Nov 09, 2009 at 08:08:18PM +0800, Andy Yan wrote:
> +void mvs_enable_msi(struct mvs_info *mvi)
> +{
> + u32 tmp;
> + pci_read_config_dword(mvi->pdev, PCR_CMD, &tmp);
> + tmp |= 1 << 10; /* disable interrupt */
> + pci_write_config_dword(mvi->pdev, PCR_CMD, tmp);
What makes you think the pci_enable_msi() code doesn't set this bit?
> + pci_read_config_dword(mvi->pdev, PCR_MSI_CTRL, &tmp);
> + tmp |= 1 << 16; /* enable MSI */
> + pci_write_config_dword(mvi->pdev, PCR_MSI_CTRL, tmp);
... and what makes you think it doesn't set this bit either?
> +static ssize_t
> +mvs_show_driver_version(struct device *cdev,
> + struct device_attribute *attr,
> + char *buffer)
> +{
> + return snprintf(buffer, PAGE_SIZE, "%s\n", DRV_VERSION);
> +}
There's a lot of code in this patch not related to enabling MSI.
--
Matthew Wilcox Intel Open Source Technology Centre
"Bill, look, we understand that you're interested in selling us this
operating system, but compare it to ours. We can't possibly take such
a retrograde step."
--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
