[flashrom-gerrit] Change in flashrom[master]: pcidev.c: Factor out pcidev_validate() into pure fn

[Nico Huber (Code Review)]

Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/37264 )

Change subject: pcidev.c: Factor out pcidev_validate() into pure fn
......................................................................

Patch Set 2:

(2 comments)

https://review.coreboot.org/c/flashrom/+/37264/2/pcidev.c 
File pcidev.c:

https://review.coreboot.org/c/flashrom/+/37264/2/pcidev.c@158 
PS2, Line 158: 			continue;
What happened to the vendor_id check?

https://review.coreboot.org/c/flashrom/+/37264/2/pcidev.c@254 
PS2, Line 254: 	return dev;
So we always return NULL now (only state that can exit the loop)? Was this tested? ;-P