More bike shedding. * Yinghai Lu <yinghai@xxxxxxxxxx>: > @@ -58,6 +103,13 @@ static void pbus_assign_resources_sorted(const struct pci_bus *bus) > res = list->res; > idx = res - &list->dev->resource[0]; > if (pci_assign_resource(list->dev, idx)) { > + if (fail_head && !pci_is_root_bus(list->dev->bus)) { > + /* > + * device need to keep flags and size > + * for next try > + */ > + add_to_failed_list(fail_head, list->dev, res); > + } I don't really think you need that comment, so just strike it completely and drop the curly braces. if (fail_head && !pci_is_root_bus(list->dev->bus)) add_to_failed_list(fail_head, list->dev, res); Reviewed-by: Alex Chiang <achiang@xxxxxx> /ac -- 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