On Mon, Apr 27, 2015 at 04:41:13PM +0300, Michael Tokarev 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 <[email protected]> > > > > Hi, > > > > Any reason not to shift down the numbering? > > > > Otherwise the whole series looks good to me. > > What does this mean, "looks good", is it your R-b? :)
Sure, with the change: Reviewed-by: Edgar E. Iglesias <[email protected]> > > >> @@ -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? > > Thanks, > > /mjt
