kernel: free does not return errors

If there are no reasons for a *_free() function to return an error, change the return type to void Signed-off-by: Federico Vaga <federico.vaga@cern.ch>

kernel: free does not return errors
If there are no reasons for a *_free() function to return an error, change the return type to void Signed-off-by: Federico Vaga <federico.vaga@cern.ch>
a9b4d07c · Federico Vaga · 38708c2a · a9b4d07c · a9b4d07c
Commit a9b4d07c authored Jul 04, 2014 by Federico Vaga
Hide whitespace changes
Inline Side-by-side

Showing with 3 additions and 4 deletions

svec-vic.c kernel/svec-vic.c +2 -3

svec.h kernel/svec.h +1 -1

No files found.
--- a/kernel/svec-vic.c
+++ b/kernel/svec-vic.c
@@ -196,7 +196,8 @@ static inline int vic_handler_count(struct vic_irq_controller *vic)
       return count;
 }

-int svec_vic_irq_free(struct svec_dev *svec, unsigned long id)
+
+void svec_vic_irq_free(struct svec_dev *svec, unsigned long id)
 {
 	int i;

@@ -218,8 +219,6 @@ int svec_vic_irq_free(struct svec_dev *svec, unsigned long id)
 		svec_vic_exit(svec->vic);
 		svec->vic = NULL;
 	}
-
-	return 0;
 }

 void svec_vic_irq_ack(struct svec_dev *svec, unsigned long id)

--- a/kernel/svec.h
+++ b/kernel/svec.h
@@ -147,7 +147,7 @@ int svec_load_golden(struct svec_dev *svec);
 /* VIC interrupt controller stuff */
 irqreturn_t svec_vic_irq_dispatch(struct svec_dev *svec);
 int svec_vic_irq_request(struct svec_dev *svec, struct fmc_device *fmc, unsigned long id, irq_handler_t handler);
-int svec_vic_irq_free(struct svec_dev *svec, unsigned long id);
+void svec_vic_irq_free(struct svec_dev *svec, unsigned long id);
 void svec_vic_irq_ack(struct svec_dev *svec, unsigned long id);

 /* Generic IRQ routines */