[PATCH] ARM: kirkwood: add missing kexec.h include

Russell King - ARM Linux linux at arm.linux.org.uk
Tue Apr 10 06:00:38 EDT 2012


On Fri, Apr 06, 2012 at 02:10:38PM +0100, Ian Campbell wrote:
> On Fri, 2012-04-06 at 16:50 +0400, Sergei Shtylyov wrote:
> > On 06.04.2012 16:30, Ian Campbell wrote:
> > > Fixes build the following error when CONFIG_KEXEC is enabled:
> > 
> >     Maybe "the following build error" instead?
> 
> Done.
> 
> > >    CC      arch/arm/mach-kirkwood/board-dt.o
> > > arch/arm/mach-kirkwood/board-dt.c: In function 'kirkwood_dt_init':
> > > arch/arm/mach-kirkwood/board-dt.c:52:2: error: 'kexec_reinit' undeclared (first use in this function)
> > > arch/arm/mach-kirkwood/board-dt.c:52:2: note: each undeclared identifier is reported only once for each function it appears in
> > 
> > > kexec.h also needs string.h.
> > 
> >     Why it doesn't include it?
> 
> Not sure, but actually I should be using linux/kexec.h not asm/kexec.h
> anyway which solves the problem too:

Because you should not be using asm/kexec.h directly but using linux/kexec.h
instead, which will include it via this path:

linux/kexec.h -> linux/compat.h -> linux/sem.h -> linux/rcupdate.h
 -> linux/cpumask.h -> linux/bitmap.h -> linux/string.h

checkpatch has been nobbled not to complain by default about using asm/%.h
includes when linux/%.h includes are also present, which is a backwards
step imho.



More information about the linux-arm-kernel mailing list