diff mbox series

[RFC,1/4] x86: apl: Add the term "Interrupt Timer Subsystem" to ITSS files

Message ID 20200122150147.25709-2-wolfgang.wallner@br-automation.com
State Accepted
Commit 7d53b5a4ceaf86ecfe1bee097f27b49557e7cf7a
Delegated to: Bin Meng
Headers show
Series x86: Move ITSS from Apollo Lake to a more generic location | expand

Commit Message

Wolfgang Wallner Jan. 22, 2020, 3:01 p.m. UTC
ITSS stands for "Interrupt Timer Subsystem", so add that term to the
description of the relevant files.

Signed-off-by: Wolfgang Wallner <wolfgang.wallner@br-automation.com>
---
ITSS stands for "Interrupt Timer Subsystem", at least according to
coreboot [1].

[1] https://coreboot.org/status/kconfig-options.html

 arch/x86/cpu/apollolake/itss.c              | 2 +-
 arch/x86/include/asm/arch-apollolake/itss.h | 2 ++
 2 files changed, 3 insertions(+), 1 deletion(-)

Comments

Simon Glass Jan. 30, 2020, 2:16 a.m. UTC | #1
On Wed, 22 Jan 2020 at 08:01, Wolfgang Wallner
<wolfgang.wallner@br-automation.com> wrote:
>
> ITSS stands for "Interrupt Timer Subsystem", so add that term to the
> description of the relevant files.
>
> Signed-off-by: Wolfgang Wallner <wolfgang.wallner@br-automation.com>
> ---
> ITSS stands for "Interrupt Timer Subsystem", at least according to
> coreboot [1].
>
> [1] https://coreboot.org/status/kconfig-options.html
>
>  arch/x86/cpu/apollolake/itss.c              | 2 +-
>  arch/x86/include/asm/arch-apollolake/itss.h | 2 ++
>  2 files changed, 3 insertions(+), 1 deletion(-)

Reviewed-by: Simon Glass <sjg@chromium.org>
Bin Meng Feb. 3, 2020, 9:33 a.m. UTC | #2
On Wed, Jan 22, 2020 at 11:01 PM Wolfgang Wallner
<wolfgang.wallner@br-automation.com> wrote:
>
> ITSS stands for "Interrupt Timer Subsystem", so add that term to the
> description of the relevant files.
>
> Signed-off-by: Wolfgang Wallner <wolfgang.wallner@br-automation.com>
> ---
> ITSS stands for "Interrupt Timer Subsystem", at least according to
> coreboot [1].
>
> [1] https://coreboot.org/status/kconfig-options.html
>
>  arch/x86/cpu/apollolake/itss.c              | 2 +-
>  arch/x86/include/asm/arch-apollolake/itss.h | 2 ++
>  2 files changed, 3 insertions(+), 1 deletion(-)
>

Reviewed-by: Bin Meng <bmeng.cn@gmail.com>
Bin Meng Feb. 3, 2020, 9:38 a.m. UTC | #3
On Mon, Feb 3, 2020 at 5:33 PM Bin Meng <bmeng.cn@gmail.com> wrote:
>
> On Wed, Jan 22, 2020 at 11:01 PM Wolfgang Wallner
> <wolfgang.wallner@br-automation.com> wrote:
> >
> > ITSS stands for "Interrupt Timer Subsystem", so add that term to the
> > description of the relevant files.
> >
> > Signed-off-by: Wolfgang Wallner <wolfgang.wallner@br-automation.com>
> > ---
> > ITSS stands for "Interrupt Timer Subsystem", at least according to
> > coreboot [1].
> >
> > [1] https://coreboot.org/status/kconfig-options.html
> >
> >  arch/x86/cpu/apollolake/itss.c              | 2 +-
> >  arch/x86/include/asm/arch-apollolake/itss.h | 2 ++
> >  2 files changed, 3 insertions(+), 1 deletion(-)
> >
>
> Reviewed-by: Bin Meng <bmeng.cn@gmail.com>

applied to u-boot-x86, thanks!
diff mbox series

Patch

diff --git a/arch/x86/cpu/apollolake/itss.c b/arch/x86/cpu/apollolake/itss.c
index 8789f8e6bb..95c9ebddc1 100644
--- a/arch/x86/cpu/apollolake/itss.c
+++ b/arch/x86/cpu/apollolake/itss.c
@@ -1,6 +1,6 @@ 
 // SPDX-License-Identifier: GPL-2.0
 /*
- * Something to do with Interrupts, but I don't know what ITSS stands for
+ * Interrupt Timer Subsystem
  *
  * Copyright (C) 2017 Intel Corporation.
  * Copyright (C) 2017 Siemens AG
diff --git a/arch/x86/include/asm/arch-apollolake/itss.h b/arch/x86/include/asm/arch-apollolake/itss.h
index 1e29503974..c75d8fe8c2 100644
--- a/arch/x86/include/asm/arch-apollolake/itss.h
+++ b/arch/x86/include/asm/arch-apollolake/itss.h
@@ -1,5 +1,7 @@ 
 /* SPDX-License-Identifier: GPL-2.0 */
 /*
+ * Interrupt Timer Subsystem
+ *
  * Copyright (C) 2017 Intel Corporation.
  * Copyright 2019 Google LLC
  *