Re: [PATCH] x86/PCI: do not tie MSI MS-7253 use_crs quirk to BIOS version

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Feb 28, 2012 at 2:31 PM, Jonathan Nieder <jrnieder@xxxxxxxxx> wrote:
> Carlos was getting
>
>        WARNING: at drivers/pci/pci.c:118 pci_ioremap_bar+0x24/0x52()
>
> when probing his sound card, and sound did not work.  After adding
> pci=use_crs to the kernel command line, no more trouble.
>
> Ok, we can add a quirk.  dmidecode output reveals that this is an MSI
> MS-7253, for which we already have a quirk, but the short-sighted
> author tied the quirk to a single BIOS version, making it not kick in
> on Carlos's machine with BIOS V1.2.  If a later BIOS update makes it
> no longer necessary to look at the _CRS info it will still be
> harmless, so let's stop trying to guess which versions have and don't
> have accurate _CRS tables.
>
> Addresses https://bugtrack.alsa-project.org/alsa-bug/view.php?id=5533
> Also see <https://bugzilla.kernel.org/show_bug.cgi?id=42619>.

Thanks, Jonathan.  I was going to do this, but you saved me the trouble :)

Reviewed-by: Bjorn Helgaas <bhelgaas@xxxxxxxxxx>

> Reported-by: Carlos Luna <caralu74@xxxxxxxxx>
> Signed-off-by: Jonathan Nieder <jrnieder@xxxxxxxxx>
> ---
> Hi,
>
> Could you squash this into the patch "x86/PCI: use host bridge _CRS
> info on MSI MS-7253", if it's not too late?  Sorry for the thinko.
>
> Thanks,
> Jonathan
>
>  arch/x86/pci/acpi.c |    1 -
>  1 file changed, 1 deletion(-)
>
> diff --git a/arch/x86/pci/acpi.c b/arch/x86/pci/acpi.c
> index 2c5cbdb5d602..d066e11159f3 100644
> --- a/arch/x86/pci/acpi.c
> +++ b/arch/x86/pci/acpi.c
> @@ -68,7 +68,6 @@ static const struct dmi_system_id pci_use_crs_table[] __initconst = {
>                        DMI_MATCH(DMI_BOARD_VENDOR, "MICRO-STAR INTERNATIONAL CO., LTD"),
>                        DMI_MATCH(DMI_BOARD_NAME, "MS-7253"),
>                        DMI_MATCH(DMI_BIOS_VENDOR, "Phoenix Technologies, LTD"),
> -                       DMI_MATCH(DMI_BIOS_VERSION, "V1.6"),
>                },
>        },
>
> --
> 1.7.9.2
>
--
To unsubscribe from this list: send the line "unsubscribe linux-pci" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [DMA Engine]     [Linux Coverity]     [Linux USB]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Greybus]

  Powered by Linux