> > pr_err? Or is this based on a module parameter? > > The macro PRINT_ERROR() has been defined in the header file > include/scst/scst_debug.h. For this macro and also for related macros > (e.g. TRACE_DBG()) the user can configure via debugfs whether or not > PID, function name and/or line number should be included in the > generated messages. Hmm, at one hand that is neat, but at other I am wondering if it would be better to use CONFIG_DYNAMIC_PRINTK functionality (which is what pr_debug could be used for) - which is a more generic framework for this. Some of these errors look relevant to the user so perhaps going through the debugfs is not proper? Or is it that there are just soo many of them that it is unclear whcih ones would be releveant? > > >> + res = -ENOMEM; > >> + goto out; > >> + } > >> + > >> + tr_id[0] = 0x00 | SCSI_TRANSPORTID_PROTOCOLID_SAS; > > > > Hmm, why the 0x00? Can't you just do an assigment? > > If I do not receive further feedback, I will queue the patch below: ok. -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
