netdev
[Top] [All Lists]

Re: [PATCH] forcedeth: Additional ethtool support

To: Manfred Spraul <manfred@xxxxxxxxxxxxxxxx>
Subject: Re: [PATCH] forcedeth: Additional ethtool support
From: Jeff Garzik <jgarzik@xxxxxxxxx>
Date: Sun, 31 Jul 2005 00:51:04 -0400
Cc: Francois Romieu <romieu@xxxxxxxxxxxxx>, Netdev <netdev@xxxxxxxxxxx>, renaud.lienhart@xxxxxxx
In-reply-to: <42D16656.6000207@colorfullife.com>
References: <42D101EC.6000608@colorfullife.com> <20050710172833.GA1951@electric-eye.fr.zoreil.com> <42D16656.6000207@colorfullife.com>
Sender: netdev-bounce@xxxxxxxxxxx
User-agent: Mozilla Thunderbird 1.0.6-1.1.fc4 (X11/20050720)
Manfred Spraul wrote:
Not for a nic without complete documentation: What if an arch uses 64-bit reads to read two registers at the same time? Not all nics like that, for example IIRC natsemi explicitely mandates 32-bit reads.
x86-64 doesn't, it uses 32-bit reads, but I don't like the idea of using memcpy to read registers.


I agree with your other remarks, updated patch attached.


Going through my pending folder, I was about to apply all the queued forcedeth patches. However, in two cases, you violated Rule #6 of

        http://linux.yyz.us/patch-format.html

Please resend ALL forcedeth patches, with proper descriptions, so I don't have to hunt through previous patch versions, and previous emails, to match up the correct description to the correct patch.

        Jeff



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