Re: [PATCH] vmwarefb 0.5.2

[Greg KH]

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.)

> +++ linux/drivers/video/vmware/guest_os.h	Tue May 14 01:32:04 2002
> @@ -0,0 +1,27 @@
> +/* $XFree86: xc/programs/Xserver/hw/xfree86/drivers/vmware/guest_os.h,v 1.1 2001/04/05 19:29:44 dawes Exp $ */
> +/* *********************************************************
> + * Copyright (C) 1999-2001 VMware, Inc.
> + * All Rights Reserved
> + * Id: guest_os.h,v 1.5 2001/01/26 23:32:15 yoel Exp $
> + * **********************************************************/

<snip>

> +++ linux/drivers/video/vmware/svga_limits.h	Tue May 14 01:31:47 2002
> @@ -0,0 +1,53 @@
> +/* $XFree86: xc/programs/Xserver/hw/xfree86/drivers/vmware/svga_limits.h,v 1.1 2001/04/05 19:29:44 dawes Exp $ */
> +/* **********************************************************
> + * Copyright (C) 1998-2001 VMware, Inc.
> + * All Rights Reserved
> + * Id: svga_limits.h,v 1.8 2001/01/26 23:32:15 yoel Exp $
> + * **********************************************************/

<snip>

> +++ 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 <[e-mail]>");
> +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.

thanks,

greg k-h