Bart, On 4/22/17 02:07, Bart Van Assche wrote: > 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> Ooops... No it is not. I will resend a v2. Thanks. -- Damien Le Moal, Western Digital
