{"id":2219287,"url":"http://patchwork.ozlabs.org/api/patches/2219287/?format=json","web_url":"http://patchwork.ozlabs.org/project/linux-pci/patch/20260402182354.12515-1-john.madieu.xa@bp.renesas.com/","project":{"id":28,"url":"http://patchwork.ozlabs.org/api/projects/28/?format=json","name":"Linux PCI development","link_name":"linux-pci","list_id":"linux-pci.vger.kernel.org","list_email":"linux-pci@vger.kernel.org","web_url":null,"scm_url":null,"webscm_url":null,"list_archive_url":"","list_archive_url_format":"","commit_url_format":""},"msgid":"<20260402182354.12515-1-john.madieu.xa@bp.renesas.com>","list_archive_url":null,"date":"2026-04-02T18:23:53","name":"[v2] PCI: rzg3s-host: Treat link-down as -ENODEV instead of error","commit_ref":null,"pull_url":null,"state":"new","archived":false,"hash":"2126dd199760c9d0093b68c91d2566c1edd5d01d","submitter":{"id":89876,"url":"http://patchwork.ozlabs.org/api/people/89876/?format=json","name":"John Madieu","email":"john.madieu.xa@bp.renesas.com"},"delegate":null,"mbox":"http://patchwork.ozlabs.org/project/linux-pci/patch/20260402182354.12515-1-john.madieu.xa@bp.renesas.com/mbox/","series":[{"id":498539,"url":"http://patchwork.ozlabs.org/api/series/498539/?format=json","web_url":"http://patchwork.ozlabs.org/project/linux-pci/list/?series=498539","date":"2026-04-02T18:23:53","name":"[v2] PCI: rzg3s-host: Treat link-down as -ENODEV instead of error","version":2,"mbox":"http://patchwork.ozlabs.org/series/498539/mbox/"}],"comments":"http://patchwork.ozlabs.org/api/patches/2219287/comments/","check":"pending","checks":"http://patchwork.ozlabs.org/api/patches/2219287/checks/","tags":{},"related":[],"headers":{"Return-Path":"\n <linux-pci+bounces-51769-incoming=patchwork.ozlabs.org@vger.kernel.org>","X-Original-To":["incoming@patchwork.ozlabs.org","linux-pci@vger.kernel.org"],"Delivered-To":"patchwork-incoming@legolas.ozlabs.org","Authentication-Results":["legolas.ozlabs.org;\n spf=pass (sender SPF authorized) smtp.mailfrom=vger.kernel.org\n (client-ip=172.234.253.10; helo=sea.lore.kernel.org;\n envelope-from=linux-pci+bounces-51769-incoming=patchwork.ozlabs.org@vger.kernel.org;\n receiver=patchwork.ozlabs.org)","smtp.subspace.kernel.org;\n arc=none smtp.client-ip=210.160.252.172","smtp.subspace.kernel.org;\n dmarc=pass (p=none dis=none) header.from=bp.renesas.com","smtp.subspace.kernel.org;\n spf=pass smtp.mailfrom=bp.renesas.com"],"Received":["from sea.lore.kernel.org (sea.lore.kernel.org [172.234.253.10])\n\t(using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)\n\t key-exchange x25519 server-signature ECDSA (secp384r1) server-digest SHA384)\n\t(No client certificate requested)\n\tby legolas.ozlabs.org (Postfix) with ESMTPS id 4fmrJb3dS8z1yGH\n\tfor <incoming@patchwork.ozlabs.org>; Fri, 03 Apr 2026 05:40:31 +1100 (AEDT)","from smtp.subspace.kernel.org (conduit.subspace.kernel.org\n [100.90.174.1])\n\tby sea.lore.kernel.org (Postfix) with ESMTP id 49A6930A84B7\n\tfor <incoming@patchwork.ozlabs.org>; Thu,  2 Apr 2026 18:33:08 +0000 (UTC)","from localhost.localdomain (localhost.localdomain [127.0.0.1])\n\tby smtp.subspace.kernel.org (Postfix) with ESMTP id 51AA436C0DC;\n\tThu,  2 Apr 2026 18:33:07 +0000 (UTC)","from relmlie6.idc.renesas.com (relmlor2.renesas.com\n [210.160.252.172])\n\tby smtp.subspace.kernel.org (Postfix) with ESMTP id C230A248896;\n\tThu,  2 Apr 2026 18:33:05 +0000 (UTC)","from unknown (HELO relmlir5.idc.renesas.com) ([10.200.68.151])\n  by relmlie6.idc.renesas.com with ESMTP; 03 Apr 2026 03:33:04 +0900","from ubuntu.adwin.renesas.com (unknown [10.226.92.32])\n\tby relmlir5.idc.renesas.com (Postfix) with ESMTP id 19AB7400068F;\n\tFri,  3 Apr 2026 03:33:00 +0900 (JST)"],"ARC-Seal":"i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116;\n\tt=1775154787; cv=none;\n b=HQA73tZVeTCEwMhc5ND0OFIi9TVU52c+0dE906IDIV/xv+sxsCbOZ4fm/5vQDlMX/ITFZaUUW0kBP9E5OGykDkd/vWm6GZMirWrS/pDGoqJMbF0sT0vtMV8M/mMTTym0DI4s6iWf6u+e7s5NZO/rdtt/q6Q8Iu+4Wo3TecS87RQ=","ARC-Message-Signature":"i=1; a=rsa-sha256; d=subspace.kernel.org;\n\ts=arc-20240116; t=1775154787; c=relaxed/simple;\n\tbh=+h1S6IY/LtS6nxa6y8su80IJrFMDgvebuxUTacdGUDM=;\n\th=From:To:Cc:Subject:Date:Message-ID:MIME-Version;\n b=YuihQxFf4g1MXEwPqSyD8xGYzwW2usTzYTgLGrFMixAHwvOvvwvLdTltHcVMyg4mTk8jEV2OckfIewdixh1OT1hA3cUSrHzEkNDzH2zIsNQ2hr1ihgQK05GroCc+YU9DhlQaUjhBRhPetyva8QVcNApk7U2/5P3Vduh/8qtI3+Y=","ARC-Authentication-Results":"i=1; smtp.subspace.kernel.org;\n dmarc=pass (p=none dis=none) header.from=bp.renesas.com;\n spf=pass smtp.mailfrom=bp.renesas.com;\n arc=none smtp.client-ip=210.160.252.172","X-CSE-ConnectionGUID":"jFfU/CcpTiu7OpgY2IVAaQ==","X-CSE-MsgGUID":"yDG8MbmLSH+gGNnoJiOQLA==","From":"John Madieu <john.madieu.xa@bp.renesas.com>","To":"claudiu.beznea.uj@bp.renesas.com,\n\tlpieralisi@kernel.org,\n\tkwilczynski@kernel.org,\n\tmani@kernel.org","Cc":"robh@kernel.org,\n\tjohn.madieu@gmail.com,\n\tbhelgaas@google.com,\n\tlinux-pci@vger.kernel.org,\n\tlinux-renesas-soc@vger.kernel.org,\n\tlinux-kernel@vger.kernel.org,\n\tJohn Madieu <john.madieu.xa@bp.renesas.com>","Subject":"[PATCH v2] PCI: rzg3s-host: Treat link-down as -ENODEV instead of\n error","Date":"Thu,  2 Apr 2026 20:23:53 +0200","Message-ID":"<20260402182354.12515-1-john.madieu.xa@bp.renesas.com>","X-Mailer":"git-send-email 2.43.0","Precedence":"bulk","X-Mailing-List":"linux-pci@vger.kernel.org","List-Id":"<linux-pci.vger.kernel.org>","List-Subscribe":"<mailto:linux-pci+subscribe@vger.kernel.org>","List-Unsubscribe":"<mailto:linux-pci+unsubscribe@vger.kernel.org>","MIME-Version":"1.0","Content-Transfer-Encoding":"8bit"},"content":"rzg3s_pcie_host_init() failing to establish a PCIe link does not\nnecessarily indicate a hardware or driver error; it may simply mean no\ncard is inserted. Demote the message from dev_err_probe() to dev_info()\nand return -ENODEV so the driver defers gracefully rather than printing\na spurious error.\n\nSigned-off-by: John Madieu <john.madieu.xa@bp.renesas.com>\n---\n\nChanges in v2:\n - Dropped Rb tag from Geert as patch structure changed\n - Moved link-down handling into rzg3s_pcie_host_init() instead of\n   masking all errors in rzg3s_pcie_host_setup() (Claudiu)\n - Kept dev_err_probe() for real init failures, only suppress for\n   -ENODEV (link down)\n\nv1:\n  - https://lore.kernel.org/all/20260401143347.8463-1-john.madieu.xa@bp.renesas.com/\n\n drivers/pci/controller/pcie-rzg3s-host.c | 9 +++++++--\n 1 file changed, 7 insertions(+), 2 deletions(-)","diff":"diff --git a/drivers/pci/controller/pcie-rzg3s-host.c b/drivers/pci/controller/pcie-rzg3s-host.c\nindex d86e7516dcc2..0acc21981ee9 100644\n--- a/drivers/pci/controller/pcie-rzg3s-host.c\n+++ b/drivers/pci/controller/pcie-rzg3s-host.c\n@@ -1384,8 +1384,11 @@ static int rzg3s_pcie_host_init(struct rzg3s_pcie_host *host)\n \t\t\t\t PCIE_LINK_WAIT_SLEEP_MS * MILLI,\n \t\t\t\t PCIE_LINK_WAIT_SLEEP_MS * MILLI *\n \t\t\t\t PCIE_LINK_WAIT_MAX_RETRIES);\n-\tif (ret)\n+\tif (ret) {\n+\t\tdev_info(host->dev, \"PCIe link down!\\n\");\n+\t\tret = -ENODEV;\n \t\tgoto config_deinit_post;\n+\t}\n \n \tval = readl_relaxed(host->axi + RZG3S_PCI_PCSTAT2);\n \tdev_info(host->dev, \"PCIe link status [0x%x]\\n\", val);\n@@ -1655,7 +1658,9 @@ rzg3s_pcie_host_setup(struct rzg3s_pcie_host *host,\n \n \tret = rzg3s_pcie_host_init(host);\n \tif (ret) {\n-\t\tdev_err_probe(dev, ret, \"Failed to initialize the HW!\\n\");\n+\t\tif (ret != -ENODEV)\n+\t\t\tdev_err_probe(dev, ret, \"Failed to initialize the HW!\\n\");\n+\n \t\tgoto teardown_irqdomain;\n \t}\n \n","prefixes":["v2"]}