From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from eggs.gnu.org ([2001:4830:134:3::10]:44756)
	by lists.gnu.org with esmtp (Exim 4.71)
	(envelope-from <peter.maydell@linaro.org>) id 1Urxzu-00012y-8Y
	for qemu-devel@nongnu.org; Wed, 26 Jun 2013 18:15:31 -0400
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <peter.maydell@linaro.org>) id 1Urxzr-0002Cb-JT
	for qemu-devel@nongnu.org; Wed, 26 Jun 2013 18:15:30 -0400
Received: from mail-la0-f48.google.com ([209.85.215.48]:59521)
	by eggs.gnu.org with esmtp (Exim 4.71)
	(envelope-from <peter.maydell@linaro.org>) id 1Urxzr-0002CT-Cg
	for qemu-devel@nongnu.org; Wed, 26 Jun 2013 18:15:27 -0400
Received: by mail-la0-f48.google.com with SMTP id lx15so14986lab.35
	for <qemu-devel@nongnu.org>; Wed, 26 Jun 2013 15:15:26 -0700 (PDT)
MIME-Version: 1.0
In-Reply-To: <51CB63C4.1080307@tribudubois.net>
References: <1372184516-32397-1-git-send-email-peter.maydell@linaro.org>
	<1372184516-32397-3-git-send-email-peter.maydell@linaro.org>
	<51C9E490.7090304@redhat.com>
	<CAFEAcA-yhQ2kzzT10wHmEJ7vGF1viKCGL77kmMGOukksvKzQ_w@mail.gmail.com>
	<51CA9086.80802@redhat.com>
	<CAEgOgz6e-trN-oAibuwUJKxD6Tn8XG0h6mM-=majpMXtS5P_Sg@mail.gmail.com>
	<51CB39C3.30707@tribudubois.net> <51CB5995.6020702@tribudubois.net>
	<51CB5AB8.9030409@redhat.com> <51CB63C4.1080307@tribudubois.net>
From: Peter Maydell <peter.maydell@linaro.org>
Date: Wed, 26 Jun 2013 23:15:05 +0100
Message-ID: <CAFEAcA-fYFCr98mWVCv85f-epwR72osMMQDCnPUxPANj1r9-9w@mail.gmail.com>
Content-Type: text/plain; charset=UTF-8
Subject: Re: [Qemu-devel] [PULL 2/8] i.MX: Implement a more complete version
	of the GPT timer.
List-Id: <qemu-devel.nongnu.org>
List-Unsubscribe: <https://lists.nongnu.org/mailman/options/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>
List-Archive: <http://lists.nongnu.org/archive/html/qemu-devel>
List-Post: <mailto:qemu-devel@nongnu.org>
List-Help: <mailto:qemu-devel-request@nongnu.org?subject=help>
List-Subscribe: <https://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=subscribe>
To: Jean-Christophe DUBOIS <jcd@tribudubois.net>
Cc: Peter Crosthwaite <peter.crosthwaite@xilinx.com>, Anthony Liguori <aliguori@us.ibm.com>, qemu-devel@nongnu.org, Paul Brook <paul@codesourcery.com>, Paolo Bonzini <pbonzini@redhat.com>, =?UTF-8?Q?Andreas_F=C3=A4rber?= <afaerber@suse.de>

On 26 June 2013 22:57, Jean-Christophe DUBOIS <jcd@tribudubois.net> wrote:
> Names don't feel really useful in the second case as they are
> indistinguishable. Is the consensus around "generic" names (like MMIO or
> "Ctrl regs") without adding reference to the device a good one for all
> platforms?

Just leave the memory region and vmstate names as they
were before your patches.

(Personally I think it's entirely fine to include the device
name in the memory region name. These strings are for debugging,
so if I print region->name in my debugger it's much more helpful
if it says "imx-serial" than if it just says "regs".)

-- PMM