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

From: Tomasz Figa
Date: Tue Jul 15 2014 - 20:57:36 EST


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.

Best regards,
Tomasz
--
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/