Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
Subject: Re: [PATCH] net/faraday: fix grammar in function
 ftgmac100_setup_clk() in ftgmac100.c
To:     Benjamin Herrenschmidt <benh@kernel.crashing.org>,
        davem@davemloft.net, andrew@lunn.ch, mchehab+samsung@kernel.org,
        andrew@aj.id.au, corbet@lwn.net
Cc:     stfrench@microsoft.com, chris@chris-wilson.co.uk,
        xiubli@redhat.com, airlied@redhat.com, tglx@linutronix.de,
        hkallweit1@gmail.com, netdev@vger.kernel.org,
        linux-kernel@vger.kernel.org
References: <20200401105624.17423-1-xianfengting221@163.com>
 <1947a3705a220ce14a2fda482c833b38a4d9fe9a.camel@kernel.crashing.org>
From:   Hu Haowen <xianfengting221@163.com>
Message-ID: <d146a02d-372e-0f54-39e7-3ebcd0c46e4f@163.com>
Date:   Thu, 2 Apr 2020 19:13:15 +0800
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101
 Thunderbird/60.9.0
MIME-Version: 1.0
In-Reply-To: <1947a3705a220ce14a2fda482c833b38a4d9fe9a.camel@kernel.crashing.org>
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Transfer-Encoding: 7bit
Content-Language: en-US
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk


On 2020/4/2 6:43 PM, Benjamin Herrenschmidt wrote:
> On Wed, 2020-04-01 at 18:56 +0800, Hu Haowen wrote:
>> "its not" is wrong. The words should be "it's not".
>>
>> Signed-off-by: Hu Haowen <xianfengting221@163.com>
> Typo more than grammer :-)


Emm...


In fact I just found the mistake by accident and decided to fix it up
because it is a standard in the kernel documention not to misspell
words :-)


>
> Acked-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>
>
>   (the offender)
>
>> ---
>>   drivers/net/ethernet/faraday/ftgmac100.c | 2 +-
>>   1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/drivers/net/ethernet/faraday/ftgmac100.c b/drivers/net/ethernet/faraday/ftgmac100.c
>> index 835b7816e372..87236206366f 100644
>> --- a/drivers/net/ethernet/faraday/ftgmac100.c
>> +++ b/drivers/net/ethernet/faraday/ftgmac100.c
>> @@ -1731,7 +1731,7 @@ static int ftgmac100_setup_clk(struct ftgmac100 *priv)
>>   	if (rc)
>>   		goto cleanup_clk;
>>   
>> -	/* RCLK is for RMII, typically used for NCSI. Optional because its not
>> +	/* RCLK is for RMII, typically used for NCSI. Optional because it's not
>>   	 * necessary if it's the AST2400 MAC, or the MAC is configured for
>>   	 * RGMII, or the controller is not an ASPEED-based controller.
>>   	 */