Skip to content

Commit c13899f

Browse files
Alex Elderkuba-moo
authored andcommitted
net: ipa: use the right accessor in ipa_endpoint_status_skip()
When extracting the destination endpoint ID from the status in ipa_endpoint_status_skip(), u32_get_bits() is used. This happens to work, but it's wrong: the structure field is only 8 bits wide instead of 32. Fix this by using u8_get_bits() to get the destination endpoint ID. Signed-off-by: Alex Elder <elder@linaro.org> Signed-off-by: Jakub Kicinski <kuba@kernel.org>
1 parent 088f8a2 commit c13899f

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

drivers/net/ipa/ipa_endpoint.c

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1164,8 +1164,8 @@ static bool ipa_endpoint_status_skip(struct ipa_endpoint *endpoint,
11641164
return true;
11651165
if (!status->pkt_len)
11661166
return true;
1167-
endpoint_id = u32_get_bits(status->endp_dst_idx,
1168-
IPA_STATUS_DST_IDX_FMASK);
1167+
endpoint_id = u8_get_bits(status->endp_dst_idx,
1168+
IPA_STATUS_DST_IDX_FMASK);
11691169
if (endpoint_id != endpoint->endpoint_id)
11701170
return true;
11711171

0 commit comments

Comments
 (0)