Date: Sun, 1 Mar 1998 13:47:59 -0800 (PST) From: asami@cs.berkeley.edu (Satoshi Asami) To: iang@digs.iafrica.com Cc: current@FreeBSD.ORG Subject: Re: Duplicate cases in /usr/src/sys/pci/pcisupport.c Message-ID: <199803012147.NAA09826@baloon.mimi.com> In-Reply-To: <E0y98KQ-0001ZC-00@brane.digs.iafrica.com> (message from Ian Freislich on Sun, 01 Mar 1998 14:58:49 %2B0200)
next in thread | previous in thread | raw e-mail | index | archive | help
* Looking in the file:
* @207:
* case 0x71008086:
* return ("Intel 82439TX System Controller (MTXC)");
* @217:
* case 0x71008086:
* return ("Intel 82439TX PCI cache memory controller");
Yikes. The second one in my commit yesterday. How did this happen?
(Dig into cvs logs, mail with Jordan, etc....)
Eek. So this was already in -current for quite some time:
===
1.54 Wed Sep 24 7:37:56 1997 by phk
Diffs to 1.53
Remove the 82371 IDE devices.
Add Intel 82439TX System Controller (MTXC)
fix a whitespace problem.
===
But he didn't bother to merge it into -stable. I am running -stable
here (where I made the diffs) and Jordan is running -stable on catfish
too, where we initially noticed that the chipset is not recognized and
also tested the diffs.
Blast. Sorry, will fix it, it is my fault not checking with -current
first.
But I really wish people will merge trivial things like this, with
"real" changes and non-real changes between -stable and -current
intermixed, it's so hard to see what's going on! :<
Satoshi
To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe freebsd-current" in the body of the message
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?199803012147.NAA09826>