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
