diff options
author | Geert Uytterhoeven <geert+renesas@glider.be> | 2019-07-31 15:29:15 +0200 |
---|---|---|
committer | Linus Walleij <linus.walleij@linaro.org> | 2019-08-05 13:29:29 +0200 |
commit | f0b0e923e07aa3efd3d5db0bd6ff421bd9a9a0d4 (patch) | |
tree | 0db183544fa9c2700a1eacb6d64d84f8d8447270 /drivers/pinctrl | |
parent | pinctrl: Remove dev_err() usage after platform_get_irq() (diff) | |
download | linux-f0b0e923e07aa3efd3d5db0bd6ff421bd9a9a0d4.tar.xz linux-f0b0e923e07aa3efd3d5db0bd6ff421bd9a9a0d4.zip |
pinctrl: devicetree: Use strlen() instead of hardcoded number
Improve readability by replacing a hardcoded number requiring a comment
by strlen().
Gcc is smart enough to evaluate the length of a constant string at
compile-time.
Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
Link: https://lore.kernel.org/r/20190731132917.17607-2-geert+renesas@glider.be
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
Diffstat (limited to 'drivers/pinctrl')
-rw-r--r-- | drivers/pinctrl/devicetree.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/drivers/pinctrl/devicetree.c b/drivers/pinctrl/devicetree.c index 88ddbb2e30de..5d6d8b1e9062 100644 --- a/drivers/pinctrl/devicetree.c +++ b/drivers/pinctrl/devicetree.c @@ -228,10 +228,8 @@ int pinctrl_dt_to_map(struct pinctrl *p, struct pinctrl_dev *pctldev) * than dynamically allocate it and have to free it later, * just point part way into the property name for the string. */ - if (ret < 0) { - /* strlen("pinctrl-") == 8 */ - statename = prop->name + 8; - } + if (ret < 0) + statename = prop->name + strlen("pinctrl-"); /* For every referenced pin configuration node in it */ for (config = 0; config < size; config++) { |