[PATCH 3/7] drm: Reindent enum drm_mode_status

Ville Syrjälä ville.syrjala at linux.intel.com
Fri Dec 4 01:15:38 PST 2015


On Fri, Dec 04, 2015 at 09:18:57AM +0100, Daniel Vetter wrote:
> On Thu, Dec 03, 2015 at 11:14:11PM +0200, ville.syrjala at linux.intel.com wrote:
> > From: Ville Syrjälä <ville.syrjala at linux.intel.com>
> > 
> > There's some random mix of spaces and tabs used within the enum
> > drm_mode_status definition. Fix it up to use just tabs.
> > 
> > Cc: Adam Jackson <ajax at redhat.com>
> > Signed-off-by: Ville Syrjälä <ville.syrjala at linux.intel.com>
> 
> Ok, this will conflict badly, ok if this doesn't go in and I do the
> reformatting in my patch to kerneldoc drm_mode_status?

Fine by me. I just added this when I noticed checkpatch complaining
about the formatting in the previous patch.

> -Daniel
> 
> > ---
> >  include/drm/drm_modes.h | 78 ++++++++++++++++++++++++-------------------------
> >  1 file changed, 39 insertions(+), 39 deletions(-)
> > 
> > diff --git a/include/drm/drm_modes.h b/include/drm/drm_modes.h
> > index 71801229ce9d..256a1bbb125c 100644
> > --- a/include/drm/drm_modes.h
> > +++ b/include/drm/drm_modes.h
> > @@ -36,45 +36,45 @@
> >   */
> >  
> >  enum drm_mode_status {
> > -    MODE_OK	= 0,	/* Mode OK */
> > -    MODE_HSYNC,		/* hsync out of range */
> > -    MODE_VSYNC,		/* vsync out of range */
> > -    MODE_H_ILLEGAL,	/* mode has illegal horizontal timings */
> > -    MODE_V_ILLEGAL,	/* mode has illegal horizontal timings */
> > -    MODE_BAD_WIDTH,	/* requires an unsupported linepitch */
> > -    MODE_NOMODE,	/* no mode with a matching name */
> > -    MODE_NO_INTERLACE,	/* interlaced mode not supported */
> > -    MODE_NO_DBLESCAN,	/* doublescan mode not supported */
> > -    MODE_NO_VSCAN,	/* multiscan mode not supported */
> > -    MODE_MEM,		/* insufficient video memory */
> > -    MODE_VIRTUAL_X,	/* mode width too large for specified virtual size */
> > -    MODE_VIRTUAL_Y,	/* mode height too large for specified virtual size */
> > -    MODE_MEM_VIRT,	/* insufficient video memory given virtual size */
> > -    MODE_NOCLOCK,	/* no fixed clock available */
> > -    MODE_CLOCK_HIGH,	/* clock required is too high */
> > -    MODE_CLOCK_LOW,	/* clock required is too low */
> > -    MODE_CLOCK_RANGE,	/* clock/mode isn't in a ClockRange */
> > -    MODE_BAD_HVALUE,	/* horizontal timing was out of range */
> > -    MODE_BAD_VVALUE,	/* vertical timing was out of range */
> > -    MODE_BAD_VSCAN,	/* VScan value out of range */
> > -    MODE_HSYNC_NARROW,	/* horizontal sync too narrow */
> > -    MODE_HSYNC_WIDE,	/* horizontal sync too wide */
> > -    MODE_HBLANK_NARROW,	/* horizontal blanking too narrow */
> > -    MODE_HBLANK_WIDE,	/* horizontal blanking too wide */
> > -    MODE_VSYNC_NARROW,	/* vertical sync too narrow */
> > -    MODE_VSYNC_WIDE,	/* vertical sync too wide */
> > -    MODE_VBLANK_NARROW,	/* vertical blanking too narrow */
> > -    MODE_VBLANK_WIDE,	/* vertical blanking too wide */
> > -    MODE_PANEL,         /* exceeds panel dimensions */
> > -    MODE_INTERLACE_WIDTH, /* width too large for interlaced mode */
> > -    MODE_ONE_WIDTH,     /* only one width is supported */
> > -    MODE_ONE_HEIGHT,    /* only one height is supported */
> > -    MODE_ONE_SIZE,      /* only one resolution is supported */
> > -    MODE_NO_REDUCED,    /* monitor doesn't accept reduced blanking */
> > -    MODE_NO_STEREO,	/* stereo modes not supported */
> > -    MODE_STALE = -3,	/* mode has become stale */
> > -    MODE_BAD = -2,	/* unspecified reason */
> > -    MODE_ERROR	= -1	/* error condition */
> > +	MODE_OK	= 0,		/* Mode OK */
> > +	MODE_HSYNC,		/* hsync out of range */
> > +	MODE_VSYNC,		/* vsync out of range */
> > +	MODE_H_ILLEGAL,		/* mode has illegal horizontal timings */
> > +	MODE_V_ILLEGAL,		/* mode has illegal horizontal timings */
> > +	MODE_BAD_WIDTH,		/* requires an unsupported linepitch */
> > +	MODE_NOMODE,		/* no mode with a matching name */
> > +	MODE_NO_INTERLACE,	/* interlaced mode not supported */
> > +	MODE_NO_DBLESCAN,	/* doublescan mode not supported */
> > +	MODE_NO_VSCAN,		/* multiscan mode not supported */
> > +	MODE_MEM,		/* insufficient video memory */
> > +	MODE_VIRTUAL_X,		/* mode width too large for specified virtual size */
> > +	MODE_VIRTUAL_Y,		/* mode height too large for specified virtual size */
> > +	MODE_MEM_VIRT,		/* insufficient video memory given virtual size */
> > +	MODE_NOCLOCK,		/* no fixed clock available */
> > +	MODE_CLOCK_HIGH,	/* clock required is too high */
> > +	MODE_CLOCK_LOW,		/* clock required is too low */
> > +	MODE_CLOCK_RANGE,	/* clock/mode isn't in a ClockRange */
> > +	MODE_BAD_HVALUE,	/* horizontal timing was out of range */
> > +	MODE_BAD_VVALUE,	/* vertical timing was out of range */
> > +	MODE_BAD_VSCAN,		/* VScan value out of range */
> > +	MODE_HSYNC_NARROW,	/* horizontal sync too narrow */
> > +	MODE_HSYNC_WIDE,	/* horizontal sync too wide */
> > +	MODE_HBLANK_NARROW,	/* horizontal blanking too narrow */
> > +	MODE_HBLANK_WIDE,	/* horizontal blanking too wide */
> > +	MODE_VSYNC_NARROW,	/* vertical sync too narrow */
> > +	MODE_VSYNC_WIDE,	/* vertical sync too wide */
> > +	MODE_VBLANK_NARROW,	/* vertical blanking too narrow */
> > +	MODE_VBLANK_WIDE,	/* vertical blanking too wide */
> > +	MODE_PANEL,		/* exceeds panel dimensions */
> > +	MODE_INTERLACE_WIDTH,	/* width too large for interlaced mode */
> > +	MODE_ONE_WIDTH,		/* only one width is supported */
> > +	MODE_ONE_HEIGHT,	/* only one height is supported */
> > +	MODE_ONE_SIZE,		/* only one resolution is supported */
> > +	MODE_NO_REDUCED,	/* monitor doesn't accept reduced blanking */
> > +	MODE_NO_STEREO,		/* stereo modes not supported */
> > +	MODE_STALE = -3,	/* mode has become stale */
> > +	MODE_BAD = -2,		/* unspecified reason */
> > +	MODE_ERROR = -1,	/* error condition */
> >  };
> >  
> >  #define DRM_MODE_TYPE_CLOCK_CRTC_C (DRM_MODE_TYPE_CLOCK_C | \
> > -- 
> > 2.4.10
> > 
> > _______________________________________________
> > dri-devel mailing list
> > dri-devel at lists.freedesktop.org
> > http://lists.freedesktop.org/mailman/listinfo/dri-devel
> 
> -- 
> Daniel Vetter
> Software Engineer, Intel Corporation
> http://blog.ffwll.ch

-- 
Ville Syrjälä
Intel OTC


More information about the dri-devel mailing list