All of lore.kernel.org
 help / color / mirror / Atom feed
* [Qemu-devel] [patch] Byte swapping bug in arm semihosting
@ 2005-08-13 15:08 Paul Brook
  0 siblings, 0 replies; only message in thread
From: Paul Brook @ 2005-08-13 15:08 UTC (permalink / raw)
  To: qemu-devel

The patch below fixes a duplicate byte swap in the arm semihosting 
implementation. ARG(x) already does the endian correction.

Paul
Index: linux-user//arm-semi.c
===================================================================
RCS file: /cvsroot/qemu/qemu/linux-user/arm-semi.c,v
retrieving revision 1.3
diff -u -p -r1.3 arm-semi.c
--- linux-user//arm-semi.c	13 May 2005 22:42:37 -0000	1.3
+++ linux-user//arm-semi.c	13 Aug 2005 15:05:30 -0000
@@ -178,7 +178,7 @@ uint32_t do_arm_semihosting(CPUState *en
                 ts->heap_limit = limit;
             }
               
-            ptr = (uint32_t *)tswap32(ARG(0));
+            ptr = (uint32_t *)ARG(0);
             ptr[0] = tswap32(ts->heap_base);
             ptr[1] = tswap32(ts->heap_limit);
             ptr[2] = tswap32(ts->stack_base);

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2005-08-13 15:37 UTC | newest]

Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2005-08-13 15:08 [Qemu-devel] [patch] Byte swapping bug in arm semihosting Paul Brook

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.