From: Gerd Hoffmann <kraxel@redhat.com>
To: Paul Brook <paul@codesourcery.com>
Cc: qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] Re: [PATCH 0/10] qdev patches.
Date: Mon, 22 Jun 2009 16:02:03 +0200 [thread overview]
Message-ID: <4A3F8EDB.5030502@redhat.com> (raw)
In-Reply-To: <4A3F4BB9.6060604@redhat.com>
Hi,
>> This is exactly the sort of fake conversion that I don't like, because
>> you
>> still require use of the old hardcoded initialization functions.
>
> I'd prefer to call them "incomplete" instead of "fake". I know that more
> work needs to be done to get the drivers into shape for a dt-driven
> machine creation.
To followup with some more backgroud on that on that: These are cases
where additional arguments are passed to the *_init() function.
Partly the reason is irq windup (i.e. passing around qemu_irqs), which
must be adapted. Didn't investigate that one yet. Probably non-trivial
to the way interrupts work (pic, lapic, ioapic, ...) in PCs.
Partly the arguments carry extra configutation info. In that case the
info should probably come as attributes from the the device tree.
Didn't look (yet) into using attributes, one of the reasons being that
the API for them is still quite unclear.
cheers,
Gerd
next prev parent reply other threads:[~2009-06-22 14:04 UTC|newest]
Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-06-17 12:59 [Qemu-devel] [PATCH 0/10] qdev patches Gerd Hoffmann
2009-06-17 12:59 ` [Qemu-devel] [PATCH 01/10] qdev: update pci device registration Gerd Hoffmann
2009-06-17 12:59 ` [Qemu-devel] [PATCH 02/10] qdev: replace bus_type enum with bus_info struct Gerd Hoffmann
2009-06-17 12:59 ` [Qemu-devel] [PATCH 03/10] qdev: remove DeviceType Gerd Hoffmann
2009-06-17 12:59 ` [Qemu-devel] [PATCH 04/10] qdev/pci: bus name Gerd Hoffmann
2009-06-17 12:59 ` [Qemu-devel] [PATCH 05/10] qdev: hook up i440fx Gerd Hoffmann
2009-06-17 12:59 ` [Qemu-devel] [PATCH 06/10] qdev: convert piix-ide, first step Gerd Hoffmann
2009-06-17 12:59 ` [Qemu-devel] [PATCH 07/10] qdev-ify: piix acpi Gerd Hoffmann
2009-06-17 12:59 ` [Qemu-devel] [PATCH 08/10] qdev-ify: uhci Gerd Hoffmann
2009-06-17 12:59 ` [Qemu-devel] [PATCH 09/10] qdev-ify: usb Gerd Hoffmann
2009-06-17 12:59 ` [Qemu-devel] [PATCH 10/10] qdev-ify: scsi Gerd Hoffmann
2009-06-18 14:39 ` [Qemu-devel] Re: [PATCH 0/10] qdev patches Gerd Hoffmann
2009-06-19 15:59 ` Gerd Hoffmann
2009-06-19 17:51 ` Paul Brook
2009-06-22 9:15 ` Gerd Hoffmann
2009-06-22 9:36 ` Avi Kivity
2009-06-22 9:57 ` Gerd Hoffmann
2009-06-22 11:25 ` Gerd Hoffmann
2009-06-22 11:29 ` Avi Kivity
2009-06-22 14:02 ` Gerd Hoffmann [this message]
2009-06-22 13:45 ` Gerd Hoffmann
2009-06-19 16:26 ` [Qemu-devel] " Paul Brook
2009-06-26 21:16 ` Markus Armbruster
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=4A3F8EDB.5030502@redhat.com \
--to=kraxel@redhat.com \
--cc=paul@codesourcery.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).