imx27_visstrim_m10: spot the mistake

commit 435ca24116bc5365a10fa0be1bf1c1e4c5680157
Author: Javier Martin [off-list ref]
Date:   Tue Apr 10 11:26:59 2012 +0200

    ARM i.MX: Visstrim_M10: Add board version detection.
    
    Visstrim_M10 boards have 3 gpios to detect the
    expansion board version attached and other 3 to
    detect the motherboard revision.
    
    Signed-off-by: Javier Martin [off-list ref]
    Signed-off-by: Sascha Hauer [off-list ref]

diff --git a/arch/arm/mach-imx/mach-imx27_visstrim_m10.c b/arch/arm/mach-imx/mach-imx27_visstrim_m10.c
index f7b074f..748ba2e 100644
--- a/arch/arm/mach-imx/mach-imx27_visstrim_m10.c
+++ b/arch/arm/mach-imx/mach-imx27_visstrim_m10.c

@@ -38,6 +38,7 @@
 #include <asm/mach-types.h>
 #include <asm/mach/arch.h>
 #include <asm/mach/time.h>
+#include <asm/system.h>
 #include <mach/common.h>
 #include <mach/iomux-mx27.h>

...

commit 9f97da78bf018206fb623cd351d454af2f105fe0
Author: David Howells [off-list ref]
Date:   Wed Mar 28 18:30:01 2012 +0100

    Disintegrate asm/system.h for ARM
    
    Disintegrate asm/system.h for ARM.
    
    Signed-off-by: David Howells [off-list ref]
    cc: Russell King [off-list ref]
    cc: linux-arm-kernel at lists.infradead.org

diff --git a/arch/arm/include/asm/system.h b/arch/arm/include/asm/system.h
index 774c41e..74542c5 100644
--- a/arch/arm/include/asm/system.h
+++ b/arch/arm/include/asm/system.h
+/* FILE TO BE DELETED. DO NOT ADD STUFF HERE! */
+#include <asm/barrier.h>
+#include <asm/compiler.h>
+#include <asm/cmpxchg.h>
+#include <asm/exec.h>
+#include <asm/switch_to.h>
+#include <asm/system_info.h>
+#include <asm/system_misc.h>

What's wrong with the above commit to mach-imx27_visstrim_m10.c ?

On Sun, Jun 24, 2012 at 05:57:10PM +0100, Russell King - ARM Linux wrote:

Sorry Russell, thank you Fabio for sending a patch.

Sascha


-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

On 25 June 2012 08:49, Sascha Hauer [off-list ref] wrote:

On Sun, Jun 24, 2012 at 05:57:10PM +0100, Russell King - ARM Linux wrote:

quoted

commit 9f97da78bf018206fb623cd351d454af2f105fe0
Author: David Howells [off-list ref]
Date: ? Wed Mar 28 18:30:01 2012 +0100

? ? Disintegrate asm/system.h for ARM

? ? Disintegrate asm/system.h for ARM.

? ? Signed-off-by: David Howells [off-list ref]
? ? cc: Russell King [off-list ref]
? ? cc: linux-arm-kernel at lists.infradead.org

diff --git a/arch/arm/include/asm/system.h b/arch/arm/include/asm/system.h
index 774c41e..74542c5 100644
--- a/arch/arm/include/asm/system.h
+++ b/arch/arm/include/asm/system.h
+/* FILE TO BE DELETED. DO NOT ADD STUFF HERE! */
+#include <asm/barrier.h>
+#include <asm/compiler.h>
+#include <asm/cmpxchg.h>
+#include <asm/exec.h>
+#include <asm/switch_to.h>
+#include <asm/system_info.h>
+#include <asm/system_misc.h>

What's wrong with the above commit to mach-imx27_visstrim_m10.c ?

Sorry Russell, thank you Fabio for sending a patch.

Sorry, thank you for spotting the mistake.

-- 
Javier Martin
Vista Silicon S.L.
CDTUC - FASE C - Oficina S-345
Avda de los Castros s/n
39005- Santander. Cantabria. Spain
+34 942 25 32 60
www.vista-silicon.com