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: <a7e51c63-a7c1-9acd-128e-02942737e726@axentia.se>
Date:   Thu, 8 Sep 2022 15:51:51 +0200
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:91.0) Gecko/20100101
 Thunderbird/91.13.0
Subject: Re: [PATCH] i2c/busses: fix repeated words in comments
Content-Language: en-US
To:     wangjianli <wangjianli@cdjrlc.com>
Cc:     linux-i2c@vger.kernel.org, linux-kernel@vger.kernel.org
References: <20220908132148.42995-1-wangjianli@cdjrlc.com>
From:   Peter Rosin <peda@axentia.se>
In-Reply-To: <20220908132148.42995-1-wangjianli@cdjrlc.com>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?Q2VkcGVadkVVUmpzcXJ2RTVIZHFkWUVSYTFMbThENWVua05sL3kvdXNLTlZN?=
 =?utf-8?B?dUpnZEV6aEFDc29nNzU4TkgxSVRKVjAzZTUwYmhqQmFWOW1mSzF5V3VHbWVK?=
 =?utf-8?B?bEhoNTVmVVQrWFE5V3hqdUkrNXBja2ltVy9UcmdTZVRRN2c4K2lTUlBOYjFC?=
 =?utf-8?B?Qk9UMk90eU5mS3NkQVJCSGlQaldlV1RhUHlwMGJoWVprTGo4c3hBNmYyYUtj?=
 =?utf-8?B?TUR0aVh6S01qRWdjdXByaTVpQWdoVjA4Njkya2hTeTFJdWJNT2d0c3graTho?=
 =?utf-8?B?aytLSE5nVm1WRUJEQ0FDcmtwVWw3bEtoSVJDd3kxVWdHQVVzS0doNElBbkpo?=
 =?utf-8?B?d3ZsUm5ua3ZteDVSUUIwd2xVT0pFOGtNTmRHbVRRSTlNcFkrVGxuMWtIeWJS?=
 =?utf-8?B?ZmZJZU1yYkJJYXI5endCSUtIckUrVlh2SHVScThJQUZmMVkwOGdVeUlDeGdH?=
 =?utf-8?B?ank5M1k1WVV5UkQybHQzYWMwVXlYSW5xbTVWbjZUMkVmZkpiTW5WQ3RDdE03?=
 =?utf-8?B?OGVkK2xtV2s0V0FxL2c0cHRHakZPOSt4OWFNaHY1SGFEc2Q2U1EvaWFKMWw1?=
 =?utf-8?B?NDlRN09PT3A2L3IwRmUxSlYyQldNMmI3K2NQOVEzQVQ1N2FGTVhvdVRSYlVW?=
 =?utf-8?B?OXdaeXliZFdJd2pmd0FwdVF6Y1lISzhuVHNpZndwbHBodzNJNzFnZ29vWWE0?=
 =?utf-8?B?SjhGS1lVSFV6UFZWb2dZMjkrRWljZjBiWUo1Y3lBSWtnY05MdkZYSTVKNnZL?=
 =?utf-8?B?ckN1ZmN5SXdRRDdxWVZzYUhEOVdIRkRLN2w2b2k0c1FSRitXaHM4R21kZDU5?=
 =?utf-8?B?SThQUjE1S3FiUHRlTTVOLzhqbzY4ZTRobW1Vc242UGlsSGRaTUhwOFkyUTBQ?=
 =?utf-8?B?YW9udGZWOXNWS1NuVG9LN3RMSEdSZTNJMVR5aEdwaFd6bzJlL2FiQ1lGcnN3?=
 =?utf-8?B?eWJXNnhoWUV6Qkp1OXUzWENoa2xFTk12NFhTeHo5QW5oREVWanpSd05sU0lE?=
 =?utf-8?B?QmV2SDRwT25nWjMzYkkxQWxWRm1McUxNODNhdkdMWnJrdTYvU2hJU1c1OVFI?=
 =?utf-8?B?VUMwMjVuaS9zT3ZTU2tFUFhNbUJ6alg4cmlWdzBOdzJTaUdyOXpZcmRzOG15?=
 =?utf-8?B?cGFhT0tremZ3VDI0Qi9KcFdCL2l5T2g2UG9JMHc1VmQ3NVlSbU0vN3RYUnlG?=
 =?utf-8?B?d09uNjVKa1p4UHExTUtZdFNJRDhySENtTjBqSyt6N1JYaDFjS3VMdmlXV3NR?=
 =?utf-8?B?OGxjODVRNmJ0YXZiWGpoeDMwQ0ZhT2lBRFlGL2Iwd2ZqS3J3Vzl1TDYxWFMw?=
 =?utf-8?B?Z2hWa3EyV0RwQ2VIejFtSHhRNHB6VThBcFlqWmJQN1FidFh2dkF1Q3VtazBX?=
 =?utf-8?B?NHo0OCtuMnJyVmZSUEhLenZSaU1JUlIzU1RMWDdSWG80N1ZoVVkvZzNKZmJG?=
 =?utf-8?B?UWE4YVk4eCsrMXBzZTZyeDQrdjVJaTMvYVpqRGtLL3ZsRXNKNGZjUjZkbXNt?=
 =?utf-8?B?c2tqbUY1Y3M1WnJoemVUalNvZjNReHU1ZXNseU9xb1FkcHJwTFhlTzlxQkpk?=
 =?utf-8?B?a1h6QnJhdzA1bjV1ejlXSUxwNzVrNlNsZmtDZ0xaaVRzZHpzWmtFejhrTlo3?=
 =?utf-8?B?bkZLYklmbnhEc2NXNVkwMXVYR08rNUdTSUNSRlVtanhKMWo3eFNYWGcrR0Mw?=
 =?utf-8?B?Q2ZpUVhQcmV5T1hFRld0aHhVeUpMZHE3b0FoZmp6end1T3BXY2lVWU42Rm1S?=
 =?utf-8?B?QjVKQW0vdXdNOHcyeWJSUlRzMUVHWUh4TTRTUEkrSDExNVhNVlUvV3hwd01o?=
 =?utf-8?B?anBlTWVvcDNRaUxjTURJZWtFQjI2WVdYVzYrTjE3aWNYdGowUDFLVC9IaDNH?=
 =?utf-8?B?b2FGT21oWml1NW1vMjZkU1lneSt2T3IvcTJOV3g1T2tuYTFlS2NBZk0vV1V4?=
 =?utf-8?B?WDdmTFYwMW8vRGdxYWcvMXFxaHBGcXFoTjhCK2pVRURGNDQxUVBhODJrMVJt?=
 =?utf-8?B?djV0ZnVvQTV6d2RBR3g1cExNV2szYmlvUURmSkhsVXh6Z2JsZWtaaVF4Vlph?=
 =?utf-8?B?dERXbll0RmpSOWU5UXFKWWxCRFQzdWdEcHlrM0E4WXN1dDVzWFRCRWU2ZnUw?=
 =?utf-8?Q?x7divzGqBACIVkdmaD2hI8Wtu?=
