On Wed, Feb 06, 2008 at 08:56:35AM +0100, Sam Ravnborg wrote: > > > > > > > > > > > > > > i8042_platform_init(): > > > > > > +#if defined(__i386__) || defined(__x86_64__) > > > > > > use #ifdef CONFIG_X86? > > > > > > > I considered it but above was tested and in line with the style of the > > rest of the file... > Then please change the rest of the file so it is consistent > with the usual style to use our CONFIG_ symbols for > conditionals like the above. >
Will do in the next update. The #if defined(__i386__) code predates CONFIG_X86_32 and was not updated when this symbol became available. Overall, do you want me to redo the pull or is it still acceptable? -- Dmitry -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/