* tip-bot for Andi Kleen <ak@xxxxxxxxxxxxxxx> wrote: > Commit-ID: c347c2092efcd7d4fc1575057db7a42c1fc15ec5 > Gitweb: http://git.kernel.org/tip/c347c2092efcd7d4fc1575057db7a42c1fc15ec5 > Author: Andi Kleen <ak@xxxxxxxxxxxxxxx> > AuthorDate: Mon, 19 Dec 2011 15:46:33 -0800 > Committer: H. Peter Anvin <hpa@xxxxxxxxxxxxxxx> > CommitDate: Mon, 19 Dec 2011 15:50:01 -0800 > > x86, cpufreq: Add support for x86 cpuinfo auto loading > diff --git a/drivers/cpufreq/gx-suspmod.c b/drivers/cpufreq/gx-suspmod.c > index ffe1f2c..5a06c0b 100644 > --- a/drivers/cpufreq/gx-suspmod.c > +++ b/drivers/cpufreq/gx-suspmod.c > @@ -82,6 +82,7 @@ > #include <linux/errno.h> > #include <linux/slab.h> > > +#include <asm/cpu_device_id.h> > #include <asm/processor-cyrix.h> > > /* PCI config registers, all at F0 */ > @@ -171,6 +172,7 @@ static struct pci_device_id gx_chipset_tbl[] __initdata = { > { PCI_VDEVICE(CYRIX, PCI_DEVICE_ID_CYRIX_5510), }, > { 0, }, > }; > +MODULE_DEVICE_TABLE(gx_chipset_tbl); Andi, you are sloppy as usual: drivers/cpufreq/gx-suspmod.c:175:35: error: macro "MODULE_DEVICE_TABLE" requires 2 arguments, but only 1 given You have not even build tested this change? Thanks, Ingo -- To unsubscribe from this list: send the line "unsubscribe linux-tip-commits" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html