… > +++ b/drivers/pci/pci-acpi.c > @@ -1677,15 +1677,12 @@ struct pci_bus *pci_acpi_scan_root(struct acpi_pci_root *root) > > root_ops = kzalloc(sizeof(*root_ops), GFP_KERNEL); > if (!root_ops) { > - kfree(ri); > - return NULL; > + goto free_ri; > } Would you like to omit curly brackets around a single statement? https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/process/coding-style.rst?h=v6.15-rc4#n197 Regards, Markus