cvs commit: src/sys/dev/sk if_sk.c if_skreg.h

Bjoern A. Zeeb bz at FreeBSD.org
Sun Apr 30 01:45:11 UTC 2006


On Sat, 29 Apr 2006, Maxim Sobolev wrote:

> Bjoern A. Zeeb wrote:
>> On Fri, 28 Apr 2006, Maxim Sobolev wrote:
>> 
>>> Joe Marcus Clarke wrote:
>>>> On Fri, 2006-04-28 at 03:17 +0000, Maxim Sobolev wrote:
>>>>> sobomax     2006-04-28 03:17:37 UTC
>>>>>
>>>>>   FreeBSD src repository
>>>>>
>>>>>   Modified files:
>>>>>     sys/dev/sk           if_sk.c if_skreg.h   Log:
>>>>>   Add some incomplete support for Marvell Yukon EC controllers based on
>>>>>   OpenBSD changes. With these changes, PHY part of the driver becomes
>> 
>> So are you going to back out your last change to sk(4)?
>
> Why would I?

because as you said it's incomplete and doesn't really help sk and
further won't help once we get myk into the tree because sk will
also successfully attach and not work for those cards then.
What was also said is that we do not want to add the code for the
PCIe (Y-II) cards to sk but want to have a different driver.
sk already has too many if() in its paths and adding different
kind of hardware to the driver won't make it any better.

If you have time and interest in getting the PCIe chipsets working you
should have a look at the myk code posted by andre and the follow-up
comments.

Thanks.

-- 
Bjoern A. Zeeb				bzeeb at Zabbadoz dot NeT


More information about the cvs-src mailing list