On 4/23/21 4:39 AM, Hannes Reinecke wrote: > The description for scsi_mode_sense() claims to return the number > of valid bytes on success, which is not what the code does. > Additionally there is no gain in returning the scsi status, as > everything the callers do is to check against scsi_result_is_good(), > which is what scsi_mode_sense() does already. > So change the calling convention to return a standard error code > on failure, and 0 on success, and adapt the description and all > callers. Reviewed-by: Bart Van Assche <bvanassche@xxxxxxx>