On Wed, 22 Jun 2011, Anthony PERARD wrote:
> On Wed, Jun 22, 2011 at 18:43, Stefano Stabellini
> <stefano.stabellini@xxxxxxxxxxxxx> wrote:
> > On Wed, 22 Jun 2011, anthony.perard@xxxxxxxxxx wrote:
> >> From: Anthony PERARD <anthony.perard@xxxxxxxxxx>
> >>
> >> Signed-off-by: Anthony PERARD <anthony.perard@xxxxxxxxxx>
> >> ---
> >> hw/piix_pci.c | 3 +++
> >> 1 files changed, 3 insertions(+), 0 deletions(-)
> >>
> >> diff --git a/hw/piix_pci.c b/hw/piix_pci.c
> >> index 26ce904..d08b31a 100644
> >> --- a/hw/piix_pci.c
> >> +++ b/hw/piix_pci.c
> >> @@ -478,6 +478,9 @@ static PCIDeviceInfo i440fx_info[] = {
> >> .no_hotplug = 1,
> >> .init = piix3_initfn,
> >> .config_write = piix3_write_config_xen,
> >> + .vendor_id = PCI_VENDOR_ID_INTEL,
> >> + .device_id = PCI_DEVICE_ID_INTEL_82371SB_0, // 82371SB PIIX3
> >> PCI-to-ISA bridge (Step A1)
> >> + .class_id = PCI_CLASS_BRIDGE_ISA,
> >> },{
> >> /* end of list */
> >> }
> >
> > shouldn't piix3_initfn take care of setting vendor_id, device_id and
> > class_id, as in the normal PIIX3 case?
>
> Not anymore. These ids have been removed from piix3_initfn and added
> to the PCIDeviceInfo of PIIX3 in the last update.
I see. Good catch. _______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel
|