2022-07-16 14:08:29

by Jilin Yuan

Subject: [PATCH] usb/storage: fix repeated words in comments

Delete the redundant word 'the'.
Delete the redundant word 'buffer'.

Signed-off-by: Jilin Yuan <[email protected]>
---
drivers/usb/storage/sddr09.c | 2 +-
drivers/usb/storage/transport.c | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/usb/storage/sddr09.c b/drivers/usb/storage/sddr09.c
index 51bcd4a43690..cb0ae82b4abc 100644
--- a/drivers/usb/storage/sddr09.c
+++ b/drivers/usb/storage/sddr09.c
@@ -1215,7 +1215,7 @@ sddr09_read_map(struct us_data *us) {
/*
* read 64 bytes for every block (actually 1 << CONTROL_SHIFT)
* but only use a 64 KB buffer
- * buffer size used must be a multiple of (1 << CONTROL_SHIFT)
+ * size used must be a multiple of (1 << CONTROL_SHIFT)
*/
#define SDDR09_READ_MAP_BUFSZ 65536

diff --git a/drivers/usb/storage/transport.c b/drivers/usb/storage/transport.c
index 64d96d210e02..aa96f97e5ac4 100644
--- a/drivers/usb/storage/transport.c
+++ b/drivers/usb/storage/transport.c
@@ -1179,7 +1179,7 @@ int usb_stor_Bulk_transport(struct scsi_cmnd *srb, struct us_data *us)
* If the device tried to send back more data than the
* amount requested, the spec requires us to transfer
* the CSW anyway. Since there's no point retrying the
- * the command, we'll return fake sense data indicating
+ * command, we'll return fake sense data indicating
* Illegal Request, Invalid Field in CDB.
*/
if (result == USB_STOR_XFER_LONG)
--
2.36.1

2022-07-16 17:41:59

by Alan Stern

Subject: Re: [PATCH] usb/storage: fix repeated words in comments

On Sat, Jul 16, 2022 at 09:52:17PM +0800, Jilin Yuan wrote:
> Delete the redundant word 'the'.
> Delete the redundant word 'buffer'.
>
> Signed-off-by: Jilin Yuan <[email protected]>
> ---
> drivers/usb/storage/sddr09.c | 2 +-
> drivers/usb/storage/transport.c | 2 +-
> 2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/usb/storage/sddr09.c b/drivers/usb/storage/sddr09.c
> index 51bcd4a43690..cb0ae82b4abc 100644
> --- a/drivers/usb/storage/sddr09.c
> +++ b/drivers/usb/storage/sddr09.c
> @@ -1215,7 +1215,7 @@ sddr09_read_map(struct us_data *us) {
> /*
> * read 64 bytes for every block (actually 1 << CONTROL_SHIFT)
> * but only use a 64 KB buffer
> - * buffer size used must be a multiple of (1 << CONTROL_SHIFT)
> + * size used must be a multiple of (1 << CONTROL_SHIFT)

This isn't a redundant word. There should be a period after the last
word ("buffer") on the preceding line.

Alan Stern