Re: [PATCH 1/1] drm/gma500: Code cleanup - inline documentation

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, Mar 20, 2014 at 6:06 PM, Arthur Borsboom <arthurborsboom@xxxxxxxxx> wrote:
Mainly styling fixes of inline documentation

Signed-off-by: Arthur Borsboom <arthurborsboom@xxxxxxxxx>

Hi Arthur

I'm not a code style fanatic, but I see a bunch of problems with this patch. If you're going through the hassle of changing stuff in accordance to CodingStyle, you at least need to make sure you're actually following it.

Maybe this should go through the kernel janitors or kernel newbies mailing lists first.

Cheers
Patrik
 
---
 drivers/gpu/drm/gma500/framebuffer.c       |  36 ++--
 drivers/gpu/drm/gma500/psb_intel_display.c |  35 ++--
 drivers/gpu/drm/gma500/psb_intel_reg.h     | 259 +++++++++++++----------------
 drivers/gpu/drm/gma500/psb_irq.c           |  64 +++----
 4 files changed, 181 insertions(+), 213 deletions(-)

diff --git a/drivers/gpu/drm/gma500/framebuffer.c b/drivers/gpu/drm/gma500/framebuffer.c
index e7fcc14..0dd015a 100644
--- a/drivers/gpu/drm/gma500/framebuffer.c
+++ b/drivers/gpu/drm/gma500/framebuffer.c
@@ -103,8 +103,10 @@ static int psbfb_pan(struct fb_var_screeninfo *var, struct fb_info *info)
         *      the actual fb is mapped. In our case that isn't quite true.
         */
        if (psbfb->gtt->npage) {
-               /* GTT roll shifts in 4K pages, we need to shift the right
-                  number of pages */
+               /*
+                * GTT roll shifts in 4K pages, we need to shift the right
+                  number of pages
+                */
                int pages = info->fix.line_length >> 12;
                psb_gtt_roll(dev, psbfb->gtt, var->yoffset * pages);
        }
@@ -229,7 +231,7 @@ static struct fb_ops psbfb_unaccel_ops = {
        .fb_ioctl = psbfb_ioctl,
 };

-/**
+/*
  *     psb_framebuffer_init    -       initialize a framebuffer
  *     @dev: our DRM device
  *     @fb: framebuffer to set up
@@ -270,7 +272,7 @@ static int psb_framebuffer_init(struct drm_device *dev,
        return 0;
 }

-/**
+/*
  *     psb_framebuffer_create  -       create a framebuffer backed by gt
  *     @dev: our DRM device
  *     @mode_cmd: the description of the requested mode
@@ -302,7 +304,7 @@ static struct drm_framebuffer *psb_framebuffer_create
        return &fb->base;
 }

-/**
+/*
  *     psbfb_alloc             -       allocate frame buffer memory
  *     @dev: the DRM device
  *     @aligned_size: space needed
@@ -327,7 +329,7 @@ static struct gtt_range *psbfb_alloc(struct drm_device *dev, int aligned_size)
        return NULL;
 }

-/**
+/*
  *     psbfb_create            -       create a framebuffer
  *     @fbdev: the framebuffer device
  *     @sizes: specification of the layout
@@ -386,9 +388,9 @@ static int psbfb_create(struct psb_fbdev *fbdev,

        if (backing == NULL) {
                /*
-                *      We couldn't get the space we wanted, fall back to the
-                *      display engine requirement instead.  The HW requires
-                *      the pitch to be 64 byte aligned
+                * We couldn't get the space we wanted, fall back to the
+                * display engine requirement instead.  The HW requires
+                * the pitch to be 64 byte aligned
                 */

                gtt_roll = 0;   /* Don't use GTT accelerated scrolling */
@@ -489,7 +491,7 @@ out_err1:
        return ret;
 }

-/**
+/*
  *     psb_user_framebuffer_create     -       create framebuffer
  *     @dev: our DRM device
  *     @filp: client file
@@ -549,9 +551,11 @@ static int psbfb_probe(struct drm_fb_helper *helper,
        if (bytespp == 3)       /* no 24bit packed */
                bytespp = 4;

-       /* If the mode will not fit in 32bit then switch to 16bit to get
+       /*
+        * If the mode will not fit in 32bit then switch to 16bit to get
           a console on full resolution. The X mode setting server will
-          allocate its own 32bit GEM framebuffer */
+          allocate its own 32bit GEM framebuffer
+        */
        if (ALIGN(sizes->fb_width * bytespp, 64) * sizes->fb_height >
                        dev_priv->vram_stolen_size) {
                 sizes->surface_bpp = 16;
@@ -633,7 +637,7 @@ static void psbfb_output_poll_changed(struct drm_device *dev)
        drm_fb_helper_hotplug_event(&fbdev->psb_fb_helper);
 }

