From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932749AbcHJDky (ORCPT ); Tue, 9 Aug 2016 23:40:54 -0400 Received: from mail-pa0-f68.google.com ([209.85.220.68]:34983 "EHLO mail-pa0-f68.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932584AbcHJDkw (ORCPT ); Tue, 9 Aug 2016 23:40:52 -0400 Subject: Re: [PATCH] w1: fix timeout_us parameter description To: Evgeniy Polyakov , Greg Kroah-Hartman References: <1470664361-9363-1-git-send-email-weiyj.lk@gmail.com> <1871470783285@web30h.yandex.ru> Cc: "linux-kernel@vger.kernel.org" From: Wei Yongjun Message-ID: Date: Wed, 10 Aug 2016 11:22:22 +0800 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.2.0 MIME-Version: 1.0 In-Reply-To: <1871470783285@web30h.yandex.ru> Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi On 08/10/2016 06:54 AM, Evgeniy Polyakov wrote: > Hi > > 08.08.2016, 16:52, "Wei Yongjun" : >> Fix 'timeout_us' parameter description. >> >> Signed-off-by: Wei Yongjun >> --- >> drivers/w1/w1.c | 4 ++-- >> 1 file changed, 2 insertions(+), 2 deletions(-) >> >> diff --git a/drivers/w1/w1.c b/drivers/w1/w1.c >> index bb34362..e213c67 100644 >> --- a/drivers/w1/w1.c >> +++ b/drivers/w1/w1.c >> @@ -53,8 +53,8 @@ int w1_max_slave_ttl = 10; >> module_param_named(timeout, w1_timeout, int, 0); >> MODULE_PARM_DESC(timeout, "time in seconds between automatic slave searches"); >> module_param_named(timeout_us, w1_timeout_us, int, 0); >> -MODULE_PARM_DESC(timeout, "time in microseconds between automatic slave" >> - " searches"); >> +MODULE_PARM_DESC(timeout_us, >> + "time in microseconds between automatic slave searches"); > I believe there will be no harm to put it to on one line, even if it crosses some obscure very-long-line rule Maybe the bad patch description confused you, this patch the typo in the first argument of MODULE_PARM_DESC(), use timeout_us instead of timeout.