On Thu, Apr 12, 2012 at 03:25:30AM +0200, Andreas Färber 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, Your patches look ok, can you please rebase on top of the added copyright notice and I'll give then a test and commit. Thanks
