Re: [PATCH v2] btrfs: change btrfs_csum_final result param type to u8

2016-10-27 Thread Domagoj Tršan
On 2016-10-24 19:05 +0200, David Sterba wrote: > > 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. > I added

Re: [PATCH v2] btrfs: change btrfs_csum_final result param type to u8

2016-10-24 Thread David Sterba
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 >

[PATCH v2] btrfs: change btrfs_csum_final result param type to u8

2016-10-21 Thread Domagoj Tršan
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); ---