On Sun, 25 Jun 2006 16:01:40 -0600 Jim Cromie wrote:
> $ diffstat add-scx200-hrt-clocksource.diff
> arch/i386/Kconfig | 24 ++++++---
> drivers/clocksource/Makefile | 5 +
> drivers/clocksource/scx200_hrt.c | 101 +++++++++++++++++++++++++++++++++++++++
> 3 files changed, 121 insertions(+), 9 deletions(-)
>
> ---
>
> diff -ruNp -X dontdiff -X exclude-diffs linux-2.6.17-mm2/arch/i386/Kconfig linux-2.6.17-mm2-hrt-sk/arch/i386/Kconfig
> --- linux-2.6.17-mm2/arch/i386/Kconfig 2006-06-25 01:09:56.000000000 -0600
> +++ linux-2.6.17-mm2-hrt-sk/arch/i386/Kconfig 2006-06-25 07:48:18.000000000 -0600
> @@ -1073,13 +1073,23 @@ config SCx200
> tristate "NatSemi SCx200 support"
> depends on !X86_VOYAGER
> help
> - This provides basic support for the National Semiconductor SCx200
> - processor. Right now this is just a driver for the GPIO pins.
> -
> - If you don't know what to do here, say N.
> -
> - This support is also available as a module. If compiled as a
> - module, it will be called scx200.
> + This provides basic support for National Semiconductor's
> + (now AMD's) Geode processors. The driver probes for the
> + PCI-IDs of several on-chip devices, so its a good dependency
it's
> + for other scx200_* drivers.
> +
> + If compiled as a module, the driver is named scx200.
> +
> +config SCx200HR_TIMER
> + tristate "NatSemi SCx200 27MHz High-Resolution Timer Support"
> + depends on SCx200 && GENERIC_TIME
> + default y
> + help
> + This driver provides a clocksource built upon the on-chip
> + 27MHz high-resolution timer. Its also a workaround for
It's
> + NSC Geode SC-1100's buggy TSC, which loses time when the
> + processor goes idle (as is done by the scheduler). The
> + other workaround is idle=poll boot option.
>
> config K8_NB
> def_bool y
> diff -ruNp -X dontdiff -X exclude-diffs linux-2.6.17-mm2/drivers/clocksource/scx200_hrt.c linux-2.6.17-mm2-hrt-sk/drivers/clocksource/scx200_hrt.c
> --- linux-2.6.17-mm2/drivers/clocksource/scx200_hrt.c 1969-12-31 17:00:00.000000000 -0700
> +++ linux-2.6.17-mm2-hrt-sk/drivers/clocksource/scx200_hrt.c 2006-06-25 07:48:18.000000000 -0600
> @@ -0,0 +1,101 @@
> +/*
> + * Copyright (C) 2006 Jim Cromie
> + *
> +
> +static int mhz27;
> +module_param(mhz27, int, 0); /* load time only */
> +MODULE_PARM_DESC(mhz27, "count at 27.0 MHz (default is 1.0 MHz)\n");
Don't need the newline. Please drop it.
> +static int ppm;
> +module_param(ppm, int, 0); /* load time only */
> +MODULE_PARM_DESC(ppm, "+-adjust to actual XO freq (ppm)\n");
Ditto.
> +static int __init init_hrt_clocksource(void)
> +{
> + /* Make sure scx200 has initd the configuration block */
I expect some people won't know what "initd" is.
Please write it for other people to read.
> + if (!scx200_cb_present())
> + return -ENODEV;
---
~Randy
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
[Index of Archives]
[Kernel Newbies]
[Netfilter]
[Bugtraq]
[Photo]
[Stuff]
[Gimp]
[Yosemite News]
[MIPS Linux]
[ARM Linux]
[Linux Security]
[Linux RAID]
[Video 4 Linux]
[Linux for the blind]
[Linux Resources]
