PERFORCE change 111420 for review

Sam Leffler sam at FreeBSD.org
Sun Dec 10 15:59:45 PST 2006


http://perforce.freebsd.org/chv.cgi?CH=111420

Change 111420 by sam at sam_ebb on 2006/12/10 23:58:11

	cleanup (and correct) comments about why the default
	pllconfig is ext66

Affected files ...

.. //depot/projects/crypto/sys/dev/hifn/hifn7751.c#6 edit

Differences ...

==== //depot/projects/crypto/sys/dev/hifn/hifn7751.c#6 (text+ko) ====

@@ -296,19 +296,11 @@
  * bus clock and in a 33MHz slot then it will be operating at half
  * speed until the correct information is provided.
  *
- * According Mike Ham of HiFn, almost every board in existence has
- * an external crystal populated at 66Mhz. Using PCI can be a problem
- * on modern motherboards, because PCI33 can have clocks from 0 to 33Mhz,
- * and some have non-PCI-compliant spread-spectrum clocks, which can confuse
- * the pll.
- *
- * On Linux, there is no way to set pllconfig except on the boot
- * command line for static kernels. Even though the string can get adjusted
- * afterwards, the device will not be reconfigured.
- *
- * On FreeBSD, you can edit the device hints by editing /boot/device.hints,
- * and this too, is loaded only at boot time. See device.hints(5).
- *
+ * We use a default setting of "ext66" because according to Mike Ham
+ * of HiFn, almost every board in existence has an external crystal
+ * populated at 66Mhz. Using PCI can be a problem on modern motherboards,
+ * because PCI33 can have clocks from 0 to 33Mhz, and some have
+ * non-PCI-compliant spread-spectrum clocks, which can confuse the pll.
  */
 static void
 hifn_getpllconfig(device_t dev, u_int *pll)


More information about the p4-projects mailing list