Skip to content

Commit 4d58403

Browse files
acpibobrafaeljw
authored andcommitted
ACPICA: Add predefined names found in the SMBus sepcification
Affects run-time (kernel) ACPICA, iASL, and acpi_help. The "SMBus Control Method Interface Specification, Version 1.0, December 10, 1999" containes predefined names: _SBA _SBI _SBR _SBT _SBW. This was done outside of the ACPI specification. This commit adds support for ACPICA to recognize these named objects as predefined named objects. ACPICA commit 2fe13bd7ba9f97d3bf25488bf1bb1b2329427093 Link: acpica/acpica@2fe13bd7 Signed-off-by: Bob Moore <robert.moore@intel.com> Signed-off-by: Erik Kaneda <erik.kaneda@intel.com> Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
1 parent 465e490 commit 4d58403

4 files changed

Lines changed: 39 additions & 7 deletions

File tree

drivers/acpi/acpica/acpredef.h

Lines changed: 32 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -101,7 +101,7 @@ enum acpi_return_package_types {
101101

102102
/* Support macros for users of the predefined info table */
103103

104-
#define METHOD_PREDEF_ARGS_MAX 4
104+
#define METHOD_PREDEF_ARGS_MAX 5
105105
#define METHOD_ARG_BIT_WIDTH 3
106106
#define METHOD_ARG_MASK 0x0007
107107
#define ARG_COUNT_IS_MINIMUM 0x8000
@@ -117,6 +117,7 @@ enum acpi_return_package_types {
117117
#define METHOD_2ARGS(a1,a2) (2 | (a1 << 3) | (a2 << 6))
118118
#define METHOD_3ARGS(a1,a2,a3) (3 | (a1 << 3) | (a2 << 6) | (a3 << 9))
119119
#define METHOD_4ARGS(a1,a2,a3,a4) (4 | (a1 << 3) | (a2 << 6) | (a3 << 9) | (a4 << 12))
120+
#define METHOD_5ARGS(a1,a2,a3,a4,a5) (5 | (a1 << 3) | (a2 << 6) | (a3 << 9) | (a4 << 12) | (a5 << 15))
120121

121122
#define METHOD_RETURNS(type) (type)
122123
#define METHOD_NO_RETURN_VALUE 0
@@ -902,9 +903,39 @@ const union acpi_predefined_info acpi_gbl_predefined_methods[] = {
902903
{{"_S4W", METHOD_0ARGS,
903904
METHOD_RETURNS(ACPI_RTYPE_INTEGER)}},
904905

906+
{{"_SBA", METHOD_0ARGS,
907+
METHOD_RETURNS(ACPI_RTYPE_PACKAGE)}}, /* Fixed-length (4 Int) */
908+
PACKAGE_INFO(ACPI_PTYPE1_FIXED, ACPI_RTYPE_INTEGER, 4, 0, 0, 0),
909+
910+
{{"_SBI", METHOD_0ARGS,
911+
METHOD_RETURNS(ACPI_RTYPE_PACKAGE)}}, /* Fixed-length (1 Int, 1 Buf) */
912+
PACKAGE_INFO(ACPI_PTYPE1_FIXED, ACPI_RTYPE_INTEGER, 1,
913+
ACPI_RTYPE_BUFFER, 1, 0),
914+
915+
{{"_SBR",
916+
METHOD_3ARGS(ACPI_TYPE_INTEGER, ACPI_TYPE_INTEGER,
917+
ACPI_TYPE_INTEGER),
918+
METHOD_RETURNS(ACPI_RTYPE_PACKAGE)}}, /* Fixed-length (2 Int) */
919+
PACKAGE_INFO(ACPI_PTYPE1_FIXED, ACPI_RTYPE_INTEGER, 2,
920+
ACPI_RTYPE_BUFFER | ACPI_RTYPE_INTEGER, 1, 0),
921+
905922
{{"_SBS", METHOD_0ARGS,
906923
METHOD_RETURNS(ACPI_RTYPE_INTEGER)}},
907924

925+
{{"_SBT",
926+
METHOD_4ARGS(ACPI_TYPE_INTEGER, ACPI_TYPE_INTEGER, ACPI_TYPE_INTEGER,
927+
ACPI_TYPE_ANY),
928+
METHOD_RETURNS(ACPI_RTYPE_PACKAGE)}}, /* Fixed-length (2 Int, 1 Buf | Int) */
929+
PACKAGE_INFO(ACPI_PTYPE1_FIXED, ACPI_RTYPE_INTEGER, 2,
930+
ACPI_RTYPE_BUFFER | ACPI_RTYPE_INTEGER, 1, 0),
931+
932+
{{"_SBW",
933+
METHOD_5ARGS(ACPI_TYPE_INTEGER, ACPI_TYPE_INTEGER, ACPI_TYPE_INTEGER,
934+
ACPI_TYPE_INTEGER, ACPI_TYPE_ANY),
935+
METHOD_RETURNS(ACPI_RTYPE_PACKAGE)}},
936+
PACKAGE_INFO(ACPI_PTYPE1_FIXED, ACPI_RTYPE_BUFFER | ACPI_RTYPE_INTEGER,
937+
1, 0, 0, 0),
938+
908939
{{"_SCP", METHOD_1ARGS(ACPI_TYPE_INTEGER) | ARG_COUNT_IS_MINIMUM,
909940
METHOD_NO_RETURN_VALUE}}, /* Acpi 1.0 allowed 1 integer arg. Acpi 3.0 expanded to 3 args. Allow both. */
910941

drivers/acpi/acpica/nsarguments.c

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -55,7 +55,9 @@ void acpi_ns_check_argument_types(struct acpi_evaluate_info *info)
5555
arg_type = METHOD_GET_NEXT_TYPE(arg_type_list);
5656
user_arg_type = info->parameters[i]->common.type;
5757

58-
if (user_arg_type != arg_type) {
58+
/* No typechecking for ACPI_TYPE_ANY */
59+
60+
if ((user_arg_type != arg_type) && (arg_type != ACPI_TYPE_ANY)) {
5961
ACPI_WARN_PREDEFINED((AE_INFO, info->full_pathname,
6062
ACPI_WARN_ALWAYS,
6163
"Argument #%u type mismatch - "

drivers/acpi/acpica/psparse.c

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -508,8 +508,8 @@ acpi_status acpi_ps_parse_aml(struct acpi_walk_state *walk_state)
508508
}
509509

510510
/*
511-
* If the transfer to the new method method call worked
512-
*, a new walk state was created -- get it
511+
* If the transfer to the new method method call worked,
512+
* a new walk state was created -- get it
513513
*/
514514
walk_state = acpi_ds_get_current_walk_state(thread);
515515
continue;

drivers/acpi/acpica/utpredef.c

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -151,7 +151,7 @@ static u32 acpi_ut_get_argument_types(char *buffer, u16 argument_types);
151151

152152
static const char *ut_external_type_names[] = /* Indexed by ACPI_TYPE_* */
153153
{
154-
", UNSUPPORTED-TYPE",
154+
", Type_ANY",
155155
", Integer",
156156
", String",
157157
", Buffer",
@@ -311,8 +311,7 @@ static u32 acpi_ut_get_argument_types(char *buffer, u16 argument_types)
311311
for (i = 0; i < arg_count; i++) {
312312
this_argument_type = METHOD_GET_NEXT_TYPE(argument_types);
313313

314-
if (!this_argument_type
315-
|| (this_argument_type > METHOD_MAX_ARG_TYPE)) {
314+
if (this_argument_type > METHOD_MAX_ARG_TYPE) {
316315
printf("**** Invalid argument type (%u) "
317316
"in predefined info structure\n",
318317
this_argument_type);

0 commit comments

Comments
 (0)