[U-Boot] [PATCH 09/19] x86: Remove ISR timer

Graeme Russ graeme.russ at gmail.com
Thu Apr 18 04:45:24 CEST 2013


Hi Simon,

On Thu, Apr 18, 2013 at 12:13 PM, Simon Glass <sjg at chromium.org> wrote:
>
> This is no longer used since we prefer the more accurate TSC timer, so
> remove the dead code.
>
> Signed-off-by: Simon Glass <sjg at chromium.org>
> ---
>  arch/x86/lib/Makefile |   1 -
>  arch/x86/lib/timer.c  | 118
--------------------------------------------------
>  2 files changed, 119 deletions(-)
>  delete mode 100644 arch/x86/lib/timer.c
>
> diff --git a/arch/x86/lib/Makefile b/arch/x86/lib/Makefile
> index 0bc8c2f..c614479 100644
> --- a/arch/x86/lib/Makefile
> +++ b/arch/x86/lib/Makefile
> @@ -36,7 +36,6 @@ COBJS-$(CONFIG_PCI) += pci_type1.o
>  COBJS-y        += relocate.o
>  COBJS-y += physmem.o
>  COBJS-y        += string.o
> -COBJS-$(CONFIG_SYS_X86_ISR_TIMER)      += timer.o
>  COBJS-$(CONFIG_SYS_X86_TSC_TIMER)      += tsc_timer.o
>  COBJS-$(CONFIG_VIDEO_VGA)      += video.o
>  COBJS-$(CONFIG_CMD_ZBOOT)      += zimage.o
> diff --git a/arch/x86/lib/timer.c b/arch/x86/lib/timer.c
> deleted file mode 100644
> index fb11bfe..0000000
> --- a/arch/x86/lib/timer.c
> +++ /dev/null
> @@ -1,118 +0,0 @@
> -/*
> - * (C) Copyright 2008,2009
> - * Graeme Russ, <graeme.russ at gmail.com>
> - *
> - * (C) Copyright 2002
> - * Daniel Engström, Omicron Ceti AB, <daniel at omicron.se>
> - *
> - * See file CREDITS for list of people who contributed to this
> - * project.
> - *
> - * This program is free software; you can redistribute it and/or
> - * modify it under the terms of the GNU General Public License as
> - * published by the Free Software Foundation; either version 2 of
> - * the License, or (at your option) any later version.
> - *
> - * This program is distributed in the hope that it will be useful,
> - * but WITHOUT ANY WARRANTY; without even the implied warranty of
> - * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
> - * GNU General Public License for more details.
> - *
> - * You should have received a copy of the GNU General Public License
> - * along with this program; if not, write to the Free Software
> - * Foundation, Inc., 59 Temple Place, Suite 330, Boston,
> - * MA 02111-1307 USA
> - */
> -
> -#include <common.h>
> -#include <malloc.h>
> -#include <asm/io.h>
> -#include <asm/i8254.h>
> -#include <asm/ibmpc.h>
> -
> -DECLARE_GLOBAL_DATA_PTR;
> -
> -struct timer_isr_function {
> -       struct timer_isr_function *next;
> -       timer_fnc_t *isr_func;
> -};
> -
> -static struct timer_isr_function *first_timer_isr;
> -static unsigned long system_ticks;
> -
> -/*
> - * register_timer_isr() allows multiple architecture and board specific
> - * functions to be called every millisecond. Keep the execution time of
> - * each function as low as possible
> - */
> -int register_timer_isr(timer_fnc_t *isr_func)
> -{
> -       struct timer_isr_function *new_func;
> -       struct timer_isr_function *temp;
> -       int flag;
> -
> -       new_func = malloc(sizeof(struct timer_isr_function));
> -
> -       if (new_func == NULL)
> -               return 1;
> -
> -       new_func->isr_func = isr_func;
> -       new_func->next = NULL;
> -
> -       /*
> -        *  Don't allow timer interrupts while the
> -        *  linked list is being modified
> -        */
> -       flag = disable_interrupts();
> -
> -       if (first_timer_isr == NULL) {
> -               first_timer_isr = new_func;
> -       } else {
> -               temp = first_timer_isr;
> -               while (temp->next != NULL)
> -                       temp = temp->next;
> -               temp->next = new_func;
> -       }
> -
> -       if (flag)
> -               enable_interrupts();
> -
> -       return 0;
> -}
> -
> -/*
> - * timer_isr() MUST be the registered interrupt handler for
> - */
> -void timer_isr(void *unused)
> -{
> -       struct timer_isr_function *temp = first_timer_isr;
> -
> -       system_ticks++;
> -
> -       /* Execute each registered function */
> -       while (temp != NULL) {
> -               temp->isr_func();
> -               temp = temp->next;
> -       }
> -}
> -
> -ulong get_timer(ulong base)
> -{
> -       return system_ticks - base;
> -}
> -
> -void timer_set_tsc_base(uint64_t new_base)
> -{
> -       gd->arch.tsc_base = new_base;
> -}
> -
> -uint64_t timer_get_tsc(void)
> -{
> -       uint64_t time_now;
> -
> -       time_now = rdtsc();
> -       if (!gd->arch.tsc_base)
> -               gd->arch.tsc_base = time_now;
> -
> -       return time_now - gd->arch.tsc_base;
> -}
> --
> 1.8.2.1
>

Acked-by: Graeme Russ <graeme.russ at gmail.com>


More information about the U-Boot mailing list