Messages in this thread | | | Date | Fri, 21 May 2021 10:10:39 +0200 | From | Michael Walle <> | Subject | Re: [PATCH v2 2/3] gpio: gpio-regmap: Use devm_add_action() |
| |
Am 2021-05-21 08:28, schrieb Matti Vaittinen: > Slightly simplify the devm_gpio_regmap_register() by using the > devm_add_action().
Hm, nice, but what bothers me a bit is that no other subsystem does it that way, eg. hwmon/hwmon.c or watchdog/watchdog_core.c. They also store just one pointer, thus could be simplified in the same way. What I don't know is if devm_add_action() was intended to be used this way. So I can't say much for this patch ;)
-michael
| |