[Intel-gfx] [RFC] [Patch 1/2] [intel-gfx] : Add the CVT algorithm in kernel space

[Ian Romanick]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

yakui_zhao wrote:
> Add the CVT algorithm in kernel space. And this function can be called to
> generate the required modeline.
> 
> Signed-off-by: Zhao Yakui <yakui.zhao at intel.com>
> ---
>  drivers/gpu/drm/drm_modes.c |  210 ++++++++++++++++++++++++++++++++++++++++++++
>  include/drm/drm_crtc.h      |    3 
>  2 files changed, 213 insertions(+)
> 
> Index: linux-2.6/drivers/gpu/drm/drm_modes.c
> ===================================================================
> --- linux-2.6.orig/drivers/gpu/drm/drm_modes.c	2009-04-21 15:48:52.000000000 +0800
> +++ linux-2.6/drivers/gpu/drm/drm_modes.c	2009-05-05 11:22:09.000000000 +0800
> @@ -59,6 +59,216 @@
>  }
>  EXPORT_SYMBOL(drm_mode_debug_printmodeline);
>  
> +#define HV_FACTOR			1000
> +/****************************************************************
> + * Function	: drm_cvt_mode
> + *
> + * Parameters	: dev -drm device
> + * 		  hdisplay - hdisplay size
> + * 		  vdisplay - vdisplay size
> + *		  vfresh - vfresh rate.
> + * 		  reduced - whether GTF calculation is simplified
> + * 		  interlaced -- whether the interlace is supported
> + * Returns	: return the display mode
> + *
> + * Function description
> + * 		: create the display mode according to the CVT algorthim.
> + * 		  It is based on the file of xserver/hw/xfree86/modes/xf86cvt
> + */

Use kerneldoc function header comment style.  I'd bet that Linus would
NAK this.
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.9 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org

iEYEARECAAYFAkoAgtEACgkQX1gOwKyEAw/yrgCeP4YeT5K45whLDuMwnqPXSWVp
n20An2zO75WyyAlxNBWqN99CrlaFcRcN
=iBlI
-----END PGP SIGNATURE-----