summaryrefslogtreecommitdiffstats
path: root/net
diff options
context:
space:
mode:
authorArnd Bergmann <arnd@arndb.de>2023-05-16 20:34:22 +0200
committerKalle Valo <kvalo@kernel.org>2023-05-17 09:24:09 +0300
commit212457ccbd60dba34f965e4ffbe62f0e4f970538 (patch)
tree5a2b5332935c22c965f38aaa21956522366a8967 /net
parentcb0ddaaa5db09d7d216fcbf0e68779be223a1128 (diff)
downloadlinux-stable-212457ccbd60dba34f965e4ffbe62f0e4f970538.tar.gz
linux-stable-212457ccbd60dba34f965e4ffbe62f0e4f970538.tar.bz2
linux-stable-212457ccbd60dba34f965e4ffbe62f0e4f970538.zip
wifi: b43: fix incorrect __packed annotation
clang warns about an unpacked structure inside of a packed one: drivers/net/wireless/broadcom/b43/b43.h:654:4: error: field data within 'struct b43_iv' is less aligned than 'union (unnamed union at /home/arnd/arm-soc/drivers/net/wireless/broadcom/b43/b43.h:651:2)' and is usually due to 'struct b43_iv' being packed, which can lead to unaligned accesses [-Werror,-Wunaligned-access] The problem here is that the anonymous union has the default alignment from its members, apparently because the original author mixed up the placement of the __packed attribute by placing it next to the struct member rather than the union definition. As the struct itself is also marked as __packed, there is no need to mark its members, so just move the annotation to the inner type instead. As Michael noted, the same problem is present in b43legacy, so change both at the same time. Acked-by: Michael Büsch <m@bues.ch> Reported-by: kernel test robot <lkp@intel.com> Reviewed-by: Simon Horman <simon.horman@corigine.com> Tested-by: Larry Finger <Larry.Finger@lwfinger.net> Link: https://lore.kernel.org/oe-kbuild-all/202305160749.ay1HAoyP-lkp@intel.com/ Signed-off-by: Arnd Bergmann <arnd@arndb.de> Signed-off-by: Kalle Valo <kvalo@kernel.org> Link: https://lore.kernel.org/r/20230516183442.536589-1-arnd@kernel.org
Diffstat (limited to 'net')
0 files changed, 0 insertions, 0 deletions