[PATCH] cris: arch-v10: kgdb: Use BAR instead of DTP0 for register P12

[PATCH] cris: arch-v10: kgdb: Use BAR instead of DTP0 for register P12

[Chen Gang]

For arch-v10, there is no DTP0 register, and at present, assembler know
BAR, so use BAR instead of DTP0, the related error (with allmodconfig):

    CC      arch/cris/arch-v10/kernel/kgdb.o
  {standard input}: Assembler messages:
  {standard input}:6: Error: Illegal operands
  {standard input}:6: Error: Illegal operands

Signed-off-by: Chen Gang <gang.chen.5i5j@gmail.com>
---
 arch/cris/arch-v10/kernel/kgdb.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/cris/arch-v10/kernel/kgdb.c b/arch/cris/arch-v10/kernel/kgdb.c
index 22d846b..5b61335 100644
--- a/arch/cris/arch-v10/kernel/kgdb.c
+++ b/arch/cris/arch-v10/kernel/kgdb.c
@@ -970,7 +970,7 @@ asm ("\n"
 "  move     $ibr,[cris_reg+0x4E]  ; P9,\n"
 "  move     $irp,[cris_reg+0x52]  ; P10,\n"
 "  move     $srp,[cris_reg+0x56]  ; P11,\n"
-"  move     $dtp0,[cris_reg+0x5A] ; P12, register BAR, assembler might not know BAR\n"
+"  move     $bar,[cris_reg+0x5A]  ; P12,\n"
 "                            ; P13, register DCCR already saved\n"
 ";; Due to the old assembler-versions BRP might not be recognized\n"
 "  .word 0xE670              ; move brp,r0\n"
@@ -1063,7 +1063,7 @@ asm ("\n"
 "  move     $ibr,[cris_reg+0x4E]  ; P9,\n"
 "  move     $irp,[cris_reg+0x52]  ; P10,\n"
 "  move     $srp,[cris_reg+0x56]  ; P11,\n"
-"  move     $dtp0,[cris_reg+0x5A] ; P12, register BAR, assembler might not know BAR\n"
+"  move     $bar,[cris_reg+0x5A]  ; P12,\n"
 "                            ; P13, register DCCR already saved\n"
 ";; Due to the old assembler-versions BRP might not be recognized\n"
 "  .word 0xE670              ; move brp,r0\n"
-- 
1.9.3

Re: [PATCH] cris: arch-v10: kgdb: Use BAR instead of DTP0 for register P12

[Hans-Peter Nilsson]

> From: Chen Gang <xili_gchen_5257@hotmail.com>
> Date: Fri, 10 Jul 2015 23:47:24 +0200

> For arch-v10, there is no DTP0 register, and at present, assembler know
> BAR, so use BAR instead of DTP0, the related error (with allmodconfig):
> 
>     CC      arch/cris/arch-v10/kernel/kgdb.o
>   {standard input}: Assembler messages:
>   {standard input}:6: Error: Illegal operands
>   {standard input}:6: Error: Illegal operands
> 
> Signed-off-by: Chen Gang <gang.chen.5i5j@gmail.com>
> ---
>  arch/cris/arch-v10/kernel/kgdb.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/arch/cris/arch-v10/kernel/kgdb.c b/arch/cris/arch-v10/kernel/kgdb.c
> index 22d846b..5b61335 100644

Acked-by: Hans-Peter Nilsson <hp@axis.com>

brgds, H-P

Re: [PATCH] cris: arch-v10: kgdb: Use BAR instead of DTP0 for register P12

[Jesper Nilsson]

Hi,

On Sat, Jul 11, 2015 at 05:47:24AM +0800, Chen Gang wrote:
> For arch-v10, there is no DTP0 register, and at present, assembler know
> BAR, so use BAR instead of DTP0, the related error (with allmodconfig):
> 
>     CC      arch/cris/arch-v10/kernel/kgdb.o
>   {standard input}: Assembler messages:
>   {standard input}:6: Error: Illegal operands
>   {standard input}:6: Error: Illegal operands
> 
> Signed-off-by: Chen Gang <gang.chen.5i5j@gmail.com>

While checking another kgdb related fix, I hit this bug too.
Applied to the for-next branch of the cris tree.

Sorry about the slow (glacial) response time and thanks!

/^JN - Jesper Nilsson
-- 
               Jesper Nilsson -- jesper.nilsson@axis.com

Re: [PATCH] cris: arch-v10: kgdb: Use BAR instead of DTP0 for register P12

[Chen Gang]

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain; charset="gb2312", Size: 1001 bytes --]

On 10/2/15 03:07, Jesper Nilsson wrote:
> Hi,
>
> On Sat, Jul 11, 2015 at 05:47:24AM +0800, Chen Gang wrote:
>> For arch-v10, there is no DTP0 register, and at present, assembler know
>> BAR, so use BAR instead of DTP0, the related error (with allmodconfig):
>>
>> CC arch/cris/arch-v10/kernel/kgdb.o
>> {standard input}: Assembler messages:
>> {standard input}:6: Error: Illegal operands
>> {standard input}:6: Error: Illegal operands
>>
>> Signed-off-by: Chen Gang <gang.chen.5i5j@gmail.com>
>
> While checking another kgdb related fix, I hit this bug too.
> Applied to the for-next branch of the cris tree.
>
> Sorry about the slow (glacial) response time and thanks!
>

It doesn't matter. Thank you for your work with the 2 patches.

--
Chen Gang (³Â¸Õ)

Open, share, and attitude like air, water, and life which God blessed
 		 	   		  ÿôèº{.nÇ+‰·Ÿ®‰­†+%ŠËÿ±éݶ\x17¥Šwÿº{.nÇ+‰·¥Š{±þG«éÿŠ{ayº\x1dʇڙë,j\a­¢f£¢·hšïêÿ‘êçz_è®\x03(­éšŽŠÝ¢j"ú\x1a¶^[m§ÿÿ¾\a«þG«éÿ¢¸?™¨è­Ú&£ø§~á¶iO•æ¬z·švØ^\x14\x04\x1a¶^[m§ÿÿÃ\fÿ¶ìÿ¢¸?–I¥