X-OriginatorOrg: axentia.se
X-MS-Exchange-CrossTenant-Network-Message-Id: 587f2cf1-3c00-47b9-7a98-08da91a1496b
X-MS-Exchange-CrossTenant-AuthSource: AM0PR02MB4436.eurprd02.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 08 Sep 2022 13:51:53.7951
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 4ee68585-03e1-4785-942a-df9c1871a234
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: dGhkTsgXVO/MUEXLWGAjL9tTv48zQEfhXcoD4Opncohu0Akrd0b5owt++jstPmdh
X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM9PR02MB6674
X-Spam-Status: No, score=-5.3 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,NICE_REPLY_A,RCVD_IN_MSPIKE_H2,
        SPF_HELO_PASS,SPF_PASS,T_SCC_BODY_TEXT_LINE autolearn=ham
        autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Hi!

The subject is the wrong, you are only fixing one comment and
you are only fixing one driver so you could just as well mention
that driver explicitly in the prefix with "i2c: viperboard: ...".

2022-09-08 at 15:21, wangjianli wrote:
> Delete the redundant word 'to'.
> 
> Signed-off-by: wangjianli <wangjianli@cdjrlc.com>
> ---
>  drivers/i2c/busses/i2c-viperboard.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/i2c/busses/i2c-viperboard.c b/drivers/i2c/busses/i2c-viperboard.c
> index 8b5322c3bce0..8f87394abbcf 100644
> --- a/drivers/i2c/busses/i2c-viperboard.c
> +++ b/drivers/i2c/busses/i2c-viperboard.c
> @@ -318,7 +318,7 @@ static int vprbrd_i2c_xfer(struct i2c_adapter *i2c, struct i2c_msg *msgs,
>  			amsg->addr = pmsg->addr;
>  			amsg->unknown1 = 0x00;
>  			amsg->len = cpu_to_le16(pmsg->len);
> -			/* send the addr, the data goes to to board */
> +			/* send the addr, the data goes to board */

I'd change it to "goes to the board" instead. 

Cheers,
Peter

>  			ret = vprbrd_i2c_addr(vb->usb_dev, amsg);
>  			if (ret < 0)
>  				error = ret;