On 11/01/2010 10:47 AM, Alexander Graf wrote:
On 01.11.2010, at 11:44, Anthony Liguori wrote:

On 11/01/2010 10:01 AM, Alexander Graf wrote:
This patch adds various header files required for the xenner kernel on 64 bit
systems.

Signed-off-by: Alexander Graf<ag...@suse.de>

I think it might make more sense to put this on a separate git.qemu.org 
repository and then use a submodule in roms/.
The main reason why I wanted to have it fully in the qemu tree is so that we can 
guarantee the qemu<->xenner interface is 100% private. I'm not sure how much 
it'll be in a flux, but I don't want to go through the hassle of defining stable ABIs 
where we don't have to :)

Yeah, I'm not suggesting it be a separate project. But we've been keeping firmware out of tree for the most part and using submodules. I like this model (even though git submodules don't always work the way you expect them to..).

I'm not passionate either way. If you feel strongly about it being in the main source tree, I don't have a problem with it. But stick it in roms/ instead of pc-bios.

Regards,

Anthony Liguori

Alex



Reply via email to