[RESEND] macintosh: Fix spelling mistake "toogle" to "toggle"

Christophe Leroy christophe.leroy at csgroup.eu
Tue Dec 3 20:19:46 AEDT 2024


Hi,

Le 03/12/2024 à 09:42, Zhu Jun a écrit :
> There is a spelling mistake, Please fix it.

This text doesn't belong to a commit message, the patch cannot be 
applied with such a message.

By the way, can you explain why it is so urgent to fix a minor typo like 
this on a so old driver that hasn't got more than half a dozen of 
commits in the last 10 years, none of them been functionnal changes ?

Anybody who reads this comment will understand it despite the typo. To 
be honest most people won't even notice the type while reading.

Before sending such patches, have in mind the cost of handling such a 
change compared to its real benefit.


> 
> Signed-off-by: Zhu Jun <zhujun2 at cmss.chinamobile.com>
> ---
>   drivers/macintosh/adbhid.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/macintosh/adbhid.c b/drivers/macintosh/adbhid.c
> index b2fe7a3dc471..943a7123786a 100644
> --- a/drivers/macintosh/adbhid.c
> +++ b/drivers/macintosh/adbhid.c
> @@ -604,7 +604,7 @@ adbhid_buttons_input(unsigned char *data, int nb, int autopoll)
>   
>   			case 0x7f:
>   			case 0xff:
> -				/* keypad overlay toogle */
> +				/* keypad overlay toggle */
>   				break;
>   
>   			default:



More information about the Linuxppc-dev mailing list