diff options
| author | Geert Uytterhoeven <[email protected]> | 2019-07-31 15:29:15 +0200 |
|---|---|---|
| committer | Linus Walleij <[email protected]> | 2019-08-05 13:29:29 +0200 |
| commit | f0b0e923e07aa3efd3d5db0bd6ff421bd9a9a0d4 (patch) | |
| tree | 0db183544fa9c2700a1eacb6d64d84f8d8447270 | |
| parent | 64c4dcbfcc4ea3ae8f898e6dac836e57acb16b6b (diff) | |
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 <[email protected]>
Link: https://lore.kernel.org/r/[email protected]
Signed-off-by: Linus Walleij <[email protected]>
| -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++) { |