From: Javier Carrasco Subject: [PATCH v2 0/2] PCI: kirin: cleanup (dev_err_probe() and scoped loop) Date: Sun, 07 Jul 2024 15:54:00 +0200 Message-Id: <20240707-pcie-kirin-dev_err_probe-v2-0-2fa94951d84d@gmail.com> X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit To: Xiaowei Song , Binghui Wang , Lorenzo Pieralisi , =?utf-8?q?Krzysztof_Wilczy=C5=84ski?= , Rob Herring , Jonathan Cameron , Bjorn Helgaas Cc: linux-pci@vger.kernel.org, linux-kernel@vger.kernel.org, Javier Carrasco Xref: photonic.trudheim.com org.kernel.vger.linux-kernel:1270125 org.kernel.vger.linux-pci:144973 Newsgroups: org.kernel.vger.linux-kernel,org.kernel.vger.linux-pci Path: photonic.trudheim.com!nntp.lore.kernel.org!not-for-mail This series removes some patterns that require multiple steps to achieve what single calls can achieve. Signed-off-by: Javier Carrasco --- Changes in v2: - Remove error value from the message "failed to parse devfn: %d\n", as it is already handled by dev_err_probe(). - Fix capitalization in error messages. - Return -ENODEV and provide more appropriate error message if no dev->of_node is available. - Link to v1: https://lore.kernel.org/r/20240706-pcie-kirin-dev_err_probe-v1-0-56df797fb8ee@gmail.com --- Javier Carrasco (2): PCI: kirin: use dev_err_probe() in probe error paths PCI: kirin: use for_each_available_child_of_node_scoped() drivers/pci/controller/dwc/pcie-kirin.c | 54 ++++++++++++--------------------- 1 file changed, 20 insertions(+), 34 deletions(-) --- base-commit: 412d6f897b7a494b373986e63a14a94d0fbd0fdb change-id: 20240705-pcie-kirin-dev_err_probe-0c9035188ff9 Best regards, -- Javier Carrasco .