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. 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