diff options
author | Nathan Chancellor <natechancellor@gmail.com> | 2019-08-11 20:13:45 -0700 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2019-08-11 21:41:48 -0700 |
commit | 125b7e0949d4e72b15c2b1a1590f8cece985a918 (patch) | |
tree | e40739b12ea0d1c7d9655c721794b77497d76693 /net | |
parent | 8874ecae2977e5a2d4f0ba301364435b81c05938 (diff) | |
download | linux-125b7e0949d4e72b15c2b1a1590f8cece985a918.tar.bz2 |
net: tc35815: Explicitly check NET_IP_ALIGN is not zero in tc35815_rx
clang warns:
drivers/net/ethernet/toshiba/tc35815.c:1507:30: warning: use of logical
'&&' with constant operand [-Wconstant-logical-operand]
if (!HAVE_DMA_RXALIGN(lp) && NET_IP_ALIGN)
^ ~~~~~~~~~~~~
drivers/net/ethernet/toshiba/tc35815.c:1507:30: note: use '&' for a
bitwise operation
if (!HAVE_DMA_RXALIGN(lp) && NET_IP_ALIGN)
^~
&
drivers/net/ethernet/toshiba/tc35815.c:1507:30: note: remove constant to
silence this warning
if (!HAVE_DMA_RXALIGN(lp) && NET_IP_ALIGN)
~^~~~~~~~~~~~~~~
1 warning generated.
Explicitly check that NET_IP_ALIGN is not zero, which matches how this
is checked in other parts of the tree. Because NET_IP_ALIGN is a build
time constant, this check will be constant folded away during
optimization.
Fixes: 82a9928db560 ("tc35815: Enable StripCRC feature")
Link: https://github.com/ClangBuiltLinux/linux/issues/608
Signed-off-by: Nathan Chancellor <natechancellor@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net')
0 files changed, 0 insertions, 0 deletions