On 14/08/2025 00:59, Rosen Penev wrote: >> >>> + u32 led_pin; >>> + >>> + if (!of_property_read_u32(np, "reg", &led_pin)) >>> + ah->led_pin = led_pin; >>> + >>> + ah->config.led_active_high = !of_property_read_bool(np, "led-active-low"); >>> + of_node_put(np); >>> + } >> >> Leaking OF node. > > Not following here. You miss proper cleanup, leaking of node. Best regards, Krzysztof