Re: [PATCH v2] pinctrl: reserve pins when states are activated

2012-10-23 Thread Linus Walleij
On Mon, Oct 22, 2012 at 10:30 PM, Stephen Warren wrote: > On 10/22/2012 02:21 AM, Linus Walleij wrote: >> If this turns out to be a severe performance bottleneck, I >> suggest to add some additional constraint API, like >> pinctrl_set_pinmux_homegeneous_pinsets(true) that will >> at runtime

Re: [PATCH v2] pinctrl: reserve pins when states are activated

2012-10-23 Thread Linus Walleij
On Mon, Oct 22, 2012 at 9:07 PM, Tony Lindgren wrote: > * Linus Walleij [121022 01:22]: >> If this turns out to be a severe performance bottleneck, I >> suggest to add some additional constraint API, like >> pinctrl_set_pinmux_homegeneous_pinsets(true) that will >> at runtime select whether the

Re: [PATCH v2] pinctrl: reserve pins when states are activated

2012-10-23 Thread Linus Walleij
On Mon, Oct 22, 2012 at 9:07 PM, Tony Lindgren t...@atomide.com wrote: * Linus Walleij linus.wall...@linaro.org [121022 01:22]: If this turns out to be a severe performance bottleneck, I suggest to add some additional constraint API, like pinctrl_set_pinmux_homegeneous_pinsets(true) that will

Re: [PATCH v2] pinctrl: reserve pins when states are activated

2012-10-23 Thread Linus Walleij
On Mon, Oct 22, 2012 at 10:30 PM, Stephen Warren swar...@wwwdotorg.org wrote: On 10/22/2012 02:21 AM, Linus Walleij wrote: If this turns out to be a severe performance bottleneck, I suggest to add some additional constraint API, like pinctrl_set_pinmux_homegeneous_pinsets(true) that will at

Re: [PATCH v2] pinctrl: reserve pins when states are activated

2012-10-22 Thread Stephen Warren
On 10/22/2012 02:21 AM, Linus Walleij wrote: > On Fri, Oct 19, 2012 at 8:10 PM, Tony Lindgren wrote: >> [Me] >>> Instead: let use reserve the pins when the state is activated >>> and drop them when the state is disabled, i.e. when we move to >>> another state. This way different devices/functions

Re: [PATCH v2] pinctrl: reserve pins when states are activated

2012-10-22 Thread Tony Lindgren
* Linus Walleij [121022 01:22]: > On Fri, Oct 19, 2012 at 8:10 PM, Tony Lindgren wrote: > > [Me] > >> Instead: let use reserve the pins when the state is activated > >> and drop them when the state is disabled, i.e. when we move to > >> another state. This way different devices/functions can use

Re: [PATCH v2] pinctrl: reserve pins when states are activated

2012-10-22 Thread Linus Walleij
On Fri, Oct 19, 2012 at 8:10 PM, Tony Lindgren wrote: > [Me] >> Instead: let use reserve the pins when the state is activated >> and drop them when the state is disabled, i.e. when we move to >> another state. This way different devices/functions can use the >> same pins at different times. > >

Re: [PATCH v2] pinctrl: reserve pins when states are activated

2012-10-22 Thread Linus Walleij
On Fri, Oct 19, 2012 at 8:10 PM, Tony Lindgren t...@atomide.com wrote: [Me] Instead: let use reserve the pins when the state is activated and drop them when the state is disabled, i.e. when we move to another state. This way different devices/functions can use the same pins at different

Re: [PATCH v2] pinctrl: reserve pins when states are activated

2012-10-22 Thread Tony Lindgren
* Linus Walleij linus.wall...@linaro.org [121022 01:22]: On Fri, Oct 19, 2012 at 8:10 PM, Tony Lindgren t...@atomide.com wrote: [Me] Instead: let use reserve the pins when the state is activated and drop them when the state is disabled, i.e. when we move to another state. This way

Re: [PATCH v2] pinctrl: reserve pins when states are activated

2012-10-22 Thread Stephen Warren
On 10/22/2012 02:21 AM, Linus Walleij wrote: On Fri, Oct 19, 2012 at 8:10 PM, Tony Lindgren t...@atomide.com wrote: [Me] Instead: let use reserve the pins when the state is activated and drop them when the state is disabled, i.e. when we move to another state. This way different

Re: [PATCH v2] pinctrl: reserve pins when states are activated

2012-10-19 Thread Tony Lindgren
* Linus Walleij [121019 06:09]: > > Instead: let use reserve the pins when the state is activated > and drop them when the state is disabled, i.e. when we move to > another state. This way different devices/functions can use the > same pins at different times. Hmm doesn't this mean that we are

Re: [PATCH v2] pinctrl: reserve pins when states are activated

2012-10-19 Thread Stephen Warren
On 10/19/2012 07:05 AM, Linus Walleij wrote: > From: Linus Walleij > > This switches the way that pins are reserved for multiplexing: > > We used to do this when the map was parsed, at the creation of > the settings inside the pinctrl handle, in pinmux_map_to_setting(). > > However this does

Re: [PATCH v2] pinctrl: reserve pins when states are activated

2012-10-19 Thread Jean-Nicolas GRAUX
Tested-by: Jean Nicolas Graux Le 10/19/2012 03:05 PM, Linus WALLEIJ a écrit : From: Linus Walleij This switches the way that pins are reserved for multiplexing: We used to do this when the map was parsed, at the creation of the settings inside the pinctrl handle, in pinmux_map_to_setting().

[PATCH v2] pinctrl: reserve pins when states are activated

2012-10-19 Thread Linus Walleij
From: Linus Walleij This switches the way that pins are reserved for multiplexing: We used to do this when the map was parsed, at the creation of the settings inside the pinctrl handle, in pinmux_map_to_setting(). However this does not work for us, because we want to use the same set of pins

[PATCH v2] pinctrl: reserve pins when states are activated

2012-10-19 Thread Linus Walleij
From: Linus Walleij linus.wall...@linaro.org This switches the way that pins are reserved for multiplexing: We used to do this when the map was parsed, at the creation of the settings inside the pinctrl handle, in pinmux_map_to_setting(). However this does not work for us, because we want to

Re: [PATCH v2] pinctrl: reserve pins when states are activated

2012-10-19 Thread Jean-Nicolas GRAUX
Tested-by: Jean Nicolas Graux jean-nicolas.gr...@stericsson.com Le 10/19/2012 03:05 PM, Linus WALLEIJ a écrit : From: Linus Walleijlinus.wall...@linaro.org This switches the way that pins are reserved for multiplexing: We used to do this when the map was parsed, at the creation of the

Re: [PATCH v2] pinctrl: reserve pins when states are activated

2012-10-19 Thread Stephen Warren
On 10/19/2012 07:05 AM, Linus Walleij wrote: From: Linus Walleij linus.wall...@linaro.org This switches the way that pins are reserved for multiplexing: We used to do this when the map was parsed, at the creation of the settings inside the pinctrl handle, in pinmux_map_to_setting().

Re: [PATCH v2] pinctrl: reserve pins when states are activated

2012-10-19 Thread Tony Lindgren
* Linus Walleij linus.wall...@stericsson.com [121019 06:09]: Instead: let use reserve the pins when the state is activated and drop them when the state is disabled, i.e. when we move to another state. This way different devices/functions can use the same pins at different times. Hmm doesn't