[PATCH] alpha: Fix double word in comments

[PATCH] alpha: Fix double word in comments

[Shaomin Deng]

Delete the rebundant word "and" in comments.

Signed-off-by: Shaomin Deng <dengshaomin@cdjrlc.com>
---
 arch/alpha/kernel/irq_i8259.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/alpha/kernel/irq_i8259.c b/arch/alpha/kernel/irq_i8259.c
index 1dcf0d9038fd..db574dcd6675 100644
--- a/arch/alpha/kernel/irq_i8259.c
+++ b/arch/alpha/kernel/irq_i8259.c
@@ -147,7 +147,7 @@ isa_no_iack_sc_device_interrupt(unsigned long vector)
 	 */
 	/* 
 	 *  The first read of gives you *all* interrupting lines.
-	 *  Therefore, read the mask register and and out those lines
+	 *  Therefore, read the mask register and out those lines
 	 *  not enabled.  Note that some documentation has 21 and a1 
 	 *  write only.  This is not true.
 	 */
-- 
2.35.1

Re: [PATCH] alpha: Fix double word in comments

[Richard Henderson]

On 8/30/22 00:42, Shaomin Deng wrote:
> Delete the rebundant word "and" in comments.

typo.

Reviewed-by: Richard Henderson <richard.henderson@linaro.org>


> 
> Signed-off-by: Shaomin Deng <dengshaomin@cdjrlc.com>
> ---
>   arch/alpha/kernel/irq_i8259.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/arch/alpha/kernel/irq_i8259.c b/arch/alpha/kernel/irq_i8259.c
> index 1dcf0d9038fd..db574dcd6675 100644
> --- a/arch/alpha/kernel/irq_i8259.c
> +++ b/arch/alpha/kernel/irq_i8259.c
> @@ -147,7 +147,7 @@ isa_no_iack_sc_device_interrupt(unsigned long vector)
>   	 */
>   	/*
>   	 *  The first read of gives you *all* interrupting lines.
> -	 *  Therefore, read the mask register and and out those lines
> +	 *  Therefore, read the mask register and out those lines
>   	 *  not enabled.  Note that some documentation has 21 and a1
>   	 *  write only.  This is not true.
>   	 */

Re: [PATCH] alpha: Fix double word in comments

[Michael Cree]

On Tue, Aug 30, 2022 at 07:52:56AM -0700, Richard Henderson wrote:
> On 8/30/22 00:42, Shaomin Deng wrote:
> > Delete the rebundant word "and" in comments.
> 
> typo.

Are you sure it's a typo and not that the first "and" is a conjunction
and the second "and" is a verb (more properly "and out") that acts on
the object, namely "those lines"?

Cheers
Michael.

> 
> Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
> 
> 
> > 
> > Signed-off-by: Shaomin Deng <dengshaomin@cdjrlc.com>
> > ---
> >   arch/alpha/kernel/irq_i8259.c | 2 +-
> >   1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > diff --git a/arch/alpha/kernel/irq_i8259.c b/arch/alpha/kernel/irq_i8259.c
> > index 1dcf0d9038fd..db574dcd6675 100644
> > --- a/arch/alpha/kernel/irq_i8259.c
> > +++ b/arch/alpha/kernel/irq_i8259.c
> > @@ -147,7 +147,7 @@ isa_no_iack_sc_device_interrupt(unsigned long vector)
> >   	 */
> >   	/*
> >   	 *  The first read of gives you *all* interrupting lines.
> > -	 *  Therefore, read the mask register and and out those lines
> > +	 *  Therefore, read the mask register and out those lines
> >   	 *  not enabled.  Note that some documentation has 21 and a1
> >   	 *  write only.  This is not true.
> >   	 */
> 

Re: [PATCH] alpha: Fix double word in comments

[Bob Tracy]

On Tue, Aug 30, 2022 at 03:42:02AM -0400, Shaomin Deng wrote:
> Delete the rebundant word "and" in comments.
> 
> Signed-off-by: Shaomin Deng <dengshaomin@cdjrlc.com>
> ---
>  arch/alpha/kernel/irq_i8259.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/arch/alpha/kernel/irq_i8259.c b/arch/alpha/kernel/irq_i8259.c
> index 1dcf0d9038fd..db574dcd6675 100644
> --- a/arch/alpha/kernel/irq_i8259.c
> +++ b/arch/alpha/kernel/irq_i8259.c
> @@ -147,7 +147,7 @@ isa_no_iack_sc_device_interrupt(unsigned long vector)
>  	 */
>  	/* 
>  	 *  The first read of gives you *all* interrupting lines.
> -	 *  Therefore, read the mask register and and out those lines
> +	 *  Therefore, read the mask register and out those lines
>  	 *  not enabled.  Note that some documentation has 21 and a1 
>  	 *  write only.  This is not true.
>  	 */
> -- 
> 2.35.1

This may have been beaten to death elsewhere, but the comment without
the second "and" doesn't make any sense, so I conclude the second "and"
is a logic operation.  As in, "... and AND out those lines...".  If that
is the case, capitalization will help disambiguate, much as I would expect
to see "OR" if that was the intended operation.  The code following the
read of the mask register (not included in the diff fragment above) should
make the meaning clear.

Accordingly, I would reject this proposed patch.

Respectfully,
--Bob