From: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
To: qemu-devel@nongnu.org
Cc: Michael@gnu.org,
Jean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com>,
Opdenacker <michael@free-electrons.com>
Subject: Re: [Qemu-devel] [patch 0/2] Add Flash support to the ARM Versatile platforms, v3
Date: Tue, 21 Oct 2008 17:08:06 +0200 [thread overview]
Message-ID: <20081021170806.203055e1@surf> (raw)
In-Reply-To: <20081020112629.GA19014@game.jcrosoft.org>
Hi,
Thanks for your feedback !
Le Mon, 20 Oct 2008 13:26:29 +0200,
Jean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com> a écrit :
> 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
--
Thomas Petazzoni, Free Electrons
Kernel, drivers and embedded Linux development,
consulting, training and support.
http://free-electrons.com
prev parent reply other threads:[~2008-10-21 15:08 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-10-13 8:22 [Qemu-devel] [patch 0/2] Add Flash support to the ARM Versatile platforms, v3 Thomas Petazzoni
2008-10-13 8:22 ` [Qemu-devel] [patch 1/2] Add Flash support to the Versatile PB platform Thomas Petazzoni
2008-10-13 20:35 ` Cedric Hombourger
2008-10-13 22:01 ` Thomas Petazzoni
2008-10-13 8:22 ` [Qemu-devel] [patch 2/2] Mention flash emulation in Versatile documentation Thomas Petazzoni
2008-10-20 8:11 ` [Qemu-devel] [patch 0/2] Add Flash support to the ARM Versatile platforms, v3 Thomas Petazzoni
2008-10-20 11:26 ` Jean-Christophe PLAGNIOL-VILLARD
2008-10-21 15:08 ` Thomas Petazzoni [this message]
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=20081021170806.203055e1@surf \
--to=thomas.petazzoni@free-electrons.com \
--cc=Michael@gnu.org \
--cc=michael@free-electrons.com \
--cc=plagnioj@jcrosoft.com \
--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).