We should call gpiochip_remove(chip) to free the resource alloced by gpiochip_add_data(chip, gpio) after the err path.
Fixes: 553b75d4bfe9 ("gpio: regmap: Allow to allocate regmap-irq device") CC: stable@vger.kernel.org Co-developed-by: WangYuli wangyl5933@chinaunicom.cn Signed-off-by: WangYuli wangyl5933@chinaunicom.cn Signed-off-by: Wentao Guan guanwentao@uniontech.com --- changelog in v2: 1. format commit message. 2. rebase to mainline now. --- --- drivers/gpio/gpio-regmap.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gpio/gpio-regmap.c b/drivers/gpio/gpio-regmap.c index f4267af00027e..c64805dcb9f88 100644 --- a/drivers/gpio/gpio-regmap.c +++ b/drivers/gpio/gpio-regmap.c @@ -328,7 +328,7 @@ struct gpio_regmap *gpio_regmap_register(const struct gpio_regmap_config *config config->regmap_irq_line, config->regmap_irq_flags, 0, config->regmap_irq_chip, &gpio->irq_chip_data); if (ret) - goto err_free_bitmap; + goto err_remove_gpiochip;
irq_domain = regmap_irq_get_domain(gpio->irq_chip_data); } else
base-commit: 3f9f0252130e7dd60d41be0802bf58f6471c691d
On Wed, Dec 03, 2025 at 10:15:58PM +0800, Wentao Guan wrote:
We should call gpiochip_remove(chip) to free the resource alloced by gpiochip_add_data(chip, gpio) after the err path.
Fixes: 553b75d4bfe9 ("gpio: regmap: Allow to allocate regmap-irq device")
As I told in v1, add the other Fixes tag, to show that the code was modified and this change has a dependency to that.
...
With that done, it LGTM.
Hello Andy:
I did not found ae495810cffe ,thought it is that: Fixes: 00aaae60faf5 ("gpio: regmap: add the .fixed_direction_output configuration parameter"),
I will add it to v3.
BRs Wentao Guan
On Wed, Dec 03, 2025 at 11:12:39PM +0800, Wentao Guan wrote:
Hello Andy:
I did not found ae495810cffe ,thought it is that: Fixes: 00aaae60faf5 ("gpio: regmap: add the .fixed_direction_output configuration parameter"),
I will add it to v3.
No, please don't. This is wrong.
$ git tag --contains ae495810cffe next-20251022 ...
$ git branch -a --contains ae495810cffe | grep gpio remotes/brgl/gpio/for-current remotes/brgl/gpio/for-next
So it will be part of v6.19-rc1, which means your patch should be based on that three.
linux-stable-mirror@lists.linaro.org