On Fri, 2017-04-21 at 18:16 +0900, damien.lemoal@xxxxxxx wrote: > Rename sd_zbc_setup_write_cmnd() to sd_zbc_write_lock_zone() to be > clear about what the function actually does. To be consistent, also > rename sd_zbc_cancel_write_cmnd() to sd_zbc_write_unlock_zone(). > > [ ... ] > /* > - * resid is optional but mostly filled in. When it's unused, > + * resid is optional but mostly filled in. When it's unused, > * its value is zero, so we assume the whole buffer transferred > */ Is this whitespace-only change useful? Otherwise this patch looks fine to me. Hence: Reviewed-by: Bart Van Assche <Bart.VanAssche@xxxxxxxxxxx>