This patch fixes the following checkpatch warnings: WARNING:RETURN_VOID: void function return statements are not generally useful WARNING:TRAILING_SEMICOLON: macros should not use a trailing semicolon Signed-off-by: Stefan Herdler <herdler@xxxxxxxxxxxxxx> --- drivers/media/pci/ttpci/budget.c | 1 - drivers/media/pci/ttpci/budget.h | 2 +- 2 files changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/media/pci/ttpci/budget.c b/drivers/media/pci/ttpci/budget.c index f14b90de6..50fce65ce 100644 --- a/drivers/media/pci/ttpci/budget.c +++ b/drivers/media/pci/ttpci/budget.c @@ -781,7 +781,6 @@ static void frontend_init(struct budget *budget) printk("budget: Frontend registration failed!\n"); dvb_frontend_detach(budget->dvb_frontend); budget->dvb_frontend = NULL; - return; } static int budget_attach(struct saa7146_dev *dev, struct saa7146_pci_extension_data *info) diff --git a/drivers/media/pci/ttpci/budget.h b/drivers/media/pci/ttpci/budget.h index 4579243c2..ae6de6a19 100644 --- a/drivers/media/pci/ttpci/budget.h +++ b/drivers/media/pci/ttpci/budget.h @@ -89,7 +89,7 @@ static struct budget_info x_var ## _info = { \ .type = x_type }; \ static struct saa7146_pci_extension_data x_var = { \ .ext_priv = &x_var ## _info, \ - .ext = &budget_extension }; + .ext = &budget_extension } #define BUDGET_TT 0 #define BUDGET_TT_HW_DISEQC 1 -- 2.34.0