From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from eggs.gnu.org ([140.186.70.92]:36437)
	by lists.gnu.org with esmtp (Exim 4.71)
	(envelope-from <peter.maydell@linaro.org>) id 1R63fB-0001yM-FM
	for qemu-devel@nongnu.org; Tue, 20 Sep 2011 12:59:18 -0400
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <peter.maydell@linaro.org>) id 1R63fA-0002Ga-Ae
	for qemu-devel@nongnu.org; Tue, 20 Sep 2011 12:59:17 -0400
Received: from mail-yi0-f45.google.com ([209.85.218.45]:35787)
	by eggs.gnu.org with esmtp (Exim 4.71)
	(envelope-from <peter.maydell@linaro.org>) id 1R63fA-0002GU-5z
	for qemu-devel@nongnu.org; Tue, 20 Sep 2011 12:59:16 -0400
Received: by yib2 with SMTP id 2so628054yib.4
	for <qemu-devel@nongnu.org>; Tue, 20 Sep 2011 09:59:15 -0700 (PDT)
MIME-Version: 1.0
In-Reply-To: <CAFEAcA-13SdTekzRPk8kCn+FFrg_K9zYYuO9ADW19nFr1GhTtw@mail.gmail.com>
References: <1316355364-23603-1-git-send-email-avi@redhat.com>
	<CAFEAcA-13SdTekzRPk8kCn+FFrg_K9zYYuO9ADW19nFr1GhTtw@mail.gmail.com>
Date: Tue, 20 Sep 2011 17:53:50 +0100
Message-ID: <CAFEAcA_4YDUtL3fko2DV0FJNSFOcTG+HUNvAWSW03QdQuka0Yg@mail.gmail.com>
From: Peter Maydell <peter.maydell@linaro.org>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable
Subject: Re: [Qemu-devel] [PATCH 00/13] Memory API conversion, batch 8
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: Avi Kivity <avi@redhat.com>
Cc: qemu-devel@nongnu.org

On 20 September 2011 17:47, Peter Maydell <peter.maydell@linaro.org> wrote:
> On 18 September 2011 15:15, Avi Kivity <avi@redhat.com> wrote:
>> Batch 7 has some issues due to the ISA conversion, so I reordered some
>> patches and am posting this for your review and testing. =C2=A0It should=
 be
>> straightforward but this stuff never is.
>>
>> Also available on git://github.com/avikivity/qemu.git memory/batch
>>
>> Avi Kivity (13):
>> =C2=A0mips_jazz: convert to memory API
>> =C2=A0mips_malta: convert to memory API
>> =C2=A0mips_mipssim: convert to memory API
>> =C2=A0mips_r4k: convert to memory API
>> =C2=A0musicpal: convert to memory API
>> =C2=A0omap1: convert to memory API (part I)
>> =C2=A0omap1: convert to memory API (part II)
>> =C2=A0omap1: convert to memory API (part III)
>> =C2=A0omap1: convert to memory API (part IV)
>> =C2=A0omap1: convert to memory API (part V)
>> =C2=A0omap_lcdc: remove imif, emiff from structure
>> =C2=A0soc_dma: drop soc_dma_port_add_mem_ram()
>> =C2=A0omap1: convert to memory API (part VI)
>
> I've tested this branch and my n810 meego test image
> still boots. I briefly eyeballed the omap patches and didn't see
> anything obviously wrong. So for patches 06-13:
> Acked-by: Peter Maydell <peter.maydell@linaro.org>

To clarify: n810 isn't an omap1 image. On the other hand I've
never seen any images for our omap1 models (sx1, cheetah), so
my criterion for omap1-affecting patches is the rather low bar
of "doesn't break omap2". If anybody has any omap1 images I'd
be happy to incorporate them into my testing (but I've asked that
here before without luck).

-- PMM