24 May
2019
24 May
'19
5:42 p.m.
On Fri, May 24, 2019 at 04:36:54PM +0100, Jon Hunter wrote:
There could well be cases where you need to explicitly call devm_kfree(), but having a quick glance at the example above, I don't see why you would call devm_kfree() here and yes looks like that code could be simplified significantly. Notice that hisi_sas_debugfs_exit() does not free any memory as it is not necessary to explicitly do so.
Cheers Jon
-- nvpublic
Thanks for your suggestions, Jon! I think I need to e-mail to those maintainers about this issue.
Thanks Gen