diff options
| author | Philippe Mathieu-Daudé <philmd@redhat.com> | 2020-12-02 14:20:38 +0100 |
|---|---|---|
| committer | Laurent Vivier <laurent@vivier.eu> | 2020-12-13 17:07:05 +0100 |
| commit | f6a3c86ebd39aaedf1ebf89629ec91b5d44b670d (patch) | |
| tree | 42943407aac4211bba2b77122f460edc30a0794d /hw/pci-host/pam.c | |
| parent | 01d152c0bfabadc6c93a39e465eb2f66f3f11527 (diff) | |
| download | qemu-f6a3c86ebd39aaedf1ebf89629ec91b5d44b670d.zip qemu-f6a3c86ebd39aaedf1ebf89629ec91b5d44b670d.tar.gz qemu-f6a3c86ebd39aaedf1ebf89629ec91b5d44b670d.tar.bz2 | |
hw/pci-host/pam: Replace magic number by PAM_REGIONS_COUNT definition
While this change helps triskaidekaphobic developers, it
is a good practice to avoid magic values and using constant
definitions instead.
Introduce the PAM_REGIONS_COUNT and use it. No logical change.
Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Reviewed-by: Julia Suvorova <jusual@redhat.com>
Message-Id: <20201202132038.1276404-1-philmd@redhat.com>
Signed-off-by: Laurent Vivier <laurent@vivier.eu>
Diffstat (limited to 'hw/pci-host/pam.c')
| -rw-r--r-- | hw/pci-host/pam.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/hw/pci-host/pam.c b/hw/pci-host/pam.c index a496205..454dd12 100644 --- a/hw/pci-host/pam.c +++ b/hw/pci-host/pam.c @@ -62,7 +62,7 @@ void init_pam(DeviceState *dev, MemoryRegion *ram_memory, void pam_update(PAMMemoryRegion *pam, int idx, uint8_t val) { - assert(0 <= idx && idx <= 12); + assert(0 <= idx && idx < PAM_REGIONS_COUNT); memory_region_set_enabled(&pam->alias[pam->current], false); pam->current = (val >> ((!(idx & 1)) * 4)) & PAM_ATTR_MASK; |