On 6/22/22 08:30, Jiang Jian wrote:
there is an unexpected word 'is' in the comments that need to be dropped file: drivers/scsi/sg.c line: 697 * but is is possible that the app intended SG_DXFER_TO_DEV, because there changed to: * but is possible that the app intended SG_DXFER_TO_DEV, because there Signed-off-by: Jiang Jian <jiangjian@xxxxxxxxxx> --- drivers/scsi/sg.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/scsi/sg.c b/drivers/scsi/sg.c index 118c7b4a8af2..047761f62b23 100644 --- a/drivers/scsi/sg.c +++ b/drivers/scsi/sg.c @@ -694,7 +694,7 @@ sg_write(struct file *filp, const char __user *buf, size_t count, loff_t * ppos) } /* * SG_DXFER_TO_FROM_DEV is functionally equivalent to SG_DXFER_FROM_DEV, - * but is is possible that the app intended SG_DXFER_TO_DEV, because there + * but is possible that the app intended SG_DXFER_TO_DEV, because there * is a non-zero input_size, so emit a warning. */ if (hp->dxfer_direction == SG_DXFER_TO_FROM_DEV) {
I do not agree with this change since the author of that comment probably wanted to write "[...] but it is possible [...]"
Bart.