Messages in this thread | | | From | Bartosz Golaszewski <> | Date | Mon, 26 Oct 2020 16:12:17 +0100 | Subject | Re: [PATCH 7/7] gpio: exar: use devm action for freeing the IDA and drop remove() |
| |
On Mon, Oct 26, 2020 at 4:02 PM Andy Shevchenko <andy.shevchenko@gmail.com> wrote: > > On Mon, Oct 26, 2020 at 4:22 PM Bartosz Golaszewski <brgl@bgdev.pl> wrote: > > > > From: Bartosz Golaszewski <bgolaszewski@baylibre.com> > > > > We can simplify the error path in probe() and drop remove() entirely if > > we provide a devm action for freeing the device ID. > > Always the same question to IDR/IDA users: > does it guarantee that when the driver is gone the IDR/IDA resources are freed? > > (It's not directly related to this patch, though) >
Yes because there's exactly one ida_free(id) per id = ida_alloc() here.
Bartosz
| |