Re: question about drivers/pinctrl/pinctrl-at91.c

2012-12-11 Thread Julia Lawall
On Tue, 11 Dec 2012, Linus Walleij wrote: > On Tue, Dec 11, 2012 at 10:04 AM, Julia Lawall wrote: > > On Tue, 11 Dec 2012, Linus Walleij wrote: > > >> I was under the impression that if you exit the probe function > >> with a negative value anything allocated with devm_* was freed > >> immediatel

Re: question about drivers/pinctrl/pinctrl-at91.c

2012-12-11 Thread Linus Walleij
On Tue, Dec 11, 2012 at 10:04 AM, Julia Lawall wrote: > On Tue, 11 Dec 2012, Linus Walleij wrote: >> I was under the impression that if you exit the probe function >> with a negative value anything allocated with devm_* was freed >> immediately, that is atleast how it's described in >> Documentat

Re: question about drivers/pinctrl/pinctrl-at91.c

2012-12-11 Thread Grant Likely
On Sat, 8 Dec 2012 16:52:42 +0100 (CET), Julia Lawall wrote: > The function at91_dt_node_to_map in drivers/pinctrl/pinctrl-at91.c > contains the following code: > > new_map = devm_kzalloc(pctldev->dev, sizeof(*new_map) * map_num, > GFP_KERNEL); > if (!new_map) >

Re: question about drivers/pinctrl/pinctrl-at91.c

2012-12-11 Thread Julia Lawall
On Tue, 11 Dec 2012, Linus Walleij wrote: > On Sat, Dec 8, 2012 at 4:52 PM, Julia Lawall wrote: > > > The function at91_dt_node_to_map in drivers/pinctrl/pinctrl-at91.c contains > > the following code: > > > >new_map = devm_kzalloc(pctldev->dev, sizeof(*new_map) * map_num, > > GFP_KERNEL)

Re: question about drivers/pinctrl/pinctrl-at91.c

2012-12-11 Thread Linus Walleij
On Sat, Dec 8, 2012 at 4:52 PM, Julia Lawall wrote: > The function at91_dt_node_to_map in drivers/pinctrl/pinctrl-at91.c contains > the following code: > >new_map = devm_kzalloc(pctldev->dev, sizeof(*new_map) * map_num, > GFP_KERNEL); > if (!new_map) > return -ENOM

question about drivers/pinctrl/pinctrl-at91.c

2012-12-08 Thread Julia Lawall
The function at91_dt_node_to_map in drivers/pinctrl/pinctrl-at91.c contains the following code: new_map = devm_kzalloc(pctldev->dev, sizeof(*new_map) * map_num, GFP_KERNEL); if (!new_map) return -ENOMEM; *map = new_map; *num_maps = map_num;