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

[Mark Hounschell]

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/