build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/31755 )
Change subject: device/pci_ops: Define pci_find_capability() just once ......................................................................
Patch Set 3:
(5 comments)
https://review.coreboot.org/#/c/31755/3/src/device/pci_ops.c File src/device/pci_ops.c:
https://review.coreboot.org/#/c/31755/3/src/device/pci_ops.c@33 PS3, Line 33: unsigned int pci_find_next_capability(pci_devfn_t dev, unsigned cap, unsigned last) Prefer 'unsigned int' to bare use of 'unsigned'
https://review.coreboot.org/#/c/31755/3/src/device/pci_ops.c@33 PS3, Line 33: unsigned int pci_find_next_capability(pci_devfn_t dev, unsigned cap, unsigned last) Prefer 'unsigned int' to bare use of 'unsigned'
https://review.coreboot.org/#/c/31755/3/src/device/pci_ops.c@35 PS3, Line 35: unsigned pos = 0; Prefer 'unsigned int' to bare use of 'unsigned'
https://review.coreboot.org/#/c/31755/3/src/device/pci_ops.c@37 PS3, Line 37: unsigned reps = 48; Prefer 'unsigned int' to bare use of 'unsigned'
https://review.coreboot.org/#/c/31755/3/src/device/pci_ops.c@84 PS3, Line 84: unsigned int pci_find_capability(pci_devfn_t dev, unsigned cap) Prefer 'unsigned int' to bare use of 'unsigned'