-/**
+/*
  *     psb_user_framebuffer_create_handle - add hamdle to a framebuffer
  *     @fb: framebuffer
  *     @file_priv: our DRM file
@@ -652,7 +656,7 @@ static int psb_user_framebuffer_create_handle(struct drm_framebuffer *fb,
        return drm_gem_handle_create(file_priv, &r->gem, handle);
 }

-/**
+/*
  *     psb_user_framebuffer_destroy    -       destruct user created fb
  *     @fb: framebuffer
  *
@@ -666,7 +670,7 @@ static void psb_user_framebuffer_destroy(struct drm_framebuffer *fb)

        /* Let DRM do its clean up */
        drm_framebuffer_cleanup(fb);
-       /*  We are no longer using the resource in GEM */
+       /* We are no longer using the resource in GEM */
        drm_gem_object_unreference_unlocked(&r->gem);
        kfree(fb);
 }
@@ -761,7 +765,7 @@ void psb_modeset_init(struct drm_device *dev)
        dev->mode_config.funcs = &psb_mode_funcs;

        /* set memory base */
-       /* Oaktrail and Poulsbo should use BAR 2*/
+       /* Oaktrail and Poulsbo should use BAR 2 */
        pci_read_config_dword(dev->pdev, PSB_BSM, (u32 *)
                                        &(dev->mode_config.fb_base));

diff --git a/drivers/gpu/drm/gma500/psb_intel_display.c b/drivers/gpu/drm/gma500/psb_intel_display.c
index 21aed85..b11cbd6 100644
--- a/drivers/gpu/drm/gma500/psb_intel_display.c
+++ b/drivers/gpu/drm/gma500/psb_intel_display.c
@@ -1,5 +1,5 @@
 /*
- * Copyright © 2006-2011 Intel Corporation
+ * Copyright © 2006-2011 Intel Corporation
  *
  * This program is free software; you can redistribute it and/or modify it
  * under the terms and conditions of the GNU General Public License,
@@ -53,7 +53,8 @@ static const struct gma_limit_t psb_intel_limits[] = {
         .m2 = {.min = 3, .max = 7},
         .p = {.min = 7, .max = 98},
         .p1 = {.min = 1, .max = 8},
-        /* The single-channel range is 25-112Mhz, and dual-channel
+        /*
+         * The single-channel range is 25-112Mhz, and dual-channel
          * is 80-224Mhz.  Prefer single channel as much as possible.
          */
         .p2 = {.dot_limit = 112000, .p2_slow = 14, .p2_fast = 7},
@@ -81,7 +82,7 @@ static void psb_intel_clock(int refclk, struct gma_clock_t *clock)
        clock->dot = clock->vco / clock->p;
 }

-/**
+/*
  * Return the pipe currently connected to the panel fitter,
  * or -1 if the panel fitter is not present or not in use
  */
@@ -226,7 +227,8 @@ static int psb_intel_crtc_mode_set(struct drm_crtc *crtc,
                udelay(150);
        }

-       /* The LVDS pin pair needs to be on before the DPLLs are enabled.
+       /*
+        * The LVDS pin pair needs to be on before the DPLLs are enabled.
         * This is an exception to the general rule that mode_set doesn't turn
         * things on.
         */
