[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <bf959b95-3e59-4863-bd92-84adee05f9e2@web.de>
Date: Tue, 22 Jul 2025 20:33:59 +0200
From: Markus Elfring <Markus.Elfring@....de>
To: Johan Hovold <johan+linaro@...nel.org>, linux-pci@...r.kernel.org,
Bjorn Helgaas <bhelgaas@...gle.com>
Cc: stable@...r.kernel.org, LKML <linux-kernel@...r.kernel.org>,
Manivannan Sadhasivam <mani@...nel.org>,
Jonathan Cameron <Jonathan.Cameron@...wei.com>
Subject: Re: [PATCH 1/3] PCI/pwrctrl: Fix device leak at registration
…
> +++ b/drivers/pci/bus.c
> @@ -362,11 +362,15 @@ void pci_bus_add_device(struct pci_dev *dev)
…
> + if (of_pci_supply_present(dn)) {
> + if (!device_link_add(&dev->dev, &pdev->dev,
> + DL_FLAG_AUTOREMOVE_CONSUMER)) {
> + pci_err(dev, "failed to add device link to power control device %s\n",
> + pdev->name);
> + }
> + }
…
How do you think about to reconsider the usage of any curly brackets
once more?
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/process/coding-style.rst?h=v6.16-rc7#n197
Regards,
Markus
Powered by blists - more mailing lists