diff mbox series

HID: amd_sfh: Convert comma to semicolon

Message ID 20240903024402.493709-1-nichen@iscas.ac.cn (mailing list archive)
State New
Delegated to: Jiri Kosina
Headers show
Series HID: amd_sfh: Convert comma to semicolon | expand

Commit Message

Chen Ni Sept. 3, 2024, 2:44 a.m. UTC
Replace a comma between expression statements by a semicolon.

Signed-off-by: Chen Ni <nichen@iscas.ac.cn>
---
 drivers/hid/amd-sfh-hid/sfh1_1/amd_sfh_init.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comments

Basavaraj Natikar Sept. 3, 2024, 9:41 a.m. UTC | #1
On 9/3/2024 8:14 AM, Chen Ni wrote:
> Replace a comma between expression statements by a semicolon.
>
> Signed-off-by: Chen Ni <nichen@iscas.ac.cn>
> ---
>  drivers/hid/amd-sfh-hid/sfh1_1/amd_sfh_init.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/hid/amd-sfh-hid/sfh1_1/amd_sfh_init.c b/drivers/hid/amd-sfh-hid/sfh1_1/amd_sfh_init.c
> index 621793d92464..db36d87d5634 100644
> --- a/drivers/hid/amd-sfh-hid/sfh1_1/amd_sfh_init.c
> +++ b/drivers/hid/amd-sfh-hid/sfh1_1/amd_sfh_init.c
> @@ -299,8 +299,8 @@ static void amd_sfh_set_ops(struct amd_mp2_dev *mp2)
>  
>  	sfh_interface_init(mp2);
>  	mp2_ops = mp2->mp2_ops;
> -	mp2_ops->clear_intr = amd_sfh_clear_intr_v2,
> -	mp2_ops->init_intr = amd_sfh_irq_init_v2,
> +	mp2_ops->clear_intr = amd_sfh_clear_intr_v2;
> +	mp2_ops->init_intr = amd_sfh_irq_init_v2;
>  	mp2_ops->suspend = amd_sfh_suspend;
>  	mp2_ops->resume = amd_sfh_resume;
>  	mp2_ops->remove = amd_mp2_pci_remove;

Looks good to me.

Acked-by: Basavaraj Natikar <Basavaraj.Natikar@amd.com>

Thanks,
--
Basavaraj
Jiri Kosina Sept. 3, 2024, 11:39 a.m. UTC | #2
On Tue, 3 Sep 2024, Chen Ni wrote:

> Replace a comma between expression statements by a semicolon.
> 
> Signed-off-by: Chen Ni <nichen@iscas.ac.cn>
> ---
>  drivers/hid/amd-sfh-hid/sfh1_1/amd_sfh_init.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/hid/amd-sfh-hid/sfh1_1/amd_sfh_init.c b/drivers/hid/amd-sfh-hid/sfh1_1/amd_sfh_init.c
> index 621793d92464..db36d87d5634 100644
> --- a/drivers/hid/amd-sfh-hid/sfh1_1/amd_sfh_init.c
> +++ b/drivers/hid/amd-sfh-hid/sfh1_1/amd_sfh_init.c
> @@ -299,8 +299,8 @@ static void amd_sfh_set_ops(struct amd_mp2_dev *mp2)
>  
>  	sfh_interface_init(mp2);
>  	mp2_ops = mp2->mp2_ops;
> -	mp2_ops->clear_intr = amd_sfh_clear_intr_v2,
> -	mp2_ops->init_intr = amd_sfh_irq_init_v2,
> +	mp2_ops->clear_intr = amd_sfh_clear_intr_v2;
> +	mp2_ops->init_intr = amd_sfh_irq_init_v2;
>  	mp2_ops->suspend = amd_sfh_suspend;
>  	mp2_ops->resume = amd_sfh_resume;

Applied, thanks.
diff mbox series

Patch

diff --git a/drivers/hid/amd-sfh-hid/sfh1_1/amd_sfh_init.c b/drivers/hid/amd-sfh-hid/sfh1_1/amd_sfh_init.c
index 621793d92464..db36d87d5634 100644
--- a/drivers/hid/amd-sfh-hid/sfh1_1/amd_sfh_init.c
+++ b/drivers/hid/amd-sfh-hid/sfh1_1/amd_sfh_init.c
@@ -299,8 +299,8 @@  static void amd_sfh_set_ops(struct amd_mp2_dev *mp2)
 
 	sfh_interface_init(mp2);
 	mp2_ops = mp2->mp2_ops;
-	mp2_ops->clear_intr = amd_sfh_clear_intr_v2,
-	mp2_ops->init_intr = amd_sfh_irq_init_v2,
+	mp2_ops->clear_intr = amd_sfh_clear_intr_v2;
+	mp2_ops->init_intr = amd_sfh_irq_init_v2;
 	mp2_ops->suspend = amd_sfh_suspend;
 	mp2_ops->resume = amd_sfh_resume;
 	mp2_ops->remove = amd_mp2_pci_remove;