On Sun, 2017-01-08 at 02:05 -0800, Christoph Hellwig wrote: > > + 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.. I have fixed the style issue so its not needed to repost this patch. Thanks for the review Christoph. Bart.-- 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