Liam Flaherty has uploaded this change for review.

View Change

debug_raiden_spi: Remove fixme with explanation

The raiden_debug_spi programmer will query the connected USB devices and
match against criteria other than the pid rather than iterate through the
vid:pid table.
The fixme has been updated to explain why the dev_entry table is empty.

TICKET=https://ticket.coreboot.org/issues/394
BUG=b:253320285
TEST=build

Change-Id: I43e364c02f42dd499d3c9ca3e0a03ead673da3e6
Signed-off-by: Liam Flaherty <liamflaherty@chromium.org>
---
M raiden_debug_spi.c
1 file changed, 21 insertions(+), 1 deletion(-)

git pull ssh://review.coreboot.org:29418/flashrom refs/changes/64/68364/1
diff --git a/raiden_debug_spi.c b/raiden_debug_spi.c
index a2e5e84..1cb2874 100644
--- a/raiden_debug_spi.c
+++ b/raiden_debug_spi.c
@@ -350,7 +350,8 @@
#include <string.h>
#include <unistd.h>

-/* FIXME: Add some programmer IDs here */
+// Table is empty as raiden_debug_spi matches against connected USB devices,
+// rather than looking for a device with a specific vid:pid.
static const struct dev_entry devs_raiden[] = {
{0},
};

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

Gerrit-Project: flashrom
Gerrit-Branch: master
Gerrit-Change-Id: I43e364c02f42dd499d3c9ca3e0a03ead673da3e6
Gerrit-Change-Number: 68364
Gerrit-PatchSet: 1
Gerrit-Owner: Liam Flaherty <liamflaherty@chromium.org>
Gerrit-MessageType: newchange