be16_to_cpu() returns a u16 value. Remove the superfluous u16 cast. That cast was introduced by commit 7dafbab3753f ("IB/hfi1: Add functions to parse BTH/IB headers"). Cc: Don Hiatt <don.hiatt@xxxxxxxxx> Cc: Dennis Dalessandro <dennis.dalessandro@xxxxxxxxx> Signed-off-by: Bart Van Assche <bvanassche@xxxxxxx> --- include/rdma/ib_hdrs.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/include/rdma/ib_hdrs.h b/include/rdma/ib_hdrs.h index 57c1ac881d08..82483120539f 100644 --- a/include/rdma/ib_hdrs.h +++ b/include/rdma/ib_hdrs.h @@ -208,7 +208,7 @@ static inline u8 ib_get_lver(struct ib_header *hdr) static inline u16 ib_get_len(struct ib_header *hdr) { - return (u16)(be16_to_cpu(hdr->lrh[2])); + return be16_to_cpu(hdr->lrh[2]); } static inline u32 ib_get_qkey(struct ib_other_headers *ohdr)