From: Peter Maydell <peter.maydell@linaro.org>
To: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>
Cc: QEMU Developers <qemu-devel@nongnu.org>,
Richard Henderson <rth@twiddle.net>
Subject: Re: [Qemu-devel] [PATCH 0/5] SPARC: fix clang warnings
Date: Fri, 2 Jan 2015 17:30:12 +0000 [thread overview]
Message-ID: <CAFEAcA9RKAzR4OOv3VuDex0zQuk38Ve-T7YK3euYis7JehU2=A@mail.gmail.com> (raw)
In-Reply-To: <54A6A3CE.8090008@ilande.co.uk>
On 2 January 2015 at 13:57, Mark Cave-Ayland
<mark.cave-ayland@ilande.co.uk> wrote:
> On 23/12/14 22:11, Peter Maydell wrote:
>> These patches fix warnings generated by clang. Patches 1-3
>> have been onlist before (and reviewed by RTH) but didn't get
>> applied I think because of a mixup between me and Mark about
>> which tree they should go in by. 4 and 5 are new.
> Oh my apologies! Given that there's not much review that I can
> personally do (and they are just compilation fixes) then I was expecting
> these to go either via master or Richard. If that is still a problem let
> me know and I'll queue them as part of my next pull request.
Where we have a subsystem maintainer I tend to prefer to route
patches via them, on the assumption that they'll do a second
level of testing (even if only of the "yep, still boots"
variety). Also it avoids potential clashes between different
patches to the same target if they all go through your tree.
The idea behind the subsystem-maintainer setup is to spread
the workload a bit :-)
-- PMM
next prev parent reply other threads:[~2015-01-02 17:30 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-12-23 22:11 [Qemu-devel] [PATCH 0/5] SPARC: fix clang warnings Peter Maydell
2014-12-23 22:11 ` [Qemu-devel] [PATCH 1/5] target-sparc: Remove unused gen_op_subi_cc and gen_op_addi_cc Peter Maydell
2014-12-23 22:11 ` [Qemu-devel] [PATCH 2/5] target-sparc: address_mask(), asi_address_mask() are TARGET_SPARC64 only Peter Maydell
2014-12-23 22:11 ` [Qemu-devel] [PATCH 3/5] target-sparc: is_translating_asi() is " Peter Maydell
2014-12-23 22:11 ` [Qemu-devel] [PATCH 4/5] target-sparc: Mark gen_load_trap_state_at_tl() as !CONFIG_USER_ONLY Peter Maydell
2014-12-23 22:11 ` [Qemu-devel] [PATCH 5/5] disas/sparc: Remove unused data sparc_opcode_archs[] Peter Maydell
2015-01-02 13:57 ` [Qemu-devel] [PATCH 0/5] SPARC: fix clang warnings Mark Cave-Ayland
2015-01-02 17:30 ` Peter Maydell [this message]
2015-01-18 21:15 ` Mark Cave-Ayland
2015-01-21 16:16 ` Mark Cave-Ayland
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='CAFEAcA9RKAzR4OOv3VuDex0zQuk38Ve-T7YK3euYis7JehU2=A@mail.gmail.com' \
--to=peter.maydell@linaro.org \
--cc=mark.cave-ayland@ilande.co.uk \
--cc=qemu-devel@nongnu.org \
--cc=rth@twiddle.net \
/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).