From owner-freebsd-mobile Tue Jul 20 17:16: 5 1999 Delivered-To: freebsd-mobile@freebsd.org Received: from afs.ntc.mita.keio.ac.jp (afs.ntc.mita.keio.ac.jp [131.113.212.3]) by hub.freebsd.org (Postfix) with ESMTP id 3993014C45 for ; Tue, 20 Jul 1999 17:15:58 -0700 (PDT) (envelope-from hosokawa@ntc.keio.ac.jp) Received: (from hosokawa@localhost) by afs.ntc.mita.keio.ac.jp (8.8.8+2.7Wbeta7/3.6Wbeta6-ntc_mailserver1.03) id JAA27543; Wed, 21 Jul 1999 09:14:56 +0900 (JST) Date: Wed, 21 Jul 1999 09:14:56 +0900 (JST) Message-Id: <199907210014.JAA27543@afs.ntc.mita.keio.ac.jp> To: nate@mt.sri.com Cc: imp@village.org, mobile@FreeBSD.ORG, hosokawa@itc.keio.ac.jp Subject: Re: usr.sbin/pccard/pccardd change for "cardio" and "cardmem" In-Reply-To: Your message of "Wed, 21 Jul 1999 03:52:24 JST". <199907201852.MAA20790@mt.sri.com> From: hosokawa@itc.keio.ac.jp (HOSOKAWA Tatsumi) X-Mailer: mnews [version 1.21] 1997-12/23(Tue) Sender: owner-freebsd-mobile@FreeBSD.ORG Precedence: bulk X-Loop: FreeBSD.org In article <199907201852.MAA20790@mt.sri.com> nate@mt.sri.com writes: >> Fix for what? What is the fix for? Just adding keywords to fix 'buggy' >> cards is not a fix, it's (potentially) a hack to workaround bogus code >> in other parts of the system that might be easily fixed. >> >> Many of the 'fixes' in PAO were *easily* fixed with just a bit of time >> with the 'bad' cards. Making the already difficult to understand code >> bigger and more complex with fixes for broken existing code is just >> silly. I don't want to incorporate such "fixes" in PAO into kernel and pccardd itself. This patch can exclude such fix out to /etc/pccard.conf and you can add commment such that "it's broken card and this is a hack for this" as plaintext. I think it would be better than hiding such workaround into kernel and pccardd binary. Most important function of "cardio" is "cardio 0". It's unstrengthen I/O address binding in CIS tupple (that's not needed for DOS compatibility). Maybe this is very useful and easier way in serial cards and Flash ATA cards without adding any new entries to pccard.conf. BTW, I admit that it's required in broken FreeBSD code in some cases. For example, it's required in merging wd.c pccard support because wd.c is tightly bound to traditional I/O address location of IDE interface. I'm trying to rewrite my wd.c patch from "pccard and PCI are spacial cases" to "traditional ISA is one of the cases" before committing it, but if we fix it essentialy, it may be includes fix for "config" system and may affect some user's environment (correct me if I am wrong. I want to fix it essentially if it's not so difficult). -- HOSOKAWA, Tatsumi Assistant Manager Information Technology Center, Keio University To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe freebsd-mobile" in the body of the message