Hi, Mark and Greg. Thanks for testing. :-) Greg, is this patch possible to merge to your staging tree? Please check for me. patch : https://lkml.org/lkml/2014/7/31/2 Thanks. regards, Daeseok Youn. 2014-08-04 21:40 GMT+09:00 Mark Hounschell <markh@xxxxxxxxxx>: > On 07/31/2014 07:14 PM, DaeSeok Youn wrote: >> >> Hi, Mark >> >> 2014-07-31 21:44 GMT+09:00 Mark Hounschell <markh@xxxxxxxxxx>: >>> >>> On 07/31/2014 12:02 AM, Daeseok Youn wrote: >>> >>>> >>>> When a configration file is parsed with dgap_parsefile(), >>>> makes nodes for saving configrations for board. >>>> >>>> Making a node will allocate node memory and strings for saving >>>> configrations with kstrdup(). >>>> >>>> So these are freed when dgap is unloaded or failed to initialize. >>>> >>>> Signed-off-by: Daeseok Youn <daeseok.youn@xxxxxxxxx> >>>> --- >>>> V2: Do not need to free for NULLNODE. >>>> >>>> I have been too busy to solve this issue, sorry for late. >>>> >>>> Mark, Can you test this patch? I try to make simple module which is >>>> testing dgap_parsefile() and dgap_cleanup_nodes(). >>>> >>> >>> I'll be happy to, but I can't do it until Monday. I'm not where the >>> hardware >>> is until then. >> >> That's OK. :-) >> > > After applying this patch I am still able to load and unload the driver at > will, and it still works for me. > > Regards > Mark > _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
