Thread (14 messages) 14 messages, 4 authors, 2016-06-21

Re: [RESEND PATCH v3 1/2] device property: Add function to search for named child of device

From: Rafael J. Wysocki <hidden>
Date: 2016-06-21 15:07:27
Also in: alsa-devel, linux-acpi, lkml

On Tuesday, June 21, 2016 11:50:01 AM 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).
Section 19.3.1 is applicable to object names and not to the data returned
by those objects.  The link names in the Hierarchical Data Extension are
data returned by _DSD (or other objects related to it).  They are general ACPI
strings and those are case-sensitive.

Thanks,
Rafael

--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help