2008-01-12 14:55:26

by Russell King

[permalink] [raw]
Subject: Fwd: Issue with arm-devel tree

It might be a good idea if there was some co-ordination with people
involved in the duplicate include removal work...

[ARM] remove duplicate includes

Signed-off-by: Lucas Woods <[email protected]>
Acked-by: Lennert Buytenhek <[email protected]>
Signed-off-by: Andrew Morton <[email protected]>
Signed-off-by: Russell King <[email protected]>

@@ -20,7 +20,6 @@
#include <linux/module.h>
#include <linux/irq.h>
#include <linux/interrupt.h>
-#include <linux/reboot.h>

#include <asm/mach-types.h>


[ARM] ixp4xx: remove double include

Remove double include.

Signed-off-by: Andre Haupt <[email protected]>
Signed-off-by: Alessandro Zummo <[email protected]>
Signed-off-by: Andrew Morton <[email protected]>
Signed-off-by: Russell King <[email protected]>

@@ -18,7 +18,6 @@
*/

#include <linux/module.h>
-#include <linux/reboot.h>
#include <linux/irq.h>
#include <linux/interrupt.h>
#include <linux/reboot.h>

I've dropped the "ixp4xx: remove double include" commit and fixed the
resulting breakage of "remove duplicate includes" change to resolve this
problem.

----- Forwarded message from Michael-Luke Jones <[email protected]> -----

To: [email protected]
From: Michael-Luke Jones <[email protected]>
Subject: Issue with arm-devel tree
Date: Sat, 12 Jan 2008 14:23:44 +0000

http://ftp.arm.linux.org.uk/pub/linux/arm/kernel/git-cur/arm:devel.mbox

contains two patches:

"Re: [ARM] remove duplicate includes" from Lucas Woods
<[email protected]>
"Re: [ARM] ixp4xx: remove double include" from Andre Haupt
<[email protected]>

which between the two of them manage to delete *both* instances of
#include <linux/reboot.h> in nslu2-power.c

Given that nslu2-power.c calls machine_power_off() and ctrl_alt_del()
this probably isn't a good thing.

Michael-Luke Jones

----- End forwarded message -----

--
Russell King


2008-01-14 13:11:14

by Andre Haupt

[permalink] [raw]
Subject: Re: Fwd: Issue with arm-devel tree

On Sat, Jan 12, 2008 at 02:54:49PM +0000, Russell King wrote:
> It might be a good idea if there was some co-ordination with people
> involved in the duplicate include removal work...

Sorry for the inconvenience! I did not realise that Lucas was working on
this and hence missed his patches.

Best regards,

Andre