Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 20 Jul 1999 18:39:05 -0600
From:      Nate Williams <nate@mt.sri.com>
To:        hosokawa@itc.keio.ac.jp (HOSOKAWA Tatsumi)
Cc:        nate@mt.sri.com, imp@village.org, mobile@FreeBSD.ORG
Subject:   Re: usr.sbin/pccard/pccardd change for "cardio" and "cardmem" 
Message-ID:  <199907210039.SAA23797@mt.sri.com>
In-Reply-To: <199907210014.JAA27543@afs.ntc.mita.keio.ac.jp>
References:  <199907201852.MAA20790@mt.sri.com> <199907210014.JAA27543@afs.ntc.mita.keio.ac.jp>

next in thread | previous in thread | raw e-mail | index | archive | help
> >> 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.
...
> /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.

Again, what is it that this patch fixes?  What bug and/or feature does
this add to FreeBSD?  It's really hard to have an opinion on the
'goodness or badness' of a piece of code when there's no description of
what the patch does?

You've explained the 'functional' change (what the code does), but not
the 'feature' change.  Can you please explain what this patch does, and
why it's necessary or helpful to have it in FreeBSD?

Ex:
  This patch allows FreeBSD to work with older ATA flash cards that have
buggy firmware, since FreeBSD does not (currently) allow multiple I/O
windows to work.

Thanks!


Nate


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?199907210039.SAA23797>