Hi Stephen, Thank you for the report. I will submit a patch for the issue soon. Since I used the net-next.git, the build failure didn't happen. This is because that the linux/device.h of the net-next.git has "#include <linux/module.h>". But, I was wrong to remove "#include <linux/module.h>" in the sh_eth.{c,h}. Thanks, Yoshihiro Shimoda 2011/09/30 10:38, Stephen Rothwell wrote: > Hi all, > > After merging the final tree, the next-20110929 build (various sh configs) > failed like this: > > drivers/net/ethernet/renesas/sh_eth.c:601: error: 'THIS_MODULE' undeclared here (not in a function) > drivers/net/ethernet/renesas/sh_eth.c:1970: error: expected declaration specifiers or '...' before string constant > drivers/net/ethernet/renesas/sh_eth.c:1970: warning: data definition has no type or storage class > drivers/net/ethernet/renesas/sh_eth.c:1970: warning: type defaults to 'int' in declaration of 'MODULE_AUTHOR' > drivers/net/ethernet/renesas/sh_eth.c:1970: warning: function declaration isn't a prototype > drivers/net/ethernet/renesas/sh_eth.c:1971: error: expected declaration specifiers or '...' before string constant > drivers/net/ethernet/renesas/sh_eth.c:1971: warning: data definition has no type or storage class > drivers/net/ethernet/renesas/sh_eth.c:1971: warning: type defaults to 'int' in declaration of 'MODULE_DESCRIPTION' > drivers/net/ethernet/renesas/sh_eth.c:1971: warning: function declaration isn't a prototype > drivers/net/ethernet/renesas/sh_eth.c:1972: error: expected declaration specifiers or '...' before string constant > drivers/net/ethernet/renesas/sh_eth.c:1972: warning: data definition has no type or storage class > drivers/net/ethernet/renesas/sh_eth.c:1972: warning: type defaults to 'int' in declaration of 'MODULE_LICENSE' > drivers/net/ethernet/renesas/sh_eth.c:1972: warning: function declaration isn't a prototype > > Presumbly revealed by the module.h split up. This file needs to include module.h. -- To unsubscribe from this list: send the line "unsubscribe linux-next" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html