2022-07-16 13:40:23

by Jilin Yuan

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

Delete the redundant word 'to'.
Delete the redundant word 'the'.
Delete the redundant word 'then'.

Signed-off-by: Jilin Yuan <[email protected]>
---
drivers/usb/host/uhci-platform.c | 2 +-
drivers/usb/host/uhci-q.c | 2 +-
drivers/usb/host/xhci-ring.c | 2 +-
3 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/drivers/usb/host/uhci-platform.c b/drivers/usb/host/uhci-platform.c
index b2049b47a08d..c7ef58c05276 100644
--- a/drivers/usb/host/uhci-platform.c
+++ b/drivers/usb/host/uhci-platform.c
@@ -20,7 +20,7 @@ static int uhci_platform_init(struct usb_hcd *hcd)
if (!uhci->rh_numports)
uhci->rh_numports = uhci_count_ports(hcd);

- /* Set up pointers to to generic functions */
+ /* Set up pointers to generic functions */
uhci->reset_hc = uhci_generic_reset_hc;
uhci->check_and_reset_hc = uhci_generic_check_and_reset_hc;

diff --git a/drivers/usb/host/uhci-q.c b/drivers/usb/host/uhci-q.c
index 35fcb826152c..e7f4c1f28349 100644
--- a/drivers/usb/host/uhci-q.c
+++ b/drivers/usb/host/uhci-q.c
@@ -345,7 +345,7 @@ static int uhci_cleanup_queue(struct uhci_hcd *uhci, struct uhci_qh *qh,
goto done;
}

- /* If the QH element pointer is UHCI_PTR_TERM then then currently
+ /* If the QH element pointer is UHCI_PTR_TERM then currently
* executing URB has already been unlinked, so this one isn't it. */
if (qh_element(qh) == UHCI_PTR_TERM(uhci))
goto done;
diff --git a/drivers/usb/host/xhci-ring.c b/drivers/usb/host/xhci-ring.c
index 46d0b9ad6f74..ad81e9a508b1 100644
--- a/drivers/usb/host/xhci-ring.c
+++ b/drivers/usb/host/xhci-ring.c
@@ -1964,7 +1964,7 @@ static void handle_port_status(struct xhci_hcd *xhci,

/*
* Check to see if xhci-hub.c is waiting on RExit to U0 transition (or
- * RExit to a disconnect state). If so, let the the driver know it's
+ * RExit to a disconnect state). If so, let the driver know it's
* out of the RExit state.
*/
if (!DEV_SUPERSPEED_ANY(portsc) && hcd->speed < HCD_USB3 &&
--
2.36.1

2022-07-16 18:03:52

by Alan Stern

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

On Sat, Jul 16, 2022 at 09:31:29PM +0800, Jilin Yuan wrote:
> Delete the redundant word 'to'.
> Delete the redundant word 'the'.
> Delete the redundant word 'then'.
>
> Signed-off-by: Jilin Yuan <[email protected]>
> ---
> drivers/usb/host/uhci-platform.c | 2 +-
> drivers/usb/host/uhci-q.c | 2 +-
> drivers/usb/host/xhci-ring.c | 2 +-
> 3 files changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/usb/host/uhci-platform.c b/drivers/usb/host/uhci-platform.c
> index b2049b47a08d..c7ef58c05276 100644
> --- a/drivers/usb/host/uhci-platform.c
> +++ b/drivers/usb/host/uhci-platform.c
> @@ -20,7 +20,7 @@ static int uhci_platform_init(struct usb_hcd *hcd)
> if (!uhci->rh_numports)
> uhci->rh_numports = uhci_count_ports(hcd);
>
> - /* Set up pointers to to generic functions */
> + /* Set up pointers to generic functions */

This isn't a mistakenly repeated word; it is a misspelled word. The
second "to" should be "the".

> uhci->reset_hc = uhci_generic_reset_hc;
> uhci->check_and_reset_hc = uhci_generic_check_and_reset_hc;
>
> diff --git a/drivers/usb/host/uhci-q.c b/drivers/usb/host/uhci-q.c
> index 35fcb826152c..e7f4c1f28349 100644
> --- a/drivers/usb/host/uhci-q.c
> +++ b/drivers/usb/host/uhci-q.c
> @@ -345,7 +345,7 @@ static int uhci_cleanup_queue(struct uhci_hcd *uhci, struct uhci_qh *qh,
> goto done;
> }
>
> - /* If the QH element pointer is UHCI_PTR_TERM then then currently
> + /* If the QH element pointer is UHCI_PTR_TERM then currently

This isn't a mistakenly repeated word; it is also a misspelling. The
second "then" should be "the".

Alan Stern