[coreboot-gerrit] Change in ...coreboot[master]: device/pci_ops: Apply some symmetry in headers

[Aaron Durbin (Code Review)]

Aaron Durbin has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/31306 )

Change subject: device/pci_ops: Apply some symmetry in headers
......................................................................

Patch Set 2:
> Patch Set 2:
> > Patch Set 2: Code-Review+2
> >
> > would be great to fix some of that other arch deps. I assume there's a big ripple?
>
> I assume you refer to the FIXME comment there? I can do some sed-magic if we agree <device/pci_ops.h> is the one to add and <arch/io.h> one to (sometimes) remove from includes.

Ya. I think arch/io should really not be in anything unless it is truly arch specific (io port access). All the memory mapped forced reads using volatile should really be arch agnostic. Either way, I like the direction the patches are taking.