> We should free 'feature->irq_ctx[idx].name' to avoid 'name' > memory leak when request_irq() failed. 1. Please choose an imperative wording for an improved change description. https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/process/submitting-patches.rst?h=v6.10-rc4#n94 2. Would you like to add any tags (like “Fixes” and “Cc”) accordingly? 3. How do you think about to use a summary phrase like “Avoid memory leak in do_set_irq_trigger()”? 4. Under which circumstances will development interests grow for increasing the application of scope-based resource management? https://elixir.bootlin.com/linux/v6.10-rc4/source/include/linux/cleanup.h#L8 Regards, Markus