Commit 10535314 authored by Sebastian Hesselbarth's avatar Sebastian Hesselbarth Committed by Rob Herring

OF: base: match each node compatible against all given matches first

Currently, of_match_node compares each given match against all node's
compatible strings with of_device_is_compatible.

To achieve multiple compatible strings per node with ordering from
specific to generic, this requires given matches to be ordered from
specific to generic. For most of the drivers this is not true and also
an alphabetical ordering is more sane there.

Therefore, this patch modifies of_match_node to match each of the node's
compatible strings against all given matches first, before checking the
next compatible string. This implies that node's compatibles are ordered
from specific to generic while given matches can be in any order.
Signed-off-by: default avatarSebastian Hesselbarth <sebastian.hesselbarth@gmail.com>
Tested-by: default avatarMeelis Roos <mroos@linux.ee>
Signed-off-by: default avatarRob Herring <rob.herring@calxeda.com>
parent 3ab72f91
...@@ -731,24 +731,42 @@ static ...@@ -731,24 +731,42 @@ static
const struct of_device_id *__of_match_node(const struct of_device_id *matches, const struct of_device_id *__of_match_node(const struct of_device_id *matches,
const struct device_node *node) const struct device_node *node)
{ {
const char *cp;
int cplen, l;
if (!matches) if (!matches)
return NULL; return NULL;
while (matches->name[0] || matches->type[0] || matches->compatible[0]) { cp = __of_get_property(node, "compatible", &cplen);
int match = 1; do {
if (matches->name[0]) const struct of_device_id *m = matches;
match &= node->name
&& !strcmp(matches->name, node->name); /* Check against matches with current compatible string */
if (matches->type[0]) while (m->name[0] || m->type[0] || m->compatible[0]) {
match &= node->type int match = 1;
&& !strcmp(matches->type, node->type); if (m->name[0])
if (matches->compatible[0]) match &= node->name
match &= __of_device_is_compatible(node, && !strcmp(m->name, node->name);
matches->compatible); if (m->type[0])
if (match) match &= node->type
return matches; && !strcmp(m->type, node->type);
matches++; if (m->compatible[0])
} match &= cp
&& !of_compat_cmp(m->compatible, cp,
strlen(m->compatible));
if (match)
return m;
m++;
}
/* Get node's next compatible string */
if (cp) {
l = strlen(cp) + 1;
cp += l;
cplen -= l;
}
} while (cp && (cplen > 0));
return NULL; return NULL;
} }
...@@ -757,7 +775,10 @@ const struct of_device_id *__of_match_node(const struct of_device_id *matches, ...@@ -757,7 +775,10 @@ const struct of_device_id *__of_match_node(const struct of_device_id *matches,
* @matches: array of of device match structures to search in * @matches: array of of device match structures to search in
* @node: the of device structure to match against * @node: the of device structure to match against
* *
* Low level utility function used by device matching. * Low level utility function used by device matching. Matching order
* is to compare each of the node's compatibles with all given matches
* first. This implies node's compatible is sorted from specific to
* generic while matches can be in any order.
*/ */
const struct of_device_id *of_match_node(const struct of_device_id *matches, const struct of_device_id *of_match_node(const struct of_device_id *matches,
const struct device_node *node) const struct device_node *node)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment