On Mon, Sep 30, 2024 at 6:32 PM Bartosz Golaszewski <brgl@xxxxxxxx> wrote: > From: Bartosz Golaszewski <bartosz.golaszewski@xxxxxxxxxx> > > The bitmap allocated in aggr_parse() is always freed before the function > returns so use __free(bitmap) to simplify it and drop the goto label. > > Signed-off-by: Bartosz Golaszewski <bartosz.golaszewski@xxxxxxxxxx> Very neat! Reviewed-by: Linus Walleij <linus.walleij@xxxxxxxxxx> Yours, Linus Walleij
Re: [PATCH] gpio: aggregator: simplify aggr_parse() with scoped bitmap
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
- Subject: Re: [PATCH] gpio: aggregator: simplify aggr_parse() with scoped bitmap
- From: Linus Walleij <linus.walleij@xxxxxxxxxx>
- Date: Wed, 2 Oct 2024 15:17:25 +0200
- In-reply-to: <20240930163207.80276-1-brgl@bgdev.pl>
- References: <20240930163207.80276-1-brgl@bgdev.pl>
- References:
- [PATCH] gpio: aggregator: simplify aggr_parse() with scoped bitmap
- From: Bartosz Golaszewski
- [PATCH] gpio: aggregator: simplify aggr_parse() with scoped bitmap
- Prev by Date: Re: [PATCH v2] gpio: free irqs that are still requested when the chip is being removed
- Next by Date: Re: [PATCH 2/3] gpio: xilinx: use helper variable to store the address of pdev->dev
- Previous by thread: [PATCH] gpio: aggregator: simplify aggr_parse() with scoped bitmap
- Next by thread: Re: [PATCH] gpio: aggregator: simplify aggr_parse() with scoped bitmap
- Index(es):
 |