Re: [Qemu-devel] [PATCH v02] add 1394 bus support

2015-04-20 Thread Itamar Tal
On Sun, Apr 19, 2015 at 8:21 PM, Andreas Färber wrote: > Hi, > > Am 19.04.2015 um 18:38 schrieb Itamar Tal: > > I've set it just after the subject field in the patch? Should I add it > > somewhere else and resubmit? > > Please compare other QEMU or Linux patches: It needs to go before --- > into

Re: [Qemu-devel] [PATCH v02] add 1394 bus support

2015-04-19 Thread Peter Maydell
On 19 April 2015 at 11:52, wrote: > From: Itamar Tal > > --- > default-configs/i386-softmmu.mak |1 + > default-configs/x86_64-softmmu.mak |1 + > hw/1394/Makefile.objs |1 + > hw/1394/hcd-ohci.c | 1754 > > hw/139

Re: [Qemu-devel] [PATCH v02] add 1394 bus support

2015-04-19 Thread Andreas Färber
Hi, Am 19.04.2015 um 18:38 schrieb Itamar Tal: > I've set it just after the subject field in the patch? Should I add it > somewhere else and resubmit? Please compare other QEMU or Linux patches: It needs to go before --- into the commit message. git commit --amend -s will place it correctly. BTW

Re: [Qemu-devel] [PATCH v02] add 1394 bus support

2015-04-19 Thread Itamar Tal
I've set it just after the subject field in the patch? Should I add it somewhere else and resubmit? On Apr 19, 2015 6:47 PM, "Andreas Färber" wrote: > Am 19.04.2015 um 12:52 schrieb itamar.t...@gmail.com: > > From: Itamar Tal > > > > --- > > Still no Signed-off-by... > > Regards, > Andreas > > -

Re: [Qemu-devel] [PATCH v02] add 1394 bus support

2015-04-19 Thread Andreas Färber
Am 19.04.2015 um 12:52 schrieb itamar.t...@gmail.com: > From: Itamar Tal > > --- Still no Signed-off-by... Regards, Andreas -- SUSE Linux GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany GF: Felix Imendörffer, Jane Smithard, Jennifer Guild, Dilip Upmanyu, Graham Norton; HRB 21284 (AG Nürnberg)

[Qemu-devel] [PATCH v02] add 1394 bus support

2015-04-19 Thread itamar . tal4
From: Itamar Tal --- default-configs/i386-softmmu.mak |1 + default-configs/x86_64-softmmu.mak |1 + hw/1394/Makefile.objs |1 + hw/1394/hcd-ohci.c | 1754 hw/1394/hcd-ohci.h | 147 +++ hw/Makefile.o