[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [V6, 2/7] nios2: Add architecture emulation support
From: |
Marek Vasut |
Subject: |
Re: [Qemu-devel] [V6, 2/7] nios2: Add architecture emulation support |
Date: |
Mon, 7 Nov 2016 19:14:34 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Icedove/45.4.0 |
On 11/07/2016 04:58 AM, Guenter Roeck wrote:
> On Tue, Oct 25, 2016 at 09:57:43PM +0200, Marek Vasut wrote:
>> From: Chris Wulff <address@hidden>
>>
>> Add support for emulating Altera NiosII R1 architecture into qemu.
>> This patch is based on previous work by Chris Wulff from 2012 and
>> updated to latest mainline QEMU.
>>
>> Signed-off-by: Marek Vasut <address@hidden>
>> Cc: Chris Wulff <address@hidden>
>> Cc: Jeff Da Silva <address@hidden>
>> Cc: Ley Foon Tan <address@hidden>
>> Cc: Sandra Loosemore <address@hidden>
>> Cc: Yves Vandervennet <address@hidden>
>> ---
>> V3: Thorough cleanup, deal with the review comments all over the place
>> V4: - Use extract32()
>> - Fix gen_goto_tb() , suppress tcg_gen_goto_tb()
>> - Clean up gen_check_supervisor() helper
>> - Use TCGMemOp type for flags
>> - Drop jump labels from wrctl/rdctl
>> - More TCG cleanup
>> V5: - Simplify load/store handling
>> - Handle loads into R_ZERO from protected page, add comment
>> V6: - Fix division opcode handling
>> - Add missing disas handling
>> - V5 review comments cleanup
>> ---
> [ ... ]
>
>> diff --git a/target-nios2/cpu.h b/target-nios2/cpu.h
>> new file mode 100644
>> index 0000000..17c9a0f
> [ ... ]
>
>> +static inline void cpu_get_tb_cpu_state(CPUNios2State *env, target_ulong
>> *pc,
>> + target_ulong *cs_base, uint32_t
>> *flags)
>> +{
>> + *pc = env->regs[R_PC];
>> + *cs_base = 0;
>> + *flags = (env->regs[CR_STATUS] & (CR_STATUS_EH | CR_STATUS_U));
>> +}
>> +
>> +#endif /* CPU_NIOS2_H */
>> +
>
> The empty line at the end results in a whitespace message from git.
Dropped, thanks. Is there anything else or is this patchset starting to
become acceptable ?
I'll wait a bit before you finish discussing the whitespace errors in
1/7 with Sandra.
--
Best regards,
Marek Vasut
- Re: [Qemu-devel] [V6, 2/7] nios2: Add architecture emulation support, Guenter Roeck, 2016/11/06
- Re: [Qemu-devel] [V6, 2/7] nios2: Add architecture emulation support,
Marek Vasut <=
- Re: [Qemu-devel] [V6, 2/7] nios2: Add architecture emulation support, Guenter Roeck, 2016/11/07
- Re: [Qemu-devel] [V6, 2/7] nios2: Add architecture emulation support, Marek Vasut, 2016/11/12
- Re: [Qemu-devel] [V6, 2/7] nios2: Add architecture emulation support, Guenter Roeck, 2016/11/12
- Re: [Qemu-devel] [V6, 2/7] nios2: Add architecture emulation support, Marek Vasut, 2016/11/13
- Re: [Qemu-devel] [V6, 2/7] nios2: Add architecture emulation support, Marek Vasut, 2016/11/13
- Re: [Qemu-devel] [V6, 2/7] nios2: Add architecture emulation support, Guenter Roeck, 2016/11/13
- Re: [Qemu-devel] [V6, 2/7] nios2: Add architecture emulation support, Marek Vasut, 2016/11/13
- Re: [Qemu-devel] [V6, 2/7] nios2: Add architecture emulation support, Guenter Roeck, 2016/11/13