From nobody Sun Feb 8 06:56:35 2026 Received: from mail-pg1-f227.google.com (mail-pg1-f227.google.com [209.85.215.227]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 817A32EC0B0 for ; Sun, 18 Jan 2026 12:40:12 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.215.227 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1768740014; cv=none; b=gmjgLP+v062WdwdDtJx7euGtGiB3QxEJELaTMD2XQLk4UsT55SbXlsKRmQBxjwHc8J3sMy8670DrYok2kYw13qJmE1uP9KRM0YNJTS6KxdINSP2feE1PbiNzjLgaKBJABYQVofBxQWROvetsprYAuD//ZOoeIuBKXezXLTenu7I= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1768740014; c=relaxed/simple; bh=qRzJzBx/RwBa0LNHNV5uj3EwIEAoqjsZQbqjP2dlWRI=; h=From:To:Cc:Subject:Date:Message-Id:In-Reply-To:References: MIME-Version; b=tlDe27R8nC11I7VmfNeeS359hj2TZ0nj1sjl7lN18iRwvwOeloXSKJ4WohyGT98CkZxAjEA5xyxCYuv922YHAZpmqYH1oHJebMP3XY7yVIYD9vVF97UKTXensSS92vfTbmLBzuf/obw+//HWm+/VGFV2CcpoUNMrLk38ofCHHAE= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=broadcom.com; spf=fail smtp.mailfrom=broadcom.com; dkim=pass (1024-bit key) header.d=broadcom.com header.i=@broadcom.com header.b=M+MSbCl8; arc=none smtp.client-ip=209.85.215.227 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=broadcom.com Authentication-Results: smtp.subspace.kernel.org; spf=fail smtp.mailfrom=broadcom.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=broadcom.com header.i=@broadcom.com header.b="M+MSbCl8" Received: by mail-pg1-f227.google.com with SMTP id 41be03b00d2f7-c227206e6dcso2277580a12.2 for ; Sun, 18 Jan 2026 04:40:12 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1768740012; x=1769344812; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:dkim-signature:x-gm-gg :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=ipxYQ6lajVVVhDpOFxNg/pK0dQuceg7/qSITiJBtMVI=; b=OzTsy+COBAawYtPGX2rMwn8efPKbaFQ3zSYtpEN7IXVtFeMqfUu1n8iUSbKNWG00IK hb1ATg+gfvOrDz4FErjHUehkq1rKBFYilHXD6dSIdZWJMD+hCPEhuo5c44X7mIZqgcj6 cI7qshflGq/s+k25xRasM/JyH/9hg0GdvJr4ulg4JVO+1U7KVjusFwNhbjm7wHSHm+yD jagU5buzcyd505v4mblF5U65bk+5nTholvfOIYdqXivmH2p5HncvanfFkTSXbnaxY1au 5jGVSWlP5+N5pGH/6dRvtBS+0spHHSmMp5pq+Tv7+M4Gw1ixkoZi7cvklBVEl17j6DbB 4Mrw== X-Gm-Message-State: AOJu0YyN/xnA4FsFB34hTjIigKHEU6Yy6I3bRPIg9pgZ6TUS9N1zG8wQ /ofBPQKaR8E+cpQxrfwgYK18pkWfE1tSzc8bTdj6kbXqj/nqbmto2duT2bn4GeAZdS9A0fBTtjS gWMwPRA6XEMF/9U4HvirrFjXij27sJkPKZoUh02EwSZkRWPuXEXHkpvCSn4pESEAPX/q158JAD5 pGCKmt8HS/PSzV24rNGBkscOH+kA4Ej2DI103hLl1Iiu1Dmgvzfr/aFerLYFz6GlFVIemphQZdr sbdPJMjOk9gqjNmruc= X-Gm-Gg: AY/fxX70PF8r7XEdtKG4+9ay3hAd1RbokW/Y7W8zl+OPajH2tb0kqB2FDLuL58YQUsP IIV/5BDebpriOjQYH4eEMgiQ1vq3WZWcNN2T3WRpE13OHIUb4eUNOI7rU8WQ82tvMqFAp9JOSqj YMdcE4fsomiCL03yioWxCEUYfrO8ne7izS+M7UMDqbjYSxKGUA27/4eYDgfbzGUVbYGwrH4ME2W KwRHBmTyrBlTrxjX+ICblB/GNYt9/+STuWZ0DiXDjuWsujCYWiSjkRktTF2oEd96EDQH35hcA58 yus4x87XLss19nP18rxEtsbUiwdndApnyx+n9qEeZWpNYRLtRBnySNXXhWubQJg9GdzYEOaD9mj fp4l1o0MLZqYslvsZCbIoQ4uCuUtFCr9j7o4qf1/ZZ2MZA/lxriwGSdvwUrvsef2L6I/Nl9MrBg MUvSxjHexlGu3Qpj7cBUWCsZ9rV2kZyJMTEhMZ8cxLXofnCqA= X-Received: by 2002:a17:902:a58a:b0:2a1:10f7:9718 with SMTP id d9443c01a7336-2a7175cbf01mr58983965ad.30.1768740011839; Sun, 18 Jan 2026 04:40:11 -0800 (PST) Received: from smtp-us-east1-p01-i01-si01.dlp.protect.broadcom.com (address-144-49-247-118.dlp.protect.broadcom.com. [144.49.247.118]) by smtp-relay.gmail.com with ESMTPS id d9443c01a7336-2a7193ac1d0sm10299515ad.56.2026.01.18.04.40.11 for (version=TLS1_2 cipher=ECDHE-ECDSA-AES128-GCM-SHA256 bits=128/128); Sun, 18 Jan 2026 04:40:11 -0800 (PST) X-Relaying-Domain: broadcom.com X-CFilter-Loop: Reflected Received: by mail-pg1-f198.google.com with SMTP id 41be03b00d2f7-c5659f40515so6167051a12.2 for ; Sun, 18 Jan 2026 04:40:10 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=broadcom.com; s=google; t=1768740010; x=1769344810; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=ipxYQ6lajVVVhDpOFxNg/pK0dQuceg7/qSITiJBtMVI=; b=M+MSbCl8wW5ed1HCHIPHoQrz69DhtubVE7+rNwnz8xKDt0fmljf71YxNCXAQSY+vLu em7of2cLD3Wq+gbDjfpaNoKkVXrE+yqDJMJRpq6Nwlh4QyF/4OszayBdmaRF1uzp3nNf ucdgkvDliO3FpNhH5tDGiG6IvULr9aJd2GKOc= X-Received: by 2002:a05:6a20:938a:b0:38d:eeb9:8f6f with SMTP id adf61e73a8af0-38dfe80be52mr7079522637.72.1768740009910; Sun, 18 Jan 2026 04:40:09 -0800 (PST) X-Received: by 2002:a05:6a20:938a:b0:38d:eeb9:8f6f with SMTP id adf61e73a8af0-38dfe80be52mr7079512637.72.1768740009545; Sun, 18 Jan 2026 04:40:09 -0800 (PST) Received: from PC-MID-R740.dhcp.broadcom.net ([192.19.234.250]) by smtp.gmail.com with ESMTPSA id 41be03b00d2f7-c5edf249c24sm6761246a12.11.2026.01.18.04.40.06 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sun, 18 Jan 2026 04:40:09 -0800 (PST) From: Pavan Chebbi To: jgg@ziepe.ca, michael.chan@broadcom.com Cc: linux-kernel@vger.kernel.org, dave.jiang@intel.com, saeedm@nvidia.com, Jonathan.Cameron@huawei.com, gospo@broadcom.com, selvin.xavier@broadcom.com, leon@kernel.org, kalesh-anakkur.purayil@broadcom.com, Pavan Chebbi , linux-rdma@vger.kernel.org Subject: [PATCH fwctl 1/5] fwctl/bnxt_en: Move common definitions to include/linux/bnxt/ Date: Sun, 18 Jan 2026 04:33:57 -0800 Message-Id: <20260118123401.3188438-2-pavan.chebbi@broadcom.com> X-Mailer: git-send-email 2.39.1 In-Reply-To: <20260118123401.3188438-1-pavan.chebbi@broadcom.com> References: <20260118123401.3188438-1-pavan.chebbi@broadcom.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable X-DetectorID-Processed: b00c1d49-9d2e-4205-b15f-d015386d3d5e Content-Type: text/plain; charset="utf-8" We have common definitions that are now going to be used by more than one component outside of bnxt (bnxt_re and fwctl) Move bnxt_ulp.h to include/linux/bnxt/ as ulp.h. Reviewed-by: Andy Gospodarek Cc: linux-rdma@vger.kernel.org Signed-off-by: Pavan Chebbi Reviewed-by: Leon Romanovsky --- drivers/infiniband/hw/bnxt_re/debugfs.c | 2 +- drivers/infiniband/hw/bnxt_re/main.c | 2 +- drivers/infiniband/hw/bnxt_re/qplib_fp.c | 2 +- drivers/infiniband/hw/bnxt_re/qplib_res.h | 2 +- drivers/net/ethernet/broadcom/bnxt/bnxt.c | 2 +- drivers/net/ethernet/broadcom/bnxt/bnxt_devlink.c | 2 +- drivers/net/ethernet/broadcom/bnxt/bnxt_ethtool.c | 2 +- drivers/net/ethernet/broadcom/bnxt/bnxt_sriov.c | 2 +- drivers/net/ethernet/broadcom/bnxt/bnxt_ulp.c | 2 +- .../broadcom/bnxt/bnxt_ulp.h =3D> include/linux/bnxt/ulp.h | 0 10 files changed, 9 insertions(+), 9 deletions(-) rename drivers/net/ethernet/broadcom/bnxt/bnxt_ulp.h =3D> include/linux/bn= xt/ulp.h (100%) diff --git a/drivers/infiniband/hw/bnxt_re/debugfs.c b/drivers/infiniband/h= w/bnxt_re/debugfs.c index 88817c86ae24..c57bbe3492a8 100644 --- a/drivers/infiniband/hw/bnxt_re/debugfs.c +++ b/drivers/infiniband/hw/bnxt_re/debugfs.c @@ -10,8 +10,8 @@ #include #include #include +#include =20 -#include "bnxt_ulp.h" #include "roce_hsi.h" #include "qplib_res.h" #include "qplib_sp.h" diff --git a/drivers/infiniband/hw/bnxt_re/main.c b/drivers/infiniband/hw/b= nxt_re/main.c index 73003ad25ee8..79ca734a1377 100644 --- a/drivers/infiniband/hw/bnxt_re/main.c +++ b/drivers/infiniband/hw/bnxt_re/main.c @@ -55,8 +55,8 @@ #include #include #include +#include =20 -#include "bnxt_ulp.h" #include "roce_hsi.h" #include "qplib_res.h" #include "qplib_sp.h" diff --git a/drivers/infiniband/hw/bnxt_re/qplib_fp.c b/drivers/infiniband/= hw/bnxt_re/qplib_fp.c index c88f049136fc..5a28946e4668 100644 --- a/drivers/infiniband/hw/bnxt_re/qplib_fp.c +++ b/drivers/infiniband/hw/bnxt_re/qplib_fp.c @@ -46,6 +46,7 @@ #include #include #include +#include #include =20 #include "roce_hsi.h" @@ -55,7 +56,6 @@ #include "qplib_sp.h" #include "qplib_fp.h" #include -#include "bnxt_ulp.h" #include "bnxt_re.h" #include "ib_verbs.h" =20 diff --git a/drivers/infiniband/hw/bnxt_re/qplib_res.h b/drivers/infiniband= /hw/bnxt_re/qplib_res.h index 2ea3b7f232a3..ebe8893937f6 100644 --- a/drivers/infiniband/hw/bnxt_re/qplib_res.h +++ b/drivers/infiniband/hw/bnxt_re/qplib_res.h @@ -39,7 +39,7 @@ #ifndef __BNXT_QPLIB_RES_H__ #define __BNXT_QPLIB_RES_H__ =20 -#include "bnxt_ulp.h" +#include =20 extern const struct bnxt_qplib_gid bnxt_qplib_gid_zero; =20 diff --git a/drivers/net/ethernet/broadcom/bnxt/bnxt.c b/drivers/net/ethern= et/broadcom/bnxt/bnxt.c index d17d0ea89c36..4481d80cdfc2 100644 --- a/drivers/net/ethernet/broadcom/bnxt/bnxt.c +++ b/drivers/net/ethernet/broadcom/bnxt/bnxt.c @@ -59,10 +59,10 @@ #include #include #include +#include =20 #include "bnxt.h" #include "bnxt_hwrm.h" -#include "bnxt_ulp.h" #include "bnxt_sriov.h" #include "bnxt_ethtool.h" #include "bnxt_dcb.h" diff --git a/drivers/net/ethernet/broadcom/bnxt/bnxt_devlink.c b/drivers/ne= t/ethernet/broadcom/bnxt/bnxt_devlink.c index 15de802bbac4..230cd95d30a2 100644 --- a/drivers/net/ethernet/broadcom/bnxt/bnxt_devlink.c +++ b/drivers/net/ethernet/broadcom/bnxt/bnxt_devlink.c @@ -13,12 +13,12 @@ #include #include #include +#include #include "bnxt.h" #include "bnxt_hwrm.h" #include "bnxt_vfr.h" #include "bnxt_devlink.h" #include "bnxt_ethtool.h" -#include "bnxt_ulp.h" #include "bnxt_ptp.h" #include "bnxt_coredump.h" #include "bnxt_nvm_defs.h" diff --git a/drivers/net/ethernet/broadcom/bnxt/bnxt_ethtool.c b/drivers/ne= t/ethernet/broadcom/bnxt/bnxt_ethtool.c index 068e191ede19..8cad7b982664 100644 --- a/drivers/net/ethernet/broadcom/bnxt/bnxt_ethtool.c +++ b/drivers/net/ethernet/broadcom/bnxt/bnxt_ethtool.c @@ -27,9 +27,9 @@ #include #include #include +#include #include "bnxt.h" #include "bnxt_hwrm.h" -#include "bnxt_ulp.h" #include "bnxt_xdp.h" #include "bnxt_ptp.h" #include "bnxt_ethtool.h" diff --git a/drivers/net/ethernet/broadcom/bnxt/bnxt_sriov.c b/drivers/net/= ethernet/broadcom/bnxt/bnxt_sriov.c index be7deb9cc410..83884af32249 100644 --- a/drivers/net/ethernet/broadcom/bnxt/bnxt_sriov.c +++ b/drivers/net/ethernet/broadcom/bnxt/bnxt_sriov.c @@ -17,9 +17,9 @@ #include #include #include +#include #include "bnxt.h" #include "bnxt_hwrm.h" -#include "bnxt_ulp.h" #include "bnxt_sriov.h" #include "bnxt_vfr.h" #include "bnxt_ethtool.h" diff --git a/drivers/net/ethernet/broadcom/bnxt/bnxt_ulp.c b/drivers/net/et= hernet/broadcom/bnxt/bnxt_ulp.c index 927971c362f1..fa513892db50 100644 --- a/drivers/net/ethernet/broadcom/bnxt/bnxt_ulp.c +++ b/drivers/net/ethernet/broadcom/bnxt/bnxt_ulp.c @@ -22,10 +22,10 @@ #include #include #include +#include =20 #include "bnxt.h" #include "bnxt_hwrm.h" -#include "bnxt_ulp.h" =20 static DEFINE_IDA(bnxt_aux_dev_ids); =20 diff --git a/drivers/net/ethernet/broadcom/bnxt/bnxt_ulp.h b/include/linux/= bnxt/ulp.h similarity index 100% rename from drivers/net/ethernet/broadcom/bnxt/bnxt_ulp.h rename to include/linux/bnxt/ulp.h --=20 2.39.1 From nobody Sun Feb 8 06:56:35 2026 Received: from mail-yw1-f225.google.com (mail-yw1-f225.google.com [209.85.128.225]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 8D2B42DCF46 for ; Sun, 18 Jan 2026 12:40:36 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.225 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1768740038; cv=none; b=iz6DZmnq9rtLPStwjUhatjqwjLTzcnVhSn2AIQsUsBlsdlCVAmEUd1Evj54lTL501kYBjCQuKrvC4WoAfnsAFjtmS+Dq1mWHAHd9qML9S+aRlMYpmrLEN5pVsOG7USRSGAWVd+SfZMmwheFaTLkt2FgJjHLkXGNvXM32rzvQPX8= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1768740038; c=relaxed/simple; bh=612l8xA/idLZ3xJNP94dG4vqM+IDV9kpBqh1tKHUt3M=; h=From:To:Cc:Subject:Date:Message-Id:In-Reply-To:References: MIME-Version; b=cZfcmYVv0G26PcrrRWj64c6E1Sgt/PngaRSBALTX5hqiHJlwommnfRF89g/XmHqjecjQGiZ2bis2IbhEtX4EPxs8yUJzIMl+7i2HP54kEZ1ycpfeLL//Hawr+jBm57HW9RZDtiZHOT7pqv6z88t/3BTPMMJO798rjcQYGNeoU0I= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=broadcom.com; spf=fail smtp.mailfrom=broadcom.com; dkim=pass (1024-bit key) header.d=broadcom.com header.i=@broadcom.com header.b=Ev7EiVO2; arc=none smtp.client-ip=209.85.128.225 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=broadcom.com Authentication-Results: smtp.subspace.kernel.org; spf=fail smtp.mailfrom=broadcom.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=broadcom.com header.i=@broadcom.com header.b="Ev7EiVO2" Received: by mail-yw1-f225.google.com with SMTP id 00721157ae682-79273a294edso33043927b3.3 for ; Sun, 18 Jan 2026 04:40:36 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1768740035; x=1769344835; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:dkim-signature:x-gm-gg :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=Y/yd2fYK3qdydV8E/E4qJFTT2fTy+J7zleU+MaEFrGQ=; b=dG3BbfpLbSjsEacgzr1byaMB7sfPbjhYFGwDFZHGZJN/HmJB95qpRX7pzps0wF17+A CKb3r7oC1z0JvtrOoFrzTDjcMIotSJgeULnNoMgkLFvUHGa4YmwLB+9c+hC6UhfqLGTd 7lDXXnKLNU3njNSCiSy92FHF6iH8Nc7e4yOR/5f7OP9z/S16cKW8zz9iIDKXY6y/EYG8 tjcB0F870CkWxwuu+lCkdTB9H/Q4VvNGu9u0mE0YNPhj4PNQ+A2eQ5BPgpAXiv7WN14l xj8ikZywHQW74l7W9d39gKIl93ptRe4nTY9F2qCATLnnOJhpJqgMMrZKOQ+HA6LwnueB RkyQ== X-Gm-Message-State: AOJu0YyKp2hRQAOnf8zt1ETXjD8Qc+CYFMZc+Ot2Bb1TK/kzm8iQlAlZ NjJ5iGbJ15HC9mmkKtnUpgOkEsWKH/YkQZtiaVjC8Aqrp0B5wUT/GosYriGp3mdN/6JFZt0CC9V U1sM8r5gCMUMbotmQvmJAndy8CKidoUEhyVkMjbaT7UqJVpiBhtbsblPzvnb5qMkhKgL6sW4XOg t6LddKAghEbUzGTIy+aSiGcrGTQ5NpcxgjfStjFfkZ4KyUu5hS49DLRNTyUtiO8/3Nt0itEy9vE D9m4+oHyvMoYEmzA/I= X-Gm-Gg: AY/fxX4L4QtTj95aZEw1LvHL3nH+yriIMjbZSeHwDn4BRbU+ooYepqNN9xSwG2292PX zB+Bp7Gh2G9/bxEiCdQhvoybW7OnG+nSW3XSZAMh2xqdmpZ2Lgzcx+X5JCYsAngZmhaKSUUn4bL oJV+56L+CDhZJ/EbNPi4fwwvc5On7t7CRO0XYSvbBMkvPTVKu7hfWMrUv39x3TrFR97TqvN6tSc Y0YvdLE4ZDowS4ll0in8ym0vcLoKqFvGmJQUog88b8iu60g31nT/u/AACx9tMLTo0vXXgRcSmGe ZYLnIwe/ukSmS02j7g5skna+5OyrFl9uCBaVMgIls6vN6FJDSQellKIwzD2gVxROB0nY/7hg5PY QMz64SmoSBc/jg8P5BEm5SzIbMOwOU3GKf7QWBVA/xn8aH4qyhz175OYy0HHqXgCEtQFHnnXb7y NtINIFr6DYbSqxfQPrjj6vXVyzuYrPFrGAHMkyQ271AGk+ZP4= X-Received: by 2002:a05:690c:45c8:b0:786:8a25:9967 with SMTP id 00721157ae682-793c53ac79fmr70612837b3.50.1768740035363; Sun, 18 Jan 2026 04:40:35 -0800 (PST) Received: from smtp-us-east1-p01-i01-si01.dlp.protect.broadcom.com (address-144-49-247-118.dlp.protect.broadcom.com. [144.49.247.118]) by smtp-relay.gmail.com with ESMTPS id 00721157ae682-793c66c7f36sm4545677b3.1.2026.01.18.04.40.34 for (version=TLS1_2 cipher=ECDHE-ECDSA-AES128-GCM-SHA256 bits=128/128); Sun, 18 Jan 2026 04:40:35 -0800 (PST) X-Relaying-Domain: broadcom.com X-CFilter-Loop: Reflected Received: by mail-pg1-f197.google.com with SMTP id 41be03b00d2f7-c5291b89733so1907903a12.0 for ; Sun, 18 Jan 2026 04:40:34 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=broadcom.com; s=google; t=1768740033; x=1769344833; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=Y/yd2fYK3qdydV8E/E4qJFTT2fTy+J7zleU+MaEFrGQ=; b=Ev7EiVO2J0dQC+L0Ux8FXHAXKAJLgsigBg8f2OGwH2voaXDj36XWMCSASysVWqNSWZ E+7NFO4gqjYgYBa8b4psCBZz+sjtrnU3CNwIVn4UqQoazOpJJ824vSzKwkjOyOo0y27d z9G/oeFJcaY4FnTgPJBIjaZENf+SVpDKopUd4= X-Received: by 2002:a05:6300:193:b0:38d:fa67:e87c with SMTP id adf61e73a8af0-38dfe7c21c0mr7101611637.59.1768740033267; Sun, 18 Jan 2026 04:40:33 -0800 (PST) X-Received: by 2002:a05:6300:193:b0:38d:fa67:e87c with SMTP id adf61e73a8af0-38dfe7c21c0mr7101599637.59.1768740032720; Sun, 18 Jan 2026 04:40:32 -0800 (PST) Received: from PC-MID-R740.dhcp.broadcom.net ([192.19.234.250]) by smtp.gmail.com with ESMTPSA id 41be03b00d2f7-c5edf249c24sm6761246a12.11.2026.01.18.04.40.29 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sun, 18 Jan 2026 04:40:32 -0800 (PST) From: Pavan Chebbi To: jgg@ziepe.ca, michael.chan@broadcom.com Cc: linux-kernel@vger.kernel.org, dave.jiang@intel.com, saeedm@nvidia.com, Jonathan.Cameron@huawei.com, gospo@broadcom.com, selvin.xavier@broadcom.com, leon@kernel.org, kalesh-anakkur.purayil@broadcom.com, Pavan Chebbi Subject: [PATCH fwctl 2/5] fwctl/bnxt_en: Refactor aux bus functions to be more generic Date: Sun, 18 Jan 2026 04:33:58 -0800 Message-Id: <20260118123401.3188438-3-pavan.chebbi@broadcom.com> X-Mailer: git-send-email 2.39.1 In-Reply-To: <20260118123401.3188438-1-pavan.chebbi@broadcom.com> References: <20260118123401.3188438-1-pavan.chebbi@broadcom.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable X-DetectorID-Processed: b00c1d49-9d2e-4205-b15f-d015386d3d5e Content-Type: text/plain; charset="utf-8" Up until now there was only one auxiliary device that bnxt created and that was for RoCE driver. bnxt fwctl is also going to use an aux bus device that bnxt should create. This requires some nomenclature changes and refactoring of the existing bnxt aux dev functions. Convert 'aux_priv' and 'edev' members of struct bnxt into arrays where each element contains supported auxbus device's data. Move struct bnxt_aux_priv from bnxt.h to ulp.h because that is where it belongs. Make aux bus init/uninit/add/del functions more generic which will loop through all the aux device types. Make bnxt_ulp_start/stop functions (the only other common functions applicable to any aux device) loop through the aux devices to update their config and states. Also, as an improvement in code, bnxt_register_dev() can skip unnecessary dereferencing of edev from bp, instead use the edev pointer from the function parameter. Future patches will reuse these functions to add an aux bus device for fwctl. Reviewed-by: Andy Gospodarek Signed-off-by: Pavan Chebbi --- drivers/net/ethernet/broadcom/bnxt/bnxt.c | 29 +- drivers/net/ethernet/broadcom/bnxt/bnxt.h | 17 +- .../net/ethernet/broadcom/bnxt/bnxt_ethtool.c | 2 +- drivers/net/ethernet/broadcom/bnxt/bnxt_ulp.c | 319 +++++++++++------- include/linux/bnxt/ulp.h | 23 +- 5 files changed, 239 insertions(+), 151 deletions(-) diff --git a/drivers/net/ethernet/broadcom/bnxt/bnxt.c b/drivers/net/ethern= et/broadcom/bnxt/bnxt.c index 4481d80cdfc2..620b05097452 100644 --- a/drivers/net/ethernet/broadcom/bnxt/bnxt.c +++ b/drivers/net/ethernet/broadcom/bnxt/bnxt.c @@ -6859,7 +6859,8 @@ int bnxt_hwrm_vnic_cfg(struct bnxt *bp, struct bnxt_v= nic_info *vnic) #endif if ((bp->flags & BNXT_FLAG_STRIP_VLAN) || def_vlan) req->flags |=3D cpu_to_le32(VNIC_CFG_REQ_FLAGS_VLAN_STRIP_MODE); - if (vnic->vnic_id =3D=3D BNXT_VNIC_DEFAULT && bnxt_ulp_registered(bp->ede= v)) + if (vnic->vnic_id =3D=3D BNXT_VNIC_DEFAULT && + bnxt_ulp_registered(bp->edev[BNXT_AUXDEV_RDMA])) req->flags |=3D cpu_to_le32(bnxt_get_roce_vnic_mode(bp)); =20 return hwrm_req_send(bp, req); @@ -7974,6 +7975,7 @@ static int bnxt_get_avail_msix(struct bnxt *bp, int n= um); =20 static int __bnxt_reserve_rings(struct bnxt *bp) { + struct bnxt_en_dev *edev =3D bp->edev[BNXT_AUXDEV_RDMA]; struct bnxt_hw_rings hwr =3D {0}; int rx_rings, old_rx_rings, rc; int cp =3D bp->cp_nr_rings; @@ -7984,7 +7986,7 @@ static int __bnxt_reserve_rings(struct bnxt *bp) if (!bnxt_need_reserve_rings(bp)) return 0; =20 - if (BNXT_NEW_RM(bp) && !bnxt_ulp_registered(bp->edev)) { + if (BNXT_NEW_RM(bp) && !bnxt_ulp_registered(edev)) { ulp_msix =3D bnxt_get_avail_msix(bp, bp->ulp_num_msix_want); if (!ulp_msix) bnxt_set_ulp_stat_ctxs(bp, 0); @@ -8035,8 +8037,7 @@ static int __bnxt_reserve_rings(struct bnxt *bp) } rx_rings =3D min_t(int, rx_rings, hwr.grp); hwr.cp =3D min_t(int, hwr.cp, bp->cp_nr_rings); - if (bnxt_ulp_registered(bp->edev) && - hwr.stat > bnxt_get_ulp_stat_ctxs(bp)) + if (bnxt_ulp_registered(edev) && hwr.stat > bnxt_get_ulp_stat_ctxs(bp)) hwr.stat -=3D bnxt_get_ulp_stat_ctxs(bp); hwr.cp =3D min_t(int, hwr.cp, hwr.stat); rc =3D bnxt_trim_rings(bp, &rx_rings, &hwr.tx, hwr.cp, sh); @@ -8075,7 +8076,7 @@ static int __bnxt_reserve_rings(struct bnxt *bp) !netif_is_rxfh_configured(bp->dev)) bnxt_set_dflt_rss_indir_tbl(bp, NULL); =20 - if (!bnxt_ulp_registered(bp->edev) && BNXT_NEW_RM(bp)) { + if (!bnxt_ulp_registered(edev) && BNXT_NEW_RM(bp)) { int resv_msix, resv_ctx, ulp_ctxs; struct bnxt_hw_resc *hw_resc; =20 @@ -11430,6 +11431,7 @@ static void bnxt_clear_int_mode(struct bnxt *bp) =20 int bnxt_reserve_rings(struct bnxt *bp, bool irq_re_init) { + struct bnxt_en_dev *edev =3D bp->edev[BNXT_AUXDEV_RDMA]; bool irq_cleared =3D false; bool irq_change =3D false; int tcs =3D bp->num_tc; @@ -11439,7 +11441,7 @@ int bnxt_reserve_rings(struct bnxt *bp, bool irq_re= _init) if (!bnxt_need_reserve_rings(bp)) return 0; =20 - if (BNXT_NEW_RM(bp) && !bnxt_ulp_registered(bp->edev)) { + if (BNXT_NEW_RM(bp) && !bnxt_ulp_registered(edev)) { int ulp_msix =3D bnxt_get_avail_msix(bp, bp->ulp_num_msix_want); =20 if (ulp_msix > bp->ulp_num_msix_want) @@ -14679,7 +14681,7 @@ int bnxt_check_rings(struct bnxt *bp, int tx, int r= x, bool sh, int tcs, hwr.cp_p5 =3D hwr.tx + rx; rc =3D bnxt_hwrm_check_rings(bp, &hwr); if (!rc && pci_msix_can_alloc_dyn(bp->pdev)) { - if (!bnxt_ulp_registered(bp->edev)) { + if (!bnxt_ulp_registered(bp->edev[BNXT_AUXDEV_RDMA])) { hwr.cp +=3D bnxt_get_ulp_msix_num(bp); hwr.cp =3D min_t(int, hwr.cp, bnxt_get_max_func_irqs(bp)); } @@ -16211,12 +16213,13 @@ static void bnxt_remove_one(struct pci_dev *pdev) if (BNXT_PF(bp)) __bnxt_sriov_disable(bp); =20 - bnxt_rdma_aux_device_del(bp); + bnxt_aux_devices_del(bp); =20 unregister_netdev(dev); bnxt_ptp_clear(bp); =20 - bnxt_rdma_aux_device_uninit(bp); + bnxt_aux_devices_uninit(bp); + bnxt_auxdev_id_free(bp, bp->auxdev_id); =20 bnxt_free_l2_filters(bp, true); bnxt_free_ntp_fltrs(bp, true); @@ -16802,7 +16805,8 @@ static int bnxt_init_one(struct pci_dev *pdev, cons= t struct pci_device_id *ent) bnxt_set_tpa_flags(bp); bnxt_init_ring_params(bp); bnxt_set_ring_params(bp); - bnxt_rdma_aux_device_init(bp); + if (!bnxt_auxdev_id_alloc(bp)) + bnxt_aux_devices_init(bp); rc =3D bnxt_set_dflt_rings(bp, true); if (rc) { if (BNXT_VF(bp) && rc =3D=3D -ENODEV) { @@ -16866,7 +16870,7 @@ static int bnxt_init_one(struct pci_dev *pdev, cons= t struct pci_device_id *ent) =20 bnxt_dl_fw_reporters_create(bp); =20 - bnxt_rdma_aux_device_add(bp); + bnxt_aux_devices_add(bp); =20 bnxt_print_device_info(bp); =20 @@ -16874,7 +16878,8 @@ static int bnxt_init_one(struct pci_dev *pdev, cons= t struct pci_device_id *ent) =20 return 0; init_err_cleanup: - bnxt_rdma_aux_device_uninit(bp); + bnxt_aux_devices_uninit(bp); + bnxt_auxdev_id_free(bp, bp->auxdev_id); bnxt_dl_unregister(bp); init_err_dl: bnxt_shutdown_tc(bp); diff --git a/drivers/net/ethernet/broadcom/bnxt/bnxt.h b/drivers/net/ethern= et/broadcom/bnxt/bnxt.h index f5f07a7e6b29..ca0052f05589 100644 --- a/drivers/net/ethernet/broadcom/bnxt/bnxt.h +++ b/drivers/net/ethernet/broadcom/bnxt/bnxt.h @@ -24,12 +24,12 @@ #include #include #include -#include #include #include #include #include #include +#include #ifdef CONFIG_TEE_BNXT_FW #include #endif @@ -2076,12 +2076,6 @@ struct bnxt_fw_health { #define BNXT_FW_IF_RETRY 10 #define BNXT_FW_SLOT_RESET_RETRY 4 =20 -struct bnxt_aux_priv { - struct auxiliary_device aux_dev; - struct bnxt_en_dev *edev; - int id; -}; - enum board_idx { BCM57301, BCM57302, @@ -2341,8 +2335,8 @@ struct bnxt { #define BNXT_CHIP_P5_AND_MINUS(bp) \ (BNXT_CHIP_P3(bp) || BNXT_CHIP_P4(bp) || BNXT_CHIP_P5(bp)) =20 - struct bnxt_aux_priv *aux_priv; - struct bnxt_en_dev *edev; + struct bnxt_aux_priv *aux_priv[__BNXT_AUXDEV_MAX]; + struct bnxt_en_dev *edev[__BNXT_AUXDEV_MAX]; =20 struct bnxt_napi **bnapi; =20 @@ -2751,6 +2745,11 @@ struct bnxt { struct bnxt_ctx_pg_info *fw_crash_mem; u32 fw_crash_len; struct bnxt_bs_trace_info bs_trace[BNXT_TRACE_MAX]; + int auxdev_id; + atomic_t auxdev_state[__BNXT_AUXDEV_MAX]; +#define BNXT_ADEV_STATE_NONE 0 +#define BNXT_ADEV_STATE_INIT 1 +#define BNXT_ADEV_STATE_ADD 2 }; =20 #define BNXT_NUM_RX_RING_STATS 8 diff --git a/drivers/net/ethernet/broadcom/bnxt/bnxt_ethtool.c b/drivers/ne= t/ethernet/broadcom/bnxt/bnxt_ethtool.c index 8cad7b982664..064d7bc4ce8d 100644 --- a/drivers/net/ethernet/broadcom/bnxt/bnxt_ethtool.c +++ b/drivers/net/ethernet/broadcom/bnxt/bnxt_ethtool.c @@ -5100,7 +5100,7 @@ static void bnxt_self_test(struct net_device *dev, st= ruct ethtool_test *etest, =20 memset(buf, 0, sizeof(u64) * bp->num_tests); if (etest->flags & ETH_TEST_FL_OFFLINE && - bnxt_ulp_registered(bp->edev)) { + bnxt_ulp_registered(bp->edev[BNXT_AUXDEV_RDMA])) { etest->flags |=3D ETH_TEST_FL_FAILED; netdev_warn(dev, "Offline tests cannot be run with RoCE driver loaded\n"= ); return; diff --git a/drivers/net/ethernet/broadcom/bnxt/bnxt_ulp.c b/drivers/net/et= hernet/broadcom/bnxt/bnxt_ulp.c index fa513892db50..1378b58da018 100644 --- a/drivers/net/ethernet/broadcom/bnxt/bnxt_ulp.c +++ b/drivers/net/ethernet/broadcom/bnxt/bnxt_ulp.c @@ -29,9 +29,31 @@ =20 static DEFINE_IDA(bnxt_aux_dev_ids); =20 +struct bnxt_aux_device { + const char *name; + bool (*is_init)(struct bnxt *bp, int idx); + bool (*is_active)(struct bnxt *bp, int idx); +}; + +static bool bnxt_auxdev_is_init(struct bnxt *bp, int idx) +{ + return atomic_read(&bp->auxdev_state[idx]) =3D=3D BNXT_ADEV_STATE_INIT; +} + +static bool bnxt_auxdev_is_active(struct bnxt *bp, int idx) +{ + return atomic_read(&bp->auxdev_state[idx]) =3D=3D BNXT_ADEV_STATE_ADD; +} + +static struct bnxt_aux_device bnxt_aux_devices[__BNXT_AUXDEV_MAX] =3D {{ + .name =3D "rdma", + .is_init =3D &bnxt_auxdev_is_init, + .is_active =3D &bnxt_auxdev_is_active, +}}; + static void bnxt_fill_msix_vecs(struct bnxt *bp, struct bnxt_msix_entry *e= nt) { - struct bnxt_en_dev *edev =3D bp->edev; + struct bnxt_en_dev *edev =3D bp->edev[BNXT_AUXDEV_RDMA]; int num_msix, i; =20 if (!edev->ulp_tbl->msix_requested) { @@ -51,61 +73,75 @@ static void bnxt_fill_msix_vecs(struct bnxt *bp, struct= bnxt_msix_entry *ent) =20 int bnxt_get_ulp_msix_num(struct bnxt *bp) { - if (bp->edev) - return bp->edev->ulp_num_msix_vec; + struct bnxt_en_dev *edev =3D bp->edev[BNXT_AUXDEV_RDMA]; + + if (edev) + return edev->ulp_num_msix_vec; return 0; } =20 void bnxt_set_ulp_msix_num(struct bnxt *bp, int num) { - if (bp->edev) - bp->edev->ulp_num_msix_vec =3D num; + struct bnxt_en_dev *edev =3D bp->edev[BNXT_AUXDEV_RDMA]; + + if (edev) + edev->ulp_num_msix_vec =3D num; } =20 int bnxt_get_ulp_msix_num_in_use(struct bnxt *bp) { - if (bnxt_ulp_registered(bp->edev)) - return bp->edev->ulp_num_msix_vec; + struct bnxt_en_dev *edev =3D bp->edev[BNXT_AUXDEV_RDMA]; + + if (bnxt_ulp_registered(edev)) + return edev->ulp_num_msix_vec; return 0; } =20 int bnxt_get_ulp_stat_ctxs(struct bnxt *bp) { - if (bp->edev) - return bp->edev->ulp_num_ctxs; + struct bnxt_en_dev *edev =3D bp->edev[BNXT_AUXDEV_RDMA]; + + if (edev) + return edev->ulp_num_ctxs; return 0; } =20 void bnxt_set_ulp_stat_ctxs(struct bnxt *bp, int num_ulp_ctx) { - if (bp->edev) - bp->edev->ulp_num_ctxs =3D num_ulp_ctx; + struct bnxt_en_dev *edev =3D bp->edev[BNXT_AUXDEV_RDMA]; + + if (edev) + edev->ulp_num_ctxs =3D num_ulp_ctx; } =20 int bnxt_get_ulp_stat_ctxs_in_use(struct bnxt *bp) { - if (bnxt_ulp_registered(bp->edev)) - return bp->edev->ulp_num_ctxs; + struct bnxt_en_dev *edev =3D bp->edev[BNXT_AUXDEV_RDMA]; + + if (bnxt_ulp_registered(edev)) + return edev->ulp_num_ctxs; return 0; } =20 void bnxt_set_dflt_ulp_stat_ctxs(struct bnxt *bp) { - if (bp->edev) { - bp->edev->ulp_num_ctxs =3D BNXT_MIN_ROCE_STAT_CTXS; + struct bnxt_en_dev *edev =3D bp->edev[BNXT_AUXDEV_RDMA]; + + if (edev) { + edev->ulp_num_ctxs =3D BNXT_MIN_ROCE_STAT_CTXS; /* Reserve one additional stat_ctx for PF0 (except * on 1-port NICs) as it also creates one stat_ctx * for PF1 in case of RoCE bonding. */ if (BNXT_PF(bp) && !bp->pf.port_id && bp->port_count > 1) - bp->edev->ulp_num_ctxs++; + edev->ulp_num_ctxs++; =20 /* Reserve one additional stat_ctx when the device is capable * of supporting port mirroring on RDMA device. */ if (BNXT_MIRROR_ON_ROCE_CAP(bp)) - bp->edev->ulp_num_ctxs++; + edev->ulp_num_ctxs++; } } =20 @@ -141,7 +177,7 @@ int bnxt_register_dev(struct bnxt_en_dev *edev, =20 edev->ulp_tbl->msix_requested =3D bnxt_get_ulp_msix_num(bp); =20 - bnxt_fill_msix_vecs(bp, bp->edev->msix_entries); + bnxt_fill_msix_vecs(bp, edev->msix_entries); exit: mutex_unlock(&edev->en_dev_lock); netdev_unlock(dev); @@ -227,20 +263,26 @@ EXPORT_SYMBOL(bnxt_send_msg); =20 void bnxt_ulp_stop(struct bnxt *bp) { - struct bnxt_aux_priv *aux_priv =3D bp->aux_priv; - struct bnxt_en_dev *edev =3D bp->edev; - - if (!edev) - return; - - mutex_lock(&edev->en_dev_lock); - if (!bnxt_ulp_registered(edev) || - (edev->flags & BNXT_EN_FLAG_ULP_STOPPED)) - goto ulp_stop_exit; + int i; =20 - edev->flags |=3D BNXT_EN_FLAG_ULP_STOPPED; - if (aux_priv) { + for (i =3D 0; i < __BNXT_AUXDEV_MAX; i++) { + struct bnxt_aux_priv *aux_priv; struct auxiliary_device *adev; + struct bnxt_en_dev *edev; + + if (!bnxt_aux_devices[i].is_active(bp, i)) + continue; + + aux_priv =3D bp->aux_priv[i]; + edev =3D bp->edev[i]; + mutex_lock(&edev->en_dev_lock); + if (!bnxt_ulp_registered(edev) || + (edev->flags & BNXT_EN_FLAG_ULP_STOPPED)) { + mutex_unlock(&edev->en_dev_lock); + continue; + } + + edev->flags |=3D BNXT_EN_FLAG_ULP_STOPPED; =20 adev =3D &aux_priv->aux_dev; if (adev->dev.driver) { @@ -251,29 +293,33 @@ void bnxt_ulp_stop(struct bnxt *bp) edev->en_state =3D bp->state; adrv->suspend(adev, pm); } + mutex_unlock(&edev->en_dev_lock); } -ulp_stop_exit: - mutex_unlock(&edev->en_dev_lock); } =20 void bnxt_ulp_start(struct bnxt *bp, int err) { - struct bnxt_aux_priv *aux_priv =3D bp->aux_priv; - struct bnxt_en_dev *edev =3D bp->edev; + int i; =20 - if (!edev || err) - return; + for (i =3D 0; i < __BNXT_AUXDEV_MAX; i++) { + struct bnxt_aux_priv *aux_priv; + struct auxiliary_device *adev; + struct bnxt_en_dev *edev; =20 - mutex_lock(&edev->en_dev_lock); - if (!bnxt_ulp_registered(edev) || - !(edev->flags & BNXT_EN_FLAG_ULP_STOPPED)) - goto ulp_start_exit; + if (!bnxt_aux_devices[i].is_active(bp, i)) + continue; =20 - if (edev->ulp_tbl->msix_requested) - bnxt_fill_msix_vecs(bp, edev->msix_entries); + aux_priv =3D bp->aux_priv[i]; + edev =3D bp->edev[i]; + mutex_lock(&edev->en_dev_lock); + if (!bnxt_ulp_registered(edev) || + !(edev->flags & BNXT_EN_FLAG_ULP_STOPPED)) { + goto clear_flag_continue; + } + + if (edev->ulp_tbl->msix_requested) + bnxt_fill_msix_vecs(bp, edev->msix_entries); =20 - if (aux_priv) { - struct auxiliary_device *adev; =20 adev =3D &aux_priv->aux_dev; if (adev->dev.driver) { @@ -283,22 +329,22 @@ void bnxt_ulp_start(struct bnxt *bp, int err) edev->en_state =3D bp->state; adrv->resume(adev); } +clear_flag_continue: + edev->flags &=3D ~BNXT_EN_FLAG_ULP_STOPPED; + mutex_unlock(&edev->en_dev_lock); } -ulp_start_exit: - edev->flags &=3D ~BNXT_EN_FLAG_ULP_STOPPED; - mutex_unlock(&edev->en_dev_lock); } =20 void bnxt_ulp_irq_stop(struct bnxt *bp) { - struct bnxt_en_dev *edev =3D bp->edev; + struct bnxt_en_dev *edev =3D bp->edev[BNXT_AUXDEV_RDMA]; struct bnxt_ulp_ops *ops; bool reset =3D false; =20 if (!edev) return; =20 - if (bnxt_ulp_registered(bp->edev)) { + if (bnxt_ulp_registered(edev)) { struct bnxt_ulp *ulp =3D edev->ulp_tbl; =20 if (!ulp->msix_requested) @@ -315,13 +361,13 @@ void bnxt_ulp_irq_stop(struct bnxt *bp) =20 void bnxt_ulp_irq_restart(struct bnxt *bp, int err) { - struct bnxt_en_dev *edev =3D bp->edev; + struct bnxt_en_dev *edev =3D bp->edev[BNXT_AUXDEV_RDMA]; struct bnxt_ulp_ops *ops; =20 if (!edev) return; =20 - if (bnxt_ulp_registered(bp->edev)) { + if (bnxt_ulp_registered(edev)) { struct bnxt_ulp *ulp =3D edev->ulp_tbl; struct bnxt_msix_entry *ent =3D NULL; =20 @@ -347,7 +393,7 @@ void bnxt_ulp_irq_restart(struct bnxt *bp, int err) void bnxt_ulp_async_events(struct bnxt *bp, struct hwrm_async_event_cmpl *= cmpl) { u16 event_id =3D le16_to_cpu(cmpl->event_id); - struct bnxt_en_dev *edev =3D bp->edev; + struct bnxt_en_dev *edev =3D bp->edev[BNXT_AUXDEV_RDMA]; struct bnxt_ulp_ops *ops; struct bnxt_ulp *ulp; =20 @@ -388,18 +434,19 @@ void bnxt_register_async_events(struct bnxt_en_dev *e= dev, } EXPORT_SYMBOL(bnxt_register_async_events); =20 -void bnxt_rdma_aux_device_uninit(struct bnxt *bp) +void bnxt_aux_devices_uninit(struct bnxt *bp) { struct bnxt_aux_priv *aux_priv; struct auxiliary_device *adev; + int idx; =20 - /* Skip if no auxiliary device init was done. */ - if (!bp->aux_priv) - return; - - aux_priv =3D bp->aux_priv; - adev =3D &aux_priv->aux_dev; - auxiliary_device_uninit(adev); + for (idx =3D 0; idx < __BNXT_AUXDEV_MAX; idx++) { + if (bnxt_aux_devices[idx].is_init(bp, idx)) { + aux_priv =3D bp->aux_priv[idx]; + adev =3D &aux_priv->aux_dev; + auxiliary_device_uninit(adev); + } + } } =20 static void bnxt_aux_dev_release(struct device *dev) @@ -408,20 +455,24 @@ static void bnxt_aux_dev_release(struct device *dev) container_of(dev, struct bnxt_aux_priv, aux_dev.dev); struct bnxt *bp =3D netdev_priv(aux_priv->edev->net); =20 - ida_free(&bnxt_aux_dev_ids, aux_priv->id); kfree(aux_priv->edev->ulp_tbl); - bp->edev =3D NULL; + bp->edev[aux_priv->id] =3D NULL; kfree(aux_priv->edev); kfree(aux_priv); - bp->aux_priv =3D NULL; + bp->aux_priv[aux_priv->id] =3D NULL; + atomic_set(&bp->auxdev_state[aux_priv->id], BNXT_ADEV_STATE_NONE); } =20 -void bnxt_rdma_aux_device_del(struct bnxt *bp) +void bnxt_aux_devices_del(struct bnxt *bp) { - if (!bp->edev) - return; + int idx; =20 - auxiliary_device_delete(&bp->aux_priv->aux_dev); + for (idx =3D 0; idx < __BNXT_AUXDEV_MAX; idx++) { + if (bnxt_aux_devices[idx].is_active(bp, idx)) { + auxiliary_device_delete(&bp->aux_priv[idx]->aux_dev); + atomic_set(&bp->auxdev_state[idx], BNXT_ADEV_STATE_INIT); + } + } } =20 static void bnxt_set_edev_info(struct bnxt_en_dev *edev, struct bnxt *bp) @@ -451,83 +502,101 @@ static void bnxt_set_edev_info(struct bnxt_en_dev *e= dev, struct bnxt *bp) edev->bar0 =3D bp->bar0; } =20 -void bnxt_rdma_aux_device_add(struct bnxt *bp) +void bnxt_aux_devices_add(struct bnxt *bp) { struct auxiliary_device *aux_dev; - int rc; - - if (!bp->edev) - return; - - aux_dev =3D &bp->aux_priv->aux_dev; - rc =3D auxiliary_device_add(aux_dev); - if (rc) { - netdev_warn(bp->dev, "Failed to add auxiliary device for ROCE\n"); - auxiliary_device_uninit(aux_dev); - bp->flags &=3D ~BNXT_FLAG_ROCE_CAP; + int rc, idx; + + for (idx =3D 0; idx < __BNXT_AUXDEV_MAX; idx++) { + if (bnxt_aux_devices[idx].is_init(bp, idx)) { + aux_dev =3D &bp->aux_priv[idx]->aux_dev; + rc =3D auxiliary_device_add(aux_dev); + if (rc) { + netdev_warn(bp->dev, "Failed to add auxiliary device for ROCE\n"); + auxiliary_device_uninit(aux_dev); + if (idx =3D=3D BNXT_AUXDEV_RDMA) + bp->flags &=3D ~BNXT_FLAG_ROCE_CAP; + continue; + } + atomic_set(&bp->auxdev_state[idx], BNXT_ADEV_STATE_ADD); + } } } =20 -void bnxt_rdma_aux_device_init(struct bnxt *bp) +void bnxt_aux_devices_init(struct bnxt *bp) { struct auxiliary_device *aux_dev; struct bnxt_aux_priv *aux_priv; struct bnxt_en_dev *edev; struct bnxt_ulp *ulp; - int rc; - - if (!(bp->flags & BNXT_FLAG_ROCE_CAP)) - return; + int rc, idx; =20 - aux_priv =3D kzalloc(sizeof(*bp->aux_priv), GFP_KERNEL); - if (!aux_priv) - goto exit; + for (idx =3D 0; idx < __BNXT_AUXDEV_MAX; idx++) { + atomic_set(&bp->auxdev_state[idx], BNXT_ADEV_STATE_NONE); =20 - aux_priv->id =3D ida_alloc(&bnxt_aux_dev_ids, GFP_KERNEL); - if (aux_priv->id < 0) { - netdev_warn(bp->dev, - "ida alloc failed for ROCE auxiliary device\n"); - kfree(aux_priv); - goto exit; - } + if (idx =3D=3D BNXT_AUXDEV_RDMA && + !(bp->flags & BNXT_FLAG_ROCE_CAP)) + continue; =20 - aux_dev =3D &aux_priv->aux_dev; - aux_dev->id =3D aux_priv->id; - aux_dev->name =3D "rdma"; - aux_dev->dev.parent =3D &bp->pdev->dev; - aux_dev->dev.release =3D bnxt_aux_dev_release; + aux_priv =3D kzalloc(sizeof(*aux_priv), GFP_KERNEL); + if (!aux_priv) + goto next_auxdev; =20 - rc =3D auxiliary_device_init(aux_dev); - if (rc) { - ida_free(&bnxt_aux_dev_ids, aux_priv->id); - kfree(aux_priv); - goto exit; - } - bp->aux_priv =3D aux_priv; + aux_dev =3D &aux_priv->aux_dev; + aux_dev->id =3D bp->auxdev_id; + aux_dev->name =3D bnxt_aux_devices[idx].name; + aux_dev->dev.parent =3D &bp->pdev->dev; + aux_dev->dev.release =3D bnxt_aux_dev_release; =20 - /* From this point, all cleanup will happen via the .release callback & - * any error unwinding will need to include a call to - * auxiliary_device_uninit. - */ - edev =3D kzalloc(sizeof(*edev), GFP_KERNEL); - if (!edev) - goto aux_dev_uninit; + rc =3D auxiliary_device_init(aux_dev); + if (rc) { + kfree(aux_priv); + goto next_auxdev; + } + bp->aux_priv[idx] =3D aux_priv; =20 - aux_priv->edev =3D edev; + /* From this point, all cleanup will happen via the .release + * callback & any error unwinding will need to include a call + * to auxiliary_device_uninit. + */ + edev =3D kzalloc(sizeof(*edev), GFP_KERNEL); + if (!edev) + goto aux_dev_uninit; =20 - ulp =3D kzalloc(sizeof(*ulp), GFP_KERNEL); - if (!ulp) - goto aux_dev_uninit; + aux_priv->edev =3D edev; + bnxt_set_edev_info(edev, bp); =20 - edev->ulp_tbl =3D ulp; - bp->edev =3D edev; - bnxt_set_edev_info(edev, bp); - bp->ulp_num_msix_want =3D bnxt_set_dflt_ulp_msix(bp); + ulp =3D kzalloc(sizeof(*ulp), GFP_KERNEL); + if (!ulp) + goto aux_dev_uninit; =20 - return; + edev->ulp_tbl =3D ulp; + bp->edev[idx] =3D edev; + if (idx =3D=3D BNXT_AUXDEV_RDMA) + bp->ulp_num_msix_want =3D bnxt_set_dflt_ulp_msix(bp); + aux_priv->id =3D idx; + atomic_set(&bp->auxdev_state[idx], BNXT_ADEV_STATE_INIT); =20 + continue; aux_dev_uninit: - auxiliary_device_uninit(aux_dev); -exit: - bp->flags &=3D ~BNXT_FLAG_ROCE_CAP; + auxiliary_device_uninit(aux_dev); +next_auxdev: + if (idx =3D=3D BNXT_AUXDEV_RDMA) + bp->flags &=3D ~BNXT_FLAG_ROCE_CAP; + } +} + +int bnxt_auxdev_id_alloc(struct bnxt *bp) +{ + bp->auxdev_id =3D ida_alloc(&bnxt_aux_dev_ids, GFP_KERNEL); + if (bp->auxdev_id < 0) + return bp->auxdev_id; + + return 0; +} + +void bnxt_auxdev_id_free(struct bnxt *bp, int id) +{ + if (bp->auxdev_id >=3D 0) + ida_free(&bnxt_aux_dev_ids, id); } diff --git a/include/linux/bnxt/ulp.h b/include/linux/bnxt/ulp.h index 3c5b8a53f715..72d6941fd57b 100644 --- a/include/linux/bnxt/ulp.h +++ b/include/linux/bnxt/ulp.h @@ -10,6 +10,8 @@ #ifndef BNXT_ULP_H #define BNXT_ULP_H =20 +#include + #define BNXT_MIN_ROCE_CP_RINGS 2 #define BNXT_MIN_ROCE_STAT_CTXS 1 =20 @@ -20,6 +22,17 @@ struct hwrm_async_event_cmpl; struct bnxt; =20 +enum bnxt_auxdev_type { + BNXT_AUXDEV_RDMA =3D 0, + __BNXT_AUXDEV_MAX +}; + +struct bnxt_aux_priv { + struct auxiliary_device aux_dev; + struct bnxt_en_dev *edev; + int id; +}; + struct bnxt_msix_entry { u32 vector; u32 ring_idx; @@ -115,14 +128,16 @@ void bnxt_ulp_sriov_cfg(struct bnxt *bp, int num_vfs); void bnxt_ulp_irq_stop(struct bnxt *bp); void bnxt_ulp_irq_restart(struct bnxt *bp, int err); void bnxt_ulp_async_events(struct bnxt *bp, struct hwrm_async_event_cmpl *= cmpl); -void bnxt_rdma_aux_device_uninit(struct bnxt *bp); -void bnxt_rdma_aux_device_del(struct bnxt *bp); -void bnxt_rdma_aux_device_add(struct bnxt *bp); -void bnxt_rdma_aux_device_init(struct bnxt *bp); +void bnxt_aux_devices_uninit(struct bnxt *bp); +void bnxt_aux_devices_del(struct bnxt *bp); +void bnxt_aux_devices_add(struct bnxt *bp); +void bnxt_aux_devices_init(struct bnxt *bp); int bnxt_register_dev(struct bnxt_en_dev *edev, struct bnxt_ulp_ops *ulp_o= ps, void *handle); void bnxt_unregister_dev(struct bnxt_en_dev *edev); int bnxt_send_msg(struct bnxt_en_dev *edev, struct bnxt_fw_msg *fw_msg); void bnxt_register_async_events(struct bnxt_en_dev *edev, unsigned long *events_bmap, u16 max_id); +int bnxt_auxdev_id_alloc(struct bnxt *bp); +void bnxt_auxdev_id_free(struct bnxt *bp, int id); #endif --=20 2.39.1 From nobody Sun Feb 8 06:56:35 2026 Received: from mail-pj1-f97.google.com (mail-pj1-f97.google.com [209.85.216.97]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id D97932EB86C for ; Sun, 18 Jan 2026 12:40:50 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.216.97 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1768740052; cv=none; b=asCO95lzG6I6AMJ0KxW+MIs7jS3Ee6/oLHSEQJbmh8m0BTm0T+KeNaoYpxN5TrvOvPCd8lwvLkkiyChpMGm+yCytqfWj+pTfuzwAlKqiza1N+ejnc/Kg+IYB1ic5Z5R76JvBHOQLChrnVC8CaAj3FR8r53EhvhPuKFkJkjbGxEo= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1768740052; c=relaxed/simple; bh=QHymrLP79c+XQE/8LFPXoEJnFY+e4wkVSqfgxyXv1tA=; h=From:To:Cc:Subject:Date:Message-Id:In-Reply-To:References: MIME-Version; b=ZEpYquQnUzHzRUKP5Tpuws21zrQa7rupaQLA2sPn813QbSRhDJlnt9Y3CihGB8EBeRNEGv+92HQtbhGfIUDys0shC1w3/lVtiL9aYaEcGzXvqwPS9Cj3pCN72RaoIvq4WdE9NETbPUnFo61e2MWq6IxL8n9L4vnHLM8rWRucl/0= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=broadcom.com; spf=fail smtp.mailfrom=broadcom.com; dkim=pass (1024-bit key) header.d=broadcom.com header.i=@broadcom.com header.b=ErORafVy; arc=none smtp.client-ip=209.85.216.97 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=broadcom.com Authentication-Results: smtp.subspace.kernel.org; spf=fail smtp.mailfrom=broadcom.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=broadcom.com header.i=@broadcom.com header.b="ErORafVy" Received: by mail-pj1-f97.google.com with SMTP id 98e67ed59e1d1-34c84ec3b6eso3116148a91.3 for ; Sun, 18 Jan 2026 04:40:50 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1768740050; x=1769344850; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:dkim-signature:x-gm-gg :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=8uwW5kTFniNzAK7B8ZBRrzacCwm+22XYIhP9CIuYilg=; b=C9FcqLGNs5BKsk578hfYAVREBTkKFGGVLgFXjMj8JnyoB6rZzdDWX038d3N59C69ps S5GkZX8LXJ2e8PRxR6g9V8bC4UZghDAFOCQcEib1dyx+b1ttTO4J4+C5+lgr8AGPmBSI 4ZeKautelvyELtgYKuGDpjbd6Mxa275O24VBG+v3JhCkgbLNMFp8lPP1X25NudhGR+6K nzwTm+RRvONtBkXONlHZUlWKoYiqS+YaYUGui742bwfS96uGbiWQ4Sy3O4RcQLs7HQL8 Lcqsh4QehWwmuqV+JhjvLzElEga4qfmHZxCH9stYq/UGCMExnsiAhOXZX/azyXaUDJVi AWOA== X-Gm-Message-State: AOJu0YwlkJQUgKicX7+dPmclJY4RmPYaEcSUXXthDPEGcGxBuFfJI3hE lvMuQWLJZHZJsxI8Fn1jmqSXNcpsq+DoepJkK+ZrVfcIdNT8f49KLjFByRh92anStBRPoHVNR3O pLITA+s858bEEtvQLm1v/XcWaqi7LiaROc5g6I0btKUnzXLIqUVhCvImRxsYlDONUuN1IlRuVug f+PwKBN5lOC+V5B5tTWYB83p1yKksWlBgJwXrDnSZmO2TNk6AVBtORyOsZzXFqWNR6YL7HTB4e5 PotvD/UwQNWBr1J4N0= X-Gm-Gg: AY/fxX7YJRU+nvsNKJkNMHMR6Deeug0CvY6IY2sXJgnLjEKKBVhTqb3E8UD/IEDqTEx TWqWOzuAcR+qZE6xwPYX95R8mMqkDZy6lp9+g9OPepHxValHdPqImNGyg7ahfkpvqVgPHcl/k2J rKDR+dnEVXltlJLIErr6hVjVSwddDfn7Vd5qBok+ez3SsJP4DxZ6vdWqtjv2NpVUDBrOZJhVj1S Y/7coouxNIFE28qvWUxGSwDiRO0pVWDlkUlkaloRkiJttCK8Am3NHXyz41TOV7iFnNjbuxP5E27 l1L1tYJPLYj8anjV/wmi7lDt0TORYN12bkgR4D5c5ama4hSacFWdPPkmh9gPxiLlKPJPreBbLGW D02nFq+WbmQF7OK+BnwDuitnRm3b2++xpCtMsjmU/MFk6mfGLrP4S4KQXynqRBzNgNTyB2v439W SrpqC5OIbOZuAbkvhBv46U6gkbfAyAJ3FY5xZvWMwK1Q== X-Received: by 2002:a17:90b:3906:b0:340:c179:3657 with SMTP id 98e67ed59e1d1-35272fbf5e1mr7086847a91.33.1768740050124; Sun, 18 Jan 2026 04:40:50 -0800 (PST) Received: from smtp-us-east1-p01-i01-si01.dlp.protect.broadcom.com (address-144-49-247-2.dlp.protect.broadcom.com. [144.49.247.2]) by smtp-relay.gmail.com with ESMTPS id 98e67ed59e1d1-3527313ffcdsm1031180a91.8.2026.01.18.04.40.49 for (version=TLS1_2 cipher=ECDHE-ECDSA-AES128-GCM-SHA256 bits=128/128); Sun, 18 Jan 2026 04:40:50 -0800 (PST) X-Relaying-Domain: broadcom.com X-CFilter-Loop: Reflected Received: by mail-pj1-f71.google.com with SMTP id 98e67ed59e1d1-34abd303b4aso6953770a91.1 for ; Sun, 18 Jan 2026 04:40:49 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=broadcom.com; s=google; t=1768740048; x=1769344848; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=8uwW5kTFniNzAK7B8ZBRrzacCwm+22XYIhP9CIuYilg=; b=ErORafVyVXoy/ZRzs8vvxGIIUnXN4IGnhMBNp9NGguRghgz0jHMjFMs+THIO7B+U6q 6NfEAo/Y7gO/ndzzqJVZiz8h94d/50NnmOPYf/DnKNJzySXT5SsMoVZOoHw0bor1aT/l nkYClBJllOJyDuBuNN0N8esRGWXinW6bfSPVM= X-Received: by 2002:a17:90b:2ec4:b0:34c:fe57:2793 with SMTP id 98e67ed59e1d1-35272f87fc2mr6957107a91.20.1768740048230; Sun, 18 Jan 2026 04:40:48 -0800 (PST) X-Received: by 2002:a17:90b:2ec4:b0:34c:fe57:2793 with SMTP id 98e67ed59e1d1-35272f87fc2mr6957098a91.20.1768740047871; Sun, 18 Jan 2026 04:40:47 -0800 (PST) Received: from PC-MID-R740.dhcp.broadcom.net ([192.19.234.250]) by smtp.gmail.com with ESMTPSA id 41be03b00d2f7-c5edf249c24sm6761246a12.11.2026.01.18.04.40.44 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sun, 18 Jan 2026 04:40:47 -0800 (PST) From: Pavan Chebbi To: jgg@ziepe.ca, michael.chan@broadcom.com Cc: linux-kernel@vger.kernel.org, dave.jiang@intel.com, saeedm@nvidia.com, Jonathan.Cameron@huawei.com, gospo@broadcom.com, selvin.xavier@broadcom.com, leon@kernel.org, kalesh-anakkur.purayil@broadcom.com, Pavan Chebbi Subject: [PATCH fwctl 3/5] fwctl/bnxt_en: Create an aux device for fwctl Date: Sun, 18 Jan 2026 04:33:59 -0800 Message-Id: <20260118123401.3188438-4-pavan.chebbi@broadcom.com> X-Mailer: git-send-email 2.39.1 In-Reply-To: <20260118123401.3188438-1-pavan.chebbi@broadcom.com> References: <20260118123401.3188438-1-pavan.chebbi@broadcom.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable X-DetectorID-Processed: b00c1d49-9d2e-4205-b15f-d015386d3d5e Content-Type: text/plain; charset="utf-8" Create an additional auxiliary device to support fwctl. The next patch will create bnxt_fwctl and bind to this device. Reviewed-by: Andy Gospodarek Signed-off-by: Pavan Chebbi --- drivers/net/ethernet/broadcom/bnxt/bnxt_ulp.c | 14 ++++++++++++-- include/linux/bnxt/ulp.h | 1 + 2 files changed, 13 insertions(+), 2 deletions(-) diff --git a/drivers/net/ethernet/broadcom/bnxt/bnxt_ulp.c b/drivers/net/et= hernet/broadcom/bnxt/bnxt_ulp.c index 1378b58da018..44cf0dfb9556 100644 --- a/drivers/net/ethernet/broadcom/bnxt/bnxt_ulp.c +++ b/drivers/net/ethernet/broadcom/bnxt/bnxt_ulp.c @@ -49,6 +49,10 @@ static struct bnxt_aux_device bnxt_aux_devices[__BNXT_AU= XDEV_MAX] =3D {{ .name =3D "rdma", .is_init =3D &bnxt_auxdev_is_init, .is_active =3D &bnxt_auxdev_is_active, +}, { + .name =3D "fwctl", + .is_init =3D &bnxt_auxdev_is_init, + .is_active =3D &bnxt_auxdev_is_active, }}; =20 static void bnxt_fill_msix_vecs(struct bnxt *bp, struct bnxt_msix_entry *e= nt) @@ -276,6 +280,11 @@ void bnxt_ulp_stop(struct bnxt *bp) aux_priv =3D bp->aux_priv[i]; edev =3D bp->edev[i]; mutex_lock(&edev->en_dev_lock); + if (i =3D=3D BNXT_AUXDEV_FWCTL) { + edev->flags |=3D BNXT_EN_FLAG_ULP_STOPPED; + mutex_unlock(&edev->en_dev_lock); + continue; + } if (!bnxt_ulp_registered(edev) || (edev->flags & BNXT_EN_FLAG_ULP_STOPPED)) { mutex_unlock(&edev->en_dev_lock); @@ -312,7 +321,7 @@ void bnxt_ulp_start(struct bnxt *bp, int err) aux_priv =3D bp->aux_priv[i]; edev =3D bp->edev[i]; mutex_lock(&edev->en_dev_lock); - if (!bnxt_ulp_registered(edev) || + if (i =3D=3D BNXT_AUXDEV_FWCTL || !bnxt_ulp_registered(edev) || !(edev->flags & BNXT_EN_FLAG_ULP_STOPPED)) { goto clear_flag_continue; } @@ -512,7 +521,8 @@ void bnxt_aux_devices_add(struct bnxt *bp) aux_dev =3D &bp->aux_priv[idx]->aux_dev; rc =3D auxiliary_device_add(aux_dev); if (rc) { - netdev_warn(bp->dev, "Failed to add auxiliary device for ROCE\n"); + netdev_warn(bp->dev, "Failed to add auxiliary device for auxdev type %= d\n", + idx); auxiliary_device_uninit(aux_dev); if (idx =3D=3D BNXT_AUXDEV_RDMA) bp->flags &=3D ~BNXT_FLAG_ROCE_CAP; diff --git a/include/linux/bnxt/ulp.h b/include/linux/bnxt/ulp.h index 72d6941fd57b..c668f3e67fef 100644 --- a/include/linux/bnxt/ulp.h +++ b/include/linux/bnxt/ulp.h @@ -24,6 +24,7 @@ struct bnxt; =20 enum bnxt_auxdev_type { BNXT_AUXDEV_RDMA =3D 0, + BNXT_AUXDEV_FWCTL, __BNXT_AUXDEV_MAX }; =20 --=20 2.39.1 From nobody Sun Feb 8 06:56:35 2026 Received: from mail-vk1-f228.google.com (mail-vk1-f228.google.com [209.85.221.228]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id DE7F932BF26 for ; Sun, 18 Jan 2026 12:41:09 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.221.228 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1768740073; cv=none; b=aqtkmdPKPxOim89uUNiYski3SBVwgJiDCF+YEYjuNi7RYP3NGDlooIOrjo5cOGzTpB1g8ydq1nTnWfkkf/IUypL788BNV+vOJhJ9dzFQNdyMq7gNaPD1LyjEbneqLUdsbbaqtJaNHXGo82mK6tDRAbNElMeJ8+OOoIC62wK9pIU= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1768740073; c=relaxed/simple; bh=bkvDLotZ22B4i2XTd7k8eKbtFzq9z0T3Ub4Tl0rE53Q=; h=From:To:Cc:Subject:Date:Message-Id:In-Reply-To:References: MIME-Version; b=R5b4sUXB9QmNEGPMQ61dQPHEZiRS/xSUrmBh73jL07q+srKD1UdGVSygY8+Usr7riBRUPnIyJhEBVFypesYjizO6Y9v1cDwOm1hdedJfEfWlWi8nfQNX43w5H5UpD8GGGYv1kcq2TcKcxIxxYcgp5NpX+JtLVXwiFBhRwK+7Toc= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=broadcom.com; spf=fail smtp.mailfrom=broadcom.com; dkim=pass (1024-bit key) header.d=broadcom.com header.i=@broadcom.com header.b=JjloGdH7; arc=none smtp.client-ip=209.85.221.228 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=broadcom.com Authentication-Results: smtp.subspace.kernel.org; spf=fail smtp.mailfrom=broadcom.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=broadcom.com header.i=@broadcom.com header.b="JjloGdH7" Received: by mail-vk1-f228.google.com with SMTP id 71dfb90a1353d-5636dce976eso2927078e0c.0 for ; Sun, 18 Jan 2026 04:41:09 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1768740069; x=1769344869; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:dkim-signature:x-gm-gg :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=08UPLl1CQ3fW/3Rx56yHOQP+JfK402Xg4OPflRA1UjM=; b=X34+eUt7YUSz2F7u8v8YfRc8+VLc+HgatO4+yly0RpR6hnQvzzk7zdv287+CXJknIz RYSmB/yMipd3tAjmeOGW/qHXA1IUzgzfPsYLdL3OGQrXVufRzYTmCdX8a+Orfqam1thm OB+pW8KgBPFikWYc0WVPek/eZvYsU3d53O/O4JyZm8+29kgAciN7pUKV4cNWMiLmvSB/ 2oyzCgqmWgG0H7CUQgpKetQJjWttRzvaARTumpWlV5fcQ5KQrOU6JOrUnEubLbCF+3wx OPK6E84YOhRfZWUWnuj4OsZ+BZ+idXEbnbjjzR46IiUHGvVfoGJZqwYTXqqpqgluvW6f ubSQ== X-Gm-Message-State: AOJu0YzZak4nntx2WSt8T/o/dvBtN/DhwayZ1eIzmY3ltu797wPJxF+s NRCjG4YRJmn/0/YERWLreSPPhQ61R4Nd0fglOHqzSKP9k7xBWua81tvX1LXSA/s+WLe3EfL0OeS kL/6U3OFH1pHgmAQ1dELSLoHgDcoFGkUQFQrclxdR3phrEjqtIvvm6paPvdqxmw8RE+SSQQ7VpA BgKz8zLKlqFgM1ZCrQysM0fwPdd6um5GC/6RkSYpcb05W2X0/M5j0jkquWFfTLQhFle1P/v/NhK vBERO3dA+VEFOxn0o0= X-Gm-Gg: AY/fxX7l3ZDcNY5nmrY4NeLET+aspuBzlFQBCa9br1+rglT0WyBbhClQ9TLBm/ZsYVR Ap1DpqDwqDLrSnZTQtNqNLStma2fqI2eGRhhSaQKU6LIZ5+mpfITsPgroi6fLwRZlpOCZ+mT0BZ yu/Xcitx5B5gdpTUvU3p2BF5+rgMBzqOaF5gbbz5RpruckVIOHnZS+aK0KUl5aYprDWBpn+gJ1f ulUlv7uaAqY9DODxYQoIe/lwFzkVMs5CeoRlTJqPDYot6u8wRiFwLrqBcifcCKlMslQXYWxwZJ3 CGuprJ/QN/i2BXbCEwp6eSHJu7+K4RhFavwvy8e9hJEOJGaRE0MOn9fE2+2pMlyKUePJrzoVv76 zhcmQGGBrw8qOrx/byvlbxwlnFbqWqEAzG9w37f4q52mykekd2AS9pHfXt4rFRrQPjesZPLD83C T+4Ad43trgeX0OMyWAHQ2HMFXcGNRQBc+qMTRzvsCPJPIgnss= X-Received: by 2002:a05:6102:a49:b0:5ee:9f71:a17c with SMTP id ada2fe7eead31-5f1a5537b30mr2385435137.14.1768740068795; Sun, 18 Jan 2026 04:41:08 -0800 (PST) Received: from smtp-us-east1-p01-i01-si01.dlp.protect.broadcom.com (address-144-49-247-118.dlp.protect.broadcom.com. [144.49.247.118]) by smtp-relay.gmail.com with ESMTPS id ada2fe7eead31-5f1a6e62eedsm1047310137.5.2026.01.18.04.41.08 for (version=TLS1_2 cipher=ECDHE-ECDSA-AES128-GCM-SHA256 bits=128/128); Sun, 18 Jan 2026 04:41:08 -0800 (PST) X-Relaying-Domain: broadcom.com X-CFilter-Loop: Reflected Received: by mail-pg1-f200.google.com with SMTP id 41be03b00d2f7-c1d27c65670so2258455a12.1 for ; Sun, 18 Jan 2026 04:41:08 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=broadcom.com; s=google; t=1768740067; x=1769344867; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=08UPLl1CQ3fW/3Rx56yHOQP+JfK402Xg4OPflRA1UjM=; b=JjloGdH7JyHGC/qEjQv2xVP/8juhzT3PDioEN+bRWQLB6uSapoH79WUog5m1iKn/D0 TMLBtFx/KJ/xB7mb5JStq0hi8WLQkJwZmDZGM4t0MZXOFfv3dGhcOGjDLE78TxIRHUlz fVqQXcQZxFyIQnkiw9guoo1Ipdw3is4BJeQsE= X-Received: by 2002:a05:6a21:3e15:b0:371:8e6d:27f9 with SMTP id adf61e73a8af0-38deeb9447bmr10547305637.33.1768740067425; Sun, 18 Jan 2026 04:41:07 -0800 (PST) X-Received: by 2002:a05:6a21:3e15:b0:371:8e6d:27f9 with SMTP id adf61e73a8af0-38deeb9447bmr10547290637.33.1768740066984; Sun, 18 Jan 2026 04:41:06 -0800 (PST) Received: from PC-MID-R740.dhcp.broadcom.net ([192.19.234.250]) by smtp.gmail.com with ESMTPSA id 41be03b00d2f7-c5edf249c24sm6761246a12.11.2026.01.18.04.41.03 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sun, 18 Jan 2026 04:41:06 -0800 (PST) From: Pavan Chebbi To: jgg@ziepe.ca, michael.chan@broadcom.com Cc: linux-kernel@vger.kernel.org, dave.jiang@intel.com, saeedm@nvidia.com, Jonathan.Cameron@huawei.com, gospo@broadcom.com, selvin.xavier@broadcom.com, leon@kernel.org, kalesh-anakkur.purayil@broadcom.com, Pavan Chebbi Subject: [PATCH fwctl 4/5] fwctl/bnxt_fwctl: Add bnxt fwctl device Date: Sun, 18 Jan 2026 04:34:00 -0800 Message-Id: <20260118123401.3188438-5-pavan.chebbi@broadcom.com> X-Mailer: git-send-email 2.39.1 In-Reply-To: <20260118123401.3188438-1-pavan.chebbi@broadcom.com> References: <20260118123401.3188438-1-pavan.chebbi@broadcom.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable X-DetectorID-Processed: b00c1d49-9d2e-4205-b15f-d015386d3d5e Content-Type: text/plain; charset="utf-8" Create bnxt_fwctl device. This will bind to bnxt's aux device. On the upper edge, it will register with the fwctl subsystem. It will make use of bnxt's ULP functions to send FW commands. Reviewed-by: Andy Gospodarek Reviewed-by: Dave Jiang Signed-off-by: Pavan Chebbi --- MAINTAINERS | 6 + drivers/fwctl/Kconfig | 11 + drivers/fwctl/Makefile | 1 + drivers/fwctl/bnxt/Makefile | 4 + drivers/fwctl/bnxt/main.c | 416 ++++++++++++++++++++++++++++++++++++ include/uapi/fwctl/bnxt.h | 64 ++++++ include/uapi/fwctl/fwctl.h | 1 + 7 files changed, 503 insertions(+) create mode 100644 drivers/fwctl/bnxt/Makefile create mode 100644 drivers/fwctl/bnxt/main.c create mode 100644 include/uapi/fwctl/bnxt.h diff --git a/MAINTAINERS b/MAINTAINERS index 12f49de7fe03..38acd5d334b2 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -10408,6 +10408,12 @@ L: linux-kernel@vger.kernel.org S: Maintained F: drivers/fwctl/pds/ =20 +FWCTL BNXT DRIVER +M: Pavan Chebbi +L: linux-kernel@vger.kernel.org +S: Maintained +F: drivers/fwctl/bnxt/ + GALAXYCORE GC0308 CAMERA SENSOR DRIVER M: Sebastian Reichel L: linux-media@vger.kernel.org diff --git a/drivers/fwctl/Kconfig b/drivers/fwctl/Kconfig index b5583b12a011..b3795a17f8f2 100644 --- a/drivers/fwctl/Kconfig +++ b/drivers/fwctl/Kconfig @@ -29,5 +29,16 @@ config FWCTL_PDS to access the debug and configuration information of the AMD/Pensando DSC hardware family. =20 + If you don't know what to do here, say N. + +config FWCTL_BNXT + tristate "bnxt control fwctl driver" + depends on BNXT + help + BNXT provides interface for the user process to access the debug and + configuration registers of the Broadcom NIC hardware family. + This will allow configuration and debug tools to work out of the box on + mainstream kernel. + If you don't know what to do here, say N. endif diff --git a/drivers/fwctl/Makefile b/drivers/fwctl/Makefile index c093b5f661d6..fdd46f3a0e4e 100644 --- a/drivers/fwctl/Makefile +++ b/drivers/fwctl/Makefile @@ -2,5 +2,6 @@ obj-$(CONFIG_FWCTL) +=3D fwctl.o obj-$(CONFIG_FWCTL_MLX5) +=3D mlx5/ obj-$(CONFIG_FWCTL_PDS) +=3D pds/ +obj-$(CONFIG_FWCTL_BNXT) +=3D bnxt/ =20 fwctl-y +=3D main.o diff --git a/drivers/fwctl/bnxt/Makefile b/drivers/fwctl/bnxt/Makefile new file mode 100644 index 000000000000..b47172761f1e --- /dev/null +++ b/drivers/fwctl/bnxt/Makefile @@ -0,0 +1,4 @@ +# SPDX-License-Identifier: GPL-2.0 +obj-$(CONFIG_FWCTL_BNXT) +=3D bnxt_fwctl.o + +bnxt_fwctl-y +=3D main.o diff --git a/drivers/fwctl/bnxt/main.c b/drivers/fwctl/bnxt/main.c new file mode 100644 index 000000000000..b7563c27475f --- /dev/null +++ b/drivers/fwctl/bnxt/main.c @@ -0,0 +1,416 @@ +// SPDX-License-Identifier: GPL-2.0 +/* + * Copyright (c) 2026, Broadcom Corporation + */ + +#include +#include +#include +#include +#include +#include +#include +#include + +struct bnxtctl_uctx { + struct fwctl_uctx uctx; + u32 uctx_caps; +}; + +struct bnxtctl_dev { + struct fwctl_device fwctl; + struct bnxt_aux_priv *aux_priv; +}; + +DEFINE_FREE(bnxtctl, struct bnxtctl_dev *, if (_T) fwctl_put(&_T->fwctl)) + +static int bnxtctl_open_uctx(struct fwctl_uctx *uctx) +{ + struct bnxtctl_uctx *bnxtctl_uctx =3D + container_of(uctx, struct bnxtctl_uctx, uctx); + + bnxtctl_uctx->uctx_caps =3D BIT(FWCTL_BNXT_QUERY_COMMANDS) | + BIT(FWCTL_BNXT_SEND_COMMAND); + return 0; +} + +static void bnxtctl_close_uctx(struct fwctl_uctx *uctx) +{ +} + +static void *bnxtctl_info(struct fwctl_uctx *uctx, size_t *length) +{ + struct bnxtctl_uctx *bnxtctl_uctx =3D + container_of(uctx, struct bnxtctl_uctx, uctx); + struct fwctl_info_bnxt *info; + + info =3D kzalloc(sizeof(*info), GFP_KERNEL); + if (!info) + return ERR_PTR(-ENOMEM); + + info->uctx_caps =3D bnxtctl_uctx->uctx_caps; + + *length =3D sizeof(*info); + return info; +} + +static bool bnxtctl_validate_rpc(struct bnxt_en_dev *edev, + struct bnxt_fw_msg *hwrm_in, + enum fwctl_rpc_scope scope) +{ + struct input *req =3D (struct input *)hwrm_in->msg; + + guard(mutex)(&edev->en_dev_lock); + if (edev->flags & BNXT_EN_FLAG_ULP_STOPPED) + return false; + + switch (le16_to_cpu(req->req_type)) { + case HWRM_FUNC_RESET: + case HWRM_PORT_CLR_STATS: + case HWRM_FW_SET_STRUCTURED_DATA: + case HWRM_PORT_PRBS_TEST: + case HWRM_FW_LIVEPATCH: + case HWRM_FW_RESET: + case HWRM_FW_SYNC: + case HWRM_FW_SET_TIME: + case HWRM_MFG_TESTS: + case HWRM_DBG_SERDES_TEST: + case HWRM_DBG_LOG_BUFFER_FLUSH: + case HWRM_DBG_DUMP: + case HWRM_DBG_ERASE_NVM: + case HWRM_DBG_CFG: + case HWRM_DBG_COREDUMP_LIST: + case HWRM_DBG_COREDUMP_INITIATE: + case HWRM_DBG_COREDUMP_RETRIEVE: + case HWRM_DBG_CRASHDUMP_HEADER: + case HWRM_DBG_CRASHDUMP_ERASE: + case HWRM_DBG_PTRACE: + case HWRM_DBG_TOKEN_CFG: + case HWRM_NVM_DEFRAG: + case HWRM_NVM_FACTORY_DEFAULTS: + case HWRM_NVM_FLUSH: + case HWRM_NVM_INSTALL_UPDATE: + case HWRM_NVM_MODIFY: + case HWRM_NVM_VERIFY_UPDATE: + case HWRM_NVM_ERASE_DIR_ENTRY: + case HWRM_NVM_MOD_DIR_ENTRY: + case HWRM_NVM_FIND_DIR_ENTRY: + case HWRM_NVM_RAW_DUMP: + return scope >=3D FWCTL_RPC_CONFIGURATION; + + case HWRM_VER_GET: + case HWRM_FW_GET_STRUCTURED_DATA: + case HWRM_ERROR_RECOVERY_QCFG: + case HWRM_FUNC_QCAPS: + case HWRM_FUNC_QCFG: + case HWRM_FUNC_QSTATS: + case HWRM_PORT_QSTATS: + case HWRM_PORT_PHY_QCFG: + case HWRM_PORT_MAC_QCFG: + case HWRM_PORT_PHY_QCAPS: + case HWRM_PORT_PHY_I2C_READ: + case HWRM_PORT_PHY_MDIO_READ: + case HWRM_QUEUE_PRI2COS_QCFG: + case HWRM_QUEUE_COS2BW_QCFG: + case HWRM_QUEUE_DSCP2PRI_QCFG: + case HWRM_VNIC_RSS_QCFG: + case HWRM_QUEUE_GLOBAL_QCFG: + case HWRM_QUEUE_ADPTV_QOS_RX_FEATURE_QCFG: + case HWRM_QUEUE_ADPTV_QOS_TX_FEATURE_QCFG: + case HWRM_QUEUE_QCAPS: + case HWRM_QUEUE_ADPTV_QOS_RX_TUNING_QCFG: + case HWRM_QUEUE_ADPTV_QOS_TX_TUNING_QCFG: + case HWRM_TUNNEL_DST_PORT_QUERY: + case HWRM_PORT_QSTATS_EXT: + case HWRM_PORT_TX_FIR_QCFG: + case HWRM_FW_LIVEPATCH_QUERY: + case HWRM_FW_QSTATUS: + case HWRM_FW_HEALTH_CHECK: + case HWRM_FW_GET_TIME: + case HWRM_PORT_DSC_DUMP: + case HWRM_PORT_EP_TX_QCFG: + case HWRM_PORT_QCFG: + case HWRM_PORT_MAC_QCAPS: + case HWRM_TEMP_MONITOR_QUERY: + case HWRM_REG_POWER_QUERY: + case HWRM_CORE_FREQUENCY_QUERY: + case HWRM_STAT_QUERY_ROCE_STATS: + case HWRM_STAT_QUERY_ROCE_STATS_EXT: + case HWRM_CFA_REDIRECT_QUERY_TUNNEL_TYPE: + case HWRM_CFA_FLOW_INFO: + case HWRM_CFA_ADV_FLOW_MGNT_QCAPS: + case HWRM_FUNC_RESOURCE_QCAPS: + case HWRM_FUNC_BACKING_STORE_QCAPS: + case HWRM_FUNC_BACKING_STORE_QCFG: + case HWRM_FUNC_QSTATS_EXT: + case HWRM_FUNC_PTP_PIN_QCFG: + case HWRM_FUNC_PTP_EXT_QCFG: + case HWRM_FUNC_BACKING_STORE_QCFG_V2: + case HWRM_FUNC_BACKING_STORE_QCAPS_V2: + case HWRM_FUNC_SYNCE_QCFG: + case HWRM_FUNC_TTX_PACING_RATE_PROF_QUERY: + case HWRM_PCIE_QSTATS: + case HWRM_MFG_OTP_QCFG: + case HWRM_MFG_FRU_EEPROM_READ: + case HWRM_MFG_GET_NVM_MEASUREMENT: + case HWRM_STAT_GENERIC_QSTATS: + case HWRM_PORT_PHY_FDRSTAT: + case HWRM_QUEUE_ADPTV_QOS_RX_QCFG: + case HWRM_QUEUE_ADPTV_QOS_TX_QCFG: + case HWRM_DBG_READ_DIRECT: + case HWRM_DBG_READ_INDIRECT: + case HWRM_DBG_RING_INFO_GET: + case HWRM_DBG_QCAPS: + case HWRM_DBG_QCFG: + case HWRM_DBG_USEQ_FLUSH: + case HWRM_DBG_USEQ_QCAPS: + case HWRM_DBG_SIM_CABLE_STATE: + case HWRM_DBG_TOKEN_QUERY_AUTH_IDS: + case HWRM_NVM_GET_VARIABLE: + case HWRM_NVM_GET_DEV_INFO: + case HWRM_NVM_GET_DIR_ENTRIES: + case HWRM_NVM_GET_DIR_INFO: + case HWRM_NVM_READ: + case HWRM_SELFTEST_QLIST: + case HWRM_SELFTEST_RETRIEVE_SERDES_DATA: + return scope >=3D FWCTL_RPC_DEBUG_READ_ONLY; + + case HWRM_PORT_PHY_I2C_WRITE: + case HWRM_MFG_FRU_WRITE_CONTROL: + case HWRM_MFG_FRU_EEPROM_WRITE: + case HWRM_DBG_WRITE_DIRECT: + case HWRM_NVM_SET_VARIABLE: + case HWRM_NVM_WRITE: + case HWRM_NVM_RAW_WRITE_BLK: + case HWRM_PORT_PHY_MDIO_WRITE: + return scope >=3D FWCTL_RPC_DEBUG_WRITE; + + default: + return false; + } +} + +static int bnxt_fw_setup_input_dma(struct bnxtctl_dev *bnxt_dev, + struct device *dev, + struct fwctl_dma_info_bnxt *msg, + struct bnxt_fw_msg *fw_msg, + int num_dma, + void **dma_virt_addr, + dma_addr_t *dma_addr) +{ + struct fwctl_dma_info_bnxt *dma_buf =3D msg; + u8 i, num_allocated =3D 0; + void *dma_ptr; + int rc; + + for (i =3D 0; i < num_dma; i++) { + if (msg->len =3D=3D 0 || msg->len > MAX_DMA_MEM_SIZE) { + rc =3D -EINVAL; + goto err; + } + dma_virt_addr[i] =3D dma_alloc_coherent(dev->parent, msg->len, + &dma_addr[i], GFP_KERNEL); + if (!dma_virt_addr[i]) { + rc =3D -ENOMEM; + goto err; + } + num_allocated++; + if (msg->dma_direction =3D=3D DEVICE_WRITE) { + if (copy_from_user(dma_virt_addr[i], + u64_to_user_ptr(msg->data), + msg->len)) { + rc =3D -EFAULT; + goto err; + } + } + dma_ptr =3D fw_msg->msg + msg->offset; + + if (!(PTR_ALIGN(dma_ptr, 8) =3D=3D dma_ptr) || + msg->offset >=3D fw_msg->msg_len) { + rc =3D -EINVAL; + goto err; + } + + *(__le64 *)(dma_ptr) =3D cpu_to_le64(dma_addr[i]); + msg++; + } + + return 0; +err: + for (i =3D 0; i < num_allocated; i++) + dma_free_coherent(dev->parent, dma_buf[i].len, + dma_virt_addr[i], dma_addr[i]); + + return rc; +} + +static void *bnxtctl_fw_rpc(struct fwctl_uctx *uctx, + enum fwctl_rpc_scope scope, + void *in, size_t in_len, size_t *out_len) +{ + struct bnxtctl_dev *bnxtctl =3D + container_of(uctx->fwctl, struct bnxtctl_dev, fwctl); + struct bnxt_aux_priv *bnxt_aux_priv =3D bnxtctl->aux_priv; + void *dma_virt_addr[MAX_NUM_DMA_INDICATIONS]; + dma_addr_t dma_addr[MAX_NUM_DMA_INDICATIONS]; + struct fwctl_dma_info_bnxt *dma_buf =3D NULL; + struct device *dev =3D &uctx->fwctl->dev; + struct fwctl_rpc_bnxt *msg =3D in; + struct bnxt_fw_msg rpc_in; + int i, rc, err =3D 0; + + rpc_in.msg =3D memdup_user(u64_to_user_ptr(msg->req), msg->req_len); + if (IS_ERR(rpc_in.msg)) + return rpc_in.msg; + + if (!bnxtctl_validate_rpc(bnxt_aux_priv->edev, &rpc_in, scope)) { + err =3D -EPERM; + goto free_msg_out; + } + + rpc_in.msg_len =3D msg->req_len; + rpc_in.resp =3D kzalloc(*out_len, GFP_KERNEL); + if (!rpc_in.resp) { + err =3D -ENOMEM; + goto free_msg_out; + } + + rpc_in.resp_max_len =3D *out_len; + if (!msg->timeout) + rpc_in.timeout =3D DFLT_HWRM_CMD_TIMEOUT; + else + rpc_in.timeout =3D msg->timeout; + + if (msg->num_dma) { + if (msg->num_dma > MAX_NUM_DMA_INDICATIONS) { + dev_err(dev, "DMA buffers exceed the number supported\n"); + err =3D -EINVAL; + goto free_msg_out; + } + + dma_buf =3D kcalloc(msg->num_dma, sizeof(*dma_buf), GFP_KERNEL); + if (!dma_buf) { + err =3D -ENOMEM; + goto free_msg_out; + } + + if (copy_from_user(dma_buf, u64_to_user_ptr(msg->payload), + msg->num_dma * sizeof(*dma_buf))) { + dev_dbg(dev, "Failed to copy payload from user\n"); + err =3D -EFAULT; + goto free_dmabuf_out; + } + + err =3D bnxt_fw_setup_input_dma(bnxtctl, dev, dma_buf, &rpc_in, + msg->num_dma, &dma_virt_addr[0], + &dma_addr[0]); + if (err) + goto free_dmabuf_out; + } + + rc =3D bnxt_send_msg(bnxt_aux_priv->edev, &rpc_in); + if (rc) { + struct output *resp =3D rpc_in.resp; + + /* Copy the response to user always, as it contains + * detailed status of the command failure + */ + if (!resp->error_code) + /* bnxt_send_msg() returned much before FW + * received the command. + */ + resp->error_code =3D rc; + + goto free_dma_out; + } + + for (i =3D 0; i < msg->num_dma; i++) { + if (dma_buf[i].dma_direction !=3D DEVICE_READ) + continue; + if (copy_to_user(u64_to_user_ptr(dma_buf[i].data), + dma_virt_addr[i], dma_buf[i].len)) { + dev_dbg(dev, "Failed to copy resp to user\n"); + err =3D -EFAULT; + break; + } + } +free_dma_out: + /* Cleanup any dma memory that bnxt_fw_setup_input_dma() may have + * allocated + */ + for (i =3D 0; i < msg->num_dma; i++) + dma_free_coherent(dev->parent, dma_buf[i].len, dma_virt_addr[i], + dma_addr[i]); +free_dmabuf_out: + kfree(dma_buf); +free_msg_out: + kfree(rpc_in.msg); + + if (err) { + kfree(rpc_in.resp); + return ERR_PTR(err); + } + + return rpc_in.resp; +} + +static const struct fwctl_ops bnxtctl_ops =3D { + .device_type =3D FWCTL_DEVICE_TYPE_BNXT, + .uctx_size =3D sizeof(struct bnxtctl_uctx), + .open_uctx =3D bnxtctl_open_uctx, + .close_uctx =3D bnxtctl_close_uctx, + .info =3D bnxtctl_info, + .fw_rpc =3D bnxtctl_fw_rpc, +}; + +static int bnxtctl_probe(struct auxiliary_device *adev, + const struct auxiliary_device_id *id) +{ + struct bnxt_aux_priv *aux_priv =3D + container_of(adev, struct bnxt_aux_priv, aux_dev); + struct bnxtctl_dev *bnxtctl __free(bnxtctl) =3D + fwctl_alloc_device(&aux_priv->edev->pdev->dev, &bnxtctl_ops, + struct bnxtctl_dev, fwctl); + int rc; + + if (!bnxtctl) + return -ENOMEM; + + bnxtctl->aux_priv =3D aux_priv; + + rc =3D fwctl_register(&bnxtctl->fwctl); + if (rc) + return rc; + + auxiliary_set_drvdata(adev, no_free_ptr(bnxtctl)); + return 0; +} + +static void bnxtctl_remove(struct auxiliary_device *adev) +{ + struct bnxtctl_dev *ctldev =3D auxiliary_get_drvdata(adev); + + fwctl_unregister(&ctldev->fwctl); + fwctl_put(&ctldev->fwctl); +} + +static const struct auxiliary_device_id bnxtctl_id_table[] =3D { + { .name =3D "bnxt_en.fwctl", }, + {} +}; +MODULE_DEVICE_TABLE(auxiliary, bnxtctl_id_table); + +static struct auxiliary_driver bnxtctl_driver =3D { + .name =3D "bnxt_fwctl", + .probe =3D bnxtctl_probe, + .remove =3D bnxtctl_remove, + .id_table =3D bnxtctl_id_table, +}; + +module_auxiliary_driver(bnxtctl_driver); + +MODULE_IMPORT_NS("FWCTL"); +MODULE_DESCRIPTION("BNXT fwctl driver"); +MODULE_AUTHOR("Pavan Chebbi "); +MODULE_AUTHOR("Andy Gospodarek "); +MODULE_LICENSE("GPL"); diff --git a/include/uapi/fwctl/bnxt.h b/include/uapi/fwctl/bnxt.h new file mode 100644 index 000000000000..5620812d839c --- /dev/null +++ b/include/uapi/fwctl/bnxt.h @@ -0,0 +1,64 @@ +/* SPDX-License-Identifier: GPL-2.0 WITH Linux-syscall-note */ +/* + * Copyright (c) 2026, Broadcom Inc + */ + +#ifndef _UAPI_FWCTL_BNXT_H_ +#define _UAPI_FWCTL_BNXT_H_ + +#include + +#define MAX_DMA_MEM_SIZE 0x10000 /*64K*/ +#define DFLT_HWRM_CMD_TIMEOUT 500 +#define DEVICE_WRITE 0 +#define DEVICE_READ 1 + +enum fwctl_bnxt_commands { + FWCTL_BNXT_QUERY_COMMANDS =3D 0, + FWCTL_BNXT_SEND_COMMAND, +}; + +/** + * struct fwctl_info_bnxt - ioctl(FWCTL_INFO) out_device_data + * @uctx_caps: The command capabilities driver accepts. + * + * Return basic information about the FW interface available. + */ +struct fwctl_info_bnxt { + __u32 uctx_caps; +}; + +#define MAX_NUM_DMA_INDICATIONS 10 + +/** + * struct fwctl_dma_info_bnxt - describe the buffer that should be DMAed + * @data: DMA-intended buffer + * @len: length of the @data + * @offset: offset at which FW (HWRM) input structure needs DMA address + * @dma_direction: DMA direction, DEVICE_READ or DEVICE_WRITE + * @unused: pad + */ +struct fwctl_dma_info_bnxt { + __aligned_u64 data; + __u32 len; + __u16 offset; + __u8 dma_direction; + __u8 unused; +}; + +/** + * struct fwctl_rpc_bnxt - describe the fwctl message for bnxt + * @req: FW (HWRM) command input structure + * @req_len: length of @req + * @timeout: if the user wants to override the driver's default, 0 otherwi= se + * @num_dma: number of DMA buffers to be added to @req + * @payload: DMA buffer details in struct fwctl_dma_info_bnxt format + */ +struct fwctl_rpc_bnxt { + __aligned_u64 req; + __u32 req_len; + __u32 timeout; + __u32 num_dma; + __aligned_u64 payload; +}; +#endif diff --git a/include/uapi/fwctl/fwctl.h b/include/uapi/fwctl/fwctl.h index 716ac0eee42d..2d6d4049c205 100644 --- a/include/uapi/fwctl/fwctl.h +++ b/include/uapi/fwctl/fwctl.h @@ -44,6 +44,7 @@ enum fwctl_device_type { FWCTL_DEVICE_TYPE_ERROR =3D 0, FWCTL_DEVICE_TYPE_MLX5 =3D 1, FWCTL_DEVICE_TYPE_CXL =3D 2, + FWCTL_DEVICE_TYPE_BNXT =3D 3, FWCTL_DEVICE_TYPE_PDS =3D 4, }; =20 --=20 2.39.1 From nobody Sun Feb 8 06:56:35 2026 Received: from mail-qv1-f99.google.com (mail-qv1-f99.google.com [209.85.219.99]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id CEA7432C31B for ; Sun, 18 Jan 2026 12:41:17 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.219.99 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1768740079; cv=none; b=jYpea9NrT5nHol9o5p7Gs0oI6uX7H8aewDaIajcS9fEVL/KYdtqglscuKLjdrw3vWw1SU1Rep5mg4FkZaz8c9S1va+g3GvOjcprAazYrEZ4VCbU1ADO2oWD8dupbBvYeuIShHAmGm3Pv9YBDq19GeXCyViJPhy7j0ibKsC6khdE= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1768740079; c=relaxed/simple; bh=p0+9v6bOKj6/t1LScVQUgu9L7VXJ0bMnHc+ppDkUBOc=; h=From:To:Cc:Subject:Date:Message-Id:In-Reply-To:References: MIME-Version; b=pSrkyA+2jRRdSrEQ6pqaj3Gz+fhk/nd7ec0mCHn9mucfUxn2yctGklZxJ5UU1nGfRIA9PkeuT0iy3fWFozA7snAgXqIvHThuXLN6oHI2JrsMgZ8hohLSVBVMUsgqu/cwlcNCRXc9GZSaYhcAHFfckxv+XMat3bJPBBvzWW2J0N4= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=broadcom.com; spf=fail smtp.mailfrom=broadcom.com; dkim=pass (1024-bit key) header.d=broadcom.com header.i=@broadcom.com header.b=STMI2sFK; arc=none smtp.client-ip=209.85.219.99 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=broadcom.com Authentication-Results: smtp.subspace.kernel.org; spf=fail smtp.mailfrom=broadcom.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=broadcom.com header.i=@broadcom.com header.b="STMI2sFK" Received: by mail-qv1-f99.google.com with SMTP id 6a1803df08f44-88a2ad13c24so31973036d6.1 for ; Sun, 18 Jan 2026 04:41:17 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1768740077; x=1769344877; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:dkim-signature:x-gm-gg :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=ylrqotwPOThrY3/hhjb9VMXvSLNMw+ZG4RF4c6zLl9k=; b=Sk37TAi8hRV6xSTBTSlGlAZ5ZkK924WjyIfZnGvViA9ybhYlScbnf9AnBmrZsZL9yN 46q+lUd/RY5PLdroKcfusW4qW8mfBl+TnwpNGqI7H9mtvB3UJmFmSyMHu+zEXwkhTf5y Cb/Ec+nwGKh1WxNYKyENI3EzOFhnd5nDhTADmVgzM+/YWjlv41lOMUVbiudxJohW69py Fxiag6m7dLX8gudyxoOt3D1CZTyD5hY1cRZ1/q1JPNexbkccCBsUSWNkRoViAd+0fpeU lChcb0SkOAv4Z+nzn2E7XakgU1osPcvJ2/RwV50Ee8ExoLcd4D9UF/kk8I2BCI2qOWYW 1IZQ== X-Gm-Message-State: AOJu0YxiaW6D7ZMXR3WuBCuk9VvvxxZNpBLero0jC1DEsVDc+0pioBeH sK4sJdcNuK/2dbiG/Gb9tf4ULQiW2vwcrOWeaTnCcopCY9VheBjZywl0wylMnK3F3Tw+I70sH4V ZWldMGzCnCcqAJtxFHQDsT0cZJljTtZb8kWx5Y3pkxlOpppIGP+8PHt7YnSPt2l8RCcdiRAQRi6 FR2at7VVtJb+BfutH2P+J/zj3EIjU34Rp12rEXNRUBLXyiHJ2we5UdY02aM7jiZHT2s0+rF0nGS 3qckQyxGvBP5bY+2yw= X-Gm-Gg: AY/fxX7tQWJO0ijbEJPHbCqW5ojiQdBbGV+Wh8W6dTrpua0suoTUux3zrDbDNkI+WXj fcpArPSXTaeqIIgy2P1YE89M1le2Uoau7PiZy/JBMKr/VIXcmK8+oQ+T/NItSJCk1xlD65FGO7Y 9b42buAKuARz1Z5HR4m0VTaAri+GtmBHFglhOHAEmOKIAgCwnFl83cZsu9yqIZKW7VNowZbttac pU5Fv4X/Zzx2GVV2y9BQEx+zVhzFnDsSCPf44wNicPsPyFJHozy1JNf0cnGvB1xXimTFs1grfc5 lLPbooN4xW+3clhZLtgDjPNHcRDY+cVCRpf4b7Jcs1PqNx5s3pKEYzcl23P5hv5Ord5XEOXNZMS gLa0czPs4RjRjOHkI5q//L75G6XNI6N1wTlcFPgNdlLiOpIMbqAK2H7ORBI0TNVV0Ut3/fbySOR qpGqqDe3aWxsFqXWK6mhjkQ3o1b+h9O+OC9XxZbPTOqF2sFDw= X-Received: by 2002:a05:6214:76e:b0:88f:ca7d:348c with SMTP id 6a1803df08f44-8942dbed6bcmr115557206d6.14.1768740076708; Sun, 18 Jan 2026 04:41:16 -0800 (PST) Received: from smtp-us-east1-p01-i01-si01.dlp.protect.broadcom.com (address-144-49-247-118.dlp.protect.broadcom.com. [144.49.247.118]) by smtp-relay.gmail.com with ESMTPS id 6a1803df08f44-8942e6cd881sm5292156d6.18.2026.01.18.04.41.16 for (version=TLS1_2 cipher=ECDHE-ECDSA-AES128-GCM-SHA256 bits=128/128); Sun, 18 Jan 2026 04:41:16 -0800 (PST) X-Relaying-Domain: broadcom.com X-CFilter-Loop: Reflected Received: by mail-pj1-f71.google.com with SMTP id 98e67ed59e1d1-34e70e2e363so2648579a91.1 for ; Sun, 18 Jan 2026 04:41:16 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=broadcom.com; s=google; t=1768740075; x=1769344875; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=ylrqotwPOThrY3/hhjb9VMXvSLNMw+ZG4RF4c6zLl9k=; b=STMI2sFKYtEYl2EcOafCCnigFJYuzTkGVruD9SYYJNtG5u6n0P+57QLu9jDs4Ljomz dvEcIWUH+O7tDH1vQE/vufhCYZDGPPy84lxDg+M/cKmyXKIM+QLzyF3tCn/0LMrMLUjl v/eszlVXL0RclPLQD95xqrIAbjSkcr0fPA944= X-Received: by 2002:a05:6a21:3a85:b0:366:14ac:e200 with SMTP id adf61e73a8af0-38dfe7b7c0bmr7695685637.62.1768740075387; Sun, 18 Jan 2026 04:41:15 -0800 (PST) X-Received: by 2002:a05:6a21:3a85:b0:366:14ac:e200 with SMTP id adf61e73a8af0-38dfe7b7c0bmr7695673637.62.1768740075002; Sun, 18 Jan 2026 04:41:15 -0800 (PST) Received: from PC-MID-R740.dhcp.broadcom.net ([192.19.234.250]) by smtp.gmail.com with ESMTPSA id 41be03b00d2f7-c5edf249c24sm6761246a12.11.2026.01.18.04.41.11 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sun, 18 Jan 2026 04:41:14 -0800 (PST) From: Pavan Chebbi To: jgg@ziepe.ca, michael.chan@broadcom.com Cc: linux-kernel@vger.kernel.org, dave.jiang@intel.com, saeedm@nvidia.com, Jonathan.Cameron@huawei.com, gospo@broadcom.com, selvin.xavier@broadcom.com, leon@kernel.org, kalesh-anakkur.purayil@broadcom.com, Pavan Chebbi Subject: [PATCH fwctl 5/5] fwctl/bnxt_fwctl: Add documentation entries Date: Sun, 18 Jan 2026 04:34:01 -0800 Message-Id: <20260118123401.3188438-6-pavan.chebbi@broadcom.com> X-Mailer: git-send-email 2.39.1 In-Reply-To: <20260118123401.3188438-1-pavan.chebbi@broadcom.com> References: <20260118123401.3188438-1-pavan.chebbi@broadcom.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable X-DetectorID-Processed: b00c1d49-9d2e-4205-b15f-d015386d3d5e Content-Type: text/plain; charset="utf-8" Add bnxt_fwctl to the driver and fwctl documentation pages. Reviewed-by: Andy Gospodarek Reviewed-by: Dave Jiang Signed-off-by: Pavan Chebbi --- .../userspace-api/fwctl/bnxt_fwctl.rst | 83 +++++++++++++++++++ Documentation/userspace-api/fwctl/fwctl.rst | 1 + Documentation/userspace-api/fwctl/index.rst | 1 + 3 files changed, 85 insertions(+) create mode 100644 Documentation/userspace-api/fwctl/bnxt_fwctl.rst diff --git a/Documentation/userspace-api/fwctl/bnxt_fwctl.rst b/Documentati= on/userspace-api/fwctl/bnxt_fwctl.rst new file mode 100644 index 000000000000..41a74a085324 --- /dev/null +++ b/Documentation/userspace-api/fwctl/bnxt_fwctl.rst @@ -0,0 +1,83 @@ +.. SPDX-License-Identifier: GPL-2.0 + +=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D +fwctl bnxt driver +=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D + +:Author: Pavan Chebbi + +Overview +=3D=3D=3D=3D=3D=3D=3D=3D + +BNXT driver makes a fwctl service available through an auxiliary_device. +The bnxt_fwctl driver binds to this device and registers itself with the +fwctl subsystem. + +The bnxt_fwctl driver is agnostic to the device firmware internals. It +uses the Upper Layer Protocol (ULP) conduit provided by bnxt to send +HardWare Resource Manager (HWRM) commands to firmware. + +These commands can query or change firmware driven device configurations +and read/write registers that are useful for debugging. + +bnxt_fwctl User API +=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D + +Each RPC request contains a message request structure (HWRM input), +its length, optional request timeout, and dma buffers' information +if the command needs any DMA. The request is then put together with +the request data and sent through bnxt's message queue to the firmware, +and the results are returned to the caller. + +A typical user application can send a FWCTL_INFO command using ioctl() +to discover bnxt_fwctl's RPC capabilities as shown below: + + ioctl(fd, FWCTL_INFO, &fwctl_info_msg); + +where fwctl_info_msg (of type struct fwctl_info) describes bnxt_info_msg +(of type struct fwctl_info_bnxt). fwctl_info_msg is set up as follows: + + size =3D sizeof(struct fwctl_info); + flags =3D 0; + device_data_len =3D sizeof(bnxt_info_msg); + out_device_data =3D (__aligned_u64)&bnxt_info_msg; + +The uctx_caps of bnxt_info_msg represents the capabilities as described +in fwctl_bnxt_commands of include/uapi/fwctl/bnxt.h + +The FW RPC itself, FWCTL_RPC can be sent using ioctl() as: + + ioctl(fd, FWCTL_RPC, &fwctl_rpc_msg); + +where fwctl_rpc_msg (of type struct fwctl_rpc) encapsulates fwctl_rpc_bnxt +(see bnxt_rpc_msg below). fwctl_rpc_bnxt members are set up as per the +requirements of specific HWRM commands described in include/bnxt/hsi.h. +An example for HWRM_VER_GET is shown below: + + struct fwctl_rpc_bnxt bnxt_rpc_msg; + struct hwrm_ver_get_output resp; + struct fwctl_rpc fwctl_rpc_msg; + struct hwrm_ver_get_input req; + + req.req_type =3D HWRM_VER_GET; + req.hwrm_intf_maj =3D HWRM_VERSION_MAJOR; + req.hwrm_intf_min =3D HWRM_VERSION_MINOR; + req.hwrm_intf_upd =3D HWRM_VERSION_UPDATE; + req.cmpl_ring =3D -1; + req.target_id =3D -1; + + bnxt_rpc_msg.req_len =3D sizeof(struct hwrm_ver_get_input); + bnxt_rpc_msg.num_dma =3D 0; + bnxt_rpc_msg.req =3D (__aligned_u64)&req; + + fwctl_rpc_msg.size =3D sizeof(struct fwctl_rpc); + fwctl_rpc_msg.scope =3D FWCTL_RPC_DEBUG_READ_ONLY; + fwctl_rpc_msg.in_len =3D sizeof(bnxt_rpc_msg) + sizeof(req); + fwctl_rpc_msg.out_len =3D sizeof(struct hwrm_ver_get_output); + fwctl_rpc_msg.in =3D (__aligned_u64)&bnxt_rpc_msg; + fwctl_rpc_msg.out =3D (__aligned_u64)&resp; + +An example python3 program that can exercise this interface can be found in +the following git repository: + +https://github.com/Broadcom/fwctl-tools diff --git a/Documentation/userspace-api/fwctl/fwctl.rst b/Documentation/us= erspace-api/fwctl/fwctl.rst index a74eab8d14c6..826817bfd54d 100644 --- a/Documentation/userspace-api/fwctl/fwctl.rst +++ b/Documentation/userspace-api/fwctl/fwctl.rst @@ -148,6 +148,7 @@ area resulting in clashes will be resolved in favour of= a kernel implementation. fwctl User API =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D =20 +.. kernel-doc:: include/uapi/fwctl/bnxt.h .. kernel-doc:: include/uapi/fwctl/fwctl.h .. kernel-doc:: include/uapi/fwctl/mlx5.h .. kernel-doc:: include/uapi/fwctl/pds.h diff --git a/Documentation/userspace-api/fwctl/index.rst b/Documentation/us= erspace-api/fwctl/index.rst index 316ac456ad3b..8062f7629654 100644 --- a/Documentation/userspace-api/fwctl/index.rst +++ b/Documentation/userspace-api/fwctl/index.rst @@ -10,5 +10,6 @@ to securely construct and execute RPCs inside device firm= ware. :maxdepth: 1 =20 fwctl + bnxt_fwctl fwctl-cxl pds_fwctl --=20 2.39.1