linux-kernel - Re: [PATCH] docs: devfreq: Fix typo 'enabeld' to 'enabled' in devfreq-event.c

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]

Message-ID: <20250527055635.qgpf2v7artictbf2@lcpd911>
Date: Tue, 27 May 2025 11:26:35 +0530
From: Dhruva Gole <d-gole@...com>
To: bhanuseshukumar <bhanuseshukumar@...il.com>
CC: <cw00.choi@...sung.com>, <myungjoo.ham@...sung.com>,
        <kyungmin.park@...sung.com>, <skhan@...uxfoundation.org>,
        <linux-pm@...r.kernel.org>, <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] docs: devfreq: Fix typo 'enabeld' to 'enabled' in
 devfreq-event.c

On May 26, 2025 at 19:08:39 +0530, bhanuseshukumar wrote:
> Fixed a typo in  description of the function 
> 
> This is me trying my way into opensource linux kernel contribution
> journey beginning with this  first patch of trivial spelling 
> mistake fix. 

We appreciate your contribution, and welcome you to the community :)

However, this is not something that's meaningful to be part of a "git
log". commit messages in the git log should convey explanation or
reasoning behind purely the content of the patch itself.
So it's fine in this case to write just "Fix a typo...."

Read more about this here:
https://docs.kernel.org/process/submitting-patches.html#explanation-body

> 
> Signed-off-by: bhanuseshukumar <bhanuseshukumar@...il.com>
> ---
>  drivers/devfreq/devfreq-event.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/devfreq/devfreq-event.c b/drivers/devfreq/devfreq-event.c
> index 70219099c604..020fe30ed5d9 100644
> --- a/drivers/devfreq/devfreq-event.c
> +++ b/drivers/devfreq/devfreq-event.c
> @@ -100,7 +100,7 @@ EXPORT_SYMBOL_GPL(devfreq_event_disable_edev);
>   * @edev	: the devfreq-event device
>   *
>   * Note that this function check whether devfreq-event dev is enabled or not.
> - * If return true, the devfreq-event dev is enabeld. If return false, the
> + * If return true, the devfreq-event dev is enabled. If return false, the
>   * devfreq-event dev is disabled.
>   */
>  bool devfreq_event_is_enabled(struct devfreq_event_dev *edev)
> -- 
> 2.34.1
> 
> 

-- 
Best regards,
Dhruva Gole
Texas Instruments Incorporated