> + if (xop->stdi > XCOPY_CSCD_DESC_ID_LIST_OFF_MAX > + || xop->dtdi > XCOPY_CSCD_DESC_ID_LIST_OFF_MAX) { Linux style would be to have the || on the first line. But I feel I'm goint too far into nitpick territory here.. -- To unsubscribe from this list: send the line "unsubscribe target-devel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html