* [PATCH] drivers/pinctrl/pinctrl-at91.c: convert kfree to devm_kfree
@ 2012-12-11 10:58 Julia Lawall
2012-12-11 13:00 ` Sergei Shtylyov
2012-12-12 11:56 ` Jean-Christophe PLAGNIOL-VILLARD
0 siblings, 2 replies; 8+ messages in thread
From: Julia Lawall @ 2012-12-11 10:58 UTC (permalink / raw)
To: Jean-Christophe Plagniol-Villard
Cc: kernel-janitors, Linus Walleij, linux-arm-kernel, linux-kernel
From: Julia Lawall <Julia.Lawall@lip6.fr>
The function at91_dt_node_to_map is ultimately called by the function
pinctrl_get, which is an exported function. Since it is possible that this
function is not called from within a probe function, for safety, the kfree
is converted to a devm_kfree, to both free the data and remove it from the
device in a failure situation.
A newline is added in the call to devm_kfree to avoid exceeding the 80
character limit.
Signed-off-by: Julia Lawall <Julia.Lawall@lip6.fr>
---
drivers/pinctrl/pinctrl-at91.c | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/drivers/pinctrl/pinctrl-at91.c b/drivers/pinctrl/pinctrl-at91.c
index c5e7571..0da8a5f 100644
--- a/drivers/pinctrl/pinctrl-at91.c
+++ b/drivers/pinctrl/pinctrl-at91.c
@@ -255,7 +255,8 @@ static int at91_dt_node_to_map(struct pinctrl_dev *pctldev,
}
map_num += grp->npins;
- new_map = devm_kzalloc(pctldev->dev, sizeof(*new_map) * map_num, GFP_KERNEL);
+ new_map = devm_kzalloc(pctldev->dev, sizeof(*new_map) * map_num,
+ GFP_KERNEL);
if (!new_map)
return -ENOMEM;
@@ -265,7 +266,7 @@ static int at91_dt_node_to_map(struct pinctrl_dev *pctldev,
/* create mux map */
parent = of_get_parent(np);
if (!parent) {
- kfree(new_map);
+ devm_kfree(pctldev->dev, new_map);
return -EINVAL;
}
new_map[0].type = PIN_MAP_TYPE_MUX_GROUP;
^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [PATCH] drivers/pinctrl/pinctrl-at91.c: convert kfree to devm_kfree
2012-12-11 10:58 [PATCH] drivers/pinctrl/pinctrl-at91.c: convert kfree to devm_kfree Julia Lawall
@ 2012-12-11 13:00 ` Sergei Shtylyov
2012-12-11 13:08 ` Julia Lawall
2012-12-12 11:56 ` Jean-Christophe PLAGNIOL-VILLARD
1 sibling, 1 reply; 8+ messages in thread
From: Sergei Shtylyov @ 2012-12-11 13:00 UTC (permalink / raw)
To: Julia Lawall
Cc: Jean-Christophe Plagniol-Villard, Linus Walleij, kernel-janitors,
linux-kernel, linux-arm-kernel
Hello.
On 11-12-2012 14:58, Julia Lawall wrote:
> From: Julia Lawall <Julia.Lawall@lip6.fr>
> The function at91_dt_node_to_map is ultimately called by the function
> pinctrl_get, which is an exported function. Since it is possible that this
> function is not called from within a probe function, for safety, the kfree
> is converted to a devm_kfree, to both free the data and remove it from the
> device in a failure situation.
> A newline is added in the call to devm_kfree to avoid exceeding the 80
devm_kzalloc() you mean?
> character limit.
> Signed-off-by: Julia Lawall <Julia.Lawall@lip6.fr>
> ---
> drivers/pinctrl/pinctrl-at91.c | 5 +++--
> 1 file changed, 3 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/pinctrl/pinctrl-at91.c b/drivers/pinctrl/pinctrl-at91.c
> index c5e7571..0da8a5f 100644
> --- a/drivers/pinctrl/pinctrl-at91.c
> +++ b/drivers/pinctrl/pinctrl-at91.c
> @@ -255,7 +255,8 @@ static int at91_dt_node_to_map(struct pinctrl_dev *pctldev,
> }
>
> map_num += grp->npins;
> - new_map = devm_kzalloc(pctldev->dev, sizeof(*new_map) * map_num, GFP_KERNEL);
> + new_map = devm_kzalloc(pctldev->dev, sizeof(*new_map) * map_num,
> + GFP_KERNEL);
WBR, Sergei
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] drivers/pinctrl/pinctrl-at91.c: convert kfree to devm_kfree
2012-12-11 13:00 ` Sergei Shtylyov
@ 2012-12-11 13:08 ` Julia Lawall
2012-12-11 18:26 ` Sergei Shtylyov
0 siblings, 1 reply; 8+ messages in thread
From: Julia Lawall @ 2012-12-11 13:08 UTC (permalink / raw)
To: Sergei Shtylyov
Cc: Julia Lawall, Jean-Christophe Plagniol-Villard, Linus Walleij,
kernel-janitors, linux-kernel, linux-arm-kernel
On Tue, 11 Dec 2012, Sergei Shtylyov wrote:
> Hello.
>
> On 11-12-2012 14:58, Julia Lawall wrote:
>
> > From: Julia Lawall <Julia.Lawall@lip6.fr>
>
> > The function at91_dt_node_to_map is ultimately called by the function
> > pinctrl_get, which is an exported function. Since it is possible that this
> > function is not called from within a probe function, for safety, the kfree
> > is converted to a devm_kfree, to both free the data and remove it from the
> > device in a failure situation.
>
> > A newline is added in the call to devm_kfree to avoid exceeding the 80
>
> devm_kzalloc() you mean?
Yes, sorry. Should I send a new patch?
thanks,
julia
> > character limit.
>
> > Signed-off-by: Julia Lawall <Julia.Lawall@lip6.fr>
>
> > ---
> > drivers/pinctrl/pinctrl-at91.c | 5 +++--
> > 1 file changed, 3 insertions(+), 2 deletions(-)
> >
> > diff --git a/drivers/pinctrl/pinctrl-at91.c b/drivers/pinctrl/pinctrl-at91.c
> > index c5e7571..0da8a5f 100644
> > --- a/drivers/pinctrl/pinctrl-at91.c
> > +++ b/drivers/pinctrl/pinctrl-at91.c
> > @@ -255,7 +255,8 @@ static int at91_dt_node_to_map(struct pinctrl_dev
> > *pctldev,
> > }
> >
> > map_num += grp->npins;
> > - new_map = devm_kzalloc(pctldev->dev, sizeof(*new_map) * map_num,
> > GFP_KERNEL);
> > + new_map = devm_kzalloc(pctldev->dev, sizeof(*new_map) * map_num,
> > + GFP_KERNEL);
>
> WBR, Sergei
>
>
>
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] drivers/pinctrl/pinctrl-at91.c: convert kfree to devm_kfree
2012-12-11 13:08 ` Julia Lawall
@ 2012-12-11 18:26 ` Sergei Shtylyov
0 siblings, 0 replies; 8+ messages in thread
From: Sergei Shtylyov @ 2012-12-11 18:26 UTC (permalink / raw)
To: Julia Lawall
Cc: Jean-Christophe Plagniol-Villard, Linus Walleij, kernel-janitors,
linux-kernel, linux-arm-kernel
Hello.
On 12/11/2012 04:08 PM, Julia Lawall wrote:
>>> From: Julia Lawall <Julia.Lawall@lip6.fr>
>>> The function at91_dt_node_to_map is ultimately called by the function
>>> pinctrl_get, which is an exported function. Since it is possible that this
>>> function is not called from within a probe function, for safety, the kfree
>>> is converted to a devm_kfree, to both free the data and remove it from the
>>> device in a failure situation.
>>
>>> A newline is added in the call to devm_kfree to avoid exceeding the 80
>>
>> devm_kzalloc() you mean?
> Yes, sorry. Should I send a new patch?
Up to the maintainer I guess...
> thanks,
> julia
WBR, Sergei
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] drivers/pinctrl/pinctrl-at91.c: convert kfree to devm_kfree
2012-12-11 10:58 [PATCH] drivers/pinctrl/pinctrl-at91.c: convert kfree to devm_kfree Julia Lawall
2012-12-11 13:00 ` Sergei Shtylyov
@ 2012-12-12 11:56 ` Jean-Christophe PLAGNIOL-VILLARD
2012-12-12 14:22 ` Julia Lawall
1 sibling, 1 reply; 8+ messages in thread
From: Jean-Christophe PLAGNIOL-VILLARD @ 2012-12-12 11:56 UTC (permalink / raw)
To: Julia Lawall
Cc: Linus Walleij, kernel-janitors, linux-kernel, linux-arm-kernel
On 11:58 Tue 11 Dec , Julia Lawall wrote:
> From: Julia Lawall <Julia.Lawall@lip6.fr>
>
> The function at91_dt_node_to_map is ultimately called by the function
> pinctrl_get, which is an exported function. Since it is possible that this
> function is not called from within a probe function, for safety, the kfree
> is converted to a devm_kfree, to both free the data and remove it from the
> device in a failure situation.
>
> A newline is added in the call to devm_kfree to avoid exceeding the 80
> character limit.
>
> Signed-off-by: Julia Lawall <Julia.Lawall@lip6.fr>
>
> ---
> drivers/pinctrl/pinctrl-at91.c | 5 +++--
> 1 file changed, 3 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/pinctrl/pinctrl-at91.c b/drivers/pinctrl/pinctrl-at91.c
> index c5e7571..0da8a5f 100644
> --- a/drivers/pinctrl/pinctrl-at91.c
> +++ b/drivers/pinctrl/pinctrl-at91.c
> @@ -255,7 +255,8 @@ static int at91_dt_node_to_map(struct pinctrl_dev *pctldev,
> }
>
> map_num += grp->npins;
> - new_map = devm_kzalloc(pctldev->dev, sizeof(*new_map) * map_num, GFP_KERNEL);
> + new_map = devm_kzalloc(pctldev->dev, sizeof(*new_map) * map_num,
> + GFP_KERNEL);
not related please drop
> if (!new_map)
> return -ENOMEM;
>
> @@ -265,7 +266,7 @@ static int at91_dt_node_to_map(struct pinctrl_dev *pctldev,
> /* create mux map */
> parent = of_get_parent(np);
> if (!parent) {
> - kfree(new_map);
> + devm_kfree(pctldev->dev, new_map);
ok
> return -EINVAL;
> }
> new_map[0].type = PIN_MAP_TYPE_MUX_GROUP;
>
Best Regards,
J.
>
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] drivers/pinctrl/pinctrl-at91.c: convert kfree to devm_kfree
2012-12-12 11:56 ` Jean-Christophe PLAGNIOL-VILLARD
@ 2012-12-12 14:22 ` Julia Lawall
2012-12-12 17:01 ` Jean-Christophe PLAGNIOL-VILLARD
2012-12-14 15:46 ` Linus Walleij
0 siblings, 2 replies; 8+ messages in thread
From: Julia Lawall @ 2012-12-12 14:22 UTC (permalink / raw)
To: Jean-Christophe PLAGNIOL-VILLARD
Cc: Linus Walleij, kernel-janitors, linux-kernel, linux-arm-kernel
From: Julia Lawall <Julia.Lawall@lip6.fr>
The function at91_dt_node_to_map is ultimately called by the function
pinctrl_get, which is an exported function. Since it is possible that this
function is not called from within a probe function, for safety, the kfree
is converted to a devm_kfree, to both free the data and remove it from the
device in a failure situation.
Signed-off-by: Julia Lawall <Julia.Lawall@lip6.fr>
---
v2: Drop cleanup of the call to devm_kzalloc at the same time.
drivers/pinctrl/pinctrl-at91.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/pinctrl/pinctrl-at91.c b/drivers/pinctrl/pinctrl-at91.c
index c5e7571..bff54bf 100644
--- a/drivers/pinctrl/pinctrl-at91.c
+++ b/drivers/pinctrl/pinctrl-at91.c
@@ -265,7 +265,7 @@ static int at91_dt_node_to_map(struct pinctrl_dev *pctldev,
/* create mux map */
parent = of_get_parent(np);
if (!parent) {
- kfree(new_map);
+ devm_kfree(pctldev->dev, new_map);
return -EINVAL;
}
new_map[0].type = PIN_MAP_TYPE_MUX_GROUP;
^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [PATCH] drivers/pinctrl/pinctrl-at91.c: convert kfree to devm_kfree
2012-12-12 14:22 ` Julia Lawall
@ 2012-12-12 17:01 ` Jean-Christophe PLAGNIOL-VILLARD
2012-12-14 15:46 ` Linus Walleij
1 sibling, 0 replies; 8+ messages in thread
From: Jean-Christophe PLAGNIOL-VILLARD @ 2012-12-12 17:01 UTC (permalink / raw)
To: Julia Lawall
Cc: Linus Walleij, kernel-janitors, linux-kernel, linux-arm-kernel
On 15:22 Wed 12 Dec , Julia Lawall wrote:
> From: Julia Lawall <Julia.Lawall@lip6.fr>
>
> The function at91_dt_node_to_map is ultimately called by the function
> pinctrl_get, which is an exported function. Since it is possible that this
> function is not called from within a probe function, for safety, the kfree
> is converted to a devm_kfree, to both free the data and remove it from the
> device in a failure situation.
>
> Signed-off-by: Julia Lawall <Julia.Lawall@lip6.fr>
Acked-by: Jean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com>
Best Regards,
J.
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] drivers/pinctrl/pinctrl-at91.c: convert kfree to devm_kfree
2012-12-12 14:22 ` Julia Lawall
2012-12-12 17:01 ` Jean-Christophe PLAGNIOL-VILLARD
@ 2012-12-14 15:46 ` Linus Walleij
1 sibling, 0 replies; 8+ messages in thread
From: Linus Walleij @ 2012-12-14 15:46 UTC (permalink / raw)
To: Julia Lawall
Cc: Jean-Christophe PLAGNIOL-VILLARD, kernel-janitors, linux-kernel,
linux-arm-kernel
On Wed, Dec 12, 2012 at 3:22 PM, Julia Lawall <julia.lawall@lip6.fr> wrote:
> From: Julia Lawall <Julia.Lawall@lip6.fr>
>
> The function at91_dt_node_to_map is ultimately called by the function
> pinctrl_get, which is an exported function. Since it is possible that this
> function is not called from within a probe function, for safety, the kfree
> is converted to a devm_kfree, to both free the data and remove it from the
> device in a failure situation.
>
> Signed-off-by: Julia Lawall <Julia.Lawall@lip6.fr>
>
> ---
> v2: Drop cleanup of the call to devm_kzalloc at the same time.
Thanks, applied with Jean-Christophe's ACK.
Yours,
Linus Walleij
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2012-12-14 15:46 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-12-11 10:58 [PATCH] drivers/pinctrl/pinctrl-at91.c: convert kfree to devm_kfree Julia Lawall
2012-12-11 13:00 ` Sergei Shtylyov
2012-12-11 13:08 ` Julia Lawall
2012-12-11 18:26 ` Sergei Shtylyov
2012-12-12 11:56 ` Jean-Christophe PLAGNIOL-VILLARD
2012-12-12 14:22 ` Julia Lawall
2012-12-12 17:01 ` Jean-Christophe PLAGNIOL-VILLARD
2012-12-14 15:46 ` Linus Walleij
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).