Mailing List Archive: Linux: Kernel

[PATCH 2.6.25.3 1/5] atl1e: Atheros L1E Gigabit Ethernet driver

Index | Next | Previous | View Threaded

Jie.Yang at Atheros

Jun 25, 2008, 10:37 PM

Post #1 of 8 (8785 views)
Permalink

[PATCH 2.6.25.3 1/5] atl1e: Atheros L1E Gigabit Ethernet driver

From: Jie Yang <jie.yang [at] atheros>

Full patch for the Atheros L1E Gigabit Ethernet driver.
Supportring AR8121, AR8113 and AR8114

Signed-off-by: Jie Yang <jie.yang @atheros.com>
---
diff -uprN -X linux-2.6.25.3.orig/Documentation/dontdiff linux-2.6.25.3.orig/drivers/net/atl1e/Makefile linux-2.6.25.3.atheros/drivers/net/atl1e/Makefile
--- linux-2.6.25.3.orig/drivers/net/atl1e/Makefile 1970-01-01 08:00:00.000000000 +0800
+++ linux-2.6.25.3.atheros/drivers/net/atl1e/Makefile 2008-06-20 11:22:34.000000000 +0800
@@ -0,0 +1,2 @@
+obj-$(CONFIG_ATL1E) += atl1e.o
+atl1e-y += atl1e_main.o atl1e_hw.o atl1e_ethtool.o atl1e_param.o
diff -uprN -X linux-2.6.25.3.orig/Documentation/dontdiff linux-2.6.25.3.orig/drivers/net/Kconfig linux-2.6.25.3.atheros/drivers/net/Kconfig
--- linux-2.6.25.3.orig/drivers/net/Kconfig 2008-05-10 12:48:50.000000000 +0800
+++ linux-2.6.25.3.atheros/drivers/net/Kconfig 2008-06-19 18:17:42.000000000 +0800
@@ -2445,6 +2445,32 @@ config ATL1
To compile this driver as a module, choose M here. The module
will be called atl1.

+config ATL1E
+ tristate "Atheros L1e Gigabit Ethernet support (EXPERIMENTAL)"
+ depends on PCI && EXPERIMENTAL
+ select CRC32
+ select MII
+ help
+ This driver supports the Atheros L1E gigabit ethernet adapter.
+
+ To compile this driver as a module, choose M here. The module
+ will be called atl1e.
+
+config ATL1E_NAPI
+ bool "Use Rx Polling (NAPI)"
+ depends on ATL1E
+ help
+ NAPI is a new driver API designed to reduce CPU and interrupt load
+ when the driver is receiving lots of packets from the card. It is
+ still somewhat experimental and thus not yet enabled by default.
+
+ If your estimated Rx load is 10kpps or more, or if the card will be
+ deployed on potentially unfriendly networks (e.g. in a firewall),
+ then say Y here.
+
+ If in doubt, say N.
+
+
endif # NETDEV_1000

#
diff -uprN -X linux-2.6.25.3.orig/Documentation/dontdiff linux-2.6.25.3.orig/drivers/net/Makefile linux-2.6.25.3.atheros/drivers/net/Makefile
--- linux-2.6.25.3.orig/drivers/net/Makefile 2008-05-10 12:48:50.000000000 +0800
+++ linux-2.6.25.3.atheros/drivers/net/Makefile 2008-06-20 09:48:38.000000000 +0800
@@ -16,6 +16,7 @@ obj-$(CONFIG_EHEA) += ehea/
obj-$(CONFIG_CAN) += can/
obj-$(CONFIG_BONDING) += bonding/
obj-$(CONFIG_ATL1) += atl1/
+obj-$(CONFIG_ATL1E) += atl1e/
obj-$(CONFIG_GIANFAR) += gianfar_driver.o
obj-$(CONFIG_TEHUTI) += tehuti.o

diff -uprN -X linux-2.6.25.3.orig/Documentation/dontdiff linux-2.6.25.3.orig/include/linux/pci_ids.h linux-2.6.25.3.atheros/include/linux/pci_ids.h
--- linux-2.6.25.3.orig/include/linux/pci_ids.h 2008-05-10 12:48:50.000000000 +0800
+++ linux-2.6.25.3.atheros/include/linux/pci_ids.h 2008-06-20 09:43:40.000000000 +0800
@@ -2175,6 +2175,7 @@

#define PCI_VENDOR_ID_ATTANSIC 0x1969
#define PCI_DEVICE_ID_ATTANSIC_L1 0x1048
+#define PCI_DEVICE_ID_ATTANSIC_L1E 0x1026

#define PCI_VENDOR_ID_JMICRON 0x197B
#define PCI_DEVICE_ID_JMICRON_JMB360 0x2360
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo [at] vger
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

