lkml.org 
[lkml]   [2002]   [May]   [15]   [last100]   RSS Feed
Views: [more markup]  [less markup]  [headers]  [forward] 
 
Messages in this thread
/
DateWed, 15 May 2002 15:19:42 +0200
FromDenis Oliver Kropp <>
SubjectRe: [PATCH] vmwarefb 0.5.2
Quoting Greg KH (greg@kroah.com):
> On Wed, May 15, 2002 at 03:02:42AM +0200, Denis Oliver Kropp wrote:
> > 
> > Hi,
> > 
> > this is an updated version of the VMware framebuffer driver.
> > It contains a fix regarding the framebuffer offset (screen_base)
> > for non-standard modes.
> 
> Some non-technical questions about this code:
> 	- please read Documentation/CodingStyle and reformat your code
> 	  based on that (scripts/Lindent will do it for you if you
> 	  want.)

Ok, will try that.

> > +++ linux/drivers/video/vmware/svga_reg.h	Tue May 14 01:31:27 2002
> > @@ -0,0 +1,298 @@
> > +/* $XFree86: xc/programs/Xserver/hw/xfree86/drivers/vmware/svga_reg.h,v 1.4 2001/09/13 08:36:24 alanh Exp $ */
> > +/* **********************************************************
> > + * Copyright (C) 1998-2001 VMware, Inc.
> > + * All Rights Reserved
> > + * $Id: svga_reg.h,v 1.16 2001/07/25 22:41:24 mgoodman Exp $
> > + * **********************************************************/
> 
> <snip>
> 
> > +#ifdef MODULE
> > +
> > +MODULE_AUTHOR("(c) 2002  Denis Oliver Kropp <dok@directfb.org>");
> > +MODULE_LICENSE("GPL");
> > +MODULE_DESCRIPTION("FBDev driver for VMware Virtual SVGA Card");
> > +MODULE_PARM(disabled, "i");
> > +MODULE_PARM_DESC(disabled, "Disable this driver's initialization.");
> > +
> > +#endif
> 
> 	- drop the #ifdef, it's not needed.
> 
> 	- you are stating that your module is under the GPL, yet it uses
> 	  two files that are marked "All Rights Reserved" with the
> 	  copyright from VMWare, Inc.  I don't think you can do that :)
> 
> Is VMWare ok with releasing those files under the GPL?  If so, I suggest
> you add that line to those files, and then everything should be fine.

I wasn't sure yet about that issue, but I will try to contact VMWare and ask.

-- 
Best regards,
  Denis Oliver Kropp

.------------------------------------------.
| DirectFB - Hardware accelerated graphics |
| http://www.directfb.org/                 |
"------------------------------------------"
                            Convergence GmbH
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2005-03-22 12:22    [from the cache]
©2003-2008