Thread (39 messages) 39 messages, 12 authors, 2024-12-26

Re: [PATCH v3 2/7] gpio: Add Nuvoton NCT6694 GPIO support

From: Bartosz Golaszewski <hidden>
Date: 2024-12-10 12:46:56
Also in: linux-can, linux-gpio, linux-hwmon, linux-i2c, linux-rtc, linux-watchdog, lkml

On Tue, Dec 10, 2024 at 11:46 AM Ming Yu [off-list ref] wrote:
This driver supports GPIO and IRQ functionality for NCT6694 MFD
device based on USB interface.

Signed-off-by: Ming Yu <tmyu0@nuvoton.com>
---
Looks much better now. Please address one more issue I just noticed.
+
+       mutex_init(&data->irq_lock);
This is never destroyed. Please use devm_mutex_init() preferably to
not add remove(). Also, the other mutex doesn't seem to be initialized
at all.

Bart
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help