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/

Reply via email to