On Tue, 2017-04-25 at 14:46 -0500, Don Brace wrote: > static int pqi_request_irqs(struct pqi_ctrl_info *ctrl_info) > { > - struct pci_dev *pdev = ctrl_info->pci_dev; > int i; > int rc; > + struct pci_dev *pdev = ctrl_info->pci_dev; Was this change necessary? If not, please leave it out. Thanks, Bart.