On Tue Aug 4 11:15:04 2026 +0000, Ashmit Kumar wrote:
> 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]>
> Signed-off-by: Hans Verkuil <[email protected]>
Patch committed.
Thanks,
Hans Verkuil
drivers/staging/media/av7110/sp8870.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
---
diff --git a/drivers/staging/media/av7110/sp8870.c
b/drivers/staging/media/av7110/sp8870.c
index 29fb4934c039..83c6ee305d62 100644
--- a/drivers/staging/media/av7110/sp8870.c
+++ b/drivers/staging/media/av7110/sp8870.c
@@ -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)
_______________________________________________
linuxtv-commits mailing list -- [email protected]
To unsubscribe send an email to [email protected]