2022-08-25 02:14:15

by Shaomin Deng

Subject: [PATCH] pinctrl: at91: Fix double word in comments

Delete the rebundant word "the" in comments.

Signed-off-by: Shaomin Deng <[email protected]>
---
drivers/pinctrl/pinctrl-at91.c | 2 +-
drivers/xen/xen-pciback/pciback_ops.c | 3 ++-
2 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/drivers/pinctrl/pinctrl-at91.c b/drivers/pinctrl/pinctrl-at91.c
index d91a010e65f5..f1234e719e58 100644
--- a/drivers/pinctrl/pinctrl-at91.c
+++ b/drivers/pinctrl/pinctrl-at91.c
@@ -1748,7 +1748,7 @@ static int at91_gpio_of_irq_setup(struct platform_device *pdev,
writel_relaxed(~0, at91_gpio->regbase + PIO_IDR);

/*
- * Let the generic code handle this edge IRQ, the the chained
+ * Let the generic code handle this edge IRQ, the chained
* handler will perform the actual work of handling the parent
* interrupt.
*/
diff --git a/drivers/xen/xen-pciback/pciback_ops.c b/drivers/xen/xen-pciback/pciback_ops.c
index e38b43b5065e..52da75bf8485 100644
--- a/drivers/xen/xen-pciback/pciback_ops.c
+++ b/drivers/xen/xen-pciback/pciback_ops.c
@@ -160,7 +160,8 @@ int xen_pcibk_enable_msi(struct xen_pcibk_device *pdev,
}

/* The value the guest needs is actually the IDT vector, not the
- * local domain's IRQ number. */
+ * local domain's IRQ number.
+ */

op->value = dev->irq ? xen_pirq_from_irq(dev->irq) : 0;

--
2.35.1

2022-08-25 06:03:32

by Claudiu Beznea

Subject: Re: [PATCH] pinctrl: at91: Fix double word in comments

On 25.08.2022 04:49, Shaomin Deng wrote:
> [Some people who received this message don't often get email from [email protected]. Learn why this is important at https://aka.ms/LearnAboutSenderIdentification ]
>
> EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe
>
> Delete the rebundant word "the" in comments.
>
> Signed-off-by: Shaomin Deng <[email protected]>
> ---
> drivers/pinctrl/pinctrl-at91.c | 2 +-
> drivers/xen/xen-pciback/pciback_ops.c | 3 ++-
> 2 files changed, 3 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/pinctrl/pinctrl-at91.c b/drivers/pinctrl/pinctrl-at91.c
> index d91a010e65f5..f1234e719e58 100644
> --- a/drivers/pinctrl/pinctrl-at91.c
> +++ b/drivers/pinctrl/pinctrl-at91.c
> @@ -1748,7 +1748,7 @@ static int at91_gpio_of_irq_setup(struct platform_device *pdev,
> writel_relaxed(~0, at91_gpio->regbase + PIO_IDR);
>
> /*
> - * Let the generic code handle this edge IRQ, the the chained
> + * Let the generic code handle this edge IRQ, the chained
> * handler will perform the actual work of handling the parent
> * interrupt.
> */
> diff --git a/drivers/xen/xen-pciback/pciback_ops.c b/drivers/xen/xen-pciback/pciback_ops.c
> index e38b43b5065e..52da75bf8485 100644
> --- a/drivers/xen/xen-pciback/pciback_ops.c
> +++ b/drivers/xen/xen-pciback/pciback_ops.c
> @@ -160,7 +160,8 @@ int xen_pcibk_enable_msi(struct xen_pcibk_device *pdev,
> }
>
> /* The value the guest needs is actually the IDT vector, not the
> - * local domain's IRQ number. */
> + * local domain's IRQ number.
> + */

This should go in a different patch.

>
> op->value = dev->irq ? xen_pirq_from_irq(dev->irq) : 0;
>
> --
> 2.35.1
>