diff options
author | Rob Herring <robh@kernel.org> | 2017-06-01 18:00:00 -0500 |
---|---|---|
committer | Rob Herring <robh@kernel.org> | 2017-06-22 12:37:46 -0500 |
commit | 95e6b1fa3311c8a7b151d38540695409048d1495 (patch) | |
tree | cb945ab7cae909f835f2bc1ed134fd4628247144 /drivers/of/base.c | |
parent | 8ee8a0e795e7a41078aa8eceb1dac1656975b9a6 (diff) | |
download | op-kernel-dev-95e6b1fa3311c8a7b151d38540695409048d1495.zip op-kernel-dev-95e6b1fa3311c8a7b151d38540695409048d1495.tar.gz |
of: use kbasename instead of open coding
Several places in DT code open code the equivalent of kbasename.
Replace them.
The behavior for root nodes in node_name_cmp will be slightly different.
Instead of comparing "/", "" will be compared. The comparison will be
the same.
Reviewed-by: Andy Shevchenko <andy.shevhchenko@gmail.com>
Signed-off-by: Rob Herring <robh@kernel.org>
Diffstat (limited to 'drivers/of/base.c')
-rw-r--r-- | drivers/of/base.c | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/drivers/of/base.c b/drivers/of/base.c index 974d512..74b0a27 100644 --- a/drivers/of/base.c +++ b/drivers/of/base.c @@ -773,10 +773,7 @@ static struct device_node *__of_find_node_by_path(struct device_node *parent, return NULL; __for_each_child_of_node(parent, child) { - const char *name = strrchr(child->full_name, '/'); - if (WARN(!name, "malformed device_node %s\n", child->full_name)) - continue; - name++; + const char *name = kbasename(child->full_name); if (strncmp(path, name, len) == 0 && (strlen(name) == len)) return child; } |