Subject: Re: [Patch 2/2]r8169: remove redundant RTL_W32

From: Ben Hutchings <[email-masked]>
Date: Thu, 03 Jun 2010 13:40:10 +0100

On Thu, 2010-06-03 at 19:27 +0800, Junchang Wang wrote:
> Writting "cmd" into "CounterAddrLow" twice is redundant.
[...]

I assume you've checked this against the datasheets for all 20 variants
of this hardware?

Ben.

-- 
Ben Hutchings, Senior Software Engineer, Solarflare Communications
Not speaking for my employer; that's the marketing department's job.
They asked us to note that Solarflare product names are trademarked.