On Thu, Apr 12, 2012 at 11:25 AM, Andreas Färber <afaer...@suse.de> wrote: > Hello John, > > Am 12.04.2012 03:14, schrieb John Williams: >> 2012/4/12 Andreas Färber <afaer...@suse.de>: >>> Move code from cpu_state_reset() to QOM mb_cpu_reset(). >>> >>> Signed-off-by: Andreas Färber <afaer...@suse.de> >>> --- >>> target-microblaze/cpu.c | 50 >>> +++++++++++++++++++++++++++++++++++++++- >>> target-microblaze/translate.c | 51 >>> +--------------------------------------- >>> 2 files changed, 51 insertions(+), 50 deletions(-) >>> >>> diff --git a/target-microblaze/cpu.c b/target-microblaze/cpu.c >>> index ea646fe..194131b 100644 >>> --- a/target-microblaze/cpu.c >>> +++ b/target-microblaze/cpu.c >>> @@ -1,6 +1,7 @@ >>> /* >>> * QEMU MicroBlaze CPU >>> * >>> + * Copyright (c) 2009 Edgar E. Iglesias >>> * Copyright (c) 2012 SUSE LINUX Products GmbH >> >> Trivial nitpick - please add >> >> Copyright (c) 2009-2012 PetaLogix Qld Pty Ltd >> >> here as well - we've been driving and funding pretty much all of the >> MicroBlaze QEMU development the last 3 years, we should be in the >> headers. > > The reason for splitting the original patches up was to make clearer for > reviewers where particular code snippets come from. > > Could you guys please prepare a patch that updates the copyright in > translate.c so that it doesn't come out of the blue here? I'll happily > rebase onto that.
Hi Andreas, Ive made this patch, Its up on the mailing list. Its based against mainline so this series will need a rebase as you suggested. Regards, Peter > > Thanks for the quick reply, > > Andreas > > -- > SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany > GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg >
