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;
Message-ID: <1a81c73e-234d-0c7d-618a-a3bcf773f1b0@linaro.org>
Date:   Tue, 21 Jun 2022 08:47:32 -0500
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101
 Thunderbird/91.9.1
Subject: Re: [PATCH] net: ipa: remove unexpected word "the"
Content-Language: en-US
To:     Jiang Jian <jiangjian@cdjrlc.com>, davem@davemloft.net,
        edumazet@google.com, kuba@kernel.org, pabeni@redhat.com
Cc:     elder@kernel.org, netdev@vger.kernel.org,
        linux-kernel@vger.kernel.org
References: <20220621085001.61320-1-jiangjian@cdjrlc.com>
From:   Alex Elder <elder@linaro.org>
In-Reply-To: <20220621085001.61320-1-jiangjian@cdjrlc.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
Precedence: bulk

On 6/21/22 3:50 AM, Jiang Jian wrote:
> there is an unexpected word "the" in the comments that need to be removed

In the future, maybe something more like "Remove duplicate 'the'
in two places."

But regardless, this looks fine to me.

Reviewed-by: Alex Elder <elder@linaro.org>

> 
> Signed-off-by: Jiang Jian <jiangjian@cdjrlc.com>
> ---
>   drivers/net/ipa/gsi_trans.c | 4 ++--
>   1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/net/ipa/gsi_trans.c b/drivers/net/ipa/gsi_trans.c
> index cf646dc8e36a..29496ca15825 100644
> --- a/drivers/net/ipa/gsi_trans.c
> +++ b/drivers/net/ipa/gsi_trans.c
> @@ -339,7 +339,7 @@ struct gsi_trans *gsi_channel_trans_alloc(struct gsi *gsi, u32 channel_id,
>   	if (!gsi_trans_tre_reserve(trans_info, tre_count))
>   		return NULL;
>   
> -	/* Allocate and initialize non-zero fields in the the transaction */
> +	/* Allocate and initialize non-zero fields in the transaction */
>   	trans = gsi_trans_pool_alloc(&trans_info->pool, 1);
>   	trans->gsi = gsi;
>   	trans->channel_id = channel_id;
> @@ -669,7 +669,7 @@ int gsi_trans_read_byte(struct gsi *gsi, u32 channel_id, dma_addr_t addr)
>   	if (!gsi_trans_tre_reserve(trans_info, 1))
>   		return -EBUSY;
>   
> -	/* Now fill the the reserved TRE and tell the hardware */
> +	/* Now fill the reserved TRE and tell the hardware */
>   
>   	dest_tre = gsi_ring_virt(tre_ring, tre_ring->index);
>   	gsi_trans_tre_fill(dest_tre, addr, 1, true, false, IPA_CMD_NONE);