All of lore.kernel.org
 help / color / mirror / Atom feed
* [RFC PATCH 03/13] Intel(R) MEI Driver
@ 2011-02-10  9:55 Oren Weil
  2011-02-10 13:48 ` Alan Cox
  2011-02-10 18:03 ` Greg KH
  0 siblings, 2 replies; 3+ messages in thread
From: Oren Weil @ 2011-02-10  9:55 UTC (permalink / raw)
  To: gregkh, akpm, linux-kernel; +Cc: david, david.woodhouse

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
+# Copyright (c) 2003-2011, Intel Corporation.
+#
+# 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.
+#
+# 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.
+#
+# 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.
+#
+obj-$(CONFIG_INTEL_MEI) += mei.o
+mei-objs := init.o
+mei-objs += interrupt.o
+mei-objs += interface.o
+mei-objs += iorw.o
+mei-objs += main.o


---------------------------------------------------------------------
Intel Israel (74) Limited

This e-mail and any attachments may contain confidential material for
the sole use of the intended recipient(s). Any review or distribution
by others is strictly prohibited. If you are not the intended
recipient, please contact the sender and delete all copies.


^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [RFC PATCH 03/13] Intel(R) MEI Driver
  2011-02-10  9:55 [RFC PATCH 03/13] Intel(R) MEI Driver Oren Weil
@ 2011-02-10 13:48 ` Alan Cox
  2011-02-10 18:03 ` Greg KH
  1 sibling, 0 replies; 3+ messages in thread
From: Alan Cox @ 2011-02-10 13:48 UTC (permalink / raw)
  To: Oren Weil; +Cc: gregkh, akpm, linux-kernel, david, david.woodhouse

On 10 Feb 2011 01:55:03 -0800
Oren Weil <oren.jer.weil@intel.com> 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

Just a note for the final submission- the kernel should always build
through a series of changes and we rely on this for git bisect so submit
the Makefile and Kconfig as the last patches that way it works out nicely.

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [RFC PATCH 03/13] Intel(R) MEI Driver
  2011-02-10  9:55 [RFC PATCH 03/13] Intel(R) MEI Driver Oren Weil
  2011-02-10 13:48 ` Alan Cox
@ 2011-02-10 18:03 ` Greg KH
  1 sibling, 0 replies; 3+ messages in thread
From: Greg KH @ 2011-02-10 18:03 UTC (permalink / raw)
  To: Oren Weil; +Cc: akpm, linux-kernel, david, david.woodhouse

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

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2011-02-10 18:06 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-02-10  9:55 [RFC PATCH 03/13] Intel(R) MEI Driver Oren Weil
2011-02-10 13:48 ` Alan Cox
2011-02-10 18:03 ` Greg KH

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.