On Thu, Jun 18, 2015 at 9:12 AM, Bjorn Helgaas <bhelgaas@xxxxxxxxxx> wrote: > The pciehp_handle_*() functions (pciehp_handle_attention_button(), etc.) > only contain a line or two of useful code, so it's clumsy to put > them in separate functions. All they so is add an event to a work queue, > and it's clearer to see that directly in the ISR. Yes. That make the code more readable. For all 4 patches: Acked-by: Yinghai Lu <yinghai@xxxxxxxxxx> -- To unsubscribe from this list: send the line "unsubscribe linux-pci" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html