On Fri, Feb 11, 2011 at 12:37:26PM +0100, Richard Schütz wrote: > > +static struct dmi_system_id __initdata samsung_dmi_table[] = { > > + { > > + .ident = "N128", > > + .matches = { > > + DMI_MATCH(DMI_SYS_VENDOR, > > + "SAMSUNG ELECTRONICS CO., LTD."), > > + DMI_MATCH(DMI_PRODUCT_NAME, "N128"), > > + DMI_MATCH(DMI_BOARD_NAME, "N128"), > > + }, > > + .callback = dmi_check_cb, > > + }, > > + { > > + .ident = "N130", > > + .matches = { > > + DMI_MATCH(DMI_SYS_VENDOR, > > + "SAMSUNG ELECTRONICS CO., LTD."), > > + DMI_MATCH(DMI_PRODUCT_NAME, "N130"), > > + DMI_MATCH(DMI_BOARD_NAME, "N130"), > > + }, > > + .callback = dmi_check_cb, > > + }, > > + { > > + .ident = "X125", > > + .matches = { > > + DMI_MATCH(DMI_SYS_VENDOR, > > + "SAMSUNG ELECTRONICS CO., LTD."), > > + DMI_MATCH(DMI_PRODUCT_NAME, "X125"), > > + DMI_MATCH(DMI_BOARD_NAME, "X125"), > > + }, > > + .callback = dmi_check_cb, > > + }, > > + { > > + .ident = "NC10", > > + .matches = { > > + DMI_MATCH(DMI_SYS_VENDOR, > > + "SAMSUNG ELECTRONICS CO., LTD."), > > + DMI_MATCH(DMI_PRODUCT_NAME, "NC10"), > > + DMI_MATCH(DMI_BOARD_NAME, "NC10"), > > + }, > > + .callback = dmi_check_cb, > > + }, > > + { > > + .ident = "NP-Q45", > > + .matches = { > > + DMI_MATCH(DMI_SYS_VENDOR, > > + "SAMSUNG ELECTRONICS CO., LTD."), > > + DMI_MATCH(DMI_PRODUCT_NAME, "SQ45S70S"), > > + DMI_MATCH(DMI_BOARD_NAME, "SQ45S70S"), > > + }, > > + .callback = dmi_check_cb, > > + }, > > + { > > + .ident = "X360", > > + .matches = { > > + DMI_MATCH(DMI_SYS_VENDOR, > > + "SAMSUNG ELECTRONICS CO., LTD."), > > + DMI_MATCH(DMI_PRODUCT_NAME, "X360"), > > + DMI_MATCH(DMI_BOARD_NAME, "X360"), > > + }, > > + .callback = dmi_check_cb, > > + }, > > + { > > + .ident = "R518", > > + .matches = { > > + DMI_MATCH(DMI_SYS_VENDOR, > > + "SAMSUNG ELECTRONICS CO., LTD."), > > + DMI_MATCH(DMI_PRODUCT_NAME, "R518"), > > + DMI_MATCH(DMI_BOARD_NAME, "R518"), > > + }, > > + .callback = dmi_check_cb, > > + }, > > + { > > + .ident = "N150/N210/N220", > > + .matches = { > > + DMI_MATCH(DMI_SYS_VENDOR, > > + "SAMSUNG ELECTRONICS CO., LTD."), > > + DMI_MATCH(DMI_PRODUCT_NAME, "N150/N210/N220"), > > + DMI_MATCH(DMI_BOARD_NAME, "N150/N210/N220"), > > + }, > > + .callback = dmi_check_cb, > > + }, > > + { > > + .ident = "R530/R730", > > + .matches = { > > + DMI_MATCH(DMI_SYS_VENDOR, "SAMSUNG ELECTRONICS CO., LTD."), > > + DMI_MATCH(DMI_PRODUCT_NAME, "R530/R730"), > > + DMI_MATCH(DMI_BOARD_NAME, "R530/R730"), > > + }, > > + .callback = dmi_check_cb, > > + }, > > + { > > + .ident = "NF110/NF210/NF310", > > + .matches = { > > + DMI_MATCH(DMI_SYS_VENDOR, "SAMSUNG ELECTRONICS CO., LTD."), > > + DMI_MATCH(DMI_PRODUCT_NAME, "NF110/NF210/NF310"), > > + DMI_MATCH(DMI_BOARD_NAME, "NF110/NF210/NF310"), > > + }, > > + .callback = dmi_check_cb, > > + }, > > + { }, > > +}; > > +MODULE_DEVICE_TABLE(dmi, samsung_dmi_table); > > Could you add > > .ident = "N145P/N250P/N260P", > .matches = { > DMI_MATCH(DMI_SYS_VENDOR, "SAMSUNG ELECTRONICS CO., LTD."), > DMI_MATCH(DMI_PRODUCT_NAME, "N145P/N250P/N260P"), > DMI_MATCH(DMI_BOARD_NAME, "N145P/N250P/N260P"), > }, > .callback = dmi_check_cb, > > there, please? I've tested the driver successfully on my N145P. Yes I can, but do you want to send it to me in a format that I can apply so you can get the proper authorship credit? Take a look at Documentation/SubmittingPatches for the details. If not, I'll be glad to do it myself, but wanted to give you the chance first. thanks, greg k-h -- 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