Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1557854

[PATCH v2 2/4] staging: android/ion: Don't use return in void function

Show all headers | View raw

Return statements are superfluous in void functions, and checkpatch
complains about them too.

Remove offending return.

Signed-off-by: Bogdan Purcareata <bogdan.purcareata@gmail.com>
---
v1 -> v2
- add changelog

 drivers/staging/android/ion/ion_of.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/drivers/staging/android/ion/ion_of.c b/drivers/staging/android/ion/ion_of.c
index 46b2bb9..7791c70 100644
--- a/drivers/staging/android/ion/ion_of.c
+++ b/drivers/staging/android/ion/ion_of.c
@@ -161,7 +161,6 @@ static int rmem_ion_device_init(struct reserved_mem *rmem, struct device *dev)
 static void rmem_ion_device_release(struct reserved_mem *rmem,
 				    struct device *dev)
 {
-	return;
 }
 
 static const struct reserved_mem_ops rmem_dma_ops = {
-- 
2.7.4

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH v2 1/4] staging: android/ion: Split function call Bogdan Purcareata <bogdan.purcareata@gmail.com> - 2017-01-12 23:10 +0100
  [PATCH v2 4/4] staging: android/ion: Use variable names in header Bogdan Purcareata <bogdan.purcareata@gmail.com> - 2017-01-12 23:10 +0100
  [PATCH v2 2/4] staging: android/ion: Don't use return in void function Bogdan Purcareata <bogdan.purcareata@gmail.com> - 2017-01-12 23:10 +0100
  [PATCH v2 3/4] staging: android/ion: Align comment mark Bogdan Purcareata <bogdan.purcareata@gmail.com> - 2017-01-12 23:10 +0100

csiph-web