linux-kernel - Re: [greybus-dev] [PATCH] staging: greybus: fix spelling mistake "entires" -> "entries"

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:   Fri, 14 Sep 2018 13:07:01 +0100
From:   Colin Ian King <colin.king@...onical.com>
To:     Alex Elder <elder@...aro.org>, Johan Hovold <johan@...nel.org>,
        Alex Elder <elder@...nel.org>,
        Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
        greybus-dev@...ts.linaro.org, devel@...verdev.osuosl.org
Cc:     kernel-janitors@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [greybus-dev] [PATCH] staging: greybus: fix spelling mistake
 "entires" -> "entries"

On 14/09/18 12:43, Alex Elder wrote:
> On 09/14/2018 06:24 AM, Colin King wrote:
>> From: Colin Ian King <colin.king@...onical.com>
>>
>> Trivial fix to spelling mistake
> 
> I hate to have two-character fixes to documentation like this.  I.e.,
> as long as you're touching the README file it might have been nice to
> review it and look for other things.  I suspect you found this in the
> program output though, and also noticed it in the README file.

Indeed, it was the program output where the mistake was first spotted.

> 
> So...  Looks good to me.
> 
> Reviewed-by: Alex Elder <elder@...aro.org>
> 
>>
>> Signed-off-by: Colin Ian King <colin.king@...onical.com>
>> ---
>>  drivers/staging/greybus/tools/README.loopback | 2 +-
>>  drivers/staging/greybus/tools/loopback_test.c | 2 +-
>>  2 files changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/drivers/staging/greybus/tools/README.loopback b/drivers/staging/greybus/tools/README.loopback
>> index 845b08dc4696..070a510cbe7c 100644
>> --- a/drivers/staging/greybus/tools/README.loopback
>> +++ b/drivers/staging/greybus/tools/README.loopback
>> @@ -79,7 +79,7 @@ Here is the summary of the available options:
>>     -t     must be one of the test names - sink, transfer or ping
>>     -i     iteration count - the number of iterations to run the test over
>>   Optional arguments
>> -   -S     sysfs location - location for greybus 'endo' entires default /sys/bus/greybus/devices/
>> +   -S     sysfs location - location for greybus 'endo' entries default /sys/bus/greybus/devices/
>>     -D     debugfs location - location for loopback debugfs entries default /sys/kernel/debug/gb_loopback/
>>     -s     size of data packet to send during test - defaults to zero
>>     -m     mask - a bit mask of connections to include example: -m 8 = 4th connection -m 9 = 1st and 4th connection etc
>> diff --git a/drivers/staging/greybus/tools/loopback_test.c b/drivers/staging/greybus/tools/loopback_test.c
>> index b82e2befe935..2fa88092514d 100644
>> --- a/drivers/staging/greybus/tools/loopback_test.c
>> +++ b/drivers/staging/greybus/tools/loopback_test.c
>> @@ -192,7 +192,7 @@ void usage(void)
>>  	"   -t     must be one of the test names - sink, transfer or ping\n"
>>  	"   -i     iteration count - the number of iterations to run the test over\n"
>>  	" Optional arguments\n"
>> -	"   -S     sysfs location - location for greybus 'endo' entires default /sys/bus/greybus/devices/\n"
>> +	"   -S     sysfs location - location for greybus 'endo' entries default /sys/bus/greybus/devices/\n"
>>  	"   -D     debugfs location - location for loopback debugfs entries default /sys/kernel/debug/gb_loopback/\n"
>>  	"   -s     size of data packet to send during test - defaults to zero\n"
>>  	"   -m     mask - a bit mask of connections to include example: -m 8 = 4th connection -m 9 = 1st and 4th connection etc\n"
>>
>