Skip to contentSkip to conversation

ARC: dw2 unwind: Fix trailing semicolon

PATCH Applied Luis de Bethencourt 2 participants 4 messages Jan 23, 2018 Synopsys ARC architecture

lore
Share & tools
lore

Luis de Bethencourt

author
#0 lore raw

PATCH 1/2 ARC: dw2 unwind: Fix trailing semicolon

The trailing semicolon is an empty statement that does no operation. Removing it since it doesn't do anything.

Signed-off-by Luis de Bethencourt <luisbg at kernel.org>

---

Hi,

After fixing the same thing in drivers/staging/rtl8723bs/, Joe Perches suggested I fix it treewide [0].

Best regards
Luis

[0] http://driverdev.linuxdriverproject.org/pipermail/driverdev-devel/2018-January/115410.html
[1] http://driverdev.linuxdriverproject.org/pipermail/driverdev-devel/2018-January/115390.html

 arch/arc/kernel/unwind.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
arch/arc/kernel/unwind.c +1 -1
index 333daab7def0..183391d4d33a 100644
@@ -366,7 +366,7 @@ static void init_unwind_hdr(struct unwind_table *table,
 	return;
 
 ret_err:
-	panic("Attention !!! Dwarf FDE parsing errors\n");;
+	panic("Attention !!! Dwarf FDE parsing errors\n");
 }
 
 #ifdef CONFIG_MODULES
Signature · 1 line
2.15.1
author
#1 lore raw

PATCH 2/2 ARC: boot log: Fix trailing semicolon

The trailing semicolon is an empty statement that does no operation. Removing it since it doesn't do anything.

Signed-off-by Luis de Bethencourt <luisbg at kernel.org>
---
 arch/arc/kernel/setup.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
arch/arc/kernel/setup.c +1 -1
index 9d27331fe69a..ec12fe1c2f07 100644
@@ -373,7 +373,7 @@ static void arc_chk_core_config(void)
 {
 	struct cpuinfo_arc *cpu = &cpuinfo_arc700[smp_processor_id()];
 	int saved = 0, present = 0;
-	char *opt_nm = NULL;;
+	char *opt_nm = NULL;
 
 	if (!cpu->extn.timer0)
 		panic("Timer0 is not present!\n");
Signature · 1 line
2.15.1
Applied
#2 lore raw

Re: [PATCH 2/2] ARC: boot log: Fix trailing semicolon

Quoted 4 lines · Luis de Bethencourt

The trailing semicolon is an empty statement that does no operation. Removing it since it doesn't do anything.

Signed-off-by Luis de Bethencourt<luisbg at kernel.org>

Applied to for-curr

Thx,
-Vineet

Applied
#3 lore raw
In reply to Luis de Bethencourt
Quoted 4 lines · Luis de Bethencourt

The trailing semicolon is an empty statement that does no operation. Removing it since it doesn't do anything.

Signed-off-by Luis de Bethencourt<luisbg at kernel.org>

Applied to for-curr

Thx,
-Vineet