[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] Re: [PATCH] apb_pci: fix header type of pbm pci host bridge
From: |
Michael S. Tsirkin |
Subject: |
[Qemu-devel] Re: [PATCH] apb_pci: fix header type of pbm pci host bridge. |
Date: |
Mon, 8 Feb 2010 12:27:53 +0200 |
User-agent: |
Mutt/1.5.19 (2009-01-05) |
On Mon, Feb 08, 2010 at 03:45:03PM +0900, Isaku Yamahata wrote:
> The change set of 776e1bbb6cf4fe66a93c1a5dd814bbb650deca00
> specifies pbm pci host bridge is type of bridge.
> It contradicts with pbm_pci_host_init().
By the way, the next below (cover letter) should be put after
--- rather than here, so that it does not end up in
commit message.
> Blue Swirl, could you please check this patch?
> To be honest I don't know about pbm pci host bridge so that
> I don't know which is correct, pbm_pci_host_init() or pbm_pci_host_info.
> I just took the older code.
>
> Cc: Blue Swirl <address@hidden>
> Cc: "Michael S. Tsirkin" <address@hidden>
> Signed-off-by: Isaku Yamahata <address@hidden>
> ---
> hw/apb_pci.c | 3 +--
> 1 files changed, 1 insertions(+), 2 deletions(-)
>
> diff --git a/hw/apb_pci.c b/hw/apb_pci.c
> index 46d5b0e..359a84f 100644
> --- a/hw/apb_pci.c
> +++ b/hw/apb_pci.c
> @@ -471,7 +471,7 @@ static int pbm_pci_host_init(PCIDevice *d)
> d->config[0x09] = 0x00; // programming i/f
> pci_config_set_class(d->config, PCI_CLASS_BRIDGE_HOST);
> d->config[0x0D] = 0x10; // latency_timer
> - d->config[PCI_HEADER_TYPE] = PCI_HEADER_TYPE_NORMAL; // header_type
> + /* header type is initialized by do_pci_register_device() */
> return 0;
> }
>
> @@ -479,7 +479,6 @@ static PCIDeviceInfo pbm_pci_host_info = {
> .qdev.name = "pbm",
> .qdev.size = sizeof(PCIDevice),
> .init = pbm_pci_host_init,
> - .header_type = PCI_HEADER_TYPE_BRIDGE,
> };
>
> static SysBusDeviceInfo pbm_host_info = {
> --
> 1.6.6.1