Re: [PATCH] pinctrl: mediatek: fix a memleak when do dt maps.

2015-11-29 Thread Linus Walleij
On Tue, Nov 17, 2015 at 11:33 PM, Hongzhou Yang wrote: > configs will kmemdup to dup_configs in pictrl util function. > So configs need to be freed. > > Signed-off-by: Hongzhou Yang > Reviewed-by: Daniel Kurtz > Reviewed-by: Matthias Brugger > --- > Fix a memleak issue. > According to Yingjo

Re: [PATCH] pinctrl: mediatek: fix a memleak when do dt maps.

2015-11-17 Thread Yingjoe Chen
On Tue, 2015-11-17 at 14:33 -0800, Hongzhou Yang wrote: > configs will kmemdup to dup_configs in pictrl util function. > So configs need to be freed. > > Signed-off-by: Hongzhou Yang > Reviewed-by: Daniel Kurtz > Reviewed-by: Matthias Brugger > --- > Fix a memleak issue. > According to Yingjo

[PATCH] pinctrl: mediatek: fix a memleak when do dt maps.

2015-11-17 Thread Hongzhou Yang
configs will kmemdup to dup_configs in pictrl util function. So configs need to be freed. Signed-off-by: Hongzhou Yang Reviewed-by: Daniel Kurtz Reviewed-by: Matthias Brugger --- Fix a memleak issue. According to Yingjoe's comment, add return checek for pinconf_generic_parse_dt_config. dri

Re: [PATCH] pinctrl: mediatek: fix a memleak when do dt maps.

2015-11-17 Thread Yingjoe Chen
On Mon, 2015-11-16 at 20:22 -0800, Hongzhou Yang wrote: > configs will kmemdup to dup_configs in pictrl util function. > So configs need to be freed. > > Signed-off-by: Hongzhou Yang > --- > Fix a memleak issue. > > drivers/pinctrl/mediatek/pinctrl-mtk-common.c | 24 ++--

Re: [PATCH] pinctrl: mediatek: fix a memleak when do dt maps.

2015-11-17 Thread Matthias Brugger
On 17/11/15 09:25, Daniel Kurtz wrote: On Tue, Nov 17, 2015 at 12:22 PM, Hongzhou Yang wrote: configs will kmemdup to dup_configs in pictrl util function. So configs need to be freed. Signed-off-by: Hongzhou Yang Reviewed-by: Daniel Kurtz Reviewed-by: Matthias Brugger --- Fix a m

Re: [PATCH] pinctrl: mediatek: fix a memleak when do dt maps.

2015-11-17 Thread Daniel Kurtz
On Tue, Nov 17, 2015 at 12:22 PM, Hongzhou Yang wrote: > configs will kmemdup to dup_configs in pictrl util function. > So configs need to be freed. > > Signed-off-by: Hongzhou Yang Reviewed-by: Daniel Kurtz > --- > Fix a memleak issue. > > drivers/pinctrl/mediatek/pinctrl-mtk-common.c | 2

[PATCH] pinctrl: mediatek: fix a memleak when do dt maps.

2015-11-16 Thread Hongzhou Yang
configs will kmemdup to dup_configs in pictrl util function. So configs need to be freed. Signed-off-by: Hongzhou Yang --- Fix a memleak issue. drivers/pinctrl/mediatek/pinctrl-mtk-common.c | 24 ++-- 1 file changed, 14 insertions(+), 10 deletions(-) diff --git a/drivers

Re: [PATCH] pinctrl: mediatek: fix a memleak when do dt maps.

2015-11-16 Thread Daniel Kurtz
On Tue, Nov 17, 2015 at 10:34 AM, Hongzhou Yang wrote: > configs will kmemdup to dup_configs in pictrl util function. > So configs need to be freed. > > Signed-off-by: Hongzhou Yang > --- > Fix a memleak issue. > > drivers/pinctrl/mediatek/pinctrl-mtk-common.c |3 ++- > 1 file changed, 2 in

[PATCH] pinctrl: mediatek: fix a memleak when do dt maps.

2015-11-16 Thread Hongzhou Yang
configs will kmemdup to dup_configs in pictrl util function. So configs need to be freed. Signed-off-by: Hongzhou Yang --- Fix a memleak issue. drivers/pinctrl/mediatek/pinctrl-mtk-common.c |3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/pinctrl/mediatek/pinctr