diff options
author | Rafał Miłecki <[email protected]> | 2022-01-11 18:29:19 +0100 |
---|---|---|
committer | Linus Walleij <[email protected]> | 2022-01-24 01:12:45 +0100 |
commit | 25d2e41cf59bd6ccd23adc2965a157053bc3ed5c (patch) | |
tree | eac7212fc2a2b77339788d83bf662749c95b898d /net/lapb/lapb_in.c | |
parent | 30cc53897470d45219fb0a5eafd0cc8b0032cd1e (diff) |
pinctrl: thunderbay: rework loops looking for groups names
Make the outer loop iterate over functions as that's the real subject.
This simplifies code (and reduces amount of lines of code) as allocating
memory for names doesn't require extra checks anymore.
While at it use local "group_names" variable. It fixes:
drivers/pinctrl/pinctrl-thunderbay.c: In function 'thunderbay_add_functions':
drivers/pinctrl/pinctrl-thunderbay.c:815:8: warning: assignment discards 'const' qualifier from pointer target type [-Wdiscarded-qualifiers]
815 | grp = func->group_names;
| ^
Ref: c26c4bfc1040 ("pinctrl: keembay: rework loops looking for groups names")
Reported-by: Nathan Chancellor <[email protected]>
Signed-off-by: Rafał Miłecki <[email protected]>
Link: https://lore.kernel.org/r/[email protected]
Signed-off-by: Linus Walleij <[email protected]>
Diffstat (limited to 'net/lapb/lapb_in.c')
0 files changed, 0 insertions, 0 deletions