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

2020-02-03 Thread Bin Meng
On Mon, Feb 3, 2020 at 5:33 PM Bin Meng  wrote:
>
> On Wed, Jan 22, 2020 at 11:01 PM Wolfgang Wallner
>  wrote:
> >
> > ITSS stands for "Interrupt Timer Subsystem", so add that term to the
> > description of the relevant files.
> >
> > Signed-off-by: Wolfgang Wallner 
> > ---
> > 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 

applied to u-boot-x86, thanks!


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

2020-02-03 Thread Bin Meng
On Wed, Jan 22, 2020 at 11:01 PM Wolfgang Wallner
 wrote:
>
> ITSS stands for "Interrupt Timer Subsystem", so add that term to the
> description of the relevant files.
>
> Signed-off-by: Wolfgang Wallner 
> ---
> 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 


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

2020-01-29 Thread Simon Glass
On Wed, 22 Jan 2020 at 08:01, Wolfgang Wallner
 wrote:
>
> ITSS stands for "Interrupt Timer Subsystem", so add that term to the
> description of the relevant files.
>
> Signed-off-by: Wolfgang Wallner 
> ---
> 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 


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

2020-01-22 Thread Wolfgang Wallner
ITSS stands for "Interrupt Timer Subsystem", so add that term to the
description of the relevant files.

Signed-off-by: Wolfgang Wallner 
---
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(-)

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
  *
-- 
2.25.0