From dcb2a1ae46ad4a5b810203abcbf5ddebbfc1741d Mon Sep 17 00:00:00 2001 From: John Baldwin Date: Mon, 28 Apr 2025 13:06:07 -0400 Subject: [PATCH] : Conditionally export table of ID names Only export the array of ID names if either _WANT_SFF_8024_ID or _WANT_SFF_8472_ID is defined. Exporting them unconditionally can trigger unused variable warnings if a consumer doesn't use the array. Reviewed by: olce, bz, brooks Differential Revision: https://reviews.freebsd.org/D49955 --- lib/libifconfig/libifconfig_sfp.c | 2 ++ sys/dev/mlx5/mlx5_core/mlx5_diagnostics.c | 2 ++ sys/net/sff8472.h | 2 ++ usr.sbin/cxgbetool/cxgbetool.c | 2 ++ 4 files changed, 8 insertions(+) diff --git a/lib/libifconfig/libifconfig_sfp.c b/lib/libifconfig/libifconfig_sfp.c index 8292135d3e4..1ba6d231a99 100644 --- a/lib/libifconfig/libifconfig_sfp.c +++ b/lib/libifconfig/libifconfig_sfp.c @@ -24,6 +24,8 @@ * SUCH DAMAGE. */ +#define _WANT_SFF_8024_ID + #include #include #include diff --git a/sys/dev/mlx5/mlx5_core/mlx5_diagnostics.c b/sys/dev/mlx5/mlx5_core/mlx5_diagnostics.c index 182be547272..9730ab3c56c 100644 --- a/sys/dev/mlx5/mlx5_core/mlx5_diagnostics.c +++ b/sys/dev/mlx5/mlx5_core/mlx5_diagnostics.c @@ -26,6 +26,8 @@ #include "opt_rss.h" #include "opt_ratelimit.h" +#define _WANT_SFF_8024_ID + #include #include #include diff --git a/sys/net/sff8472.h b/sys/net/sff8472.h index a33dfed84b0..33b6dddb5b9 100644 --- a/sys/net/sff8472.h +++ b/sys/net/sff8472.h @@ -415,6 +415,7 @@ enum { SFF_8024_ID_LAST = SFF_8024_ID_QSFP_CMIS }; +#if defined(_WANT_SFF_8024_ID) || defined(_WANT_SFF_8472_ID) static const char *sff_8024_id[SFF_8024_ID_LAST + 1] = { "Unknown", "GBIC", @@ -448,6 +449,7 @@ static const char *sff_8024_id[SFF_8024_ID_LAST + 1] = { "x8MiniLink", "QSFP+(CIMS)" }; +#endif /* Keep compatibility with old definitions */ #define SFF_8472_ID_UNKNOWN SFF_8024_ID_UNKNOWN diff --git a/usr.sbin/cxgbetool/cxgbetool.c b/usr.sbin/cxgbetool/cxgbetool.c index 8f58fe8107d..c3bd883b39f 100644 --- a/usr.sbin/cxgbetool/cxgbetool.c +++ b/usr.sbin/cxgbetool/cxgbetool.c @@ -25,6 +25,8 @@ * SUCH DAMAGE. */ +#define _WANT_SFF_8472_ID + #include #include #include