Re: [PATCH V2] staging: dgap: introduce dgap_cleanup_nodes()

[DaeSeok Youn]

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
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/