On Sat, Nov 18, 2017 at 06:21:35PM +0800, Lijun Ou wrote: > struct hns_roce_v2_cqe { > - unsigned int byte_4; > - unsigned int rkey_immtdata; > - unsigned int byte_12; > - unsigned int byte_16; > - unsigned int byte_cnt; > - unsigned int smac; > - unsigned int byte_28; > - unsigned int byte_32; > + __le32 byte_4; > + union { > + __le32 rkey; > + __be32 immtdata; > + }; Yes, that looks exactly right. Reviewed-by: Jason Gunthorpe <jgg@xxxxxxxx> Jason -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html