netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 4/4] ehea: Kconfig and Makefile
@ 2006-06-07 17:06 Jan-Bernd Themann
  2006-06-07 17:59 ` Randy.Dunlap
  0 siblings, 1 reply; 2+ messages in thread
From: Jan-Bernd Themann @ 2006-06-07 17:06 UTC (permalink / raw)
  To: netdev; +Cc: meder, raisch, themann, tklein

Signed-off-by:  Jan-Bernd Themann <themann@de.ibm.com>


drivers/net/ehea/Kconfig  |    6 ++++++
drivers/net/ehea/Makefile |   45 +++++++++++++++++++++++++++++++++++++++++++++
2 files changed, 51 insertions(+)



--- linux-2.6.16-rc5-orig/drivers/net/ehea/Makefile    1969-12-31 16:00:00.000000000 -0800
+++ kernel/drivers/net/ehea/Makefile    2006-06-07 07:01:14.634178704 -0700
@@ -0,0 +1,45 @@
+# Server eHEA ethernet device driver for Linux on POWER
+#
+#
+#  linux/drivers/net/ehea/Makefile
+#
+#  eHEA ethernet device driver for IBM eServer System p
+#
+#  (C) Copyright IBM Corp. 2006
+#
+#  Authors:
+#       Christoph Raisch <raisch@de.ibm.com>
+#       Jan-Bernd Themann <themann@de.ibm.com>
+#       Heiko-Joerg Schick <schickhj@de.ibm.com>
+#       Thomas Klein <tklein@de.ibm.com>
+#
+#
+# This program is free software; you can redistribute it and/or modify
+# it under the terms of the GNU General Public License as published by
+# the Free Software Foundation; either version 2, or (at your option)
+# any later version.
+#
+# This program is distributed in the hope that 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., 675 Mass Ave, Cambridge, MA 02139, USA.
+
+
+
+
+# To compile module type:
+# make -C /root/linux-2.6.5-7.244 SUBDIRS=`pwd` V=1 modules CONFIG_EHEA=m
+# make -C /usr/src/linux-2.6.17-rc5 SUBDIRS=`pwd` V=1 modules CONFIG_EHEA=m
+
+ehea_mod-objs = ehea_main.o ehea_phyp.o ehea_qmr.o ehea_ethtool.o ehea_phyp.o
+
+obj-$(CONFIG_EHEA) += ehea_mod.o
+
+#CFLAGS += -Werror
+
+
+
--- linux-2.6.16-rc5-orig/drivers/net/ehea/Kconfig    1969-12-31 16:00:00.000000000 -0800
+++ kernel/drivers/net/ehea/Kconfig    2006-06-07 07:01:14.637178248 -0700
@@ -0,0 +1,6 @@
+config EHEA
+       tristate "eHEA Ethernet support"
+       depends on IBMEBUS
+       ---help---
+       This driver supports the IBM pSeries
+       Ethernet adapter




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

* Re: [PATCH 4/4] ehea: Kconfig and Makefile
  2006-06-07 17:06 [PATCH 4/4] ehea: Kconfig and Makefile Jan-Bernd Themann
@ 2006-06-07 17:59 ` Randy.Dunlap
  0 siblings, 0 replies; 2+ messages in thread
From: Randy.Dunlap @ 2006-06-07 17:59 UTC (permalink / raw)
  To: Jan-Bernd Themann; +Cc: netdev, meder, raisch, themann, tklein

On Wed, 07 Jun 2006 19:06:03 +0200 Jan-Bernd Themann wrote:

> Signed-off-by:  Jan-Bernd Themann <themann@de.ibm.com>
> 
> 
> drivers/net/ehea/Kconfig  |    6 ++++++
> drivers/net/ehea/Makefile |   45 +++++++++++++++++++++++++++++++++++++++++++++
> 2 files changed, 51 insertions(+)
> 
> 
> 
> --- linux-2.6.16-rc5-orig/drivers/net/ehea/Makefile    1969-12-31 16:00:00.000000000 -0800
> +++ kernel/drivers/net/ehea/Makefile    2006-06-07 07:01:14.634178704 -0700
> @@ -0,0 +1,45 @@
> +# Server eHEA ethernet device driver for Linux on POWER
> +#
> +#
> +#  linux/drivers/net/ehea/Makefile
> +#
> +#  eHEA ethernet device driver for IBM eServer System p
> +#
> +#  (C) Copyright IBM Corp. 2006
> +#
> +#  Authors:
> +#       Christoph Raisch <raisch@de.ibm.com>
> +#       Jan-Bernd Themann <themann@de.ibm.com>
> +#       Heiko-Joerg Schick <schickhj@de.ibm.com>
> +#       Thomas Klein <tklein@de.ibm.com>
> +#
> +#
> +# This program is free software; you can redistribute it and/or modify
> +# it under the terms of the GNU General Public License as published by
> +# the Free Software Foundation; either version 2, or (at your option)
> +# any later version.
> +#
> +# This program is distributed in the hope that 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., 675 Mass Ave, Cambridge, MA 02139, USA.
> +
> +
> +
> +
> +# To compile module type:
> +# make -C /root/linux-2.6.5-7.244 SUBDIRS=`pwd` V=1 modules CONFIG_EHEA=m
> +# make -C /usr/src/linux-2.6.17-rc5 SUBDIRS=`pwd` V=1 modules CONFIG_EHEA=m
> +
> +ehea_mod-objs = ehea_main.o ehea_phyp.o ehea_qmr.o ehea_ethtool.o ehea_phyp.o
> +
> +obj-$(CONFIG_EHEA) += ehea_mod.o
> +
> +#CFLAGS += -Werror
> +
> +
> +

45 lines for a 5-line makefile.  :(

You need to integrate the Makefile and Kconfig into the upper-level
directory Makefile and Kconfig files.  Don't expect/require people
to enter thing like this:
> +# make -C /root/linux-2.6.5-7.244 SUBDIRS=`pwd` V=1 modules CONFIG_EHEA=m
> +# make -C /usr/src/linux-2.6.17-rc5 SUBDIRS=`pwd` V=1 modules CONFIG_EHEA=m


> --- linux-2.6.16-rc5-orig/drivers/net/ehea/Kconfig    1969-12-31 16:00:00.000000000 -0800
> +++ kernel/drivers/net/ehea/Kconfig    2006-06-07 07:01:14.637178248 -0700
> @@ -0,0 +1,6 @@
> +config EHEA
> +       tristate "eHEA Ethernet support"
> +       depends on IBMEBUS
> +       ---help---
> +       This driver supports the IBM pSeries
> +       Ethernet adapter

That last sentence can be on one line, please.

---
~Randy

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

end of thread, other threads:[~2006-06-07 17:57 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2006-06-07 17:06 [PATCH 4/4] ehea: Kconfig and Makefile Jan-Bernd Themann
2006-06-07 17:59 ` Randy.Dunlap

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).