qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: "KONRAD Frédéric" <fred.konrad@greensocs.com>
To: Peter Maydell <peter.maydell@linaro.org>
Cc: amit.shah@redhat.com, cornelia.huck@de.ibm.com,
	Anthony Liguori <aliguori@us.ibm.com>,
	mark.burton@greensocs.com, qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] [PATCH v6 0/7] virtio-serial refactoring.
Date: Tue, 09 Apr 2013 17:05:10 +0200	[thread overview]
Message-ID: <51642E26.1010702@greensocs.com> (raw)
In-Reply-To: <CAFEAcA_7ShyfgaOsmBr5gvZaBxcq+mkuEMnBQ-0KDU2+ycv-hw@mail.gmail.com>

On 09/04/2013 16:53, Peter Maydell wrote:
> On 9 April 2013 13:53,  <fred.konrad@greensocs.com> wrote:
>> From: KONRAD Frederic <fred.konrad@greensocs.com>
>>
>> This is the next part of virtio-refactoring.
>>
>> Basically it creates virtio-serial device which extends virtio-device.
>> Then a virtio-serial can be connected on a virtio-bus.
>> virtio-serial-pci, virtio-serial-s390 and virtio-serial-ccw are created too,
>> they extend respectively virtio-pci, virtio-s390-device, virtio-ccw-device and
>> have a virtio-serial.
>>
>> Tested-by: Cornelia Huck <cornelia.huck@de.ibm.com>
>> Reviewed-by: Cornelia Huck <cornelia.huck@de.ibm.com>
>> Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
> Note to Anthony -- the way Fred has put the R-b: tags in
> his cover letter here seems to confuse your 'patches'
> script. 'patches apply' applies the patches and adds these
> tags to every patch in the series, but
>    patches list id:1365512016-21944-1-git-send-email-fred.konrad@greensocs.com
> status:reviewed
>
> produces no output.
>
> I flag this up because this patch series will probably not appear
> on your radar otherwise.
>
> Fred: the usual convention is to put reviewed-by
> tags on the patches themselves when resending, even
> if the reviewers were lazy and replied to the coverletter.
>
> thanks
> -- PMM
Sorry for that, I'm not too aware of this patches scripts and thought it 
had the same result.

Do you want me to resend with the r-b tags inside each patches?

Fred

  reply	other threads:[~2013-04-09 15:05 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-04-09 12:53 [Qemu-devel] [PATCH v6 0/7] virtio-serial refactoring fred.konrad
2013-04-09 12:53 ` [Qemu-devel] [PATCH v6 1/7] virtio-serial: add the virtio-serial device fred.konrad
2013-04-09 12:53 ` [Qemu-devel] [PATCH v6 2/7] virtio-serial-pci: switch to the new API fred.konrad
2013-04-09 12:53 ` [Qemu-devel] [PATCH v6 3/7] virtio-serial-s390: " fred.konrad
2013-04-09 12:53 ` [Qemu-devel] [PATCH v6 4/7] virtio-serial-ccw: " fred.konrad
2013-04-09 12:53 ` [Qemu-devel] [PATCH v6 5/7] virtio-serial: cleanup: init and exit functions fred.konrad
2013-04-09 12:53 ` [Qemu-devel] [PATCH v6 6/7] virtio-serial: cleanup: use QOM casts fred.konrad
2013-04-09 12:53 ` [Qemu-devel] [PATCH v6 7/7] virtio-serial: cleanup: remove qdev field fred.konrad
2013-04-09 14:53 ` [Qemu-devel] [PATCH v6 0/7] virtio-serial refactoring Peter Maydell
2013-04-09 15:05   ` KONRAD Frédéric [this message]
2013-04-09 15:07   ` Anthony Liguori
2013-04-09 15:20     ` Peter Maydell
2013-04-10 10:34 ` Amit Shah
2013-04-15 21:10 ` Anthony Liguori

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=51642E26.1010702@greensocs.com \
    --to=fred.konrad@greensocs.com \
    --cc=aliguori@us.ibm.com \
    --cc=amit.shah@redhat.com \
    --cc=cornelia.huck@de.ibm.com \
    --cc=mark.burton@greensocs.com \
    --cc=peter.maydell@linaro.org \
    --cc=qemu-devel@nongnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).