diff options
author | Hugo Villeneuve <hvilleneuve@dimonoff.com> | 2025-07-03 15:18:29 -0400 |
---|---|---|
committer | Bartosz Golaszewski <bartosz.golaszewski@linaro.org> | 2025-07-04 10:24:03 +0200 |
commit | 30e0fd3c0273dc106320081793793a424f1f1950 (patch) | |
tree | 28e8ebf9dbf96e9ea8a340464be934fee8908d0e /net/switchdev/switchdev.c | |
parent | d0b3b7b22dfa1f4b515fd3a295b3fd958f9e81af (diff) |
gpiolib: fix performance regression when using gpio_chip_get_multiple()
commit 74abd086d2ee ("gpiolib: sanitize the return value of
gpio_chip::get_multiple()") altered the value returned by
gc->get_multiple() in case it is positive (> 0), but failed to return
for other cases (<= 0).
This may result in the "if (gc->get)" block being executed and thus
negates the performance gain that is normally obtained by using
gc->get_multiple().
Fix by returning the result of gc->get_multiple() if it is <= 0.
Also move the "ret" variable to the scope where it is used, which as an
added bonus fixes an indentation error introduced by the aforementioned
commit.
Fixes: 74abd086d2ee ("gpiolib: sanitize the return value of gpio_chip::get_multiple()")
Cc: stable@vger.kernel.org
Signed-off-by: Hugo Villeneuve <hvilleneuve@dimonoff.com>
Link: https://lore.kernel.org/r/20250703191829.2952986-1-hugo@hugovil.com
Signed-off-by: Bartosz Golaszewski <bartosz.golaszewski@linaro.org>
Diffstat (limited to 'net/switchdev/switchdev.c')
0 files changed, 0 insertions, 0 deletions