> + unsigned int sector_sz = SCpnt->device->sector_size;
Can you spell out size?
> + /*
> + * In case of bogus fw or device, we could end up having
> + * unaligned partial completion. Check this here.
> + */
> + resid = scsi_get_resid(SCpnt);
> + if (SCpnt->request->cmd_type == REQ_TYPE_FS &&
> + resid & (sector_sz - 1)) {
->done is only called for fs requests, so you can remove this check.
