Bart, On 4/22/17 01:58, Bart Van Assche wrote: > On Fri, 2017-04-21 at 18:16 +0900, damien.lemoal@xxxxxxx wrote: >> @@ -2363,42 +2365,32 @@ EXPORT_SYMBOL(scsi_command_normalize_sense); >> * Return value: >> * 1 if information field found, 0 if not found. >> */ >> -int scsi_get_sense_info_fld(const u8 * sense_buffer, int sb_len, >> - u64 * info_out) >> +bool scsi_get_sense_info_fld(const u8 * sense_buffer, int sb_len, >> + u64 * info_out) > > Hello Damien, > > If you have to repost this patch please address the checkpatch complaints > of the following category triggered by the above declaration: > > ERROR: "foo * bar" should be "foo *bar" > > Otherwise this patch looks fine to me. Hence: > > Reviewed-by: Bart Van Assche <Bart.VanAssche@xxxxxxxxxxx> OK. Will do. -- Damien Le Moal, Western Digital