From: Thomas Huth <thuth@redhat.com>
To: Laurent Vivier <laurent@vivier.eu>, qemu-devel@nongnu.org
Cc: Fam Zheng <famz@redhat.com>,
Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>,
Max Reitz <mreitz@redhat.com>,
qemu-arm@nongnu.org, Paolo Bonzini <pbonzini@redhat.com>,
John Snow <jsnow@redhat.com>,
David Gibson <david@gibson.dropbear.id.au>
Subject: Re: [Qemu-arm] [PATCH v2 3/5] m68k: Update the 68k sections in the MAINTAINERS file
Date: Tue, 8 Nov 2016 13:49:11 +0100 [thread overview]
Message-ID: <46d63cec-57f2-e433-b4ef-e117fde46c68@redhat.com> (raw)
In-Reply-To: <118e4f8d-c837-46aa-da6a-5fb135dc3c07@vivier.eu>
On 08.11.2016 13:21, Laurent Vivier wrote:
> Le 08/11/2016 à 13:17, Thomas Huth a écrit :
>> disas/m68k.c obviously belong to the m68k CPU section in
>> the MAINTAINERS file, but remove the hw/m68k/ directory
>> here since it only contains machine (not CPU) related
>> files, as requested by Laurent. Add the machine related
>> files to the right machine sections instead.
>
> I think you can also add include/hw/m68k/mcf.h in the machine section.
That header currently contains prototypes for both, the 5206 and the
5208 machine... and I'd like to avoid adding that twice. We should
likely clean that up one day, but since it's currently orphaned anyway,
it's IMHO not that important to have this listed right now.
Thomas
next prev parent reply other threads:[~2016-11-08 12:49 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-11-08 12:17 [Qemu-arm] [PATCH 0/5] Fixes for the MAINTAINERS file Thomas Huth
2016-11-08 12:17 ` [Qemu-arm] [PATCH 1/5] MAINTAINERS: Add some ARM related files to the corresponding sections Thomas Huth
2016-11-08 12:17 ` [Qemu-arm] [PATCH 2/5] sparc: Add slavio_misc.c and eccmemctl.c to the MAINTAINERS file Thomas Huth
2016-11-08 12:17 ` [Qemu-arm] [PATCH v2 3/5] m68k: Update the 68k sections in " Thomas Huth
2016-11-08 12:21 ` [Qemu-devel] " Laurent Vivier
2016-11-08 12:49 ` Thomas Huth [this message]
2016-11-08 13:46 ` Laurent Vivier
2016-11-08 12:17 ` [Qemu-arm] [PATCH 4/5] MAINTAINERS: Add an entry for the CHRP NVRAM files Thomas Huth
2016-11-09 0:38 ` David Gibson
2016-11-08 12:17 ` [Qemu-arm] [PATCH 5/5] MAINTAINERS: Add Fam and Jsnow for Bitmap support Thomas Huth
2016-11-08 17:00 ` [Qemu-devel] [PATCH 0/5] Fixes for the MAINTAINERS file Stefan Hajnoczi
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=46d63cec-57f2-e433-b4ef-e117fde46c68@redhat.com \
--to=thuth@redhat.com \
--cc=david@gibson.dropbear.id.au \
--cc=famz@redhat.com \
--cc=jsnow@redhat.com \
--cc=laurent@vivier.eu \
--cc=mark.cave-ayland@ilande.co.uk \
--cc=mreitz@redhat.com \
--cc=pbonzini@redhat.com \
--cc=qemu-arm@nongnu.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).