Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 21 Jan 2000 01:01:02 -0700
From:      Warner Losh <imp@village.org>
To:        Tatsumi Hosokawa <hosokawa@itc.keio.ac.jp>
Cc:        mobile@FreeBSD.ORG
Subject:   Re: if_sn now works on -current (Re: One more serious problem with -current PCCARD support ) 
Message-ID:  <200001210801.BAA09842@harmony.village.org>
In-Reply-To: Your message of "Fri, 21 Jan 2000 02:54:40 %2B0900." <863drs1vdr.wl@ringo.FromTo.Cc> 
References:  <863drs1vdr.wl@ringo.FromTo.Cc>  <86r9fisfe2.wl@ringo.FromTo.Cc> <86u2kesizg.wl@ringo.FromTo.Cc> <200001180742.AAA15379@harmony.village.org> <864sc820u4.wl@ringo.FromTo.Cc> 

next in thread | previous in thread | raw e-mail | index | archive | help
In message <863drs1vdr.wl@ringo.FromTo.Cc> Tatsumi Hosokawa writes:
: card "Megahertz" "CC10BT/2"
:         config  0x1 "sn0" ?
:         ether attr2
:         insert  logger -s Megahertz X-Jack Ethernet inserted
:         insert  /etc/pccard_ether $device
:         remove  logger -s Megahertz X-Jack Ethernet removed
:         remove  /etc/pccard_ether_remove $device

Looks good.

: This patch adds "attr2" keyword to /etc/pccard.conf and I'm afraid
: that it can break feature freeze.  If it's not acceptable, I'll commit
: it later.

That is fine by me.  When you commit this, you may want to add that I
said it was OK.

: Index: sys/dev/sn/if_sn_pccard.c

I think you missed the kludge I have in if_sn.c:

	if (1) {
		/* XXX The pccard probe routine for megahearts needs to */
		/* XXX snag this from your info 2 */
		int j;

		for (j = 0; j < 3; j++) {
			u_short	w;

			w = (u_short)sn_pccard_macaddr[j * 2] | 
				(((u_short)sn_pccard_macaddr[j * 2 + 1]) << 8);
			outw(BASE + IAR_ADDR0_REG_W + j * 2, w);
		}
	}

which needs to be replaced by more appropriate code.

Warner


To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe freebsd-mobile" in the body of the message




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200001210801.BAA09842>