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

Reply via email to