Re: [PATCH v5] net: ethernet: add driver for Aurora VLSI NB8800 Ethernet controller

From: David Miller
Date: Tue Nov 10 2015 - 16:06:23 EST


From: Måns Rullgård <mans@xxxxxxxxx>
Date: Tue, 10 Nov 2015 20:53:19 +0000

> David Miller <davem@xxxxxxxxxxxxx> writes:
>
>> From: Måns Rullgård <mans@xxxxxxxxx>
>> 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 linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/