From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from eggs.gnu.org ([2001:4830:134:3::10]:60011)
	by lists.gnu.org with esmtp (Exim 4.71)
	(envelope-from <peter.maydell@linaro.org>) id 1aVdDd-0007Av-Jc
	for qemu-devel@nongnu.org; Tue, 16 Feb 2016 05:50:58 -0500
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <peter.maydell@linaro.org>) id 1aVdDZ-0001t6-Px
	for qemu-devel@nongnu.org; Tue, 16 Feb 2016 05:50:57 -0500
Received: from mail-vk0-x22d.google.com ([2607:f8b0:400c:c05::22d]:36415)
	by eggs.gnu.org with esmtp (Exim 4.71)
	(envelope-from <peter.maydell@linaro.org>) id 1aVdDZ-0001sY-5d
	for qemu-devel@nongnu.org; Tue, 16 Feb 2016 05:50:53 -0500
Received: by mail-vk0-x22d.google.com with SMTP id c3so129352060vkb.3
	for <qemu-devel@nongnu.org>; Tue, 16 Feb 2016 02:50:52 -0800 (PST)
MIME-Version: 1.0
In-Reply-To: <1452521812-5664-1-git-send-email-pmallapp@broadcom.com>
References: <1452521812-5664-1-git-send-email-pmallapp@broadcom.com>
From: Peter Maydell <peter.maydell@linaro.org>
Date: Tue, 16 Feb 2016 10:50:33 +0000
Message-ID: <CAFEAcA-uYMXYZJ9ARhawvbLpqwOpDPppyjskCus7QfC8119yVA@mail.gmail.com>
Content-Type: text/plain; charset=UTF-8
Subject: Re: [Qemu-devel] [PATCH RFC 0/4] ARM SMMUv3 Emulation
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: pmallapp@broadcom.com
Cc: "Edgar E. Iglesias" <edgar.iglesias@gmail.com>, Peter Crosthwaite <crosthwaite.peter@gmail.com>, QEMU Developers <qemu-devel@nongnu.org>, Alistair Francis <alistair.francis@xilinx.com>

On 11 January 2016 at 14:16,  <pmallapp@broadcom.com> wrote:
> From: Prem Mallappa <pmallapp@broadcom.com>
>
> Implementation Notes:
>
>         - SMMUv3 model, as per ARM SMMUv3 11.0 spec

I haven't reviewed any of this yet, but 11.0 is not the current
revision of the SMMUv3 spec, so you should probably start by
updating your code to the current version. Otherwise code review
will be full of "this isn't what the spec says"...

thanks
-- PMM