Message ID | 1412669964-2517-1-git-send-email-alex.hung@canonical.com |
---|---|
State | Accepted |
Headers | show |
On 07/10/14 09:19, Alex Hung wrote: > Signed-off-by: Alex Hung <alex.hung@canonical.com> > --- > src/acpi/method/method.c | 68 +++++++++++++++++++++++++++++++++++++++++++++--- > 1 file changed, 65 insertions(+), 3 deletions(-) > > diff --git a/src/acpi/method/method.c b/src/acpi/method/method.c > index cb873cb..703bd07 100644 > --- a/src/acpi/method/method.c > +++ b/src/acpi/method/method.c > @@ -38,7 +38,7 @@ > * _ALC 9.2.5 Y > * _ALI 9.2.2 Y > * _ALP 9.2.6 Y > - * _ALR 9.2.5 N > + * _ALR 9.2.5 Y > * _ALT 9.2.3 Y > * _ALx 11.4.2 N > * _ART 11.4.3 Y > @@ -3555,7 +3555,69 @@ method_test_integer(_ALC, METHOD_OPTIONAL) > method_test_integer(_ALI, METHOD_OPTIONAL) > method_test_integer(_ALT, METHOD_OPTIONAL) > > -/* TODO _ALR */ > +static void method_test_ALR_return( > + fwts_framework *fw, > + char *name, > + ACPI_BUFFER *buf, > + ACPI_OBJECT *obj, > + void *private) > +{ > + uint32_t i; > + bool failed = false; > + > + FWTS_UNUSED(private); > + > + if (method_check_type(fw, name, buf, ACPI_TYPE_PACKAGE) != FWTS_OK) > + return; > + > + /* Could be one or more sub-packages */ > + for (i = 0; i < obj->Package.Count; i++) { > + ACPI_OBJECT *pkg; > + uint32_t adjustment = 0, illuminance = 0; > + > + pkg = &obj->Package.Elements[i]; > + if (pkg->Package.Count != 2) { > + fwts_failed(fw, LOG_LEVEL_MEDIUM, > + "Method_ALRBadSubPackageElementCount", > + "%s sub-package %" PRIu32 " was expected to " > + "have 2 elements, got %" PRIu32 " elements instead.", > + name, i, pkg->Package.Count); > + failed = true; > + } else { > + /* elements should be listed in monotonically increasing order */ > + if (pkg->Package.Elements[0].Type != ACPI_TYPE_INTEGER || > + adjustment > pkg->Package.Elements[0].Integer.Value) { > + fwts_failed(fw, LOG_LEVEL_MEDIUM, > + "Method_ALRBadSubPackageReturnType", > + "%s sub-package %" PRIu32 > + " element 0 is an invalid integer.", > + name, i); > + failed = true; > + } > + > + if (pkg->Package.Elements[1].Type != ACPI_TYPE_INTEGER || > + illuminance > pkg->Package.Elements[1].Integer.Value) { > + fwts_failed(fw, LOG_LEVEL_MEDIUM, > + "Method_ALRBadSubPackageReturnType", > + "%s sub-package %" PRIu32 > + " element 1 is an invalid integer.", > + name, i); > + failed = true; > + } > + adjustment = pkg->Package.Elements[0].Integer.Value; > + illuminance = pkg->Package.Elements[1].Integer.Value; > + } > + } > + > + if (!failed) > + method_passed_sane(fw, name, "package"); > +} > + > +static int method_test_ALR(fwts_framework *fw) > +{ > + return method_evaluate_method(fw, METHOD_OPTIONAL, > + "_ALR", NULL, 0, method_test_ALR_return, NULL); > +} > > static int method_test_ALP(fwts_framework *fw) > { > @@ -5825,7 +5887,7 @@ static fwts_framework_minor_test method_tests[] = { > { method_test_ALI, "Test _ALI (Ambient Light Illuminance)." }, > { method_test_ALT, "Test _ALT (Ambient Light Temperature)." }, > { method_test_ALP, "Test _ALP (Ambient Light Polling). "}, > - /* { method_test_ALR, "Test _ALR (Ambient Light Response). "}, */ > + { method_test_ALR, "Test _ALR (Ambient Light Response). "}, > > /* Section 9.3 Battery Device */ > > Thanks Alex! Acked-by: Colin Ian King <colin.king@canonical.com>
On Tue, Oct 7, 2014 at 4:21 PM, Colin Ian King <colin.king@canonical.com> wrote: > On 07/10/14 09:19, Alex Hung wrote: >> Signed-off-by: Alex Hung <alex.hung@canonical.com> >> --- >> src/acpi/method/method.c | 68 +++++++++++++++++++++++++++++++++++++++++++++--- >> 1 file changed, 65 insertions(+), 3 deletions(-) >> >> diff --git a/src/acpi/method/method.c b/src/acpi/method/method.c >> index cb873cb..703bd07 100644 >> --- a/src/acpi/method/method.c >> +++ b/src/acpi/method/method.c >> @@ -38,7 +38,7 @@ >> * _ALC 9.2.5 Y >> * _ALI 9.2.2 Y >> * _ALP 9.2.6 Y >> - * _ALR 9.2.5 N >> + * _ALR 9.2.5 Y >> * _ALT 9.2.3 Y >> * _ALx 11.4.2 N >> * _ART 11.4.3 Y >> @@ -3555,7 +3555,69 @@ method_test_integer(_ALC, METHOD_OPTIONAL) >> method_test_integer(_ALI, METHOD_OPTIONAL) >> method_test_integer(_ALT, METHOD_OPTIONAL) >> >> -/* TODO _ALR */ >> +static void method_test_ALR_return( >> + fwts_framework *fw, >> + char *name, >> + ACPI_BUFFER *buf, >> + ACPI_OBJECT *obj, >> + void *private) >> +{ >> + uint32_t i; >> + bool failed = false; >> + >> + FWTS_UNUSED(private); >> + >> + if (method_check_type(fw, name, buf, ACPI_TYPE_PACKAGE) != FWTS_OK) >> + return; >> + >> + /* Could be one or more sub-packages */ >> + for (i = 0; i < obj->Package.Count; i++) { >> + ACPI_OBJECT *pkg; >> + uint32_t adjustment = 0, illuminance = 0; >> + >> + pkg = &obj->Package.Elements[i]; >> + if (pkg->Package.Count != 2) { >> + fwts_failed(fw, LOG_LEVEL_MEDIUM, >> + "Method_ALRBadSubPackageElementCount", >> + "%s sub-package %" PRIu32 " was expected to " >> + "have 2 elements, got %" PRIu32 " elements instead.", >> + name, i, pkg->Package.Count); >> + failed = true; >> + } else { >> + /* elements should be listed in monotonically increasing order */ >> + if (pkg->Package.Elements[0].Type != ACPI_TYPE_INTEGER || >> + adjustment > pkg->Package.Elements[0].Integer.Value) { >> + fwts_failed(fw, LOG_LEVEL_MEDIUM, >> + "Method_ALRBadSubPackageReturnType", >> + "%s sub-package %" PRIu32 >> + " element 0 is an invalid integer.", >> + name, i); >> + failed = true; >> + } >> + >> + if (pkg->Package.Elements[1].Type != ACPI_TYPE_INTEGER || >> + illuminance > pkg->Package.Elements[1].Integer.Value) { >> + fwts_failed(fw, LOG_LEVEL_MEDIUM, >> + "Method_ALRBadSubPackageReturnType", >> + "%s sub-package %" PRIu32 >> + " element 1 is an invalid integer.", >> + name, i); >> + failed = true; >> + } >> + adjustment = pkg->Package.Elements[0].Integer.Value; >> + illuminance = pkg->Package.Elements[1].Integer.Value; >> + } >> + } >> + >> + if (!failed) >> + method_passed_sane(fw, name, "package"); >> +} >> + >> +static int method_test_ALR(fwts_framework *fw) >> +{ >> + return method_evaluate_method(fw, METHOD_OPTIONAL, >> + "_ALR", NULL, 0, method_test_ALR_return, NULL); >> +} >> >> static int method_test_ALP(fwts_framework *fw) >> { >> @@ -5825,7 +5887,7 @@ static fwts_framework_minor_test method_tests[] = { >> { method_test_ALI, "Test _ALI (Ambient Light Illuminance)." }, >> { method_test_ALT, "Test _ALT (Ambient Light Temperature)." }, >> { method_test_ALP, "Test _ALP (Ambient Light Polling). "}, >> - /* { method_test_ALR, "Test _ALR (Ambient Light Response). "}, */ >> + { method_test_ALR, "Test _ALR (Ambient Light Response). "}, >> >> /* Section 9.3 Battery Device */ >> >> > Thanks Alex! > > Acked-by: Colin Ian King <colin.king@canonical.com> > Acked-by: Keng-Yu Lin <kengyu@canonical.com>
diff --git a/src/acpi/method/method.c b/src/acpi/method/method.c index cb873cb..703bd07 100644 --- a/src/acpi/method/method.c +++ b/src/acpi/method/method.c @@ -38,7 +38,7 @@ * _ALC 9.2.5 Y * _ALI 9.2.2 Y * _ALP 9.2.6 Y - * _ALR 9.2.5 N + * _ALR 9.2.5 Y * _ALT 9.2.3 Y * _ALx 11.4.2 N * _ART 11.4.3 Y @@ -3555,7 +3555,69 @@ method_test_integer(_ALC, METHOD_OPTIONAL) method_test_integer(_ALI, METHOD_OPTIONAL) method_test_integer(_ALT, METHOD_OPTIONAL) -/* TODO _ALR */ +static void method_test_ALR_return( + fwts_framework *fw, + char *name, + ACPI_BUFFER *buf, + ACPI_OBJECT *obj, + void *private) +{ + uint32_t i; + bool failed = false; + + FWTS_UNUSED(private); + + if (method_check_type(fw, name, buf, ACPI_TYPE_PACKAGE) != FWTS_OK) + return; + + /* Could be one or more sub-packages */ + for (i = 0; i < obj->Package.Count; i++) { + ACPI_OBJECT *pkg; + uint32_t adjustment = 0, illuminance = 0; + + pkg = &obj->Package.Elements[i]; + if (pkg->Package.Count != 2) { + fwts_failed(fw, LOG_LEVEL_MEDIUM, + "Method_ALRBadSubPackageElementCount", + "%s sub-package %" PRIu32 " was expected to " + "have 2 elements, got %" PRIu32 " elements instead.", + name, i, pkg->Package.Count); + failed = true; + } else { + /* elements should be listed in monotonically increasing order */ + if (pkg->Package.Elements[0].Type != ACPI_TYPE_INTEGER || + adjustment > pkg->Package.Elements[0].Integer.Value) { + fwts_failed(fw, LOG_LEVEL_MEDIUM, + "Method_ALRBadSubPackageReturnType", + "%s sub-package %" PRIu32 + " element 0 is an invalid integer.", + name, i); + failed = true; + } + + if (pkg->Package.Elements[1].Type != ACPI_TYPE_INTEGER || + illuminance > pkg->Package.Elements[1].Integer.Value) { + fwts_failed(fw, LOG_LEVEL_MEDIUM, + "Method_ALRBadSubPackageReturnType", + "%s sub-package %" PRIu32 + " element 1 is an invalid integer.", + name, i); + failed = true; + } + adjustment = pkg->Package.Elements[0].Integer.Value; + illuminance = pkg->Package.Elements[1].Integer.Value; + } + } + + if (!failed) + method_passed_sane(fw, name, "package"); +} + +static int method_test_ALR(fwts_framework *fw) +{ + return method_evaluate_method(fw, METHOD_OPTIONAL, + "_ALR", NULL, 0, method_test_ALR_return, NULL); +} static int method_test_ALP(fwts_framework *fw) { @@ -5825,7 +5887,7 @@ static fwts_framework_minor_test method_tests[] = { { method_test_ALI, "Test _ALI (Ambient Light Illuminance)." }, { method_test_ALT, "Test _ALT (Ambient Light Temperature)." }, { method_test_ALP, "Test _ALP (Ambient Light Polling). "}, - /* { method_test_ALR, "Test _ALR (Ambient Light Response). "}, */ + { method_test_ALR, "Test _ALR (Ambient Light Response). "}, /* Section 9.3 Battery Device */
Signed-off-by: Alex Hung <alex.hung@canonical.com> --- src/acpi/method/method.c | 68 +++++++++++++++++++++++++++++++++++++++++++++--- 1 file changed, 65 insertions(+), 3 deletions(-)