On Fri, Oct 21, 2016 at 12:47:02PM +0100, Domagoj Tršan wrote: > csum member of struct btrfs_super_block has array type of u8. It makes sense > that function btrfs_csum_final should be also declared to accept u8 *. I > changed the declaration of method void btrfs_csum_final(u32 crc, char > *result); > to void btrfs_csum_final(u32 crc, u8 *result);
Ok that's better, just the signed-off-by line is missing. I would add it myself on behalf of contributors I know, but as a matter of practice, fix it please and resend the patch. Using the "git commit -s" adds the line for you. It is a formality, established in the linux kernel development. If you need more explanatino what and why, please refer to the following docs. http://elinux.org/Developer_Certificate_Of_Origin https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/tree/Documentation/SubmittingPatches -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html