lkml.org 
[lkml]   [2002]   [May]   [15]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
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 13:22    [W:0.070 / U:0.232 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site