Re: [PATCH] scsi: Format scsi_track_queue_full() return values as bullet list

From: Bart Van Assche
Date: Thu Jun 26 2025 - 12:16:36 EST


On 6/25/25 9:18 PM, Bagas Sanjaya wrote:
- * Returns: 0 - No change needed, >0 - Adjust queue depth to this new depth,
- * -1 - Drop back to untagged operation using host->cmd_per_lun
- * as the untagged command depth
+ * Returns: * 0 - No change needed
+ * * >0 - Adjust queue depth to this new depth,
+ * * -1 - Drop back to untagged operation using host->cmd_per_lun
+ * as the untagged command depth
*
* Lock Status: None held on entry
*

Here is an example from Documentation/doc-guide/kernel-doc.rst:

* Return:
* * %0 - OK to runtime suspend the device
* * %-EBUSY - Device should not be runtime suspended

Wouldn't it be better to follow that example and to move the list under
'Returns:' and to move it more to the left?

Thanks,

Bart.