2022-11-02 06:25:46

by chen zhang

[permalink] [raw]
Subject: [PATCH v2] platform/x86: ISST: Fix typo in comments

Fix spelling typo in comments. Change "interace" to "interface".

Reported-by: k2ci <[email protected]>
Signed-off-by: chen zhang <[email protected]>
---
v2: update the commit
There is a bug with my company's mailbox with kylinos.cn, and sometimes
I can't receive reply emails. I cc my personal 163 mailbox, so that I can
receive feedback from reviewer on time.
---
drivers/platform/x86/intel/speed_select_if/isst_if_common.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/platform/x86/intel/speed_select_if/isst_if_common.c b/drivers/platform/x86/intel/speed_select_if/isst_if_common.c
index fd102678c75f..c1d7f4f38765 100644
--- a/drivers/platform/x86/intel/speed_select_if/isst_if_common.c
+++ b/drivers/platform/x86/intel/speed_select_if/isst_if_common.c
@@ -623,7 +623,7 @@ static long isst_if_def_ioctl(struct file *file, unsigned int cmd,

/* Lock to prevent module registration when already opened by user space */
static DEFINE_MUTEX(punit_misc_dev_open_lock);
-/* Lock to allow one share misc device for all ISST interace */
+/* Lock to allow one share misc device for all ISST interface */
static DEFINE_MUTEX(punit_misc_dev_reg_lock);
static int misc_usage_count;
static int misc_device_ret;
--
2.25.1



2022-11-02 15:14:09

by Randy Dunlap

[permalink] [raw]
Subject: Re: [PATCH v2] platform/x86: ISST: Fix typo in comments

Hi--

On 11/1/22 23:06, chen zhang wrote:
> Fix spelling typo in comments. Change "interace" to "interface".
>
> Reported-by: k2ci <[email protected]>
> Signed-off-by: chen zhang <[email protected]>
> ---
> v2: update the commit
> There is a bug with my company's mailbox with kylinos.cn, and sometimes
> I can't receive reply emails. I cc my personal 163 mailbox, so that I can
> receive feedback from reviewer on time.
> ---
> drivers/platform/x86/intel/speed_select_if/isst_if_common.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/platform/x86/intel/speed_select_if/isst_if_common.c b/drivers/platform/x86/intel/speed_select_if/isst_if_common.c
> index fd102678c75f..c1d7f4f38765 100644
> --- a/drivers/platform/x86/intel/speed_select_if/isst_if_common.c
> +++ b/drivers/platform/x86/intel/speed_select_if/isst_if_common.c
> @@ -623,7 +623,7 @@ static long isst_if_def_ioctl(struct file *file, unsigned int cmd,
>
> /* Lock to prevent module registration when already opened by user space */
> static DEFINE_MUTEX(punit_misc_dev_open_lock);
> -/* Lock to allow one share misc device for all ISST interace */
> +/* Lock to allow one share misc device for all ISST interface */

How about:

+/* Lock to allow one shared misc device for all ISST interfaces */

> static DEFINE_MUTEX(punit_misc_dev_reg_lock);
> static int misc_usage_count;
> static int misc_device_ret;

--
~Randy