Re: [PATCH] pinctrl: update GPIO range doc
On Mon, Jun 17, 2013 at 7:20 AM, Rob Landley wrote: > On 06/16/2013 05:18:29 AM, Linus Walleij wrote: >> >> From: Linus Walleij >> >> This updates the GPIO range documentation with the API changes >> for sparse/random/arbitrary pin-to-GPIO mappings. > > Acked-by: Rob Landley > > Why isn't this part of the pinctrl series you posted right afterwards? (It > should go in through that maintainer...) Well I'm the pinctrl maintainer so I keep it all in my head :-D >> +in gpio number, and use the ranges internals to retrive a pin number. >> After > > Range's perhaps? Thanks, fixed it up. Yours, Linus Walleij -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] pinctrl: update GPIO range doc
On Sun, Jun 16, 2013 at 12:18:29PM +0200, Linus Walleij wrote: > From: Linus Walleij > > This updates the GPIO range documentation with the API changes > for sparse/random/arbitrary pin-to-GPIO mappings. Oops, sorry, I should have thought of updating this with my previous patch. Thanks for stepping in. Reviewed-by: Christian Ruppert > Cc: Christian Ruppert > Signed-off-by: Linus Walleij > --- > Documentation/pinctrl.txt | 24 +--- > 1 file changed, 21 insertions(+), 3 deletions(-) > > diff --git a/Documentation/pinctrl.txt b/Documentation/pinctrl.txt > index f6e664b..4548743 100644 > --- a/Documentation/pinctrl.txt > +++ b/Documentation/pinctrl.txt > @@ -350,6 +350,23 @@ chip b: > - GPIO range : [48 .. 55] > - pin range : [64 .. 71] > > +The above examples assume the mapping between the GPIOs and pins is > +linear. If the mapping is sparse or haphazard, an array of arbitrary pin > +numbers can be encoded in the range like this: > + > +static const unsigned range_pins[] = { 14, 1, 22, 17, 10, 8, 6, 2 }; > + > +static struct pinctrl_gpio_range gpio_range = { > + .name = "chip", > + .id = 0, > + .base = 32, > + .pins = _pins, > + .npins = ARRAY_SIZE(range_pins), > + .gc = > +}; > + > +In this case the pin_base property will be ignored. > + > When GPIO-specific functions in the pin control subsystem are called, these > ranges will be used to look up the appropriate pin controller by inspecting > and matching the pin to the pin ranges across all controllers. When a > @@ -357,9 +374,9 @@ pin controller handling the matching range is found, > GPIO-specific functions > will be called on that specific pin controller. > > For all functionalities dealing with pin biasing, pin muxing etc, the pin > -controller subsystem will subtract the range's .base offset from the passed > -in gpio number, and add the ranges's .pin_base offset to retrive a pin > number. > -After that, the subsystem passes it on to the pin control driver, so the > driver > +controller subsystem will look up the corresponding pin number from the > passed > +in gpio number, and use the ranges internals to retrive a pin number. After > +that, the subsystem passes it on to the pin control driver, so the driver > will get an pin number into its handled number range. Further it is also > passed > the range ID value, so that the pin controller knows which range it should > deal with. > @@ -368,6 +385,7 @@ Calling pinctrl_add_gpio_range from pinctrl driver is > DEPRECATED. Please see > section 2.1 of Documentation/devicetree/bindings/gpio/gpio.txt on how to bind > pinctrl and gpio drivers. > > + > PINMUX interfaces > = > > -- > 1.7.11.3 > -- Christian Ruppert , /| Tel: +41/(0)22 816 19-42 //| 3, Chemin du Pré-Fleuri _// | bilis Systems CH-1228 Plan-les-Ouates -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] pinctrl: update GPIO range doc
On Sun, Jun 16, 2013 at 12:18:29PM +0200, Linus Walleij wrote: From: Linus Walleij linus.wall...@linaro.org This updates the GPIO range documentation with the API changes for sparse/random/arbitrary pin-to-GPIO mappings. Oops, sorry, I should have thought of updating this with my previous patch. Thanks for stepping in. Reviewed-by: Christian Ruppert christian.rupp...@abilis.com Cc: Christian Ruppert christian.rupp...@abilis.com Signed-off-by: Linus Walleij linus.wall...@linaro.org --- Documentation/pinctrl.txt | 24 +--- 1 file changed, 21 insertions(+), 3 deletions(-) diff --git a/Documentation/pinctrl.txt b/Documentation/pinctrl.txt index f6e664b..4548743 100644 --- a/Documentation/pinctrl.txt +++ b/Documentation/pinctrl.txt @@ -350,6 +350,23 @@ chip b: - GPIO range : [48 .. 55] - pin range : [64 .. 71] +The above examples assume the mapping between the GPIOs and pins is +linear. If the mapping is sparse or haphazard, an array of arbitrary pin +numbers can be encoded in the range like this: + +static const unsigned range_pins[] = { 14, 1, 22, 17, 10, 8, 6, 2 }; + +static struct pinctrl_gpio_range gpio_range = { + .name = chip, + .id = 0, + .base = 32, + .pins = range_pins, + .npins = ARRAY_SIZE(range_pins), + .gc = chip; +}; + +In this case the pin_base property will be ignored. + When GPIO-specific functions in the pin control subsystem are called, these ranges will be used to look up the appropriate pin controller by inspecting and matching the pin to the pin ranges across all controllers. When a @@ -357,9 +374,9 @@ pin controller handling the matching range is found, GPIO-specific functions will be called on that specific pin controller. For all functionalities dealing with pin biasing, pin muxing etc, the pin -controller subsystem will subtract the range's .base offset from the passed -in gpio number, and add the ranges's .pin_base offset to retrive a pin number. -After that, the subsystem passes it on to the pin control driver, so the driver +controller subsystem will look up the corresponding pin number from the passed +in gpio number, and use the ranges internals to retrive a pin number. After +that, the subsystem passes it on to the pin control driver, so the driver will get an pin number into its handled number range. Further it is also passed the range ID value, so that the pin controller knows which range it should deal with. @@ -368,6 +385,7 @@ Calling pinctrl_add_gpio_range from pinctrl driver is DEPRECATED. Please see section 2.1 of Documentation/devicetree/bindings/gpio/gpio.txt on how to bind pinctrl and gpio drivers. + PINMUX interfaces = -- 1.7.11.3 -- Christian Ruppert , christian.rupp...@abilis.com /| Tel: +41/(0)22 816 19-42 //| 3, Chemin du Pré-Fleuri _// | bilis Systems CH-1228 Plan-les-Ouates -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] pinctrl: update GPIO range doc
On Mon, Jun 17, 2013 at 7:20 AM, Rob Landley r...@landley.net wrote: On 06/16/2013 05:18:29 AM, Linus Walleij wrote: From: Linus Walleij linus.wall...@linaro.org This updates the GPIO range documentation with the API changes for sparse/random/arbitrary pin-to-GPIO mappings. Acked-by: Rob Landley r...@landley.net Why isn't this part of the pinctrl series you posted right afterwards? (It should go in through that maintainer...) Well I'm the pinctrl maintainer so I keep it all in my head :-D +in gpio number, and use the ranges internals to retrive a pin number. After Range's perhaps? Thanks, fixed it up. Yours, Linus Walleij -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] pinctrl: update GPIO range doc
On 06/16/2013 05:18:29 AM, Linus Walleij wrote: From: Linus Walleij This updates the GPIO range documentation with the API changes for sparse/random/arbitrary pin-to-GPIO mappings. Acked-by: Rob Landley Why isn't this part of the pinctrl series you posted right afterwards? (It should go in through that maintainer...) For all functionalities dealing with pin biasing, pin muxing etc, the pin -controller subsystem will subtract the range's .base offset from the passed -in gpio number, and add the ranges's .pin_base offset to retrive a pin number. -After that, the subsystem passes it on to the pin control driver, so the driver +controller subsystem will look up the corresponding pin number from the passed +in gpio number, and use the ranges internals to retrive a pin number. After Range's perhaps? Rob-- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] pinctrl: update GPIO range doc
On 06/16/2013 05:18:29 AM, Linus Walleij wrote: From: Linus Walleij linus.wall...@linaro.org This updates the GPIO range documentation with the API changes for sparse/random/arbitrary pin-to-GPIO mappings. Acked-by: Rob Landley r...@landley.net Why isn't this part of the pinctrl series you posted right afterwards? (It should go in through that maintainer...) For all functionalities dealing with pin biasing, pin muxing etc, the pin -controller subsystem will subtract the range's .base offset from the passed -in gpio number, and add the ranges's .pin_base offset to retrive a pin number. -After that, the subsystem passes it on to the pin control driver, so the driver +controller subsystem will look up the corresponding pin number from the passed +in gpio number, and use the ranges internals to retrive a pin number. After Range's perhaps? Rob-- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/