Attention is currently required from: Nico Huber, Edward O'Callaghan.

Edward O'Callaghan uploaded patch set #3 to this change.

View Change

pcidev: Construct a API to handle phantom pci controllers

On Intel the SPI PCI device can have hiden PCI vendor
and device IDs. So we need a PCI access method that works
even when the OS doesn't know the PCI device.

This however does not generalise since it would bring along other
constraints (e.g. on PCI domains, extended PCIe config space).

Fortunately the logic exists inside chipset_enable.c however
the interaction with the rest of the pcidev workers are quite
invasive and hard to reason about. Therefore here we implement
a definable API to manage this special casing by having methods
define on a handle with a known life-time.

BUG=b:220950271
TEST=`sudo ./flashrom -p internal -r /tmp/bios`
on Found chipset "Intel Kaby Lake U w/ iHDCP2.2 Prem.".

Change-Id: Ic75573a14c9997a2d0347cd683ffc0c78d1b3a4f
Signed-off-by: Edward O'Callaghan <quasisec@google.com>
---
M chipset_enable.c
M include/programmer.h
M pcidev.c
3 files changed, 82 insertions(+), 30 deletions(-)

git pull ssh://review.coreboot.org:29418/flashrom refs/changes/51/62951/3

To view, visit change 62951. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: flashrom
Gerrit-Branch: master
Gerrit-Change-Id: Ic75573a14c9997a2d0347cd683ffc0c78d1b3a4f
Gerrit-Change-Number: 62951
Gerrit-PatchSet: 3
Gerrit-Owner: Edward O'Callaghan <quasisec@chromium.org>
Gerrit-Reviewer: Nico Huber <nico.h@gmx.de>
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>
Gerrit-CC: Paul Menzel <paulepanter@mailbox.org>
Gerrit-CC: Subrata Banik <subratabanik@google.com>
Gerrit-CC: Thomas Heijligen <src@posteo.de>
Gerrit-Attention: Nico Huber <nico.h@gmx.de>
Gerrit-Attention: Edward O'Callaghan <quasisec@chromium.org>
Gerrit-MessageType: newpatchset