> + > +static struct sym53c500_cmd_priv *SYM53c500_priv(struct scsi_cmnd *cmd) > { > return scsi_cmd_priv(cmd); > } I don't even think we need this helper, all callers can just use scsi_cmd_priv directly. Otherwise looks good: Reviewed-by: Christoph Hellwig <hch@xxxxxx>