On Thu, 2009-05-07 at 06:01 +0800, Eric Anholt wrote:
> On Tue, 2009-05-05 at 15:19 +0800, 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 <[email protected]>
> 
> This code appears to be a copy-and-paste with small modifications of
> xf86cvt.c, but the license of that code isn't respected.
Yes. Most is copied from xserver/hw/xfree86/mode/xf86CVT(xf86GTF). 
What I done is that the float-point calculation is replaced by using
integer.

How to pay attention to the license of xf86CVT code?
   Is it OK that the following is added in the function description?

/*
 * Copyright 2005-2006 Luc Verhaegen.
 *
 * Permission is hereby granted, free of charge, to any person obtaining
a
 * copy of this software and associated documentation files (the
"Software"),
 * to deal in the Software without restriction, including without
limitation
 * the rights to use, copy, modify, merge, publish, distribute,
sublicense,
 * and/or sell copies of the Software, and to permit persons to whom the
 * Software is furnished to do so, subject to the following conditions:
*/

Is the function comment OK ?
   If the style is incorrect, I will change it based on the description
in kernel-doc-nano-Howto.txt.

Thanks.
> 


------------------------------------------------------------------------------
The NEW KODAK i700 Series Scanners deliver under ANY circumstances! Your
production scanning environment may not be a perfect world - but thanks to
Kodak, there's a perfect scanner to get the job done! With the NEW KODAK i700
Series Scanner you'll get full speed at 300 dpi even with all image 
processing features enabled. http://p.sf.net/sfu/kodak-com
--
_______________________________________________
Dri-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/dri-devel

Reply via email to