> +static int report_forzen_detected(struct pci_dev *dev, void *data) s/forzen/frozen/ ? > static pci_ers_result_t broadcast_error_message(struct pci_dev *dev, > + pci_ers_result_t result, char *error_mesg, > + int (*cb)(struct pci_dev *, void *)) > { > pci_printk(KERN_DEBUG, dev, "broadcast %s message\n", error_mesg); > + pci_walk_bus(dev->subordinate, cb, &result); > + return result; > } Is there any point in keeping this helper?