> +/** > + * scsi_host_busy_iter - Iterate over all busy commands > + * @shost: Pointer to Scsi_Host. > + * @fn: Function to call on each busy command > + * @priv: Data pointer passed to @fn > + **/ Can you add the context information from the commit log to the kerneldoc comment her? > +typedef bool (scsi_host_busy_iter_fn)(struct scsi_cmnd *, void *, bool); Is there much of a point in having this typedef? > + > +void scsi_host_busy_iter(struct Scsi_Host *, > + scsi_host_busy_iter_fn *fn, void *priv); Any reason to spell out to argument names, but not the third one?
