diff mbox

acpi: XENV: fix a few spelling mistakes and fix up copyright

Message ID 1436345803-8265-1-git-send-email-colin.king@canonical.com
State Accepted
Headers show

Commit Message

Colin Ian King July 8, 2015, 8:56 a.m. UTC
From: Colin Ian King <colin.king@canonical.com>

This is a new test and not derived from the Linux-ready Firmware
Developer Kit, so remove the "Copyright (C) 2006, Intel Corporation"
from the copyright.

Also, fix up some typos / spelling mistakes

Signed-off-by: Colin Ian King <colin.king@canonical.com>
---
 src/acpi/xenv/xenv.c | 9 +++------
 1 file changed, 3 insertions(+), 6 deletions(-)

Comments

Ivan Hu July 9, 2015, 1:03 a.m. UTC | #1
On 2015年07月08日 16:56, Colin King wrote:
> From: Colin Ian King <colin.king@canonical.com>
>
> This is a new test and not derived from the Linux-ready Firmware
> Developer Kit, so remove the "Copyright (C) 2006, Intel Corporation"
> from the copyright.
>
> Also, fix up some typos / spelling mistakes
>
> Signed-off-by: Colin Ian King <colin.king@canonical.com>
> ---
>   src/acpi/xenv/xenv.c | 9 +++------
>   1 file changed, 3 insertions(+), 6 deletions(-)
>
> diff --git a/src/acpi/xenv/xenv.c b/src/acpi/xenv/xenv.c
> index c33942b..db8379a 100644
> --- a/src/acpi/xenv/xenv.c
> +++ b/src/acpi/xenv/xenv.c
> @@ -1,9 +1,6 @@
>   /*
> - * Copyright (C) 2006, Intel Corporation
>    * Copyright (C) 2010-2015 Canonical
>    *
> - * This code was originally part of the Linux-ready Firmware Developer Kit
> - *
>    * 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
> @@ -67,8 +64,8 @@ static int xenv_test1(fwts_framework *fw)
>   	}
>   
>   	fwts_log_info_verbatum(fw, "XENV Table:");
> -	fwts_log_info_verbatum(fw, "  GNT Start Address                0x%16.16" PRIx64, xenv->gnt_start);
> -	fwts_log_info_verbatum(fw, "  GNT Sizne                        0x%16.16" PRIx64, xenv->gnt_size);
> +	fwts_log_info_verbatum(fw, "  GNT Start Address:               0x%16.16" PRIx64, xenv->gnt_start);
> +	fwts_log_info_verbatum(fw, "  GNT Size:                        0x%16.16" PRIx64, xenv->gnt_size);
>   	fwts_log_info_verbatum(fw, "  Evtchn Intr:                     0x%8.8"   PRIx32, xenv->evtchn_intr);
>   	fwts_log_info_verbatum(fw, "  Evtchn Intr Flags:               0x%2.2"   PRIx8,  xenv->evtchn_intr_flags);
>   
> @@ -88,7 +85,7 @@ static int xenv_test1(fwts_framework *fw)
>   }
>   
>   static fwts_framework_minor_test xenv_tests[] = {
> -	{ xenv_test1, "Validate XEVN table." },
> +	{ xenv_test1, "Validate XENV table." },
>   	{ NULL, NULL }
>   };
>   
Acked-by: Ivan Hu<ivan.hu@canonical.com>
Alex Hung July 9, 2015, 12:05 p.m. UTC | #2
On 07/08/2015 04:56 PM, Colin King wrote:
> From: Colin Ian King <colin.king@canonical.com>
> 
> This is a new test and not derived from the Linux-ready Firmware
> Developer Kit, so remove the "Copyright (C) 2006, Intel Corporation"
> from the copyright.
> 
> Also, fix up some typos / spelling mistakes
> 
> Signed-off-by: Colin Ian King <colin.king@canonical.com>
> ---
>  src/acpi/xenv/xenv.c | 9 +++------
>  1 file changed, 3 insertions(+), 6 deletions(-)
> 
> diff --git a/src/acpi/xenv/xenv.c b/src/acpi/xenv/xenv.c
> index c33942b..db8379a 100644
> --- a/src/acpi/xenv/xenv.c
> +++ b/src/acpi/xenv/xenv.c
> @@ -1,9 +1,6 @@
>  /*
> - * Copyright (C) 2006, Intel Corporation
>   * Copyright (C) 2010-2015 Canonical
>   *
> - * This code was originally part of the Linux-ready Firmware Developer Kit
> - *
>   * 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
> @@ -67,8 +64,8 @@ static int xenv_test1(fwts_framework *fw)
>  	}
>  
>  	fwts_log_info_verbatum(fw, "XENV Table:");
> -	fwts_log_info_verbatum(fw, "  GNT Start Address                0x%16.16" PRIx64, xenv->gnt_start);
> -	fwts_log_info_verbatum(fw, "  GNT Sizne                        0x%16.16" PRIx64, xenv->gnt_size);
> +	fwts_log_info_verbatum(fw, "  GNT Start Address:               0x%16.16" PRIx64, xenv->gnt_start);
> +	fwts_log_info_verbatum(fw, "  GNT Size:                        0x%16.16" PRIx64, xenv->gnt_size);
>  	fwts_log_info_verbatum(fw, "  Evtchn Intr:                     0x%8.8"   PRIx32, xenv->evtchn_intr);
>  	fwts_log_info_verbatum(fw, "  Evtchn Intr Flags:               0x%2.2"   PRIx8,  xenv->evtchn_intr_flags);
>  
> @@ -88,7 +85,7 @@ static int xenv_test1(fwts_framework *fw)
>  }
>  
>  static fwts_framework_minor_test xenv_tests[] = {
> -	{ xenv_test1, "Validate XEVN table." },
> +	{ xenv_test1, "Validate XENV table." },
>  	{ NULL, NULL }
>  };
>  
> 

Acked-by: Alex Hung <alex.hung@canonical.com>
diff mbox

Patch

diff --git a/src/acpi/xenv/xenv.c b/src/acpi/xenv/xenv.c
index c33942b..db8379a 100644
--- a/src/acpi/xenv/xenv.c
+++ b/src/acpi/xenv/xenv.c
@@ -1,9 +1,6 @@ 
 /*
- * Copyright (C) 2006, Intel Corporation
  * Copyright (C) 2010-2015 Canonical
  *
- * This code was originally part of the Linux-ready Firmware Developer Kit
- *
  * 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
@@ -67,8 +64,8 @@  static int xenv_test1(fwts_framework *fw)
 	}
 
 	fwts_log_info_verbatum(fw, "XENV Table:");
-	fwts_log_info_verbatum(fw, "  GNT Start Address                0x%16.16" PRIx64, xenv->gnt_start);
-	fwts_log_info_verbatum(fw, "  GNT Sizne                        0x%16.16" PRIx64, xenv->gnt_size);
+	fwts_log_info_verbatum(fw, "  GNT Start Address:               0x%16.16" PRIx64, xenv->gnt_start);
+	fwts_log_info_verbatum(fw, "  GNT Size:                        0x%16.16" PRIx64, xenv->gnt_size);
 	fwts_log_info_verbatum(fw, "  Evtchn Intr:                     0x%8.8"   PRIx32, xenv->evtchn_intr);
 	fwts_log_info_verbatum(fw, "  Evtchn Intr Flags:               0x%2.2"   PRIx8,  xenv->evtchn_intr_flags);
 
@@ -88,7 +85,7 @@  static int xenv_test1(fwts_framework *fw)
 }
 
 static fwts_framework_minor_test xenv_tests[] = {
-	{ xenv_test1, "Validate XEVN table." },
+	{ xenv_test1, "Validate XENV table." },
 	{ NULL, NULL }
 };