[PATCH 2/2] PCI/pwrctrl: tc9563: Remove unnecessary semicolons

Manivannan Sadhasivam posted 2 patches 1 week, 4 days ago
[PATCH 2/2] PCI/pwrctrl: tc9563: Remove unnecessary semicolons
Posted by Manivannan Sadhasivam 1 week, 4 days ago
As reported by the LKP bot, semicolons are not needed at the end of the
switch and if blocks. Hence, remove them.

This fixes the below cocci warnings:

  cocci warnings: (new ones prefixed by >>)
  >> drivers/pci/pwrctrl/pci-pwrctrl-tc9563.c:351:2-3: Unneeded semicolon
     drivers/pci/pwrctrl/pci-pwrctrl-tc9563.c:414:2-3: Unneeded semicolon
     drivers/pci/pwrctrl/pci-pwrctrl-tc9563.c:316:2-3: Unneeded semicolon

Reported-by: kernel test robot <lkp@intel.com>
Closes: https://lore.kernel.org/oe-kbuild-all/202511200555.M4TX84jK-lkp@intel.com
Signed-off-by: Manivannan Sadhasivam <mani@kernel.org>
---
 drivers/pci/pwrctrl/pci-pwrctrl-tc9563.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/drivers/pci/pwrctrl/pci-pwrctrl-tc9563.c b/drivers/pci/pwrctrl/pci-pwrctrl-tc9563.c
index 46339a23204f..ec423432ac65 100644
--- a/drivers/pci/pwrctrl/pci-pwrctrl-tc9563.c
+++ b/drivers/pci/pwrctrl/pci-pwrctrl-tc9563.c
@@ -313,7 +313,7 @@ static int tc9563_pwrctrl_set_tx_amplitude(struct tc9563_pwrctrl_ctx *ctx,
 		break;
 	default:
 		return -EINVAL;
-	};
+	}
 
 	struct tc9563_pwrctrl_reg_setting tx_amp_seq[] = {
 		{TC9563_PORT_ACCESS_ENABLE, port_access},
@@ -348,7 +348,7 @@ static int tc9563_pwrctrl_disable_dfe(struct tc9563_pwrctrl_ctx *ctx,
 		break;
 	default:
 		return -EINVAL;
-	};
+	}
 
 	struct tc9563_pwrctrl_reg_setting disable_dfe_seq[] = {
 		{TC9563_PORT_ACCESS_ENABLE, port_access},
@@ -411,7 +411,7 @@ static int tc9563_pwrctrl_parse_device_dt(struct tc9563_pwrctrl_ctx *ctx, struct
 	if (!of_device_is_available(node)) {
 		cfg->disable_port = true;
 		return 0;
-	};
+	}
 
 	ret = of_property_read_u32(node, "aspm-l0s-entry-delay-ns", &cfg->l0s_delay);
 	if (ret && ret != -EINVAL)
-- 
2.48.1
Re: [PATCH 2/2] PCI/pwrctrl: tc9563: Remove unnecessary semicolons
Posted by Bartosz Golaszewski 1 week, 4 days ago
On Thu, 20 Nov 2025 07:51:16 +0100, Manivannan Sadhasivam
<manivannan.sadhasivam@oss.qualcomm.com> said:
> As reported by the LKP bot, semicolons are not needed at the end of the
> switch and if blocks. Hence, remove them.
>
> This fixes the below cocci warnings:
>
>   cocci warnings: (new ones prefixed by >>)
>   >> drivers/pci/pwrctrl/pci-pwrctrl-tc9563.c:351:2-3: Unneeded semicolon
>      drivers/pci/pwrctrl/pci-pwrctrl-tc9563.c:414:2-3: Unneeded semicolon
>      drivers/pci/pwrctrl/pci-pwrctrl-tc9563.c:316:2-3: Unneeded semicolon
>
> Reported-by: kernel test robot <lkp@intel.com>
> Closes: https://lore.kernel.org/oe-kbuild-all/202511200555.M4TX84jK-lkp@intel.com
> Signed-off-by: Manivannan Sadhasivam <mani@kernel.org>
> ---
>  drivers/pci/pwrctrl/pci-pwrctrl-tc9563.c | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
>

Acked-by: Bartosz Golaszewski <bartosz.golaszewski@linaro.org>