> - memset(h->buff, 0, stpg_len); > - h->buff[4] = TPGS_STATE_OPTIMIZED & 0x0f; > - h->buff[6] = (h->group_id >> 8) & 0xff; > - h->buff[7] = h->group_id & 0xff; > + memset(stpg_data, 0, stpg_len); > + stpg_data[4] = TPGS_STATE_OPTIMIZED & 0x0f; > + put_unaligned_be16(group_id, &stpg_data[6]); Unrelated get/put_unaligned changes again. > - if (!scsi_normalize_sense(h->sense, SCSI_SENSE_BUFFERSIZE, > + if (!(driver_byte(retval) & DRIVER_SENSE) || > + !scsi_normalize_sense(h->sense, SCSI_SENSE_BUFFERSIZE, Where does this come from? > + (!h->pref) && no need for braces here. -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html