[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <20151110.160615.1511568572619688788.davem@davemloft.net>
Date: Tue, 10 Nov 2015 16:06:15 -0500 (EST)
From: David Miller <davem@...emloft.net>
To: mans@...sr.com
Cc: eric.dumazet@...il.com, linux-kernel@...r.kernel.org,
netdev@...r.kernel.org, slash.tmp@...e.fr
Subject: Re: [PATCH v5] net: ethernet: add driver for Aurora VLSI NB8800
Ethernet controller
From: Måns Rullgård <mans@...sr.com>
Date: Tue, 10 Nov 2015 20:53:19 +0000
> David Miller <davem@...emloft.net> writes:
>
>> From: Måns Rullgård <mans@...sr.com>
>> Date: Tue, 10 Nov 2015 18:05:15 +0000
>>
>>> Because I haven't been following the netdev list closely for the last
>>> five years, and no documentation I read mentioned this function. I can
>>> certainly change it.
>>
>> It is always advisable to mimick what other drivers do and use them as
>> a reference, rather than depend upon documentation which by definition
>> is always going to be out of sync with the source tree.
>
> Sure. The trick is to pick the right driver(s) to use as reference.
> Quite a few of them don't use that function.
If you really are stumped on this matter, start at least with the
ixgbe driver. In fact pretty much every Intel ethernet driver is
a reasonable reference. Others to check out are bnx2x and mlx5.
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists