On Thu, 23 Apr 2026 11:40:23 +0800
Zaiyu Wang <[email protected]> wrote:

> The original I2C access flow in the module information retrieval
> process was flawed. Correct the implementation to properly fetch
> module info.
> 
> Cc: [email protected]
> 
> Signed-off-by: Zaiyu Wang <[email protected]>

Build errors. Must fix.
clang -Idrivers/libtmp_rte_net_txgbe.a.p -Idrivers -I../drivers 
-Idrivers/net/txgbe -I../drivers/net/txgbe -Idrivers/net/txgbe/base 
-I../drivers/net/txgbe/base -Ilib/ethdev -I../lib/ethdev -Ilib/eal/common 
-I../lib/eal/common -I. -I.. -Iconfig -I../config -Ilib/eal/include 
-I../lib/eal/include -Ilib/eal/linux/include -I../lib/eal/linux/include 
-Ilib/eal/x86/include -I../lib/eal/x86/include -I../kernel/linux -Ilib/eal 
-I../lib/eal -Ilib/kvargs -I../lib/kvargs -Ilib/log -I../lib/log -Ilib/metrics 
-I../lib/metrics -Ilib/telemetry -I../lib/telemetry -Ilib/argparse 
-I../lib/argparse -Ilib/net -I../lib/net -Ilib/mbuf -I../lib/mbuf -Ilib/mempool 
-I../lib/mempool -Ilib/ring -I../lib/ring -Ilib/meter -I../lib/meter 
-Idrivers/bus/pci -I../drivers/bus/pci -I../drivers/bus/pci/linux -Ilib/pci 
-I../lib/pci -Idrivers/bus/vdev -I../drivers/bus/vdev -Ilib/hash -I../lib/hash 
-Ilib/rcu -I../lib/rcu -Ilib/security -I../lib/security -Ilib/cryptodev 
-I../lib/cryptodev -Xclang -fcolor-diagnostics -pipe -D
 _FILE_OFFSET_BITS=64 -Wall -Winvalid-pch -Wextra -Werror -std=c11 -O3 -include 
rte_config.h -Wvla -Wcast-qual -Wcomma -Wdeprecated -Wformat 
-Wformat-nonliteral -Wformat-security -Wmissing-declarations 
-Wmissing-prototypes -Wnested-externs -Wold-style-definition -Wpointer-arith 
-Wshadow -Wsign-compare -Wstrict-prototypes -Wundef -Wwrite-strings 
-Wno-missing-field-initializers -D_GNU_SOURCE -fPIC -march=native -mrtm 
-DALLOW_EXPERIMENTAL_API -DALLOW_INTERNAL_API -Wno-format-truncation 
-Wno-address-of-packed-member -Wno-vla -Wno-shadow 
-DRTE_LOG_DEFAULT_LOGTYPE=pmd.net.txgbe -DRTE_ANNOTATE_LOCKS -Wthread-safety 
-MD -MQ drivers/libtmp_rte_net_txgbe.a.p/net_txgbe_base_txgbe_e56_bp.c.o -MF 
drivers/libtmp_rte_net_txgbe.a.p/net_txgbe_base_txgbe_e56_bp.c.o.d -o 
drivers/libtmp_rte_net_txgbe.a.p/net_txgbe_base_txgbe_e56_bp.c.o -c 
../drivers/net/txgbe/base/txgbe_e56_bp.c
../drivers/net/txgbe/base/txgbe_e56_bp.c:352:13: error: variable 
'CMVAR_SEC_LOW_TH' is used uninitialized whenever 'if' condition is false 
[-Werror,-Wsometimes-uninitialized]
352 |         } else if (speed == 25) {
|                    ^~~~~~~~~~~
../drivers/net/txgbe/base/txgbe_e56_bp.c:376:21: note: uninitialized use occurs 
here
376 |         if (SECOND_CODE <= CMVAR_SEC_LOW_TH) {
|                            ^~~~~~~~~~~~~~~~
../drivers/net/txgbe/base/txgbe_e56_bp.c:352:9: note: remove the 'if' if its 
condition is always true
352 |         } else if (speed == 25) {
|                ^~~~~~~~~~~~~~~~
../drivers/net/txgbe/base/txgbe_e56_bp.c:322:22: note: initialize the variable 
'CMVAR_SEC_LOW_TH' to silence this warning
322 |         int CMVAR_SEC_LOW_TH;
|                             ^
|                              = 0
1 error generated.

Reply via email to