minigbm: Fix switch statements
Our switch statements were inconsistent. Let's use kernel
style switch statements since we already follow many kernel
coding guidelines in this repo (8-space tabs, mostly 80
line limit).
Change-Id: I2b6ccc2369160bf49bbdf6479aebec7dada2e2fa
Reviewed-on: https://chromium-review.googlesource.com/377883
Commit-Ready: Gurchetan Singh <gurchetansingh@chromium.org>
Tested-by: Gurchetan Singh <gurchetansingh@chromium.org>
Reviewed-by: Stéphane Marchesin <marcheu@chromium.org>
diff --git a/i915.c b/i915.c
index 4942246..e2646ad 100644
--- a/i915.c
+++ b/i915.c
@@ -75,26 +75,26 @@
struct i915_device *i915_drv = (struct i915_device *)drv->priv;
uint32_t width_alignment = 4, height_alignment = 4;
- switch(tiling_mode) {
- default:
- case I915_TILING_NONE:
- width_alignment = 64 / bpp;
- break;
+ switch (tiling_mode) {
+ default:
+ case I915_TILING_NONE:
+ width_alignment = 64 / bpp;
+ break;
- case I915_TILING_X:
+ case I915_TILING_X:
+ width_alignment = 512 / bpp;
+ height_alignment = 8;
+ break;
+
+ case I915_TILING_Y:
+ if (i915_drv->gen == 3) {
width_alignment = 512 / bpp;
height_alignment = 8;
- break;
-
- case I915_TILING_Y:
- if (i915_drv->gen == 3) {
- width_alignment = 512 / bpp;
- height_alignment = 8;
- } else {
- width_alignment = 128 / bpp;
- height_alignment = 32;
- }
- break;
+ } else {
+ width_alignment = 128 / bpp;
+ height_alignment = 32;
+ }
+ break;
}
if (i915_drv->gen > 3) {
@@ -222,13 +222,13 @@
drv_format_t i915_resolve_format(drv_format_t format)
{
switch (format) {
- case DRV_FORMAT_FLEX_IMPLEMENTATION_DEFINED:
- /*HACK: See b/28671744 */
- return DRV_FORMAT_XBGR8888;
- case DRV_FORMAT_FLEX_YCbCr_420_888:
- return DRV_FORMAT_YVU420;
- default:
- return format;
+ case DRV_FORMAT_FLEX_IMPLEMENTATION_DEFINED:
+ /*HACK: See b/28671744 */
+ return DRV_FORMAT_XBGR8888;
+ case DRV_FORMAT_FLEX_YCbCr_420_888:
+ return DRV_FORMAT_YVU420;
+ default:
+ return format;
}
}