Youness Alaoui posted comments on this change.
Patch set 1:
(2 comments)
Yes, the code was and is still checking the `pacc` pointer in
It's not really nuts, it makes sense because the pci_access gets referenced in the pci_dev, and is used, so if the pacc is cleaned up and destroyed, then calling any pci_dev API will cause an error. The mistake here is that 'pacc' is not set to NULL after cleanup in its shutdown routine.
Of course, this makes no more sense if the pacc variable is different like in the case of pch100, that check becomes wrong.
Patch Set #2, Line 200: int rpci_write_word(struct pci_dev *dev, int reg, uint16_t data);
That's not accurate... how about:
Sounds good to me. Thanks
To view, visit change 20784. To unsubscribe, visit settings.