[Qemu-devel] [PATCH] pci: fix i82801b11 bridge

[Qemu-devel] [PATCH] pci: fix i82801b11 bridge

[Gerd Hoffmann]

Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
---
 hw/pci-bridge/i82801b11.c |    1 +
 1 file changed, 1 insertion(+)

diff --git a/hw/pci-bridge/i82801b11.c b/hw/pci-bridge/i82801b11.c
index 8a5e426..14cd7fd 100644
--- a/hw/pci-bridge/i82801b11.c
+++ b/hw/pci-bridge/i82801b11.c
@@ -90,6 +90,7 @@ static void i82801b11_bridge_class_init(ObjectClass *klass, void *data)
     k->device_id = PCI_DEVICE_ID_INTEL_82801BA_11;
     k->revision = ICH9_D2P_A2_REVISION;
     k->init = i82801b11_bridge_initfn;
+    k->config_write = pci_bridge_write_config;
     set_bit(DEVICE_CATEGORY_BRIDGE, dc->categories);
 }
 
-- 
1.7.9.7

Re: [Qemu-devel] [PATCH] pci: fix i82801b11 bridge

[Andreas Färber]

Am 05.08.2013 10:50, schrieb Gerd Hoffmann:
> Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
> ---
>  hw/pci-bridge/i82801b11.c |    1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/hw/pci-bridge/i82801b11.c b/hw/pci-bridge/i82801b11.c
> index 8a5e426..14cd7fd 100644
> --- a/hw/pci-bridge/i82801b11.c
> +++ b/hw/pci-bridge/i82801b11.c
> @@ -90,6 +90,7 @@ static void i82801b11_bridge_class_init(ObjectClass *klass, void *data)
>      k->device_id = PCI_DEVICE_ID_INTEL_82801BA_11;
>      k->revision = ICH9_D2P_A2_REVISION;
>      k->init = i82801b11_bridge_initfn;
> +    k->config_write = pci_bridge_write_config;
>      set_bit(DEVICE_CATEGORY_BRIDGE, dc->categories);
>  }
>  

We have an abstract TYPE_PCI_BRIDGE now, suggest to set it in its
class_init instead if that's a generic bridge function.

Probably should be for-1.6?

Cheers,
Andreas

-- 
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg

Re: [Qemu-devel] [PATCH] pci: fix i82801b11 bridge

[Gerd Hoffmann]

On 08/05/13 11:45, Andreas Färber wrote:
> Am 05.08.2013 10:50, schrieb Gerd Hoffmann:
>> Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
>> ---
>>  hw/pci-bridge/i82801b11.c |    1 +
>>  1 file changed, 1 insertion(+)
>>
>> diff --git a/hw/pci-bridge/i82801b11.c b/hw/pci-bridge/i82801b11.c
>> index 8a5e426..14cd7fd 100644
>> --- a/hw/pci-bridge/i82801b11.c
>> +++ b/hw/pci-bridge/i82801b11.c
>> @@ -90,6 +90,7 @@ static void i82801b11_bridge_class_init(ObjectClass *klass, void *data)
>>      k->device_id = PCI_DEVICE_ID_INTEL_82801BA_11;
>>      k->revision = ICH9_D2P_A2_REVISION;
>>      k->init = i82801b11_bridge_initfn;
>> +    k->config_write = pci_bridge_write_config;
>>      set_bit(DEVICE_CATEGORY_BRIDGE, dc->categories);
>>  }
>>  
> 
> We have an abstract TYPE_PCI_BRIDGE now, suggest to set it in its
> class_init instead if that's a generic bridge function.

Well, depends on the bridge device.  When they have stuff beside core
pci bridge functionality (such as shpc hotplug support like the
'pci-bridge' device) they need their private function.  But those can
still override config_write in the device init I guess.

/me leaves that to mst to decide.  Beside that it should most likely go
into a separate cleanup patch as there are other devices which hook
pci_bridge_write_config the same way.

> Probably should be for-1.6?

Yes.

cheers,
  Gerd

Re: [Qemu-devel] [PATCH] pci: fix i82801b11 bridge

[Michael S. Tsirkin]

On Mon, Aug 05, 2013 at 12:33:14PM +0200, Gerd Hoffmann wrote:
> On 08/05/13 11:45, Andreas Färber wrote:
> > Am 05.08.2013 10:50, schrieb Gerd Hoffmann:
> >> Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
> >> ---
> >>  hw/pci-bridge/i82801b11.c |    1 +
> >>  1 file changed, 1 insertion(+)
> >>
> >> diff --git a/hw/pci-bridge/i82801b11.c b/hw/pci-bridge/i82801b11.c
> >> index 8a5e426..14cd7fd 100644
> >> --- a/hw/pci-bridge/i82801b11.c
> >> +++ b/hw/pci-bridge/i82801b11.c
> >> @@ -90,6 +90,7 @@ static void i82801b11_bridge_class_init(ObjectClass *klass, void *data)
> >>      k->device_id = PCI_DEVICE_ID_INTEL_82801BA_11;
> >>      k->revision = ICH9_D2P_A2_REVISION;
> >>      k->init = i82801b11_bridge_initfn;
> >> +    k->config_write = pci_bridge_write_config;
> >>      set_bit(DEVICE_CATEGORY_BRIDGE, dc->categories);
> >>  }
> >>  
> > 
> > We have an abstract TYPE_PCI_BRIDGE now, suggest to set it in its
> > class_init instead if that's a generic bridge function.
> 
> Well, depends on the bridge device.  When they have stuff beside core
> pci bridge functionality (such as shpc hotplug support like the
> 'pci-bridge' device) they need their private function.  But those can
> still override config_write in the device init I guess.
> 
> /me leaves that to mst to decide.  Beside that it should most likely go
> into a separate cleanup patch as there are other devices which hook
> pci_bridge_write_config the same way.

Yea, it's probably not 1.6 material.


> > Probably should be for-1.6?
> 
> Yes.
> 
> cheers,
>   Gerd
>