shemminger at vyatta

Jun 25, 2008, 10:48 PM

Post #2 of 8 (8497 views)
Permalink

Re: [PATCH 2.6.25.3 1/5] atl1e: Atheros L1E Gigabit Ethernet driver [In reply to]

On Thu, 26 Jun 2008 13:37:23 +0800
Jie Yang <Jie.Yang [at] Atheros> wrote:

> From: Jie Yang <jie.yang [at] atheros>
>
> Full patch for the Atheros L1E Gigabit Ethernet driver.
> Supportring AR8121, AR8113 and AR8114
>
> Signed-off-by: Jie Yang <jie.yang @atheros.com>
> ---
> diff -uprN -X linux-2.6.25.3.orig/Documentation/dontdiff linux-2.6.25.3.orig/drivers/net/atl1e/Makefile linux-2.6.25.3.atheros/drivers/net/atl1e/Makefile
> --- linux-2.6.25.3.orig/drivers/net/atl1e/Makefile 1970-01-01 08:00:00.000000000 +0800
> +++ linux-2.6.25.3.atheros/drivers/net/atl1e/Makefile 2008-06-20 11:22:34.000000000 +0800
> @@ -0,0 +1,2 @@
> +obj-$(CONFIG_ATL1E) += atl1e.o
> +atl1e-y += atl1e_main.o atl1e_hw.o atl1e_ethtool.o atl1e_param.o
> diff -uprN -X linux-2.6.25.3.orig/Documentation/dontdiff linux-2.6.25.3.orig/drivers/net/Kconfig linux-2.6.25.3.atheros/drivers/net/Kconfig
> --- linux-2.6.25.3.orig/drivers/net/Kconfig 2008-05-10 12:48:50.000000000 +0800
> +++ linux-2.6.25.3.atheros/drivers/net/Kconfig 2008-06-19 18:17:42.000000000 +0800
> @@ -2445,6 +2445,32 @@ config ATL1
> To compile this driver as a module, choose M here. The module
> will be called atl1.
>
> +config ATL1E
> + tristate "Atheros L1e Gigabit Ethernet support (EXPERIMENTAL)"
> + depends on PCI && EXPERIMENTAL
> + select CRC32
> + select MII
> + help
> + This driver supports the Atheros L1E gigabit ethernet adapter.
> +
> + To compile this driver as a module, choose M here. The module
> + will be called atl1e.
> +
> +config ATL1E_NAPI
> + bool "Use Rx Polling (NAPI)"
> + depends on ATL1E
> + help
> + NAPI is a new driver API designed to reduce CPU and interrupt load
> + when the driver is receiving lots of packets from the card. It is
> + still somewhat experimental and thus not yet enabled by default.
> +
> + If your estimated Rx load is 10kpps or more, or if the card will be
> + deployed on potentially unfriendly networks (e.g. in a firewall),
> + then say Y here.
> +
> + If in doubt, say N.
> +
> +
> endif # NETDEV_1000

Congratulations on the best new vendor driver submission to date.

I would prefer that the driver be NAPI only, because configuration choices
are hard for users and distribution to make correct choice. Plus it means two
choices for your QA to test.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo [at] vger
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

davem at davemloft

Jun 25, 2008, 10:55 PM

Post #3 of 8 (8491 views)
Permalink

Re: [PATCH 2.6.25.3 1/5] atl1e: Atheros L1E Gigabit Ethernet driver [In reply to]

From: Stephen Hemminger <shemminger [at] vyatta>
Date: Wed, 25 Jun 2008 22:48:29 -0700

> I would prefer that the driver be NAPI only, because configuration
> choices are hard for users and distribution to make correct
> choice. Plus it means two choices for your QA to test.

I agree with Stephen.

It's absolutely to provide a non-NAPI option, please just remove
the option and the conditionals from the code.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo [at] vger
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

w at 1wt

Jun 25, 2008, 10:59 PM

Post #4 of 8 (8486 views)
Permalink

Re: [PATCH 2.6.25.3 1/5] atl1e: Atheros L1E Gigabit Ethernet driver [In reply to]

On Wed, Jun 25, 2008 at 10:55:50PM -0700, David Miller wrote:
> From: Stephen Hemminger <shemminger [at] vyatta>
> Date: Wed, 25 Jun 2008 22:48:29 -0700
>
> > I would prefer that the driver be NAPI only, because configuration
> > choices are hard for users and distribution to make correct
> > choice. Plus it means two choices for your QA to test.
>
> I agree with Stephen.
>
> It's absolutely to provide a non-NAPI option, please just remove
> the option and the conditionals from the code.