@@ -238,7 +240,8 @@ static int psb_intel_crtc_mode_set(struct drm_crtc *crtc,
                        lvds |= LVDS_PIPEB_SELECT;

                lvds |= LVDS_PORT_EN | LVDS_A0A2_CLKA_POWER_UP;
-               /* Set the B0-B3 data pairs corresponding to
+               /*
+                * Set the B0-B3 data pairs corresponding to
                 * whether we're going to
                 * set the DPLLs for dual-channel mode or not.
                 */
@@ -246,7 +249,8 @@ static int psb_intel_crtc_mode_set(struct drm_crtc *crtc,
                if (clock.p2 == 7)
                        lvds |= LVDS_B0B3_POWER_UP | LVDS_CLKB_POWER_UP;

-               /* It would be nice to set 24 vs 18-bit mode (LVDS_A3_POWER_UP)
+               /*
+                * It would be nice to set 24 vs 18-bit mode (LVDS_A3_POWER_UP)
                 * appropriately here, but we need to look more
                 * thoroughly into how panels behave in the two modes.
                 */
@@ -261,7 +265,7 @@ static int psb_intel_crtc_mode_set(struct drm_crtc *crtc,
        /* Wait for the clocks to stabilize. */
        udelay(150);

-       /* write it again -- the BIOS does, after all */
+       /* Write it again -- the BIOS does, after all */
        REG_WRITE(map->dpll, dpll);

        REG_READ(map->dpll);
@@ -280,7 +284,8 @@ static int psb_intel_crtc_mode_set(struct drm_crtc *crtc,
                  ((adjusted_mode->crtc_vblank_end - 1) << 16));
        REG_WRITE(map->vsync, (adjusted_mode->crtc_vsync_start - 1) |
                  ((adjusted_mode->crtc_vsync_end - 1) << 16));
-       /* pipesrc and dspsize control the size that is scaled from,
+       /*
+        * pipesrc and dspsize control the size that is scaled from,
         * which should always be the user's requested size.
         */
        REG_WRITE(map->size,
@@ -371,7 +376,8 @@ static int psb_intel_crtc_clock_get(struct drm_device *dev,
                psb_intel_clock(48000, &clock);
        }

-       /* XXX: It would be nice to validate the clocks, but we can't reuse
+       /*
+        * XXX: It would be nice to validate the clocks, but we can't reuse
         * i830PllIsValid() because it relies on the xf86_config connector
         * configuration being accurate, which it isn't necessarily.
         */
@@ -379,7 +385,7 @@ static int psb_intel_crtc_clock_get(struct drm_device *dev,
        return clock.dot;
 }

-/** Returns the currently programmed mode of the given pipe. */
+/* Returns the currently programmed mode of the given pipe. */
 struct drm_display_mode *psb_intel_crtc_mode_get(struct drm_device *dev,
                                             struct drm_crtc *crtc)
 {
@@ -466,7 +472,8 @@ static void psb_intel_cursor_init(struct drm_device *dev,
        struct gtt_range *cursor_gt;

        if (dev_priv->ops->cursor_needs_phys) {
-               /* Allocate 4 pages of stolen mem for a hardware cursor. That
+               /*
+                * Allocate 4 pages of stolen mem for a hardware cursor. That
                 * is enough for the 64 x 64 ARGB cursors we support.
                 */
                cursor_gt = psb_gtt_alloc_range(dev, 4 * PAGE_SIZE, "cursor", 1,
@@ -495,8 +502,10 @@ void psb_intel_crtc_init(struct drm_device *dev, int pipe,
        int i;
        uint16_t *r_base, *g_base, *b_base;

-       /* We allocate a extra array of drm_connector pointers
-        * for fbdev after the crtc */
+       /*
+        * We allocate an extra array of drm_connector pointers
+        * for fbdev after the crtc
+        */
        gma_crtc = kzalloc(sizeof(struct gma_crtc) +
                        (INTELFB_CONN_LIMIT * sizeof(struct drm_connector *)),
                        GFP_KERNEL);
diff --git a/drivers/gpu/drm/gma500/psb_intel_reg.h b/drivers/gpu/drm/gma500/psb_intel_reg.h
index 0be30e4..ec32a06 100644
--- a/drivers/gpu/drm/gma500/psb_intel_reg.h
+++ b/drivers/gpu/drm/gma500/psb_intel_reg.h
@@ -17,9 +17,7 @@
 #ifndef __PSB_INTEL_REG_H__
 #define __PSB_INTEL_REG_H__

-/*
- * GPIO regs
- */
+/* GPIO regs */
 #define GPIOA                  0x5010
 #define GPIOB                  0x5014
 #define GPIOC                  0x5018
@@ -254,7 +252,7 @@
 #define DPLL_LOCK                      (1 << 15)       /* CDV */

 /*
- *  The i830 generation, in DAC/serial mode, defines p1 as two plus this
+ * The i830 generation, in DAC/serial mode, defines p1 as two plus this
  * bitfield, or just 2 if PLL_P1_DIVIDE_BY_TWO is set.
  */
 # define DPLL_FPA01_P1_POST_DIV_MASK_I830      0x001f0000
@@ -292,9 +290,7 @@
 #define SDVO_MULTIPLIER_SHIFT_HIRES    4
 #define SDVO_MULTIPLIER_SHIFT_VGA      0

-/*
- * PLL_MD
- */
+/* PLL_MD */
 /* Pipe A SDVO/UDI clock multiplier/divider register for G965. */
 #define DPLL_A_MD              0x0601c
 /* Pipe B SDVO/UDI clock multiplier/divider register for G965. */
@@ -385,7 +381,7 @@
 #define TV_HOTPLUG_INT_EN              (1 << 18)
 #define CRT_HOTPLUG_INT_EN             (1 << 9)
 #define CRT_HOTPLUG_FORCE_DETECT       (1 << 3)
-/* CDV.. */
+/* CDV */
 #define CRT_HOTPLUG_ACTIVATION_PERIOD_64       (1 << 8)
 #define CRT_HOTPLUG_DAC_ON_TIME_2M             (0 << 7)
 #define CRT_HOTPLUG_DAC_ON_TIME_4M             (1 << 7)
@@ -419,7 +415,7 @@
 #define SDVO_COLOR_RANGE_16_235                (1 << 8)
 #define SDVO_AUDIO_ENABLE              (1 << 6)

-/**
+/*
  * 915G/GM SDVO pixel multiplier.
  *
  * Programmed value is multiplier - 1, up to 5x.
@@ -702,9 +698,7 @@ struct dpst_guardband {
 #define VGA_2X_MODE                    (1 << 30)
 #define VGA_PIPE_B_SELECT              (1 << 29)

-/*
- * Overlay registers
- */
+/* Overlay registers */
 #define OV_C_OFFSET            0x08000
 #define OV_OVADD               0x30000
 #define OV_DOVASTA             0x30008
@@ -739,9 +733,7 @@ struct dpst_guardband {
 #define SWF5                   0x71424
 #define SWF6                   0x71428

-/*
- * 855 scratch registers.
- */
+/* 855 scratch registers */
 #define SWF00                  0x70410
 #define SWF01                  0x70414
 #define SWF02                  0x70418
@@ -762,10 +754,7 @@ struct dpst_guardband {
 #define SWF31                  0x72418
 #define SWF32                  0x7241c

-
-/*
- * Palette registers
- */
+/* Palette registers */
 #define PALETTE_A              0x0a000
 #define PALETTE_B              0x0a800
 #define PALETTE_C              0x0ac00
@@ -789,17 +778,13 @@ struct dpst_guardband {
 #define CURCBASE               0x700e4
 #define CURCPOS                        0x700e8

-/*
- * Interrupt Registers
- */
+/* Interrupt Registers */
 #define IER                    0x020a0
 #define IIR                    0x020a4
 #define IMR                    0x020a8
 #define ISR                    0x020ac

-/*
- * MOORESTOWN delta registers
- */
+/* MOORESTOWN delta registers */
 #define MRST_DPLL_A            0x0f014
 #define MDFLD_DPLL_B           0x0f018
 #define MDFLD_INPUT_REF_SEL            (1 << 14)
@@ -814,9 +799,7 @@ struct dpst_guardband {
 #define MDFLD_DPLL_DIV1                0x0f04c
 #define MRST_PERF_MODE         0x020f4

-/*
- * MEDFIELD HDMI registers
- */
+/* MEDFIELD HDMI registers */
 #define HDMIPHYMISCCTL         0x61134
 #define HDMI_PHY_POWER_DOWN            0x7f
 #define HDMIB_CONTROL          0x61140
@@ -862,13 +845,9 @@ struct dpst_guardband {
 #define MRST_DSPBBASE          0x7119c
 #define MDFLD_DSPCBASE         0x7219c

-/*
- * Moorestown registers.
- */
+/* Moorestown registers */

-/*
- *     MIPI IP registers
- */
+/* MIPI IP registers */
 #define MIPIC_REG_OFFSET               0x800

 #define DEVICE_READY_REG               0xb000
@@ -998,9 +977,7 @@ struct dpst_guardband {
 #define DBI_BW_CTRL_REG                        0xb084
 #define CLK_LANE_SWT_REG               0xb088

-/*
- * MIPI Adapter registers
- */
+/* MIPI Adapter registers */
 #define MIPI_CONTROL_REG               0xb104
 #define MIPI_2X_CLOCK_BITS                     ((1 << 0) | (1 << 1))
 #define MIPI_DATA_ADDRESS_REG          0xb108
@@ -1020,161 +997,149 @@ struct dpst_guardband {
 /* DBI COMMANDS */
 #define soft_reset                     0x01
 /*
- *     The display module performs a software reset.
- *     Registers are written with their SW Reset default values.
+ * The display module performs a software reset.
+ * Registers are written with their SW Reset default values.
  */
 #define get_power_mode                 0x0a
-/*
- *     The display module returns the current power mode
- */
+/* The display module returns the current power mode */
 #define get_address_mode               0x0b
-/*
- *     The display module returns the current status.
- */
+/* The display module returns the current status */
 #define get_pixel_format               0x0c
 /*
- *     This command gets the pixel format for the RGB image data
- *     used by the interface.
+ * This command gets the pixel format for the RGB image data
+ * used by the interface.
  */
 #define get_display_mode               0x0d
-/*
- *     The display module returns the Display Image Mode status.
- */
+/* The display module returns the Display Image Mode status */
 #define get_signal_mode                        0x0e
-/*
- *     The display module returns the Display Signal Mode.
- */
+/* The display module returns the Display Signal Mode */
 #define get_diagnostic_result          0x0f
 /*
- *     The display module returns the self-diagnostic results following
- *     a Sleep Out command.
+ * The display module returns the self-diagnostic results following
+ * a Sleep Out command.
  */
 #define enter_sleep_mode               0x10
 /*
- *     This command causes the display module to enter the Sleep mode.
- *     In this mode, all unnecessary blocks inside the display module are
- *     disabled except interface communication. This is the lowest power
- *     mode the display module supports.
+ * This command causes the display module to enter the Sleep mode.
+ * In this mode, all unnecessary blocks inside the display module are
+ * disabled except interface communication. This is the lowest power
+ * mode the display module supports.
  */
 #define exit_sleep_mode                        0x11
 /*
- *     This command causes the display module to exit Sleep mode.
- *     All blocks inside the display module are enabled.
+ * This command causes the display module to exit Sleep mode.
+ * All blocks inside the display module are enabled.
  */
 #define enter_partial_mode             0x12
 /*
- *     This command causes the display module to enter the Partial Display
- *     Mode. The Partial Display Mode window is described by the
- *     set_partial_area command.
+ * This command causes the display module to enter the Partial Display
+ * Mode. The Partial Display Mode window is described by the
+ * set_partial_area command.
  */
 #define enter_normal_mode              0x13
 /*
- *     This command causes the display module to enter the Normal mode.
- *     Normal Mode is defined as Partial Display mode and Scroll mode are off
+ * This command causes the display module to enter the Normal mode.
+ * Normal Mode is defined as Partial Display mode and Scroll mode are off
  */
 #define exit_invert_mode               0x20
 /*
- *     This command causes the display module to stop inverting the image
- *     data on the display device. The frame memory contents remain unchanged.
- *     No status bits are changed.
+ * This command causes the display module to stop inverting the image
+ * data on the display device. The frame memory contents remain unchanged.
+ * No status bits are changed.
  */
 #define enter_invert_mode              0x21
 /*
- *     This command causes the display module to invert the image data only on
- *     the display device. The frame memory contents remain unchanged.
- *     No status bits are changed.
+ * This command causes the display module to invert the image data only on
+ * the display device. The frame memory contents remain unchanged.
+ * No status bits are changed.
  */
 #define set_gamma_curve                        0x26
 /*
- *     This command selects the desired gamma curve for the display device.
- *     Four fixed gamma curves are defined in section DCS spec.
+ * This command selects the desired gamma curve for the display device.
+ * Four fixed gamma curves are defined in section DCS spec.
  */
 #define set_display_off                        0x28
-/* ************************************************************************* *\
-This command causes the display module to stop displaying the image data
-on the display device. The frame memory contents remain unchanged.
-No status bits are changed.
-\* ************************************************************************* */
+/*
+ * This command causes the display module to stop displaying the image data
+ * on the display device. The frame memory contents remain unchanged.
+ * No status bits are changed.
+ */
 #define set_display_on                 0x29
-/* ************************************************************************* *\
-This command causes the display module to start displaying the image data
-on the display device. The frame memory contents remain unchanged.
-No status bits are changed.
-\* ************************************************************************* */
+/*
+ * This command causes the display module to start displaying the image data
+ * on the display device. The frame memory contents remain unchanged.
+ * No status bits are changed.
+ */
 #define set_column_address             0x2a
 /*
- *     This command defines the column extent of the frame memory accessed by
- *     the hostprocessor with the read_memory_continue and
- *     write_memory_continue commands.
- *     No status bits are changed.
+ * This command defines the column extent of the frame memory accessed by
+ * the hostprocessor with the read_memory_continue and
+ * write_memory_continue commands.
+ * No status bits are changed.
  */
 #define set_page_addr                  0x2b
 /*
- *     This command defines the page extent of the frame memory accessed by
- *     the host processor with the write_memory_continue and
- *     read_memory_continue command.
- *     No status bits are changed.
+ * This command defines the page extent of the frame memory accessed by
+ * the host processor with the write_memory_continue and
+ * read_memory_continue command.
+ * No status bits are changed.
  */
 #define write_mem_start                        0x2c
 /*
- *     This command transfers image data from the host processor to the
- *     display modules frame memory starting at the pixel location specified
- *     by preceding set_column_address and set_page_address commands.
+ * This command transfers image data from the host processor to the
+ * display modules frame memory starting at the pixel location specified
+ * by preceding set_column_address and set_page_address commands.
  */
 #define set_partial_area               0x30
 /*
- *     This command defines the Partial Display mode s display area.
- *     There are two parameters associated with this command, the first
- *     defines the Start Row (SR) and the second the End Row (ER). SR and ER
- *     refer to the Frame Memory Line Pointer.
+ * This command defines the Partial Display mode s display area.
+ * There are two parameters associated with this command, the first
+ * defines the Start Row (SR) and the second the End Row (ER). SR and ER
+ * refer to the Frame Memory Line Pointer.
  */
 #define set_scroll_area                        0x33
-/*
- *     This command defines the display modules Vertical Scrolling Area.
- */
+/* This command defines the display modules Vertical Scrolling Area. */
 #define set_tear_off                   0x34
 /*
- *     This command turns off the display modules Tearing Effect output
- *     signal on the TE signal line.
+ * This command turns off the display modules Tearing Effect output
+ * signal on the TE signal line.
  */
 #define set_tear_on                    0x35
 /*
- *     This command turns on the display modules Tearing Effect output signal
- *     on the TE signal line.
+ * This command turns on the display modules Tearing Effect output signal
+ * on the TE signal line.
  */
 #define set_address_mode               0x36
 /*
- *     This command sets the data order for transfers from the host processor
- *     to display modules frame memory,bits B[7:5] and B3, and from the
- *     display modules frame memory to the display device, bits B[2:0] and B4.
+ * This command sets the data order for transfers from the host processor
+ * to display modules frame memory,bits B[7:5] and B3, and from the
+ * display modules frame memory to the display device, bits B[2:0] and B4.
  */
 #define set_scroll_start               0x37
 /*
- *     This command sets the start of the vertical scrolling area in the frame
- *     memory. The vertical scrolling area is fully defined when this command
- *     is used with the set_scroll_area command The set_scroll_start command
- *     has one parameter, the Vertical Scroll Pointer. The VSP defines the
- *     line in the frame memory that is written to the display device as the
- *     first line of the vertical scroll area.
+ * This command sets the start of the vertical scrolling area in the frame
+ * memory. The vertical scrolling area is fully defined when this command
+ * is used with the set_scroll_area command The set_scroll_start command
+ * has one parameter, the Vertical Scroll Pointer. The VSP defines the
+ * line in the frame memory that is written to the display device as the
+ * first line of the vertical scroll area.
  */
 #define exit_idle_mode                 0x38
-/*
- *     This command causes the display module to exit Idle mode.
- */
+/* This command causes the display module to exit Idle mode. */
 #define enter_idle_mode                        0x39
 /*
- *     This command causes the display module to enter Idle Mode.
- *     In Idle Mode, color _expression_ is reduced. Colors are shown on the
- *     display device using the MSB of each of the R, G and B color
- *     components in the frame memory
+ * This command causes the display module to enter Idle Mode.
+ * In Idle Mode, color _expression_ is reduced. Colors are shown on the
+ * display device using the MSB of each of the R, G and B color
+ * components in the frame memory
  */
 #define set_pixel_format               0x3a
 /*
- *     This command sets the pixel format for the RGB image data used by the
- *     interface.
- *     Bits D[6:4]  DPI Pixel Format Definition
- *     Bits D[2:0]  DBI Pixel Format Definition
- *     Bits D7 and D3 are not used.
+ * This command sets the pixel format for the RGB image data used by the
+ * interface.
+ * Bits D[6:4]  DPI Pixel Format Definition
+ * Bits D[2:0]  DBI Pixel Format Definition
+ * Bits D7 and D3 are not used.
  */
 #define DCS_PIXEL_FORMAT_3bpp          0x1
 #define DCS_PIXEL_FORMAT_8bpp          0x2
@@ -1186,23 +1151,23 @@ No status bits are changed.
 #define write_mem_cont                 0x3c

 /*
- *     This command transfers image data from the host processor to the
- *     display module's frame memory continuing from the pixel location
- *     following the previous write_memory_continue or write_memory_start
- *     command.
+ * This command transfers image data from the host processor to the
+ * display module's frame memory continuing from the pixel location
+ * following the previous write_memory_continue or write_memory_start
+ * command.
  */
 #define set_tear_scanline              0x44
 /*
- *     This command turns on the display modules Tearing Effect output signal
- *     on the TE signal line when the display module reaches line N.
+ * This command turns on the display modules Tearing Effect output signal
+ * on the TE signal line when the display module reaches line N.
  */
 #define get_scanline                   0x45
 /*
- *     The display module returns the current scanline, N, used to update the
- *      display device. The total number of scanlines on a display device is
- *     defined as VSYNC + VBP + VACT + VFP.The first scanline is defined as
- *     the first line of V Sync and is denoted as Line 0.
- *     When in Sleep Mode, the value returned by get_scanline is undefined.
+ * The display module returns the current scanline, N, used to update the
+ * display device. The total number of scanlines on a display device is
+ * defined as VSYNC + VBP + VACT + VFP.The first scanline is defined as
+ * the first line of V Sync and is denoted as Line 0.
+ * When in Sleep Mode, the value returned by get_scanline is undefined.
  */

 /* MCS or Generic COMMANDS */
@@ -1404,7 +1369,8 @@ No status bits are changed.
 #define   DP_VOLTAGE_MASK              (7 << 25)
 #define   DP_VOLTAGE_SHIFT             25

-/* Signal pre-emphasis levels, like voltages, the other end tells us what
+/*
+ * Signal pre-emphasis levels, like voltages, the other end tells us what
  * they want
  */
 #define   DP_PRE_EMPHASIS_0            (0 << 22)
@@ -1423,29 +1389,30 @@ No status bits are changed.
 /* Mystic DPCD version 1.1 special mode */
 #define   DP_ENHANCED_FRAMING          (1 << 18)

-/** locked once port is enabled */
+/* locked once port is enabled */
 #define   DP_PORT_REVERSAL             (1 << 15)

-/** sends the clock on lane 15 of the PEG for debug */
+/* sends the clock on lane 15 of the PEG for debug */
 #define   DP_CLOCK_OUTPUT_ENABLE       (1 << 13)

 #define   DP_SCRAMBLING_DISABLE                (1 << 12)
 #define   DP_SCRAMBLING_DISABLE_IRONLAKE       (1 << 7)

-/** limit RGB values to avoid confusing TVs */
+/* limit RGB values to avoid confusing TVs */
 #define   DP_COLOR_RANGE_16_235                (1 << 8)

-/** Turn on the audio link */
+/* Turn on the audio link */
 #define   DP_AUDIO_OUTPUT_ENABLE       (1 << 6)

-/** vs and hs sync polarity */
+/* vs and hs sync polarity */
 #define   DP_SYNC_VS_HIGH              (1 << 4)
 #define   DP_SYNC_HS_HIGH              (1 << 3)

-/** A fantasy */
+/* A fantasy */
 #define   DP_DETECTED                  (1 << 2)

-/** The aux channel provides a way to talk to the
+/*
+ * The aux channel provides a way to talk to the
  * signal sink for DDC etc. Max packet size supported
  * is 20 bytes in each direction, hence the 5 fixed
  * data registers
diff --git a/drivers/gpu/drm/gma500/psb_irq.c b/drivers/gpu/drm/gma500/psb_irq.c
index 624eb36..ba3fd4f 100644
--- a/drivers/gpu/drm/gma500/psb_irq.c
+++ b/drivers/gpu/drm/gma500/psb_irq.c
@@ -19,8 +19,6 @@
  * develop this driver.
  *
  **************************************************************************/
-/*
- */

 #include <drm/drmP.h>
 #include "psb_drv.h"
@@ -30,10 +28,7 @@
 #include "psb_irq.h"
 #include "mdfld_output.h"

-/*
- * inline functions
- */
-
+/* inline functions */
 static inline u32
 psb_pipestat(int pipe)
 {
@@ -139,10 +134,7 @@ static void mid_disable_pipe_event(struct drm_psb_private *dev_priv, int pipe)
        }
 }

-/**
- * Display controller interrupt handler for pipe event.
- *
- */
+/* Display controller interrupt handler for pipe event */
 static void mid_pipe_event_handler(struct drm_device *dev, int pipe)
 {
        struct drm_psb_private *dev_priv =
@@ -163,8 +155,10 @@ static void mid_pipe_event_handler(struct drm_device *dev, int pipe)

        spin_unlock(&dev_priv->irqmask_lock);

-       /* Clear the 2nd level interrupt status bits
-        * Sometimes the bits are very sticky so we repeat until they unstick */
+       /*
+        * Clear the 2nd level interrupt status bits
+        * Sometimes the bits are very sticky so we repeat until they unstick
+        */
        for (i = 0; i < 0xffff; i++) {
                PSB_WVDC32(PSB_RVDC32(pipe_stat_reg), pipe_stat_reg);
                pipe_clear = PSB_RVDC32(pipe_stat_reg) & pipe_status;
@@ -185,9 +179,7 @@ static void mid_pipe_event_handler(struct drm_device *dev, int pipe)
                drm_handle_vblank(dev, pipe);
 }

-/*
- * Display controller interrupt handler.
- */
+/* Display controller interrupt handler */
 static void psb_vdc_interrupt(struct drm_device *dev, uint32_t vdc_stat)
 {
        if (vdc_stat & _PSB_IRQ_ASLE)
@@ -200,9 +192,7 @@ static void psb_vdc_interrupt(struct drm_device *dev, uint32_t vdc_stat)
                mid_pipe_event_handler(dev, 1);
 }

-/*
- * SGX interrupt handler
- */
+/* SGX interrupt handler */
 static void psb_sgx_interrupt(struct drm_device *dev, u32 stat_1, u32 stat_2)
 {
        struct drm_psb_private *dev_priv = dev->dev_private;
@@ -267,7 +257,8 @@ irqreturn_t psb_irq_handler(int irq, void *arg)
        if (vdc_stat & (_PSB_PIPE_EVENT_FLAG|_PSB_IRQ_ASLE))
                dsp_int = 1;

-       /* FIXME: Handle Medfield
+       /*
+        * FIXME: Handle Medfield
        if (vdc_stat & _MDFLD_DISP_ALL_IRQ_FLAG)
                dsp_int = 1;
        */
@@ -292,8 +283,10 @@ irqreturn_t psb_irq_handler(int irq, void *arg)
                handled = 1;
        }

-       /* Note: this bit has other meanings on some devices, so we will
-          need to address that later if it ever matters */
+       /*
+        * Note: this bit has other meanings on some devices, so we will
+          need to address that later if it ever matters
+        */
        if (hotplug_int && dev_priv->ops->hotplug) {
                handled = dev_priv->ops->hotplug(dev);
                REG_WRITE(PORT_HOTPLUG_STAT, REG_READ(PORT_HOTPLUG_STAT));
@@ -336,7 +329,7 @@ void psb_irq_preinstall(struct drm_device *dev)
                dev_priv->vdc_irq_mask |= _MDFLD_PIPEC_EVENT_FLAG;
        */

-       /* Revisit this area - want per device masks ? */
+       /* TODO: Revisit this area - want per device masks ? */
        if (dev_priv->ops->hotplug)
                dev_priv->vdc_irq_mask |= _PSB_IRQ_DISP_HOTSYNC;
        dev_priv->vdc_irq_mask |= _PSB_IRQ_ASLE | _PSB_IRQ_SGX_FLAG;
@@ -461,7 +454,7 @@ int psb_irq_enable_dpst(struct drm_device *dev)

        spin_lock_irqsave(&dev_priv->irqmask_lock, irqflags);

-       /* enable DPST */
+       /* Enable DPST */
        mid_enable_pipe_event(dev_priv, 0);
        psb_irq_turn_on_dpst(dev);

@@ -507,9 +500,7 @@ int psb_irq_disable_dpst(struct drm_device *dev)
        return 0;
 }

-/*
- * It is used to enable VBLANK interrupt
- */
+/* It is used to enable VBLANK interrupt */
 int psb_enable_vblank(struct drm_device *dev, int pipe)
 {
        struct drm_psb_private *dev_priv = dev->dev_private;
@@ -517,8 +508,10 @@ int psb_enable_vblank(struct drm_device *dev, int pipe)
        uint32_t reg_val = 0;
        uint32_t pipeconf_reg = mid_pipeconf(pipe);

-       /* Medfield is different - we should perhaps extract out vblank
-          and blacklight etc ops */
+       /*
+        * Medfield is different - we should perhaps extract out vblank
+        * and blacklight etc ops
+        */
        if (IS_MFLD(dev))
                return mdfld_enable_te(dev, pipe);

@@ -546,9 +539,7 @@ int psb_enable_vblank(struct drm_device *dev, int pipe)
        return 0;
 }

-/*
- * It is used to disable VBLANK interrupt
- */
+/* It is used to disable VBLANK interrupt */
 void psb_disable_vblank(struct drm_device *dev, int pipe)
 {
        struct drm_psb_private *dev_priv = dev->dev_private;
@@ -570,9 +561,7 @@ void psb_disable_vblank(struct drm_device *dev, int pipe)
        spin_unlock_irqrestore(&dev_priv->irqmask_lock, irqflags);
 }

-/*
- * It is used to enable TE interrupt
- */
+/* It is used to enable TE interrupt */
 int mdfld_enable_te(struct drm_device *dev, int pipe)
 {
        struct drm_psb_private *dev_priv =
@@ -599,9 +588,7 @@ int mdfld_enable_te(struct drm_device *dev, int pipe)
        return 0;
 }

-/*
- * It is used to disable TE interrupt
- */
+/* It is used to disable TE interrupt */
 void mdfld_disable_te(struct drm_device *dev, int pipe)
 {
        struct drm_psb_private *dev_priv =
@@ -619,7 +606,8 @@ void mdfld_disable_te(struct drm_device *dev, int pipe)
        spin_unlock_irqrestore(&dev_priv->irqmask_lock, irqflags);
 }

-/* Called from drm generic code, passed a 'crtc', which
+/*
+ * Called from drm generic code, passed a 'crtc', which
  * we use as a pipe index
  */
 u32 psb_get_vblank_counter(struct drm_device *dev, int pipe)
--
1.9.0


_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/dri-devel

[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux