[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 0/9] target-mips: Add Enhanced Virtual Addressin
From: |
James Hogan |
Subject: |
Re: [Qemu-devel] [PATCH 0/9] target-mips: Add Enhanced Virtual Addressing (EVA) support |
Date: |
Tue, 6 Sep 2016 13:09:37 +0100 |
User-agent: |
Mutt/1.5.24 (2015-08-30) |
On Tue, Sep 06, 2016 at 04:18:38AM -0700, address@hidden wrote:
> Checking PATCH 1/9: target-mips: Add CP0_Ebase.WG (write gate) support...
> ERROR: space prohibited after that '&' (ctx:WxW)
> #104: FILE: target-mips/op_helper.c:1529:
> + if (arg1 & (1 << CP0EBase_WG) & env->CP0_EBase_rw_bitmask) {
> ^
>
> ERROR: space prohibited after that '&' (ctx:WxW)
> #116: FILE: target-mips/op_helper.c:1540:
> + if (arg1 & (1 << CP0EBase_WG) & env->CP0_EBase_rw_bitmask) {
> ^
its a bitwise and operator, not an address-of operator, so I presumed
this was a false positive.
> Checking PATCH 6/9: target-mips: Add an MMU mode for ERL...
> ERROR: trailing statements should be on next line
> #94: FILE: target-mips/op_helper.c:97:
> + case 3: cpu_##insn##_error_ra(env, addr, val, retaddr); break; \
And this is merely consistent with the surrounding lines so I didn't
bother fixing it.
Cheers
James
signature.asc
Description: Digital signature
- [Qemu-devel] [PATCH 5/9] target-mips: Abstract mmu_idx from hflags, (continued)
- [Qemu-devel] [PATCH 5/9] target-mips: Abstract mmu_idx from hflags, James Hogan, 2016/09/06
- [Qemu-devel] [PATCH 4/9] target-mips: Check memory permissions with mem_idx, James Hogan, 2016/09/06
- [Qemu-devel] [PATCH 1/9] target-mips: Add CP0_Ebase.WG (write gate) support, James Hogan, 2016/09/06
- [Qemu-devel] [PATCH 6/9] target-mips: Add an MMU mode for ERL, James Hogan, 2016/09/06
- [Qemu-devel] [PATCH 2/9] target-mips: Prepare loads/stores for EVA, James Hogan, 2016/09/06
- [Qemu-devel] [PATCH 7/9] target-mips: Add segmentation control registers, James Hogan, 2016/09/06
- [Qemu-devel] [PATCH 9/9] target-mips: Add EVA support to P5600, James Hogan, 2016/09/06
- [Qemu-devel] [PATCH 8/9] target-mips: Implement segmentation control, James Hogan, 2016/09/06
- [Qemu-devel] [PATCH 3/9] target-mips: Decode EVA load & store instructions, James Hogan, 2016/09/06
- Re: [Qemu-devel] [PATCH 0/9] target-mips: Add Enhanced Virtual Addressing (EVA) support, no-reply, 2016/09/06
- Re: [Qemu-devel] [PATCH 0/9] target-mips: Add Enhanced Virtual Addressing (EVA) support,
James Hogan <=