[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH microblaze v1 5/6] mb: cpu: Delete EXCP_NMI
From: |
Peter Crosthwaite |
Subject: |
Re: [Qemu-devel] [PATCH microblaze v1 5/6] mb: cpu: Delete EXCP_NMI |
Date: |
Mon, 27 Apr 2015 07:13:12 -0700 |
On Mon, Apr 27, 2015 at 6:41 AM, Michael Tokarev <address@hidden> wrote:
> 27.04.2015 12:28, Edgar E. Iglesias wrote:
>> On Sun, Apr 26, 2015 at 12:10:23PM -0700, Peter Crosthwaite wrote:
>>> This define is unused. Remove.
>>>
>>> Signed-off-by: Peter Crosthwaite <address@hidden>
>>
>> Hi,
>>
>> Any reason not to shift down the numbering?
None. Although my goal was to make no code change in the trivial patch series.
>>
>> Otherwise the whole series looks good to me.
>
> What does this mean, "looks good", is it your R-b? :)
>
>>> @@ -36,7 +36,6 @@ typedef struct CPUMBState CPUMBState;
>>>
>>> #define ELF_MACHINE EM_MICROBLAZE
>>>
>>> -#define EXCP_NMI 1
>>> #define EXCP_MMU 2
>>> #define EXCP_IRQ 3
>>> #define EXCP_BREAK 4
>
> I can renumber this on commit just fine, there's no need
> to resend whole thing, is it okay with you Peter?
>
It's ok with me. Thanks.
Regards,
Peter
> Thanks,
>
> /mjt
>
- [Qemu-devel] [PATCH microblaze v1 0/6] Microblaze Cleanups, Peter Crosthwaite, 2015/04/26
- [Qemu-devel] [PATCH microblaze v1 1/6] defconfigs: Piggyback microblazeel on microblaze, Peter Crosthwaite, 2015/04/26
- [Qemu-devel] [PATCH microblaze v1 2/6] mb: mmu: Delete flip_um fn prototype, Peter Crosthwaite, 2015/04/26
- [Qemu-devel] [PATCH microblaze v1 3/6] mb: cpu: Remote unused cpu_get_pc, Peter Crosthwaite, 2015/04/26
- [Qemu-devel] [PATCH microblaze v1 4/6] mb: cpu: Remove unused CC_OP enum, Peter Crosthwaite, 2015/04/26
- [Qemu-devel] [PATCH microblaze v1 5/6] mb: cpu: Delete EXCP_NMI, Peter Crosthwaite, 2015/04/26
- [Qemu-devel] [PATCH microblaze v1 6/6] mb: cpu: delete unused cpu_interrupts_enabled, Peter Crosthwaite, 2015/04/26
- Re: [Qemu-devel] [PATCH microblaze v1 0/6] Microblaze Cleanups, Michael Tokarev, 2015/04/27