|
From: | Anthony Liguori |
Subject: | Re: [Qemu-devel] [PATCH 0/5] qdev: device capabilities |
Date: | Fri, 04 Sep 2009 10:08:09 -0500 |
User-agent: | Thunderbird 2.0.0.23 (X11/20090825) |
Gerd Hoffmann wrote:
Hi, This patch series adds device capabilities to qdev devices. First patch adds the infrastructure, next patches add the capabilities. New member of this patch series is #5 which adds a watchdog capability and uses it to kill the superfluous private driver list in watchdog.c. Individual patches have more detailed descriptions.
I still don't understand why this is needed. It's duplicating data that should already be present in the device model.
A device has a capability because it supports a particular interface. Doesn't it make more sense to tie that interface to qdev instead of having to explicitly state the device has that interface?
Regards, Anthony Liguori
cheers, Gerd
[Prev in Thread] | Current Thread | [Next in Thread] |