On 2020-01-22 23:25, Can Guo wrote: > + /* getting Specification Version in big endian format */ > + hba->dev_info.spec_version = desc_buf[DEVICE_DESC_PARAM_SPEC_VER] << 8 | > + desc_buf[DEVICE_DESC_PARAM_SPEC_VER + 1]; Please use get_unaligned_be16() instead of open-coding it. Thanks, Bart.