All of lore.kernel.org
 help / color / mirror / Atom feed
* [Qemu-devel] [PATCH] mips: Make CP0.Config4 and CP0.Config5 registers signed
@ 2014-11-04 15:37 Maciej W. Rozycki
  2014-11-05 15:25 ` Leon Alrae
  0 siblings, 1 reply; 2+ messages in thread
From: Maciej W. Rozycki @ 2014-11-04 15:37 UTC (permalink / raw)
  To: qemu-devel; +Cc: Leon Alrae, Aurelien Jarno

Make the data type used for the CP0.Config4 and CP0.Config5 registers 
and their mask signed, for consistency with the remaining 32-bit CP0 
registers, like CP0.Config0, etc.

Signed-off-by: Maciej W. Rozycki <macro@codesourcery.com>
---
qemu-mips-config-int32_t.diff
Index: qemu-git-trunk/target-mips/cpu.h
===================================================================
--- qemu-git-trunk.orig/target-mips/cpu.h	2014-11-02 01:05:19.000000000 +0000
+++ qemu-git-trunk/target-mips/cpu.h	2014-11-02 01:08:26.527563002 +0000
@@ -372,11 +372,11 @@ struct CPUMIPSState {
 #define CP0C3_MT   2
 #define CP0C3_SM   1
 #define CP0C3_TL   0
-    uint32_t CP0_Config4;
-    uint32_t CP0_Config4_rw_bitmask;
+    int32_t CP0_Config4;
+    int32_t CP0_Config4_rw_bitmask;
 #define CP0C4_M    31
-    uint32_t CP0_Config5;
-    uint32_t CP0_Config5_rw_bitmask;
+    int32_t CP0_Config5;
+    int32_t CP0_Config5_rw_bitmask;
 #define CP0C5_M          31
 #define CP0C5_K          30
 #define CP0C5_CV         29

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [Qemu-devel] [PATCH] mips: Make CP0.Config4 and CP0.Config5 registers signed
  2014-11-04 15:37 [Qemu-devel] [PATCH] mips: Make CP0.Config4 and CP0.Config5 registers signed Maciej W. Rozycki
@ 2014-11-05 15:25 ` Leon Alrae
  0 siblings, 0 replies; 2+ messages in thread
From: Leon Alrae @ 2014-11-05 15:25 UTC (permalink / raw)
  To: Maciej W. Rozycki, qemu-devel; +Cc: Aurelien Jarno

On 04/11/2014 15:37, Maciej W. Rozycki wrote:
> Make the data type used for the CP0.Config4 and CP0.Config5 registers 
> and their mask signed, for consistency with the remaining 32-bit CP0 
> registers, like CP0.Config0, etc.
> 
> Signed-off-by: Maciej W. Rozycki <macro@codesourcery.com>
> ---
> qemu-mips-config-int32_t.diff
> Index: qemu-git-trunk/target-mips/cpu.h
> ===================================================================
> --- qemu-git-trunk.orig/target-mips/cpu.h	2014-11-02 01:05:19.000000000 +0000
> +++ qemu-git-trunk/target-mips/cpu.h	2014-11-02 01:08:26.527563002 +0000
> @@ -372,11 +372,11 @@ struct CPUMIPSState {
>  #define CP0C3_MT   2
>  #define CP0C3_SM   1
>  #define CP0C3_TL   0
> -    uint32_t CP0_Config4;
> -    uint32_t CP0_Config4_rw_bitmask;
> +    int32_t CP0_Config4;
> +    int32_t CP0_Config4_rw_bitmask;
>  #define CP0C4_M    31
> -    uint32_t CP0_Config5;
> -    uint32_t CP0_Config5_rw_bitmask;
> +    int32_t CP0_Config5;
> +    int32_t CP0_Config5_rw_bitmask;
>  #define CP0C5_M          31
>  #define CP0C5_K          30
>  #define CP0C5_CV         29
> 

Reviewed-by: Leon Alrae <leon.alrae@imgtec.com>

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2014-11-05 15:25 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-11-04 15:37 [Qemu-devel] [PATCH] mips: Make CP0.Config4 and CP0.Config5 registers signed Maciej W. Rozycki
2014-11-05 15:25 ` Leon Alrae

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.