On Tue, Dec 16, 2008 at 05:01:38PM +0100, Jiri Kosina wrote: > On Tue, 16 Dec 2008, Ben Dooks wrote: > > > > #ifdef CONFIG_HID_COMPAT > > > #define HID_COMPAT_LOAD_DRIVER(name) \ > > > +/* prototype to avoid sparse warning */ \ > > > +extern void hid_compat_##name(void); \ > > > void hid_compat_##name(void) { } \ > > surely this could simply be changed to 'static void > > hid_compat_##name(void)' if it is only being defined to be an empty > > This will cause gcc emit warnings about unused symbols. sorry, 'static inline void' then. > -- > Jiri Kosina > SUSE Labs > -- > To unsubscribe from this list: send the line "unsubscribe linux-kernel" in > the body of a message to majordomo@xxxxxxxxxxxxxxx > More majordomo info at http://vger.kernel.org/majordomo-info.html > Please read the FAQ at http://www.tux.org/lkml/ -- Ben (ben@xxxxxxxxx, http://www.fluff.org/) 'a smiley only costs 4 bytes' -- To unsubscribe from this list: send the line "unsubscribe linux-input" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html