Re: [Qemu-devel] [PATCH 1/2] versatile_pci: convert to symbolic names

[Anthony Liguori <anthony@codemonkey.ws>]

On 02/08/2010 03:41 PM, Michael S. Tsirkin wrote:
> This converts versatile_pci to use symbolic
> constants. Verified by comparing binary to
> original one.
>
> Signed-off-by: Michael S. Tsirkin<mst@redhat.com>
>    

Acked-by: Anthony Liguori <aliguori@us.ibm.com>

Regards,

Anthony Liguori

> ---
>   hw/versatile_pci.c |   17 ++++++++++-------
>   1 files changed, 10 insertions(+), 7 deletions(-)
>
> diff --git a/hw/versatile_pci.c b/hw/versatile_pci.c
> index 153c651..e58b7f4 100644
> --- a/hw/versatile_pci.c
> +++ b/hw/versatile_pci.c
> @@ -147,14 +147,17 @@ static int versatile_pci_host_init(PCIDevice *d)
>       pci_config_set_vendor_id(d->config, PCI_VENDOR_ID_XILINX);
>       /* Both boards have the same device ID.  Oh well.  */
>       pci_config_set_device_id(d->config, PCI_DEVICE_ID_XILINX_XC2VP30);
> -    d->config[0x04] = 0x00;
> -    d->config[0x05] = 0x00;
> -    d->config[0x06] = 0x20;
> -    d->config[0x07] = 0x02;
> -    d->config[0x08] = 0x00; // revision
> -    d->config[0x09] = 0x00; // programming i/f
> +    /* TODO: no need to clear command */
> +    pci_set_byte(d->config + PCI_COMMAND, 0x00);
> +    pci_set_byte(d->config + PCI_COMMAND + 1, 0x00);
> +    /* TODO: convert to set_word */
> +    pci_set_byte(d->config + PCI_STATUS, PCI_STATUS_66MHZ);
> +    pci_set_byte(d->config + PCI_STATUS + 1, PCI_STATUS_DEVSEL_MEDIUM>>  8);
> +    /* TODO: no need to clear revision/prog ifc */
> +    pci_set_byte(d->config + PCI_REVISION_ID, 0x00);
> +    pci_set_byte(d->config + PCI_CLASS_PROG, 0x00);
>       pci_config_set_class(d->config, PCI_CLASS_PROCESSOR_CO);
> -    d->config[0x0D] = 0x10; // latency_timer
> +    pci_set_byte(d->config + PCI_LATENCY_TIMER, 0x10);
>       return 0;
>   }
>
>