From: Yannis Damigos
This patch fixes the following comparison with NULL issues:
CHECK: Comparison to NULL could be written "compatible[i].name"
+ for (i = 0; compatible[i].name != NULL; i++) {
CHECK: Comparison to NULL could be written "!compatible[i].name"
+ if (compatible[i].name
From: Yannis Damigos
This patch removes the return statement of void function
WARNING: void function return statements are not generally useful
+ return;
+}
Signed-off-by: Yannis Damigos
---
drivers/staging/android/ion/ion_of.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/
From: Yannis Damigos
This patch fixes the following whitespace issues:
CHECK: Alignment should match open parenthesis
+int ion_parse_dt_heap_common(struct device_node *heap_node,
+ struct ion_platform_heap *heap,
CHECK: Alignment should match open parenthesis
+ pr_in
From: Yannis Damigos
This patch fixes the following blank line issues:
CHECK: Please don't use multiple blank lines
+
+
CHECK: Please use a blank line after function/struct/union/enum
declarations
+}
+RESERVEDMEM_OF_DECLARE(ion, "ion-region", rmem_ion_setup);
Signed-off-by: Yannis Damigos
---
From: Yannis Damigos
This patch fixes the following switch statement indentation issue:
ERROR: switch and case should be at the same indent
+ switch (heap->type) {
+ case ION_HEAP_TYPE_CARVEOUT:
+ case ION_HEAP_TYPE_CHUNK:
[...]
+ default:
Signed-
