Subject: Re: CVS commit: src/sys/dev/pci
To: Izumi Tsutsui <tsutsui@ceres.dti.ne.jp>
From: Jaromir Dolecek <jdolecek@NetBSD.org>
List: source-changes
Date: 03/05/2005 16:05:09
On Sat, Mar 05, 2005 at 08:35:54PM +0900, Izumi Tsutsui wrote:
> In article <20050220183433.53CDC2DA1D@cvs.netbsd.org>
> jdolecek@netbsd.org wrote:
> 
> > Modified Files:
> > 	src/sys/dev/pci: files.pci
> > Added Files:
> > 	src/sys/dev/pci: if_vge.c if_vgereg.h if_vgevar.h
> > 
> > Log Message:
> > Add device driver support for the VIA Networking Technologies VT6122
> > gigabit ethernet chip and integrated 10/100/1000 copper PHY.
> > 
> > Obtained from: FreeBSD
> 
> Is this tested on big endian machines?

Apparently not (certainly not by me) - the driver has been committed
to FreeBSD tree with commit message containing:

"""
The vge driver has been added to GENERIC for i386, pc98 and amd64,
but not to sparc or ia64 since I don't have the ability to test
it there.
"""

> Some FreeBSD guy always uses strange byteswap ops on reading
> values from 93Cx6 based EEPROM, which returns 16bit words.
> (attached patch is not tested)

Assuming eaddr is stored in network byte order and the returned
value being in network byte order too, the code might actually
be just fine as-is. If you had a chance to try this on big-endian
machine, let me know :)

Jaromir
-- 
Jaromir Dolecek <jdolecek@NetBSD.org>            http://www.NetBSD.cz/
-=- We can walk our road together if our goals are all the same;     -=-
-=- We can run alone and free if we pursue a different aim.          -=-