2007-01-11 01:29:59

by Jay Cliburn

[permalink] [raw]
Subject: [PATCH 1/4] atl1: Build files for Attansic L1 driver


From: Jay Cliburn <[email protected]>
From: Chris Snook <[email protected]>

This patch contains the build files for the Attansic L1 gigabit ethernet
adapter driver.

Signed-off-by: Jay Cliburn <[email protected]>
Signed-off-by: Chris Snook <[email protected]>
---

Kconfig | 11 +++++++++++
Makefile | 1 +
atl1/Makefile | 30 ++++++++++++++++++++++++++++++
3 files changed, 42 insertions(+)

diff --git a/drivers/net/Kconfig b/drivers/net/Kconfig
index 8aa8dd0..92a5efe 100644
--- a/drivers/net/Kconfig
+++ b/drivers/net/Kconfig
@@ -2348,6 +2348,17 @@ config QLA3XXX
To compile this driver as a module, choose M here: the module
will be called qla3xxx.

+config ATL1
+ tristate "Attansic(R) L1 Gigabit Ethernet support (EXPERIMENTAL)"
+ depends on NET_PCI && PCI && EXPERIMENTAL
+ select CRC32
+ select MII
+ help
+ This driver supports Attansic L1 gigabit ethernet adapter.
+
+ To compile this driver as a module, choose M here. The module
+ will be called atl1.
+
endmenu

#
diff --git a/drivers/net/Makefile b/drivers/net/Makefile
index 4c0d4e5..d0beced 100644
--- a/drivers/net/Makefile
+++ b/drivers/net/Makefile
@@ -8,6 +8,7 @@ obj-$(CONFIG_IXGB) += ixgb/
obj-$(CONFIG_CHELSIO_T1) += chelsio/
obj-$(CONFIG_EHEA) += ehea/
obj-$(CONFIG_BONDING) += bonding/
+obj-$(CONFIG_ATL1) += atl1/
obj-$(CONFIG_GIANFAR) += gianfar_driver.o

gianfar_driver-objs := gianfar.o \
diff --git a/drivers/net/atl1/Makefile b/drivers/net/atl1/Makefile
new file mode 100644
index 0000000..1a10b91
--- /dev/null
+++ b/drivers/net/atl1/Makefile
@@ -0,0 +1,30 @@
+################################################################################
+#
+# Attansic L1 gigabit ethernet driver
+# Copyright(c) 2005 - 2006 Attansic 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.
+#
+# The full GNU General Public License is included in this distribution in
+# the file called "COPYING".
+#
+################################################################################
+
+#
+# Makefile for the Attansic L1 gigabit ethernet driver
+#
+
+obj-$(CONFIG_ATL1) += atl1.o
+
+atl1-objs := atl1_main.o atl1_hw.o atl1_ethtool.o atl1_param.o


2007-01-11 09:19:37

by Christoph Hellwig

[permalink] [raw]
Subject: Re: [PATCH 1/4] atl1: Build files for Attansic L1 driver

On Wed, Jan 10, 2007 at 06:40:51PM -0600, Jay Cliburn wrote:
> --- /dev/null
> +++ b/drivers/net/atl1/Makefile
> @@ -0,0 +1,30 @@
> +################################################################################
> +#
> +# Attansic L1 gigabit ethernet driver
> +# Copyright(c) 2005 - 2006 Attansic 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.
> +#
> +# The full GNU General Public License is included in this distribution in
> +# the file called "COPYING".
> +#
> +################################################################################

I don't think anyone can claim copyright on two lines of actual kbuild code.

> +#
> +# Makefile for the Attansic L1 gigabit ethernet driver
> +#

This comment is antirely superflous.

> +obj-$(CONFIG_ATL1) += atl1.o
> +
> +atl1-objs := atl1_main.o atl1_hw.o atl1_ethtool.o atl1_param.o

Thi should be atl1-y += ...

In short the whole contents of this file should be:

---------------- snip ----------------
obj-$(CONFIG_ATL1) += atl1.o
atl1-y += atl1_main.o atl1_hw.o atl1_ethtool.o atl1_param.o
---------------- snip ----------------

2007-01-11 09:32:27

by Chris Snook

[permalink] [raw]
Subject: Re: [PATCH 1/4] atl1: Build files for Attansic L1 driver

Christoph Hellwig wrote:
> On Wed, Jan 10, 2007 at 06:40:51PM -0600, Jay Cliburn wrote:
>> --- /dev/null
>> +++ b/drivers/net/atl1/Makefile
>> @@ -0,0 +1,30 @@
>> +################################################################################
>> +#
>> +# Attansic L1 gigabit ethernet driver
>> +# Copyright(c) 2005 - 2006 Attansic 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.
>> +#
>> +# The full GNU General Public License is included in this distribution in
>> +# the file called "COPYING".
>> +#
>> +################################################################################
>
> I don't think anyone can claim copyright on two lines of actual kbuild code.
>
>> +#
>> +# Makefile for the Attansic L1 gigabit ethernet driver
>> +#
>
> This comment is antirely superflous.
>
>> +obj-$(CONFIG_ATL1) += atl1.o
>> +
>> +atl1-objs := atl1_main.o atl1_hw.o atl1_ethtool.o atl1_param.o
>
> Thi should be atl1-y += ...
>
> In short the whole contents of this file should be:
>
> ---------------- snip ----------------
> obj-$(CONFIG_ATL1) += atl1.o
> atl1-y += atl1_main.o atl1_hw.o atl1_ethtool.o atl1_param.o
> ---------------- snip ----------------
>

Good point. The original Attansic driver had a whole bunch of legacy
compat crap, documentation build targets left over from e1000, etc.
which we've been modifying and mostly just removing. We don't really
need this for merging. If Attansic wants to maintain something out of
tree for legacy kernels, they might want to reinsert this, but we really
don't need it.

Thanks for pointing this out.

-- Chris

2007-01-11 09:49:42

by Jeff Garzik

[permalink] [raw]
Subject: Re: [PATCH 1/4] atl1: Build files for Attansic L1 driver

FWIW Jay is not the vendor, just someone who is helping clean up an ugly
vendor driver.

Jeff