On Jun 20, 2007 17:26 +0530, Girish Shilamkar wrote: > crc = crc16(crc, (__u8 *)gdp + offset, > - sbi->s_es->s_desc_size - offset); > + le16_to_cpu(sbi->s_es->s_desc_size) > + - offset);
Minor nit - please put '-' at the end of the previous line, and align "offset" with '(sbi-> ...'. Also, we're missing the signoff: Signed-off-by: Andreas Dilger <[EMAIL PROTECTED]> Signed-off-by: Girish Shilamkar <[EMAIL PROTECTED]> Cheers, Andreas -- Andreas Dilger Principal Software Engineer Cluster File Systems, Inc. - To unsubscribe from this list: send the line "unsubscribe linux-ext4" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html