> > Please test this patch.
> > drivers/acpi/bus.c | 14 +++-----------
> > 1 files changed, 3 insertions(+), 11 deletions(-)
> >
> > diff --git a/drivers/acpi/bus.c b/drivers/acpi/bus.c
> > index 5c221ab..a70ca8e 100644
> > --- a/drivers/acpi/bus.c
> > +++ b/drivers/acpi/bus.c
> > @@ -80,23 +80,15 @@ static int set_copy_dsdt(const struct dmi_system_id *id)
> >
> > static struct dmi_system_id dsdt_dmi_table[] __initdata = {
> > /*
> > - * Insyde BIOS on some TOSHIBA machines corrupt the DSDT.
> > + * Invoke DSDT corruption work-around on all Toshiba Satellite.
> > * https://bugzilla.kernel.org/show_bug.cgi?id=14679
> > */
> > {
> > .callback = set_copy_dsdt,
> > - .ident = "TOSHIBA Satellite A505",
> > + .ident = "TOSHIBA Satellite",
> > .matches = {
> > DMI_MATCH(DMI_SYS_VENDOR, "TOSHIBA"),
> > - DMI_MATCH(DMI_PRODUCT_NAME, "Satellite A505"),
> > - },
> > - },
> > - {
> > - .callback = set_copy_dsdt,
> > - .ident = "TOSHIBA Satellite L505D",
> > - .matches = {
> > - DMI_MATCH(DMI_SYS_VENDOR, "TOSHIBA"),
> Isn't that a bit too broad? Maybe we should keep "TOSHIBA" in case
> another vendor decides to add "Satellite" in its DMI product name.
We kept it -- it is the 1st line above without a '-' in fron of it.
> > - DMI_MATCH(DMI_PRODUCT_NAME, "Satellite L505D"),
> > + DMI_MATCH(DMI_PRODUCT_NAME, "Satellite"),
> > },
> > },
> > {}
>
> Anyway, this works well on Toshiba Satellite C650D.
>
> Tested-by: Anisse Astier <anisse@xxxxxxxxx>
Thanks very much for testing.
cheers,
Len Brown, Intel Open Souce Technology Center
--
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
