netdev
[Top] [All Lists]

Re: [PATCH] mv643xx enet update + ethtool support

To: Dale Farnsworth <dale@xxxxxxxxxxxxxx>
Subject: Re: [PATCH] mv643xx enet update + ethtool support
From: Jeff Garzik <jgarzik@xxxxxxxxx>
Date: Thu, 24 Feb 2005 00:02:11 -0500
Cc: netdev@xxxxxxxxxxx, Ralf Baechle <ralf@xxxxxxxxxxxxxx>, Manish Lachwani <mlachwani@xxxxxxxxxx>, Brian Waite <brian@xxxxxxxxxxxxxx>, "Steven J. Hill" <sjhill@xxxxxxxxxxxxxxxxxx>, Benjamin Herrenschmidt <benh@xxxxxxxxxxxxxxxxxxx>
In-reply-to: <20050217224239.GA16609@xyzzy>
References: <20050217224239.GA16609@xyzzy>
Sender: netdev-bounce@xxxxxxxxxxx
User-agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.7.3) Gecko/20040922
Dale Farnsworth wrote:
Here is an update to mv643xx ethernet support. There are several small bugfixes. The two larger issues are
the beginning of ethtool support and the fact that I've disabled
hardware tcp/udp checksum generation.  It now looks like there's
a hardware bug with the checksums, but I'm still characterizing it.

ChangeSets before 1.2065 are already in -netdev.
I've included a cumulative patch of the new changesets below.

Thanks,
-Dale Farnsworth

Please do a

        bk pull bk://dfarnsworth.bkbits.net/linux-2.5-mv643xx-enet

Pulled.  Two comments:

1) Your descriptions MUST follow the "one line summary" format found in other changesets.

When Linus, myself and others use automated tools to summarize a large number of changesets, the first line from each changeset description is used. You should include an identifier indicating which area of the kernel tree you are modifying ("mv643xx" in this case), and a one-line summary of the changeset. For example, when I check in net driver changesets, they look like

        [netdrvr 8139too] fix bugs, whitespace change

        * fix RX error overflow case
        * Update PCI initialization code to better use PCI API
        * trim trailing whitespace (chomp)

For now, I have (with great patience) hand-edited all 30+ changesets to fix this up.

See rule #1 of http://linux.yyz.us/patch-format.html. Even though that URL applies to patches, not BitKeeper, the one-line-summary and $subsystem portions are applicable to you.


+void mv643xx_set_ethtool_ops(struct net_device *netdev)
+{
+       SET_ETHTOOL_OPS(netdev, &mv643xx_ethtool_ops);
+}
+

2) This function should not exist.

        Jeff



<Prev in Thread] Current Thread [Next in Thread>