On 2020-06-06 11:26, Avri Altman wrote: >> + data_seg_len = be32_to_cpu(lrbp->ucd_rsp_ptr->header.dword_2) >> + & MASK_RSP_UPIU_DATA_SEG_LEN; > get_unaligned instead of be32_to_cpu ? Since sparse checks that the argument of be32_to_cpu() has type __be32 and since no such check is performed for get_unaligned_*(), please keep the be32_to_cpu(). Thanks, Bart.