Dear Carlos, Yes, you are right. I copy/paste the wrong word all over the patch. I just submitted a patch for it, thanks for your report. Best regards, AceLan Kao. 2012/8/3 Carlos Alberto Lopez Perez <clopez@xxxxxxxxxx>: > On 13/07/12 10:39, AceLan Kao wrote: >> Add the following machines into quirk, >> Isnpiron 5420, Isnpiron 5520, Isnpiron 5720, >> Isnpiron 7420, Isnpiron 7520, Isnpiron 7720 >> >> Signed-off-by: AceLan Kao <acelan.kao@xxxxxxxxxxxxx> >> --- >> drivers/platform/x86/dell-laptop.c | 54 ++++++++++++++++++++++++++++++++++++ >> 1 file changed, 54 insertions(+) >> >> diff --git a/drivers/platform/x86/dell-laptop.c b/drivers/platform/x86/dell-laptop.c >> index 5f78aac..4e96e8c 100644 >> --- a/drivers/platform/x86/dell-laptop.c >> +++ b/drivers/platform/x86/dell-laptop.c >> @@ -206,6 +206,60 @@ static struct dmi_system_id __devinitdata dell_quirks[] = { >> }, >> .driver_data = &quirk_dell_vostro_v130, >> }, >> + { >> + .callback = dmi_matched, >> + .ident = "Dell Inspiron 5420", >> + .matches = { >> + DMI_MATCH(DMI_SYS_VENDOR, "Dell Inc."), >> + DMI_MATCH(DMI_PRODUCT_NAME, "Isnpiron 5420"), >> + }, >> + .driver_data = &quirk_dell_vostro_v130, >> + }, >> + { >> + .callback = dmi_matched, >> + .ident = "Dell Inspiron 5520", >> + .matches = { >> + DMI_MATCH(DMI_SYS_VENDOR, "Dell Inc."), >> + DMI_MATCH(DMI_PRODUCT_NAME, "Isnpiron 5520"), >> + }, >> + .driver_data = &quirk_dell_vostro_v130, >> + }, >> + { >> + .callback = dmi_matched, >> + .ident = "Dell Inspiron 5720", >> + .matches = { >> + DMI_MATCH(DMI_SYS_VENDOR, "Dell Inc."), >> + DMI_MATCH(DMI_PRODUCT_NAME, "Isnpiron 5720"), >> + }, >> + .driver_data = &quirk_dell_vostro_v130, >> + }, >> + { >> + .callback = dmi_matched, >> + .ident = "Dell Inspiron 7420", >> + .matches = { >> + DMI_MATCH(DMI_SYS_VENDOR, "Dell Inc."), >> + DMI_MATCH(DMI_PRODUCT_NAME, "Isnpiron 7420"), >> + }, >> + .driver_data = &quirk_dell_vostro_v130, >> + }, >> + { >> + .callback = dmi_matched, >> + .ident = "Dell Inspiron 7520", >> + .matches = { >> + DMI_MATCH(DMI_SYS_VENDOR, "Dell Inc."), >> + DMI_MATCH(DMI_PRODUCT_NAME, "Isnpiron 7520"), >> + }, >> + .driver_data = &quirk_dell_vostro_v130, >> + }, >> + { >> + .callback = dmi_matched, >> + .ident = "Dell Inspiron 7720", >> + .matches = { >> + DMI_MATCH(DMI_SYS_VENDOR, "Dell Inc."), >> + DMI_MATCH(DMI_PRODUCT_NAME, "Isnpiron 7720"), >> + }, >> + .driver_data = &quirk_dell_vostro_v130, >> + }, >> { } >> }; >> > > Perhaps this needs a sed "s/Isnpiron/Inspiron/g" ??? > -- Chia-Lin Kao(AceLan) http://blog.acelan.idv.tw/ E-Mail: acelan.kaoATcanonical.com (s/AT/@/) -- To unsubscribe from this list: send the line "unsubscribe platform-driver-x86" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html