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.suthikulpa...@amd.com] > Sent: Monday, March 30, 2015 7:17 AM > To: Moore, Robert; r...@rjwysocki.net; mika.westerb...@linux.intel.com; > Zheng, Lv; hanjun....@linaro.org > Cc: l...@kernel.org; hdego...@redhat.com; t...@kernel.org; > mj...@srcf.ucam.org; gre...@linuxfoundation.org; al.st...@linaro.org; > graeme.greg...@linaro.org; leo.du...@amd.com; linux-...@vger.kernel.org; > linux-a...@vger.kernel.org; linux-kernel@vger.kernel.org; linaro- > a...@lists.linaro.org > 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-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
