On Fri, 2008-05-16 at 14:46 +1000, Stephen Rothwell wrote: > Hi Remi, > > On Thu, 15 May 2008 17:24:28 -0700 Remi Machet <[EMAIL PROTECTED]> wrote: > > > > +#include <linux/stddef.h> > > +#include <linux/kernel.h> > > +#include <linux/delay.h> > > +#include <linux/interrupt.h> > > +#include <linux/seq_file.h> > > + > > +#include <asm/machdep.h> > > +#include <asm/prom.h> > > +#include <asm/system.h> > > +#include <linux/time.h> > > +#include <asm/time.h> > > +#include <linux/kexec.h> > > Normally we don'y mix linux/ and asm/ includes (put all the linux ones at > the top). > > You need to include <linux/of.h> because you use the device tree accessor > functions. > Sorry, I made the mistake when correcting a few checkpatch.pl warnings. I will fix that and re-submit.
Thanks, Remi _______________________________________________ Linuxppc-dev mailing list Linuxppc-dev@ozlabs.org https://ozlabs.org/mailman/listinfo/linuxppc-dev