minigbm: add clang-format and presubmit hooks

Running the presubmit.sh script will apply our rules to every file in
the repo. Exclude gbm.h from the formatting requirements since this file
was taken from other open-source projects, and diffing will be easier
without our formatting rules.

In addition, special case drivers where the order of includes matters.

BUG=none
TEST=Verified the following commands succeed:

emerge-cyan minigbm/arc-cros-gralloc
emerge-oak minigbm/arc-cros-gralloc
emerge-veyron_minnie-cheets minigbm/arc-cros-gralloc
emerge-peach_pi minigbm
emerge-nyan_big minigbm
emerge-jadeite minigbm

Change-Id: I6ce93fb1930da254d13d5017766c17341870ccc9
Reviewed-on: https://chromium-review.googlesource.com/447319
Commit-Ready: Gurchetan Singh <gurchetansingh@chromium.org>
Tested-by: Gurchetan Singh <gurchetansingh@chromium.org>
Reviewed-by: Gurchetan Singh <gurchetansingh@chromium.org>

1 files changed, 31 insertions, 49 deletions

diff --git a/rockchip.c b/rockchip.c
index 42d3f76..4d10581 100644
--- a/rockchip.c
+++ b/rockchip.c
@@ -7,25 +7,23 @@
 #ifdef DRV_ROCKCHIP
 
 #include <errno.h>
+#include <rockchip_drm.h>
 #include <stdbool.h>
 #include <stdio.h>
 #include <string.h>
 #include <sys/mman.h>
 #include <xf86drm.h>
-#include <rockchip_drm.h>
 
 #include "drv_priv.h"
 #include "helpers.h"
 #include "util.h"
 
-static const uint32_t supported_formats[] = {
-	DRM_FORMAT_ABGR8888, DRM_FORMAT_ARGB8888, DRM_FORMAT_NV12,
-	DRM_FORMAT_RGB565, DRM_FORMAT_XBGR8888, DRM_FORMAT_XRGB8888,
-	DRM_FORMAT_YVU420, DRM_FORMAT_YVU420_ANDROID
-};
+static const uint32_t supported_formats[] = { DRM_FORMAT_ABGR8888, DRM_FORMAT_ARGB8888,
+					      DRM_FORMAT_NV12,     DRM_FORMAT_RGB565,
+					      DRM_FORMAT_XBGR8888, DRM_FORMAT_XRGB8888,
+					      DRM_FORMAT_YVU420,   DRM_FORMAT_YVU420_ANDROID };
 
