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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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




[Index of Archives]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux