glamo_fb: Cosmetic cleanup

Remove dead code and coding style fixes. Purely cosmetic.

Signed-off-by: Harald Welte <laforge@openmoko.org>

1 files changed, 21 insertions, 57 deletions

diff --git a/drivers/mfd/glamo/glamo-fb.c b/drivers/mfd/glamo/glamo-fb.c
index 7e800f571ce..7ffafda2dcf 100644
--- a/drivers/mfd/glamo/glamo-fb.c
+++ b/drivers/mfd/glamo/glamo-fb.c
@@ -1,6 +1,6 @@
 /* Smedia Glamo 336x/337x driver
  *
- * (C) 2007 by Openmoko, Inc.
+ * (C) 2007-2008 by Openmoko, Inc.
  * Author: Harald Welte <laforge@openmoko.org>
  * All rights reserved.
  *
@@ -237,11 +237,10 @@ enum orientation {
 };
 
 
-static void rotate_lcd(struct glamofb_handle *glamo,
-                       __u32 rotation)
+/* the caller has to enxure lock_cmd is held and we are in cmd mode */
+static void __rotate_lcd(struct glamofb_handle *glamo, __u32 rotation)
 {
 	int glamo_rot;
-	unsigned long flags;
 
 	switch (rotation) {
 		case FB_ROTATE_UR:
@@ -261,14 +260,6 @@ static void rotate_lcd(struct glamofb_handle *glamo,
 			break;
 	}
 
-	/*
- 	 * ha ha we are only called when we are in cmd mode already
-	 * printk(KERN_ERR"rotate_lcd spin_lock_irqsave\n");
-	 * spin_lock_irqsave(&glamo->lock_cmd, flags);
-	 *
-	 * if (glamofb_cmd_mode(glamo, 1))
-	 *		goto out_unlock;
-	 */
 	reg_set_bit_mask(glamo,
 			 GLAMO_REG_LCD_WIDTH,
 			 GLAMO_LCD_ROT_MODE_MASK,
@@ -278,12 +269,6 @@ static void rotate_lcd(struct glamofb_handle *glamo,
 			 GLAMO_LCD_MODE1_ROTATE_EN,
 			 (glamo_rot != GLAMO_LCD_ROT_MODE_0)?
 				 GLAMO_LCD_MODE1_ROTATE_EN : 0);
-/*	glamofb_cmd_mode(glamo, 0);
-
-out_unlock:
-	printk(KERN_ERR"rotate_lcd spin_unlock_irqrestore\n");
-	spin_unlock_irqrestore(&glamo->lock_cmd, flags);
-*/
 }
 
 static enum orientation get_orientation(struct fb_var_screeninfo *var)
@@ -296,14 +281,17 @@ static enum orientation get_orientation(struct fb_var_screeninfo *var)
 
 static int will_orientation_change(struct fb_var_screeninfo *var)
 {
-	enum  orientation orient = get_orientation(var);
+	enum orientation orient = get_orientation(var);
+
 	switch (orient) {
 		case ORIENTATION_LANDSCAPE:
-			if (var->rotate == FB_ROTATE_UR || var->rotate == FB_ROTATE_UD)
+			if (var->rotate == FB_ROTATE_UR ||
+			    var->rotate == FB_ROTATE_UD)
 				return 1;
 			break;
 		case ORIENTATION_PORTRAIT:
-			if (var->rotate == FB_ROTATE_CW || var->rotate == FB_ROTATE_CCW)
+			if (var->rotate == FB_ROTATE_CW ||
+			    var->rotate == FB_ROTATE_CCW)
 				return 1;
 			break;
 	}
@@ -316,8 +304,6 @@ static void glamofb_update_lcd_controller(struct glamofb_handle *glamo,
 	int sync, bp, disp, fp, total, xres, yres, pitch, orientation_changing;
 	unsigned long flags;
 
-/*	GLAMO_LOG("enter: glamo:%#x, var:%#x\n", (unsigned)glamo, (unsigned)var);
-*/
 	if (!glamo || !var)
 		return;
 
@@ -334,27 +320,18 @@ static void glamofb_update_lcd_controller(struct glamofb_handle *glamo,
 
 	xres = var->xres;
 	yres = var->yres;
-/*	GLAMO_LOG("xres:%d, yres:%d, rotate:%d\n", xres, yres, var->rotate);
-*/
-	/*
-	 * figure out if orientation is going to change
-	 */
+
+	/* figure out if orientation is going to change */
 	orientation_changing = will_orientation_change(var);
-/*	GLAMO_LOG("orientation_changing:%d\n", orientation_changing);
-*/
-        /*
-         * adjust the pitch according to new orientation to come
-         */
+
+        /* adjust the pitch according to new orientation to come */
         if (orientation_changing) {
 		pitch = var->yres * var->bits_per_pixel / 8;
         } else {
 		pitch = var->xres * var->bits_per_pixel / 8;
         }
-/*	GLAMO_LOG("pitch:%d\n", pitch);
-*/
-	/*
-	 * set the awaiten LCD geometry
-	 */
+
+	/* set the awaiten LCD geometry */
 	reg_set_bit_mask(glamo,
 			 GLAMO_REG_LCD_WIDTH,
 			 GLAMO_LCD_WIDTH_MASK,
@@ -368,16 +345,10 @@ static void glamofb_update_lcd_controller(struct glamofb_handle *glamo,
 			 GLAMO_LCD_PITCH_MASK,
 			 pitch);
 
-/*	GLAMO_LOG("mark:\n");*/
-	/*
-	 * honour the rotation request
-	 */
-	rotate_lcd(glamo, var->rotate);
+	/* honour the rotation request */
+	__rotate_lcd(glamo, var->rotate);
 
-	/*
-	 * update the reported geometry
-	 * of the framebuffer.
-	 */
+	/* update the reported geometry of the framebuffer. */
 	if (orientation_changing) {
 		var->xres_virtual = var->xres = yres;
 		var->yres_virtual = var->yres = xres;
@@ -386,11 +357,7 @@ static void glamofb_update_lcd_controller(struct glamofb_handle *glamo,
 		var->yres_virtual = var->yres = yres;
 	}
 
-/*	GLAMO_LOG("reported res:(%d,%d)\n", var->xres, var->yres);
-*/
-	/*
-	 * update scannout timings
-	 */
+	/* update scannout timings */
 	sync = 0;
 	bp = sync + var->hsync_len;
 	disp = bp + var->left_margin;
@@ -408,8 +375,6 @@ static void glamofb_update_lcd_controller(struct glamofb_handle *glamo,
 	reg_set_bit_mask(glamo, GLAMO_REG_LCD_HORIZ_DISP_END,
 			 GLAMO_LCD_HV_RETR_DISP_END_MASK, fp);
 
-/*	GLAMO_LOG("mark:\n");
-*/
 	sync = 0;
 	bp = sync + var->vsync_len;
 	disp = bp + var->upper_margin;
@@ -427,10 +392,8 @@ static void glamofb_update_lcd_controller(struct glamofb_handle *glamo,
 	reg_set_bit_mask(glamo, GLAMO_REG_LCD_VERT_DISP_END,
 			 GLAMO_LCD_HV_RETR_DISP_END_MASK, fp);
 
-/*	GLAMO_LOG("mark:\n"); */
 	glamofb_cmd_mode(glamo, 0);
 
-/*	GLAMO_LOG("leave:\n"); */
 out_unlock:
 	printk(KERN_ERR"glamofb_update_lcd_controller spin_unlock_irqrestore\n");
 	spin_unlock_irqrestore(&glamo->lock_cmd, flags);
@@ -463,7 +426,8 @@ static int glamofb_blank(int blank_mode, struct fb_info *info)
 	return 0;
 }
 
-static inline unsigned int chan_to_field(unsigned int chan, struct fb_bitfield *bf)
+static inline unsigned int chan_to_field(unsigned int chan,
+					 struct fb_bitfield *bf)
 {
 	chan &= 0xffff;
 	chan >>= 16 - bf->length;