Change in flashrom[master]: pcidev: Move pci_get_dev() logic into canonical place

[Nico Huber (Code Review)]

Attention is currently required from: Edward O'Callaghan.
Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/59276 )

Change subject: pcidev: Move pci_get_dev() logic into canonical place
......................................................................

Patch Set 3:

(1 comment)

File board_enable.c:

https://review.coreboot.org/c/flashrom/+/59276/comment/f98b7a1f_d0152967 
PS3, Line 1097: 		dev = pcidev_clonedev(dev);
> I didn't understand what is actionable here? […]

Yeah, sorry for the earlier suggestion, I didn't realize how odd things
are.

It should be obvious what the function does. So yes, putting the 1 in the
name would work. (I'm not sure if this case is worth a function, though.)

I'd prefer something like `pcidev_get_function1(dev)`, or
`pcidev_get_fn1(dev)` or `pcidev_get_function(dev, fn)`. We really
shoudn't make the code more confusing.