Re: [RFC PATCH 03/13] Intel(R) MEI Driver

[Greg KH]

On Thu, Feb 10, 2011 at 01:55:03AM -0800, Oren Weil wrote:
> diff --git a/drivers/char/mei/Makefile b/drivers/char/mei/Makefile
> new file mode 100644
> index 0000000..a9ac1cb
> --- /dev/null
> +++ b/drivers/char/mei/Makefile
> @@ -0,0 +1,23 @@
> +#
> +# Kbuild - Intel(R) Management Engine Interface (Intel(R) MEI) Linux driver

This is not a Kbuild file, it's a Makefile file.

> +# Copyright (c) 2003-2011, Intel Corporation.

Really?  This file is originally based on 2003?  I find that very hard
to believe.

> +#
> +# This program is free software; you can redistribute it and/or modify it
> +# under the terms and conditions of the GNU General Public License,
> +# version 2, as published by the Free Software Foundation.

Makefiles are not "programs", please remove this from the Makefile, it
is not needed.

> +#
> +# This program is distributed in the hope it will be useful, but WITHOUT
> +# ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
> +# FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License for
> +# more details.

This paragraph is not needed.

> +#
> +# You should have received a copy of the GNU General Public License along with
> +# this program; if not, write to the Free Software Foundation, Inc.,
> +# 51 Franklin St - Fifth Floor, Boston, MA 02110-1301 USA.

NEVER put this paragraph in, unless you are going to be willing to
update the file every time the FSF moves office buildings for the rest
of your life.

greg k-h
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/