On Tue, 2017-05-30 at 10:54 -0700, Himanshu Madhani wrote: > - ctio->u.status1.flags = (atio->u.isp24.attr << 9) | > - cpu_to_le16(CTIO7_FLAGS_STATUS_MODE_1 | CTIO7_FLAGS_SEND_STATUS); > + temp = (atio->u.isp24.attr << 9)| > + CTIO7_FLAGS_STATUS_MODE_1 | CTIO7_FLAGS_SEND_STATUS; > + ctio->u.status1.flags = cpu_to_le16(temp); This patch not only includes the functional changes explained in the patch description but also several changes that do not change any functionality like the above. Please keep functional and non-functional changes separate for future patch submissions. Thanks, Bart.