diff options
author | 2021-03-15 16:51:48 -0500 | |
---|---|---|
committer | 2021-03-15 20:41:58 -0700 | |
commit | 9d131d044f89a808f947d49f1a1865815c610158 (patch) | |
tree | 0ded386f9628a1d35756d26252eb8cb306e6419e /net/lapb/lapb_timer.c | |
parent | net: qualcomm: rmnet: simplify some byte order logic (diff) | |
download | wireguard-linux-9d131d044f89a808f947d49f1a1865815c610158.tar.xz wireguard-linux-9d131d044f89a808f947d49f1a1865815c610158.zip |
net: qualcomm: rmnet: kill RMNET_MAP_GET_*() accessor macros
The following macros, defined in "rmnet_map.h", assume a socket
buffer is provided as an argument without any real indication this
is the case.
RMNET_MAP_GET_MUX_ID()
RMNET_MAP_GET_CD_BIT()
RMNET_MAP_GET_PAD()
RMNET_MAP_GET_CMD_START()
RMNET_MAP_GET_LENGTH()
What they hide is pretty trivial accessing of fields in a structure,
and it's much clearer to see this if we do these accesses directly.
So rather than using these accessor macros, assign a local
variable of the map header pointer type to the socket buffer data
pointer, and derereference that pointer variable.
In "rmnet_map_data.c", use sizeof(object) rather than sizeof(type)
in one spot. Also, there's no need to byte swap 0; it's all zeros
irrespective of endianness.
Signed-off-by: Alex Elder <elder@linaro.org>
Reviewed-by: Bjorn Andersson <bjorn.andersson@linaro.org>
Reviewed-by: Alexander Duyck <alexanderduyck@fb.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/lapb/lapb_timer.c')
0 files changed, 0 insertions, 0 deletions