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

From: Bagas Sanjaya
Date: Thu Jun 26 2025 - 19:08:42 EST


On 6/26/25 23:16, Bart Van Assche wrote:
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?


Ack.

Thanks.

--
An old man doll... just what I always wanted! - Clara