This patch fixes two coding style issues found with checkpatch.pl. Both
of them are warnings.
uapi/ion.h had an unaligned block comment. ion/hisilicon/hi6220_ion.c
had unnecessary braces around a single statement block.
One warning in uapi/ion.h was left in place that wanted a typedef
(ion_user_handle_t) to be removed. I think the typedef makes sense so
I left it intact.
Signed-off-by: Antti Keränen <detegr@xxxxxxxxx>
---
drivers/staging/android/ion/hisilicon/hi6220_ion.c | 4 ++--
drivers/staging/android/uapi/ion.h | 2 +-
2 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/staging/android/ion/hisilicon/hi6220_ion.c b/drivers/staging/android/ion/hisilicon/hi6220_ion.c
index 659aa71..bd45151 100644
--- a/drivers/staging/android/ion/hisilicon/hi6220_ion.c
+++ b/drivers/staging/android/ion/hisilicon/hi6220_ion.c
@@ -82,9 +82,9 @@ static int hi6220_ion_remove(struct platform_device *pdev)
ipdev = platform_get_drvdata(pdev);
- for (i = 0; i < ipdev->data->nr; i++) {
+ for (i = 0; i < ipdev->data->nr; i++)
ion_heap_destroy(ipdev->heaps[i]);
- }
+
ion_destroy_platform_data(ipdev->data);
ion_device_destroy(ipdev->idev);
diff --git a/drivers/staging/android/uapi/ion.h b/drivers/staging/android/uapi/ion.h
index 647f130..e629394 100644
--- a/drivers/staging/android/uapi/ion.h
+++ b/drivers/staging/android/uapi/ion.h
@@ -57,7 +57,7 @@ enum ion_heap_type {
* cached, ion will do cache
* maintenance when the buffer is
* mapped for dma
- */
+ */
#define ION_FLAG_CACHED_NEEDS_SYNC 2 /*
* mappings of this buffer will created
* at mmap time, if this is set
--
2.9.3
_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
