From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43)
	id 1KsIqJ-0008Ri-Vn
	for qemu-devel@nongnu.org; Tue, 21 Oct 2008 11:08:20 -0400
Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43)
	id 1KsIqH-0008Qn-Bx
	for qemu-devel@nongnu.org; Tue, 21 Oct 2008 11:08:18 -0400
Received: from [199.232.76.173] (port=51675 helo=monty-python.gnu.org)
	by lists.gnu.org with esmtp (Exim 4.43) id 1KsIqH-0008Qk-8F
	for qemu-devel@nongnu.org; Tue, 21 Oct 2008 11:08:17 -0400
Received: from mail.free-electrons.com ([88.191.76.200]:51770)
	by monty-python.gnu.org with esmtp (Exim 4.60)
	(envelope-from <thomas.petazzoni@free-electrons.com>)
	id 1KsIqG-0000b7-JU
	for qemu-devel@nongnu.org; Tue, 21 Oct 2008 11:08:17 -0400
Date: Tue, 21 Oct 2008 17:08:06 +0200
From: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
Subject: Re: [Qemu-devel] [patch 0/2] Add Flash support to the ARM Versatile
	platforms, v3
Message-ID: <20081021170806.203055e1@surf>
In-Reply-To: <20081020112629.GA19014@game.jcrosoft.org>
References: <20081013082238.042379564@free-electrons.com>
	<20081020101150.5ecfdfc3@surf>
	<20081020112629.GA19014@game.jcrosoft.org>
Mime-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable
Reply-To: qemu-devel@nongnu.org
List-Id: qemu-devel.nongnu.org
List-Unsubscribe: <http://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>
List-Archive: <http://lists.gnu.org/pipermail/qemu-devel>
List-Post: <mailto:qemu-devel@nongnu.org>
List-Help: <mailto:qemu-devel-request@nongnu.org?subject=help>
List-Subscribe: <http://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=subscribe>
To: qemu-devel@nongnu.org
Cc: Michael@gnu.org, Jean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com>, Opdenacker <michael@free-electrons.com>

Hi,

Thanks for your feedback !

Le Mon, 20 Oct 2008 13:26:29 +0200,
Jean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com> a =C3=A9crit :

> IMHO we need to check that you U-Boot patch could run on a real
> hardware before modify qemu. As I sais before we need to be reflect
> the hardware at maximun

My U-Boot patch is only secondary: the flash emulation can simply be
used by the Linux kernel, for example.

In my U-Boot patch, the only thing related to the flash is the switch
from a Versatile-specific flash driver to the generic CFI driver. The
rest of the modifications have nothing to do with the flash, they are
just hacks/workarounds to get U-Boot to run under Qemu and are not
related in any way with this patch (network driver things and timing
things).

These issues should be worked out as a separate Qemu patch if we really
want Qemu to run an unmodified U-Boot for the Versatile platform, but
IMO, that shouldn't prevent the flash emulation from being merged in
Qemu.

Sincerly,

Thomas
--=20
Thomas Petazzoni, Free Electrons
Kernel, drivers and embedded Linux development,
consulting, training and support.
http://free-electrons.com