Skip to contentSkip to conversation

staging: media: av7110: remove unnecessary parentheses from return

PATCH Superseded Ashmit Kumar 1 participant 1 message Aug 4

Subsystems (2)
Media (V4L/DVB) Staging
lore
Share & tools
lore

Ashmit Kumar

author
#0 lore raw

PATCH v3 1/1 staging: media: av7110: remove unnecessary parentheses from return

The Linux kernel coding style specifies that return is not a function and therefore parentheses are not required.

Signed-off-by Ashmit Kumar <[email protected]>

---
Changes in v3:
- Formatted as a standalone patch (removed 2/3 series numbering).

 drivers/staging/media/av7110/sp8870.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
drivers/staging/media/av7110/sp8870.c +2 -2
index 29fb4934c03..83c6ee305d6 100644
@@ -92,7 +92,7 @@ static int sp8870_readreg(struct sp8870_state *state, u16 reg)
 		return -1;
 	}
 
-	return (b1[0] << 8 | b1[1]);
+	return b1[0] << 8 | b1[1];
 }
 
 static int sp8870_firmware_upload(struct sp8870_state *state, const struct firmware *fw)

@@ -167,7 +167,7 @@ static void sp8870_microcontroller_start(struct sp8870_state *state)
 
 static int sp8870_read_data_valid_signal(struct sp8870_state *state)
 {
-	return (sp8870_readreg(state, 0x0D02) > 0);
+	return sp8870_readreg(state, 0x0D02) > 0;
 }
 
 static int configure_reg0xc05(struct dtv_frontend_properties *p, u16 *reg0xc05)
Signature · 1 line
2.51.0.windows.1