Youness Alaoui posted comments on this change.
Patch set 1:
(3 comments)
Patch Set #1, Line 262: struct pci_dev * dev;
No whitespace after the asterisk, please.
oups, I noticed it but forgot to fix it.
Add a comment that `dev` passed to rpci_* has to be compatible
Added in the header file. The code was already checking for validity of 'pacc', so it was already a requirement before my change, but I've changed things to make it more robust.
Patch Set #1, Line 307: a->domain, a->bus, a->dev, a->func); \
Bail out if `undo_pci_write_data->dev` is NULL or check it
Done
To view, visit change 20784. To unsubscribe, visit settings.