Thanks,
Avri

> -----Original Message-----
> From: Bart Van Assche
> Sent: Wednesday, August 08, 2018 7:27 PM
> To: [email protected]; Avri Altman ; [email protected];
> [email protected]; [email protected]; [email protected];
> [email protected]
> Cc: Vinayak Holikatti ; Avi Shchislowski ; Alex Lemberg ; Stanislav Nijnikov ;
> [email protected]
> Subject: Re: [PATCH v2 2/8] scsi: ufs: Add ufs-bsg module
> 
> On Sun, 2018-08-05 at 14:39 +0300, Avri Altman wrote:
> > A LLD companion for the ufs scsi transport.
> 
> This description is misleading. How about changing this into "Add a bsg
> endpoint that supports UPIUs"?
Done.

> 
> diff --git a/drivers/scsi/ufs/ufs_bsg.c b/drivers/scsi/ufs/ufs_bsg.c
> > new file mode 100644
> > index 0000000..6f1941b
> > --- /dev/null
> > +++ b/drivers/scsi/ufs/ufs_bsg.c
> > @@ -0,0 +1,127 @@
> > +// SPDX-License-Identifier: GPL-2.0
> > +/*
> > + * SCSI transport companion for UFS HBA
> 
> This description is misleading too. Please consider to change this into
> something like "bsg endpoint that supports UPIUs".
Done.

> 
> Thanks,
> 
> Bart.

Reply via email to