On Thu, Dec 8, 2016 at 3:35 PM, Arvind Yadav <arvind.yadav.cs@xxxxxxxxx> wrote: > In functions pxa2xx_build_functions, the memory allocated for > 'functions' is live within the function only. After the > allocation it is immediately freed with devm_kfree. There is > no need to allocate memory for 'functions' with devm function > so replace devm_kcalloc with kcalloc and devm_kfree with kfree. > > Signed-off-by: Arvind Yadav <arvind.yadav.cs@xxxxxxxxx> I want the maintainer Robert Jarzmik to review this before I do anything with it. Yours, Linus Walleij -- To unsubscribe from this list: send the line "unsubscribe linux-gpio" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH V1] pinctrl:pxa:pinctrl-pxa2xx:- No need of devm functions
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
- Subject: Re: [PATCH V1] pinctrl:pxa:pinctrl-pxa2xx:- No need of devm functions
- From: Linus Walleij <linus.walleij@xxxxxxxxxx>
- Date: Tue, 27 Dec 2016 13:49:06 +0100
- In-reply-to: <1481207730-6332-1-git-send-email-arvind.yadav.cs@gmail.com>
- References: <1481207730-6332-1-git-send-email-arvind.yadav.cs@gmail.com>
- Follow-Ups:
- Re: [PATCH V1] pinctrl:pxa:pinctrl-pxa2xx:- No need of devm functions
- From: Robert Jarzmik
- Re: [PATCH V1] pinctrl:pxa:pinctrl-pxa2xx:- No need of devm functions
- References:
- [PATCH V1] pinctrl:pxa:pinctrl-pxa2xx:- No need of devm functions
- From: Arvind Yadav
- [PATCH V1] pinctrl:pxa:pinctrl-pxa2xx:- No need of devm functions
- Prev by Date: [PATCH v2 11/12] ARM: imx: correct i.mx6sll dram io low power mode
- Next by Date: Re: [PATCH v2 05/12] Document: dt: binding: imx: update pinctrl doc for imx6sll
- Previous by thread: Re: [PATCH V1] pinctrl:pxa:pinctrl-pxa2xx:- No need of devm functions
- Next by thread: Re: [PATCH V1] pinctrl:pxa:pinctrl-pxa2xx:- No need of devm functions
- Index(es):
 |