linux-stable.git/drivers/pinctrl/core.c, branch linux-3.13.y

pinctrl: protect pinctrl_list add

2014-02-20T19:10:11+00:00

commit 7b320cb1ed2dbd2c5f2a778197baf76fd6bf545a upstream.

We have few fedora bug reports about list corruption on pinctrl,
for example:
https://bugzilla.redhat.com/show_bug.cgi?id=1051918

Most likely corruption happen due lack of protection of pinctrl_list
when adding new nodes to it. Patch corrects that.

Fixes: 42fed7ba44e ("pinctrl: move subsystem mutex to pinctrl_dev struct")
Signed-off-by: Stanislaw Gruszka 
Acked-by: Stephen Warren 
Signed-off-by: Linus Walleij 
Signed-off-by: Greg Kroah-Hartman

pinctrl/gpio: non-linear GPIO ranges accesible from gpiolib

2013-10-16T13:33:50+00:00

This patch adds the infrastructure required to register non-linear gpio
ranges through gpiolib and the standard GPIO device tree bindings.

Signed-off-by: Christian Ruppert 
Signed-off-by: Linus Walleij

Merge tag 'v3.11-rc7' into devel

2013-08-29T07:46:30+00:00

Merged in this to avoid conflicts with the big locking fixes
from upstream.

Signed-off-by: Linus Walleij 

Conflicts:
	drivers/pinctrl/pinctrl-sunxi.c

pinctrl: core: Add proper mutex lock in pinctrl_request_gpio

2013-08-28T11:13:05+00:00

This one is missed in commit 42fed7ba "pinctrl: move subsystem mutex to
pinctrl_dev struct".

I think this fixes the race between pin_free() and pin_request() calls.
It protects accessing the members of pctldev->desc.
(e.g. update desc->mux_usecount, desc->gpio_owner, desc->mux_owner, etc)
Current code grabs pctldev->mutex before calling pinmux_free_gpio(),
but did not grab the mutex while calling pinmux_request_gpio().

Signed-off-by: Axel Lin 
Signed-off-by: Linus Walleij

pinctrl: core: Remove unnecessary test for desc->name

2013-08-23T06:56:29+00:00

The implementation in pinctrl_register_one_pin() ensures pindesc->name is always
not NULL before insert the pindesc to radix tree.
If the desc return from pin_desc_get is not NULL, desc->name is always not NULL.

Signed-off-by: Axel Lin 
Signed-off-by: Linus Walleij

pinctrl: core: Hold pctldev->mutex mutex lock while traversing gpio_ranges list

2013-08-23T06:56:28+00:00

Hold pctldev->mutex mutex_lock when traverse the list.

Signed-off-by: Axel Lin 
Signed-off-by: Linus Walleij

pinctrl: core: Hold pinctrldev_list_mutex mutex while traversing pinctrldev_list

2013-08-23T06:56:27+00:00

This one is missed in commit 44d5f7bb "pinctrl: sink pinctrldev_list_mutex".

Signed-off-by: Axel Lin 
Signed-off-by: Linus Walleij

pinctrl: fix a memleak when freeing maps

2013-07-25T15:18:36+00:00

We forgot to free the node itself when free:ing a map.

Reported-by: xulinuxkernel 
Reviewed-by: Stephen Warren 
Signed-off-by: Linus Walleij

pinctrl: Remove duplicate code in pinctrl_pm_select_state functions

2013-07-22T23:23:27+00:00

There's no need to duplicate essentially the same functions. Let's
introduce static int pinctrl_pm_select_state() and make the other
related functions call that.

This allows us to add support later on for multiple active states,
and more optimized dynamic remuxing.

Note that we still need to export the various pinctrl_pm_select
functions as we want to keep struct pinctrl_state private to the
pinctrl code, and cannot replace those with inline functions.

Cc: Felipe Balbi 
Cc: Stephen Warren 
Signed-off-by: Grygorii Strashko 
Signed-off-by: Tony Lindgren 
Signed-off-by: Linus Walleij

pinctrl: Staticize local symbols

2013-06-18T14:54:45+00:00

Symbols referenced only in this file are made static.

Signed-off-by: Sachin Kamat 
Cc: Patrice Chotard 
Signed-off-by: Linus Walleij