On Tue, Jun 21, 2016 at 11:50:01AM +0000, Opensource [Adam Thomson] wrote:

21 June 2016 12:42, Rafael J. Wysocki wrote:

quoted

quoted

quoted

+static inline bool acpi_data_node_match(struct fwnode_handle *fwnode,
+					const char *name)
+{
+	return is_acpi_data_node(fwnode) ?
+		(!strcasecmp(to_acpi_data_node(fwnode)->name, name)) : false;
+}

Looks fine to me.

One question - is it expected that matching ACPI data nodes is always
case insensitive?

That would not be a correct expectation in theory, although I don't think it
really matters in practice.

From my reading of the Hierarchical Data Extension and ACPI Spec, I thought
that was the case (section 19.3.1 ASL Names - ASL names are not case-sensitive
and will be converted to upper case). Am I misreading the documents/missing
something else?

Those are names in the ASL code itself.

What we are talking here are actually just string values (name of the
data node).