Hi On 08/10/2016 06:54 AM, Evgeniy Polyakov wrote: > Hi > > 08.08.2016, 16:52, "Wei Yongjun" <weiyj...@gmail.com>: >> Fix 'timeout_us' parameter description. >> >> Signed-off-by: Wei Yongjun <weiyj...@gmail.com> >> --- >> 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.