diff options
| author | Eric Engestrom <[email protected]> | 2019-10-02 13:42:14 +0100 |
|---|---|---|
| committer | Eric Engestrom <[email protected]> | 2019-10-07 21:30:16 +0100 |
| commit | b9157ea415d9734e67678d2d5a26798d6fa03a02 (patch) | |
| tree | 26a050c870d52b94e5c0cd8d4fefa055179313e1 /src | |
| parent | 5be6c8959c1c8354423ca4fee2c3da4532100a5c (diff) | |
loader: use ARRAY_SIZE instead of NULL sentinel
Signed-off-by: Eric Engestrom <[email protected]>
Acked-by: Lionel Landwerlin <[email protected]>
Diffstat (limited to 'src')
| -rw-r--r-- | src/loader/loader.c | 4 | ||||
| -rw-r--r-- | src/loader/pci_id_driver_map.h | 5 |
2 files changed, 3 insertions, 6 deletions
diff --git a/src/loader/loader.c b/src/loader/loader.c index 873f55c8a9c..c828105c951 100644 --- a/src/loader/loader.c +++ b/src/loader/loader.c @@ -56,6 +56,8 @@ #endif #endif +#include "util/macros.h" + #define __IS_LOADER #include "pci_id_driver_map.h" @@ -465,7 +467,7 @@ loader_get_driver_for_fd(int fd) return driver; } - for (i = 0; driver_map[i].driver; i++) { + for (i = 0; i < ARRAY_SIZE(driver_map); i++) { if (vendor_id != driver_map[i].vendor_id) continue; diff --git a/src/loader/pci_id_driver_map.h b/src/loader/pci_id_driver_map.h index 63fb1ae01b6..f71548a1c4e 100644 --- a/src/loader/pci_id_driver_map.h +++ b/src/loader/pci_id_driver_map.h @@ -4,10 +4,6 @@ #include <stdbool.h> #include <stddef.h> -#ifndef ARRAY_SIZE -#define ARRAY_SIZE(a) (sizeof(a) / sizeof((a)[0])) -#endif - #ifndef __IS_LOADER # error "Only include from loader.c" #endif @@ -106,7 +102,6 @@ static const struct { { 0x10de, "nouveau", NULL, -1, }, { 0x1af4, "virtio_gpu", virtio_gpu_chip_ids, ARRAY_SIZE(virtio_gpu_chip_ids) }, { 0x15ad, "vmwgfx", vmwgfx_chip_ids, ARRAY_SIZE(vmwgfx_chip_ids) }, - { 0x0000, NULL, NULL, 0 }, }; #endif /* _PCI_ID_DRIVER_MAP_H_ */ |