diff mbox series

[2/2] acpi: sbbr: add mcfg and pptt to --sbbr option

Message ID 20210604194743.580256-2-alex.hung@canonical.com
State Accepted
Headers show
Series [1/2] acpi: remove SBBR mandatory tests in various tables | expand

Commit Message

Alex Hung June 4, 2021, 7:47 p.m. UTC
Both tables are SBBR mandatory

Signed-off-by: Alex Hung <alex.hung@canonical.com>
---
 src/acpi/mcfg/mcfg.c | 3 ++-
 src/acpi/pptt/pptt.c | 2 +-
 2 files changed, 3 insertions(+), 2 deletions(-)

Comments

ivanhu June 7, 2021, 8:50 a.m. UTC | #1
On 6/5/21 3:47 AM, Alex Hung wrote:
> Both tables are SBBR mandatory
> 
> Signed-off-by: Alex Hung <alex.hung@canonical.com>
> ---
>  src/acpi/mcfg/mcfg.c | 3 ++-
>  src/acpi/pptt/pptt.c | 2 +-
>  2 files changed, 3 insertions(+), 2 deletions(-)
> 
> diff --git a/src/acpi/mcfg/mcfg.c b/src/acpi/mcfg/mcfg.c
> index 7cef0f43..1c8b4b13 100644
> --- a/src/acpi/mcfg/mcfg.c
> +++ b/src/acpi/mcfg/mcfg.c
> @@ -252,6 +252,7 @@ static fwts_framework_ops mcfg_ops = {
>  	.minor_tests = mcfg_tests
>  };
>  
> -FWTS_REGISTER("mcfg", &mcfg_ops, FWTS_TEST_ANYTIME, FWTS_FLAG_BATCH | FWTS_FLAG_ROOT_PRIV | FWTS_FLAG_ACPI)
> +FWTS_REGISTER("mcfg", &mcfg_ops, FWTS_TEST_ANYTIME,
> +	      FWTS_FLAG_BATCH | FWTS_FLAG_ROOT_PRIV | FWTS_FLAG_ACPI  | FWTS_FLAG_SBBR)
>  
>  #endif
> diff --git a/src/acpi/pptt/pptt.c b/src/acpi/pptt/pptt.c
> index 56394660..1d25798a 100644
> --- a/src/acpi/pptt/pptt.c
> +++ b/src/acpi/pptt/pptt.c
> @@ -189,6 +189,6 @@ static fwts_framework_ops pptt_ops = {
>  	.minor_tests = pptt_tests
>  };
>  
> -FWTS_REGISTER("pptt", &pptt_ops, FWTS_TEST_ANYTIME, FWTS_FLAG_BATCH | FWTS_FLAG_ACPI)
> +FWTS_REGISTER("pptt", &pptt_ops, FWTS_TEST_ANYTIME, FWTS_FLAG_BATCH | FWTS_FLAG_ACPI | FWTS_FLAG_SBBR)
>  
>  #endif
> 

Acked-by: Ivan Hu <ivan.hu@canonical.com>
Colin Ian King June 14, 2021, 6:01 p.m. UTC | #2
On 04/06/2021 20:47, Alex Hung wrote:
> Both tables are SBBR mandatory
> 
> Signed-off-by: Alex Hung <alex.hung@canonical.com>
> ---
>  src/acpi/mcfg/mcfg.c | 3 ++-
>  src/acpi/pptt/pptt.c | 2 +-
>  2 files changed, 3 insertions(+), 2 deletions(-)
> 
> diff --git a/src/acpi/mcfg/mcfg.c b/src/acpi/mcfg/mcfg.c
> index 7cef0f43..1c8b4b13 100644
> --- a/src/acpi/mcfg/mcfg.c
> +++ b/src/acpi/mcfg/mcfg.c
> @@ -252,6 +252,7 @@ static fwts_framework_ops mcfg_ops = {
>  	.minor_tests = mcfg_tests
>  };
>  
> -FWTS_REGISTER("mcfg", &mcfg_ops, FWTS_TEST_ANYTIME, FWTS_FLAG_BATCH | FWTS_FLAG_ROOT_PRIV | FWTS_FLAG_ACPI)
> +FWTS_REGISTER("mcfg", &mcfg_ops, FWTS_TEST_ANYTIME,
> +	      FWTS_FLAG_BATCH | FWTS_FLAG_ROOT_PRIV | FWTS_FLAG_ACPI  | FWTS_FLAG_SBBR)
>  
>  #endif
> diff --git a/src/acpi/pptt/pptt.c b/src/acpi/pptt/pptt.c
> index 56394660..1d25798a 100644
> --- a/src/acpi/pptt/pptt.c
> +++ b/src/acpi/pptt/pptt.c
> @@ -189,6 +189,6 @@ static fwts_framework_ops pptt_ops = {
>  	.minor_tests = pptt_tests
>  };
>  
> -FWTS_REGISTER("pptt", &pptt_ops, FWTS_TEST_ANYTIME, FWTS_FLAG_BATCH | FWTS_FLAG_ACPI)
> +FWTS_REGISTER("pptt", &pptt_ops, FWTS_TEST_ANYTIME, FWTS_FLAG_BATCH | FWTS_FLAG_ACPI | FWTS_FLAG_SBBR)
>  
>  #endif
> 
Acked-by: Colin Ian King <colin.king@canonical.com>
diff mbox series

Patch

diff --git a/src/acpi/mcfg/mcfg.c b/src/acpi/mcfg/mcfg.c
index 7cef0f43..1c8b4b13 100644
--- a/src/acpi/mcfg/mcfg.c
+++ b/src/acpi/mcfg/mcfg.c
@@ -252,6 +252,7 @@  static fwts_framework_ops mcfg_ops = {
 	.minor_tests = mcfg_tests
 };
 
-FWTS_REGISTER("mcfg", &mcfg_ops, FWTS_TEST_ANYTIME, FWTS_FLAG_BATCH | FWTS_FLAG_ROOT_PRIV | FWTS_FLAG_ACPI)
+FWTS_REGISTER("mcfg", &mcfg_ops, FWTS_TEST_ANYTIME,
+	      FWTS_FLAG_BATCH | FWTS_FLAG_ROOT_PRIV | FWTS_FLAG_ACPI  | FWTS_FLAG_SBBR)
 
 #endif
diff --git a/src/acpi/pptt/pptt.c b/src/acpi/pptt/pptt.c
index 56394660..1d25798a 100644
--- a/src/acpi/pptt/pptt.c
+++ b/src/acpi/pptt/pptt.c
@@ -189,6 +189,6 @@  static fwts_framework_ops pptt_ops = {
 	.minor_tests = pptt_tests
 };
 
-FWTS_REGISTER("pptt", &pptt_ops, FWTS_TEST_ANYTIME, FWTS_FLAG_BATCH | FWTS_FLAG_ACPI)
+FWTS_REGISTER("pptt", &pptt_ops, FWTS_TEST_ANYTIME, FWTS_FLAG_BATCH | FWTS_FLAG_ACPI | FWTS_FLAG_SBBR)
 
 #endif