Also I think it's useless to provide one patch per source file. It'd
be better to have one whole patch, or several ones if some features
may be added incrementally.

Willy

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

Jie.Yang at Atheros

Jun 25, 2008, 11:25 PM

Post #5 of 8 (8506 views)
Permalink

RE: [PATCH 2.6.25.3 1/5] atl1e: Atheros L1E Gigabit Ethernet driver [In reply to]

From: Willy Tarreau [w [at] 1wt]
Date: Thursday, June 26, 2008 1:59 PM

> On Wed, Jun 25, 2008 at 10:55:50PM -0700, David Miller wrote:
> > From: Stephen Hemminger <shemminger [at] vyatta>
> > Date: Wed, 25 Jun 2008 22:48:29 -0700
> >
> > > I would prefer that the driver be NAPI only, because
> configuration
> > > choices are hard for users and distribution to make
> correct choice.
> > > Plus it means two choices for your QA to test.
> >
> > I agree with Stephen.
> >
> > It's absolutely to provide a non-NAPI option, please just
> remove the
> > option and the conditionals from the code.
>
> Also I think it's useless to provide one patch per source
> file. It'd be better to have one whole patch, or several ones
> if some features may be added incrementally.
>
> Willy
>
>

Ok, thanks for comments. I will change code on these comments.

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

jacliburn at bellsouth

Jun 26, 2008, 7:45 PM

Post #6 of 8 (8476 views)
Permalink

Re: [PATCH 2.6.25.3 1/5] atl1e: Atheros L1E Gigabit Ethernet driver [In reply to]

Patches 1 and 2 reviewed tonight. I intend to review the other 3 in
coming days.

On Thu, 26 Jun 2008 13:37:23 +0800
Jie Yang <Jie.Yang [at] Atheros> wrote:

> From: Jie Yang <jie.yang [at] atheros>
>
> Full patch for the Atheros L1E Gigabit Ethernet driver.
> Supportring AR8121, AR8113 and AR8114

Please generate the patch against current mainline (or at least
current -rc); it fails to apply cleanly as is.

>
> Signed-off-by: Jie Yang <jie.yang @atheros.com>
> ---
> diff -uprN -X linux-2.6.25.3.orig/Documentation/dontdiff
> linux-2.6.25.3.orig/drivers/net/atl1e/Makefile
> linux-2.6.25.3.atheros/drivers/net/atl1e/Makefile ---
> linux-2.6.25.3.orig/drivers/net/atl1e/Makefile 1970-01-01
> 08:00:00.000000000 +0800 +++
> linux-2.6.25.3.atheros/drivers/net/atl1e/Makefile 2008-06-20
> 11:22:34.000000000 +0800 @@ -0,0 +1,2 @@ +obj-$(CONFIG_ATL1E) +=
> atl1e.o +atl1e-y += atl1e_main.o atl1e_hw.o
> atl1e_ethtool.o atl1e_param.o diff -uprN -X

Use tabs, not spaces, for indentation. Run scripts/checkpatch.pl on
each patchfile and try to eliminate as many errors as you can. There
are hundreds of checkpatch-reported errors in your submitted patches.

> linux-2.6.25.3.orig/Documentation/dontdiff
> linux-2.6.25.3.orig/drivers/net/Kconfig
> linux-2.6.25.3.atheros/drivers/net/Kconfig ---
> linux-2.6.25.3.orig/drivers/net/Kconfig 2008-05-10
> 12:48:50.000000000 +0800 +++
> linux-2.6.25.3.atheros/drivers/net/Kconfig 2008-06-19
> 18:17:42.000000000 +0800 @@ -2445,6 +2445,32 @@ config ATL1 To
> compile this driver as a module, choose M here. The module will be
> called atl1.
>
> +config ATL1E
> + tristate "Atheros L1e Gigabit Ethernet support (EXPERIMENTAL)"
> + depends on PCI && EXPERIMENTAL
> + select CRC32
> + select MII
> + help
> + This driver supports the Atheros L1E gigabit ethernet
> adapter. +
> + To compile this driver as a module, choose M here. The
> module
> + will be called atl1e.
> +
> +config ATL1E_NAPI
> + bool "Use Rx Polling (NAPI)"
> + depends on ATL1E
> + help
> + NAPI is a new driver API designed to reduce CPU and
> interrupt load
> + when the driver is receiving lots of packets from the card.
> It is
> + still somewhat experimental and thus not yet enabled by
> default. +
> + If your estimated Rx load is 10kpps or more, or if the card
> will be
> + deployed on potentially unfriendly networks (e.g. in a
> firewall),
> + then say Y here.
> +
> + If in doubt, say N.
> +
> +

