> + struct scatterlist *sg = *cur_sg ?: > + (struct scatterlist *)scsi_sglist(srb); > +
No need for the cast here. And I have to say I hate that GNU C
non-standard shortshut in ? :.
Why not simply:
struct scatterlist *sg = *cur_sg;
if (!sg)
sg = scsi_sglist(srb);
Which is a little more verbose, but much more readabe.
The rest of the patch looks fine to me.
