Re: [PATCH 13/19] ARM: s5pv210: move debug-macro.S into the common space

From: Kukjin Kim
Date: Fri Jul 18 2014 - 15:38:19 EST


On 07/16/14 09:56, Tomasz Figa wrote:
On 16.07.2014 02:53, Kukjin Kim wrote:
Kukjin Kim wrote:

On 07/05/14 02:48, Tomasz Figa wrote:
Move debug-macro.S from mach/include to include/debug where
all other common debug macros are.

Signed-off-by: Tomasz Figa<t.figa@xxxxxxxxxxx>
---
arch/arm/Kconfig.debug | 12 +++++--
arch/arm/include/debug/s5pv210.S | 34 ++++++++++++++++++++
arch/arm/mach-s5pv210/include/mach/debug-macro.S | 41 ------------------------
3 files changed, 44 insertions(+), 43 deletions(-)
create mode 100644 arch/arm/include/debug/s5pv210.S
delete mode 100644 arch/arm/mach-s5pv210/include/mach/debug-macro.S

[...]

Tomasz,

I couldn't apply this one from this your series because of conflict with
others. Can you please respin this one?

One more note, since I didn't apply this, there is a build breakage for
s5pv210_defconfig now...

arch/arm/kernel/debug.S:24:33: fatal error: mach/debug-macro.S: No such file or directory
compilation terminated.
make[2]: *** [arch/arm/kernel/debug.o] Error 1
make[1]: *** [arch/arm/kernel] Error 2
make[1]: *** Waiting for unfinished jobs....

Hmm? Are you sure previous patches applied correctly? I have tested this
series patch by patch on all affected configs and it built fine back
then. Maybe some conflict, I'll see tomorrow.

Tomasz, I've applied this whole series including this again. Can you please check my branch before sending pull-request to arm-soc for 3.17?

Thanks,
Kukjin
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/