Remove extra blank line.

> endif # NETDEV_1000
>
> #
> diff -uprN -X linux-2.6.25.3.orig/Documentation/dontdiff
> linux-2.6.25.3.orig/drivers/net/Makefile
> linux-2.6.25.3.atheros/drivers/net/Makefile ---
> linux-2.6.25.3.orig/drivers/net/Makefile 2008-05-10
> 12:48:50.000000000 +0800 +++
> linux-2.6.25.3.atheros/drivers/net/Makefile 2008-06-20
> 09:48:38.000000000 +0800 @@ -16,6 +16,7 @@ obj-$(CONFIG_EHEA) +=
> ehea/ obj-$(CONFIG_CAN) += can/ obj-$(CONFIG_BONDING) += bonding/
> obj-$(CONFIG_ATL1) += atl1/ +obj-$(CONFIG_ATL1E) += atl1e/
> obj-$(CONFIG_GIANFAR) += gianfar_driver.o obj-$(CONFIG_TEHUTI) +=
> tehuti.o
>
> diff -uprN -X linux-2.6.25.3.orig/Documentation/dontdiff
> linux-2.6.25.3.orig/include/linux/pci_ids.h
> linux-2.6.25.3.atheros/include/linux/pci_ids.h ---
> linux-2.6.25.3.orig/include/linux/pci_ids.h 2008-05-10
> 12:48:50.000000000 +0800 +++
> linux-2.6.25.3.atheros/include/linux/pci_ids.h 2008-06-20
> 09:43:40.000000000 +0800 @@ -2175,6 +2175,7 @@
>
> #define PCI_VENDOR_ID_ATTANSIC 0x1969
> #define PCI_DEVICE_ID_ATTANSIC_L1 0x1048
> +#define PCI_DEVICE_ID_ATTANSIC_L1E 0x1026
>
> #define PCI_VENDOR_ID_JMICRON 0x197B
> #define PCI_DEVICE_ID_JMICRON_JMB360 0x2360
> --
> To unsubscribe from this list: send the line "unsubscribe netdev" in
> the body of a message to majordomo [at] vger
> More majordomo info at http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo [at] vger
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

Jie.Yang at Atheros

Jun 26, 2008, 10:24 PM

Post #7 of 8 (8462 views)
Permalink

RE: [PATCH 2.6.25.3 1/5] atl1e: Atheros L1E Gigabit Ethernet driver [In reply to]

From: Jay Cliburn [acliburn [at] bellsouth]
Date: Friday, June 27, 2008 10:45 AM

> Patches 1 and 2 reviewed tonight. I intend to review the
> other 3 in coming days.

Thanks for your help.
I will update source code on these comments.

> On Thu, 26 Jun 2008 13:37:23 +0800
> Jie Yang <Jie.Yang [at] Atheros> wrote:
>
> > From: Jie Yang <jie.yang [at] atheros>
> >
> > Full patch for the Atheros L1E Gigabit Ethernet driver.
> > Supportring AR8121, AR8113 and AR8114
>
> Please generate the patch against current mainline (or at
> least current -rc); it fails to apply cleanly as is.
>

how about use the git
git://git.kernel.org/pub/scm/linux/kernel/git/jgarzik/libata-dev.git

Best wishes
jie
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo [at] vger
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

jeff at garzik

Jun 26, 2008, 10:35 PM

Post #8 of 8 (8454 views)
Permalink

Re: [PATCH 2.6.25.3 1/5] atl1e: Atheros L1E Gigabit Ethernet driver [In reply to]

Jie Yang wrote:
> From: Jay Cliburn [acliburn [at] bellsouth]
> Date: Friday, June 27, 2008 10:45 AM
>
>> Patches 1 and 2 reviewed tonight. I intend to review the
>> other 3 in coming days.
>
> Thanks for your help.
> I will update source code on these comments.
>
>> On Thu, 26 Jun 2008 13:37:23 +0800
>> Jie Yang <Jie.Yang [at] Atheros> wrote:
>>
>>> From: Jie Yang <jie.yang [at] atheros>
>>>
>>> Full patch for the Atheros L1E Gigabit Ethernet driver.
>>> Supportring AR8121, AR8113 and AR8114
>> Please generate the patch against current mainline (or at
>> least current -rc); it fails to apply cleanly as is.
>>
>
> how about use the git
> git://git.kernel.org/pub/scm/linux/kernel/git/jgarzik/libata-dev.git

You'll want to use

git://git.kernel.org/pub/scm/linux/kernel/git/davem/net-next-2.6.git

Regards,

Jeff

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

Index | Next | Previous | View Threaded

Interested in having your list archived? Contact Gossamer Threads