[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] Re: [PATCH 04/25] pci: use the symbolic constant, PCI_ROM_A
From: |
Michael S. Tsirkin |
Subject: |
[Qemu-devel] Re: [PATCH 04/25] pci: use the symbolic constant, PCI_ROM_ADDRESS_ENABLE instead of 1. |
Date: |
Sun, 4 Oct 2009 12:06:41 +0200 |
User-agent: |
Mutt/1.5.19 (2009-01-05) |
On Sat, Oct 03, 2009 at 05:15:56AM +0900, Isaku Yamahata wrote:
> use the symbolic constant, PCI_ROM_ADDRESS_ENABLE instead of 1.
>
> Signed-off-by: Isaku Yamahata <address@hidden>
Acked-by: Michael S. Tsirkin <address@hidden>
> ---
> hw/pci.c | 2 +-
> hw/pci.h | 2 ++
> 2 files changed, 3 insertions(+), 1 deletions(-)
>
> diff --git a/hw/pci.c b/hw/pci.c
> index 40035e6..1016893 100644
> --- a/hw/pci.c
> +++ b/hw/pci.c
> @@ -422,7 +422,7 @@ void pci_register_bar(PCIDevice *pci_dev, int region_num,
> if (region_num == PCI_ROM_SLOT) {
> addr = 0x30;
> /* ROM enable bit is writeable */
> - wmask |= 1;
> + wmask |= PCI_ROM_ADDRESS_ENABLE;
> } else {
> addr = 0x10 + region_num * 4;
> }
> diff --git a/hw/pci.h b/hw/pci.h
> index aa3090e..def45ae 100644
> --- a/hw/pci.h
> +++ b/hw/pci.h
> @@ -131,6 +131,8 @@ typedef struct PCIIORegion {
> #define PCI_SUBVENDOR_ID 0x2c /* obsolete, use
> PCI_SUBSYSTEM_VENDOR_ID */
> #define PCI_SUBDEVICE_ID 0x2e /* obsolete, use PCI_SUBSYSTEM_ID */
>
> +#define PCI_ROM_ADDRESS_ENABLE 0x01
> +
Please put this just after PCI_ROM_ADDRESS just like it appears in
pci_regs.h, to make it easier to find:
#define PCI_ROM_ADDRESS 0x30 /* Bits 31..11 are address, 10..1
reserved */
#define PCI_ROM_ADDRESS_ENABLE 0x01
> /* Bits in the PCI Status Register (PCI 2.3 spec) */
> #define PCI_STATUS_RESERVED1 0x007
> #define PCI_STATUS_INT_STATUS 0x008
> --
> 1.6.0.2
>
>
- [Qemu-devel] Re: [PATCH 12/25] pci: 64bit bar support., (continued)
[Qemu-devel] [PATCH 08/25] pci: use helper functions to access pci config space., Isaku Yamahata, 2009/10/02
[Qemu-devel] [PATCH 10/25] pci: introduce FMT_pcibus for printf format for pcibus_t., Isaku Yamahata, 2009/10/02
[Qemu-devel] [PATCH 14/25] pci: factor out the logic to get pci device from address., Isaku Yamahata, 2009/10/02
[Qemu-devel] [PATCH 25/25] pci/monitor: print out bridge's filtering values and so on., Isaku Yamahata, 2009/10/02
[Qemu-devel] [PATCH 04/25] pci: use the symbolic constant, PCI_ROM_ADDRESS_ENABLE instead of 1., Isaku Yamahata, 2009/10/02
- [Qemu-devel] Re: [PATCH 04/25] pci: use the symbolic constant, PCI_ROM_ADDRESS_ENABLE instead of 1.,
Michael S. Tsirkin <=
[Qemu-devel] [PATCH 15/25] pci_host.h: split non-inline static function in pci_host.h into pci_host.c, Isaku Yamahata, 2009/10/02
[Qemu-devel] [PATCH 20/25] pci: factor out config update logic., Isaku Yamahata, 2009/10/02
[Qemu-devel] [PATCH 18/25] pci: add helper functions for pci config write function., Isaku Yamahata, 2009/10/02
[Qemu-devel] [PATCH 09/25] pci: introduce pcibus_t to represent pci bus address/size instead of uint32_t, Isaku Yamahata, 2009/10/02
[Qemu-devel] [PATCH 11/25] pci: typedef pcibus_t as uint64_t instead of uint32_t., Isaku Yamahata, 2009/10/02
[Qemu-devel] [PATCH 07/25] pci: helper functions to access PCIDevice::config, Isaku Yamahata, 2009/10/02
[Qemu-devel] [PATCH 23/25] pci: add helper function to initialize wmask., Isaku Yamahata, 2009/10/02