On 4/26/21 5:24 PM, Christoph Hellwig wrote: >> +static inline void translate_msg_byte(struct scsi_cmnd *cmd, u8 msg) > > I think a comment describing the function would be very helpful. > Also maybe scsi_message_to_host_byte might be a more descriptive name? > Can do. Cheers, Hannes -- Dr. Hannes Reinecke Kernel Storage Architect hare@xxxxxxx +49 911 74053 688 SUSE Software Solutions Germany GmbH, 90409 Nürnberg GF: F. Imendörffer, HRB 36809 (AG Nürnberg)