Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Subject: Re: [PATCH] ethernet/sfc: fix repeated words in comments
To:     wangjianli <wangjianli@cdjrlc.com>, davem@davemloft.net,
        edumazet@google.com, kuba@kernel.org
Cc:     netdev@vger.kernel.org, linux-kernel@vger.kernel.org
References: <20220908124630.24314-1-wangjianli@cdjrlc.com>
From:   Edward Cree <ecree.xilinx@gmail.com>
Message-ID: <30f2488d-9af3-fe8d-6e6f-713a7d38800b@gmail.com>
Date:   Thu, 8 Sep 2022 17:07:47 +0100
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101
 Thunderbird/78.14.0
MIME-Version: 1.0
In-Reply-To: <20220908124630.24314-1-wangjianli@cdjrlc.com>
Content-Type: text/plain; charset=utf-8
Content-Language: en-GB
Content-Transfer-Encoding: 7bit
Precedence: bulk

On 08/09/2022 13:46, wangjianli wrote:
> Delete the redundant word 'in'.
> 
> Signed-off-by: wangjianli <wangjianli@cdjrlc.com>
> ---
>  drivers/net/ethernet/sfc/bitfield.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/net/ethernet/sfc/bitfield.h b/drivers/net/ethernet/sfc/bitfield.h
> index 1f981dfe4bdc..0b502d1c3c9e 100644
> --- a/drivers/net/ethernet/sfc/bitfield.h
> +++ b/drivers/net/ethernet/sfc/bitfield.h
> @@ -117,7 +117,7 @@ typedef union efx_oword {
>   *
>   *   ( element ) << 4
>   *
> - * The result will contain the relevant bits filled in in the range
> + * The result will contain the relevant bits filled in the range
>   * [0,high-low), with garbage in bits [high-low+1,...).
>   */
>  #define EFX_EXTRACT_NATIVE(native_element, min, max, low, high)		\
> 

Nack.
"filled in" is a phrasal verb, so the existing text is correct.  Stet.

#ifdef RANT
NGL, getting kinda sick of these bogus comment text 'fixes' from people
 who clearly don't have enough mastery of English to copyedit it.
(Previous one from this author was actually wrong too but I didn't catch
 it at the time.)
English is a tricksy language, why would someone with a limited
 understanding of it think that grammar fixes are the best use of their
 time and energy?
I can't help suspecting that this is a box-ticking exercise, where a
 certain corporate culture has a standard requirement that engineers
 must get X number of Linux / opensource commits in order to get
 promoted, and this kind of mindless patch is the easiest way for them
 to achieve that.
#endif

-ed