-static int afbc_bo_from_format(struct bo *bo, uint32_t width, uint32_t height,
-			       uint32_t format)
+static int afbc_bo_from_format(struct bo *bo, uint32_t width, uint32_t height, uint32_t format)
 {
 	/* We've restricted ourselves to four bytes per pixel. */
 	const uint32_t pixel_size = 4;
@@ -55,10 +53,8 @@ static int afbc_bo_from_format(struct bo *bo, uint32_t width, uint32_t height,
 	 * alignement for the body plane. */
 	const uint32_t body_plane_alignment = 1024;
 
-	const uint32_t body_plane_offset =
-		ALIGN(header_plane_size, body_plane_alignment);
-	const uint32_t total_size =
-		body_plane_offset + body_plane_size;
+	const uint32_t body_plane_offset = ALIGN(header_plane_size, body_plane_alignment);
+	const uint32_t total_size = body_plane_offset + body_plane_size;
 
 	bo->strides[0] = width_in_blocks * block_width * pixel_size;
 	bo->sizes[0] = total_size;
@@ -82,16 +78,13 @@ static int rockchip_add_kms_item(struct driver *drv, const struct kms_item *item
 	for (i = 0; i < drv->backend->combos.size; i++) {
 		combo = &drv->backend->combos.data[i];
 		if (combo->format == item->format) {
-			if (item->modifier ==
-			    DRM_FORMAT_MOD_CHROMEOS_ROCKCHIP_AFBC) {
-				flags = BO_USE_RENDERING | BO_USE_SCANOUT |
-					BO_USE_TEXTURE;
+			if (item->modifier == DRM_FORMAT_MOD_CHROMEOS_ROCKCHIP_AFBC) {
+				flags = BO_USE_RENDERING | BO_USE_SCANOUT | BO_USE_TEXTURE;
 				metadata.modifier = item->modifier;
 				metadata.tiling = 0;
 				metadata.priority = 2;
 
-				ret = drv_add_combination(drv, item[i].format,
-							  &metadata, flags);
+				ret = drv_add_combination(drv, item[i].format, &metadata, flags);
 				if (ret)
 					return ret;
 			} else {
@@ -114,16 +107,13 @@ static int rockchip_init(struct driver *drv)
 	metadata.priority = 1;
 	metadata.modifier = DRM_FORMAT_MOD_NONE;
 
-	ret = drv_add_combinations(drv, supported_formats,
-				   ARRAY_SIZE(supported_formats), &metadata,
+	ret = drv_add_combinations(drv, supported_formats, ARRAY_SIZE(supported_formats), &metadata,
 				   BO_COMMON_USE_MASK);
 	if (ret)
 		return ret;
 
-	drv_modify_combination(drv, DRM_FORMAT_XRGB8888, &metadata,
-			       BO_USE_CURSOR | BO_USE_SCANOUT);
-	drv_modify_combination(drv, DRM_FORMAT_ARGB8888, &metadata,
-			       BO_USE_CURSOR | BO_USE_SCANOUT);
+	drv_modify_combination(drv, DRM_FORMAT_XRGB8888, &metadata, BO_USE_CURSOR | BO_USE_SCANOUT);
+	drv_modify_combination(drv, DRM_FORMAT_ARGB8888, &metadata, BO_USE_CURSOR | BO_USE_SCANOUT);
 
 	items = drv_query_kms(drv, &num_items);
 	if (!items || !num_items)
@@ -152,10 +142,8 @@ static bool has_modifier(const uint64_t *list, uint32_t count, uint64_t modifier
 	return false;
 }
 
-static int rockchip_bo_create_with_modifiers(struct bo *bo,
-					     uint32_t width, uint32_t height,
-					     uint32_t format,
-					     const uint64_t *modifiers,
+static int rockchip_bo_create_with_modifiers(struct bo *bo, uint32_t width, uint32_t height,
+					     uint32_t format, const uint64_t *modifiers,
 					     uint32_t count)
 {
 	int ret;
@@ -170,11 +158,9 @@ static int rockchip_bo_create_with_modifiers(struct bo *bo,
 		uint32_t aligned_height = DIV_ROUND_UP(h_mbs * 16 * 3, 2);
 
 		drv_bo_from_format(bo, aligned_width, height, format);
-		bo->total_size = bo->strides[0] * aligned_height
-				 + w_mbs * h_mbs * 128;
+		bo->total_size = bo->strides[0] * aligned_height + w_mbs * h_mbs * 128;
 	} else if (width <= 2560 &&
-		   has_modifier(modifiers, count,
-				DRM_FORMAT_MOD_CHROMEOS_ROCKCHIP_AFBC)) {
+		   has_modifier(modifiers, count, DRM_FORMAT_MOD_CHROMEOS_ROCKCHIP_AFBC)) {
 		/* If the caller has decided they can use AFBC, always
 		 * pick that */
 		afbc_bo_from_format(bo, width, height, format);
@@ -193,8 +179,7 @@ static int rockchip_bo_create_with_modifiers(struct bo *bo,
 		 * luma plane to 128 bytes.
 		 */
 		stride = drv_stride_from_format(format, width, 0);
-		if (format == DRM_FORMAT_YVU420 ||
-		    format == DRM_FORMAT_YVU420_ANDROID)
+		if (format == DRM_FORMAT_YVU420 || format == DRM_FORMAT_YVU420_ANDROID)
 			stride = ALIGN(stride, 128);
 		else
 			stride = ALIGN(stride, 64);
@@ -205,12 +190,12 @@ static int rockchip_bo_create_with_modifiers(struct bo *bo,
 	memset(&gem_create, 0, sizeof(gem_create));
 	gem_create.size = bo->total_size;
 
-	ret = drmIoctl(bo->drv->fd, DRM_IOCTL_ROCKCHIP_GEM_CREATE,
-		       &gem_create);
+	ret = drmIoctl(bo->drv->fd, DRM_IOCTL_ROCKCHIP_GEM_CREATE, &gem_create);
 
 	if (ret) {
 		fprintf(stderr, "drv: DRM_IOCTL_ROCKCHIP_GEM_CREATE failed "
-				"(size=%llu)\n", gem_create.size);
+				"(size=%llu)\n",
+			gem_create.size);
 		return ret;
 	}
 
@@ -220,13 +205,13 @@ static int rockchip_bo_create_with_modifiers(struct bo *bo,
 	return 0;
 }
 
-static int rockchip_bo_create(struct bo *bo, uint32_t width, uint32_t height,
-			      uint32_t format, uint32_t flags)
+static int rockchip_bo_create(struct bo *bo, uint32_t width, uint32_t height, uint32_t format,
+			      uint32_t flags)
 {
 	uint64_t modifiers[] = { DRM_FORMAT_MOD_NONE };
 
-	return rockchip_bo_create_with_modifiers(bo, width, height, format,
-						 modifiers, ARRAY_SIZE(modifiers));
+	return rockchip_bo_create_with_modifiers(bo, width, height, format, modifiers,
+						 ARRAY_SIZE(modifiers));
 }
 
 static void *rockchip_bo_map(struct bo *bo, struct map_info *data, size_t plane)
@@ -242,18 +227,16 @@ static void *rockchip_bo_map(struct bo *bo, struct map_info *data, size_t plane)
 	memset(&gem_map, 0, sizeof(gem_map));
 	gem_map.handle = bo->handles[0].u32;
 
-	ret = drmIoctl(bo->drv->fd, DRM_IOCTL_ROCKCHIP_GEM_MAP_OFFSET,
-		       &gem_map);
+	ret = drmIoctl(bo->drv->fd, DRM_IOCTL_ROCKCHIP_GEM_MAP_OFFSET, &gem_map);
 	if (ret) {
-		fprintf(stderr,
-			"drv: DRM_IOCTL_ROCKCHIP_GEM_MAP_OFFSET failed\n");
+		fprintf(stderr, "drv: DRM_IOCTL_ROCKCHIP_GEM_MAP_OFFSET failed\n");
 		return MAP_FAILED;
 	}
 
 	data->length = bo->total_size;
 
-	return mmap(0, bo->total_size, PROT_READ | PROT_WRITE, MAP_SHARED,
-		    bo->drv->fd, gem_map.offset);
+	return mmap(0, bo->total_size, PROT_READ | PROT_WRITE, MAP_SHARED, bo->drv->fd,
+		    gem_map.offset);
 }
 
 static uint32_t rockchip_resolve_format(uint32_t format)
@@ -269,8 +252,7 @@ static uint32_t rockchip_resolve_format(uint32_t format)
 	}
 }
 
-struct backend backend_rockchip =
-{
+struct backend backend_rockchip = {
 	.name = "rockchip",
 	.init = rockchip_init,
 	.bo_create = rockchip_bo_create,