AE_AML_OPERAND_TYPE -- used to indicate an exception raised by a type issue related to the AML code. Used to differentiate between a programmer error (AE_TYPE). > -----Original Message----- > From: Suravee Suthikulanit [mailto:suravee.suthikulpanit@xxxxxxx] > Sent: Monday, March 30, 2015 7:17 AM > To: Moore, Robert; rjw@xxxxxxxxxxxxx; mika.westerberg@xxxxxxxxxxxxxxx; > Zheng, Lv; hanjun.guo@xxxxxxxxxx > Cc: lenb@xxxxxxxxxx; hdegoede@xxxxxxxxxx; tj@xxxxxxxxxx; > mjg59@xxxxxxxxxxxxx; gregkh@xxxxxxxxxxxxxxxxxxx; al.stone@xxxxxxxxxx; > graeme.gregory@xxxxxxxxxx; leo.duran@xxxxxxx; linux-ide@xxxxxxxxxxxxxxx; > linux-acpi@xxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx; linaro- > acpi@xxxxxxxxxxxxxxxx > Subject: Re: [V7 PATCH 1/3] ACPICA: Add ACPI _CLS processing > > On 3/27/2015 12:51 PM, Moore, Robert wrote: > > + cls_objects = obj_desc->package.elements; > > + > > + if (obj_desc->common.type == ACPI_TYPE_PACKAGE && > > + obj_desc->package.count == 3 && > > + cls_objects[0]->common.type == ACPI_TYPE_INTEGER && > > + cls_objects[1]->common.type == ACPI_TYPE_INTEGER && > > + cls_objects[2]->common.type == ACPI_TYPE_INTEGER) { > > + > > + /* Allocate a buffer for the CLS */ > > + cls = ACPI_ALLOCATE_ZEROED(sizeof(struct acpi_pnp_device_id) + > > + (acpi_size) 7); > > > > I would like to see an error returned if an object or subobject is of > the incorrect type. > > Then, the caller knows not to attempt to look at it. > > > > Ok. I will return AE_TYPE if the condition is false here. > > Thanks, > > Suravee -- To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
