net: stmmac: Avoid one more sometimes uninitialized Clang warning
[ Upstream commit 1f5d861f7fefa971b2c6e766f77932c86419a319 ] When building with -Wsometimes-uninitialized, Clang warns: drivers/net/ethernet/stmicro/stmmac/stmmac_ptp.c:111:2: error: variable 'ns' is used uninitialized whenever 'if' condition is false [-Werror,-Wsometimes-uninitialized] drivers/net/ethernet/stmicro/stmmac/stmmac_ptp.c:111:2: error: variable 'ns' is used uninitialized whenever '&&' condition is false [-Werror,-Wsometimes-uninitialized] Clang is concerned with the use of stmmac_do_void_callback (which stmmac_get_systime wraps), as it may fail to initialize these values if the if condition was ever false (meaning the callback doesn't exist). It's not wrong because the callback is what initializes ns. While it's unlikely that the callback is going to disappear at some point and make that condition false, we can easily avoid this warning by zero initializing the variable. Link: https://github.com/ClangBuiltLinux/linux/issues/384 Fixes: df103170854e ("net: stmmac: Avoid sometimes uninitialized Clang warnings") Suggested-by: Nick Desaulniers <ndesaulniers@google.com> Signed-off-by: Nathan Chancellor <natechancellor@gmail.com> Reviewed-by: Nick Desaulniers <ndesaulniers@google.com> Signed-off-by: David S. Miller <davem@davemloft.net> Signed-off-by: Sasha Levin <sashal@kernel.org>
This commit is contained in:
parent
972e31ba4b
commit
d1d2ca98b7
1 changed files with 1 additions and 1 deletions
|
@ -105,7 +105,7 @@ static int stmmac_get_time(struct ptp_clock_info *ptp, struct timespec64 *ts)
|
||||||
struct stmmac_priv *priv =
|
struct stmmac_priv *priv =
|
||||||
container_of(ptp, struct stmmac_priv, ptp_clock_ops);
|
container_of(ptp, struct stmmac_priv, ptp_clock_ops);
|
||||||
unsigned long flags;
|
unsigned long flags;
|
||||||
u64 ns;
|
u64 ns = 0;
|
||||||
|
|
||||||
spin_lock_irqsave(&priv->ptp_lock, flags);
|
spin_lock_irqsave(&priv->ptp_lock, flags);
|
||||||
stmmac_get_systime(priv, priv->ptpaddr, &ns);
|
stmmac_get_systime(priv, priv->ptpaddr, &ns);
|
||||||
|
|
Loading…
Reference in a new issue