linux-toradex.git/drivers/pinctrl/pinctrl-st.c, branch v4.9.123

pinctrl: st: add irq_request/release_resources callbacks

2017-12-25T13:23:39+00:00

[ Upstream commit e855fa9a65c40788b5069abb0d094537daa22e05 ]

When using GPIO as IRQ source, the GPIO must be configured
in INPUT. Callbacks dedicated for this was missing in
pinctrl-st driver.

This fix the following kernel error when trying to lock a gpio
as IRQ:

[    7.521095] gpio gpiochip7: (PIO11): gpiochip_lock_as_irq: tried to flag a GPIO set as output for IRQ
[    7.526018] gpio gpiochip7: (PIO11): unable to lock HW IRQ 6 for IRQ
[    7.529405] genirq: Failed to request resources for 0-0053 (irq 81) on irqchip GPIO

Signed-off-by: Patrice Chotard 
Signed-off-by: Linus Walleij 
Signed-off-by: Sasha Levin 
Signed-off-by: Greg Kroah-Hartman

pinctrl: st: don't specify default interrupt trigger

2016-10-31T20:25:15+00:00

Thanks to 332e99d5ae4 which now alerts of default
trigger usage when configuring interrupts.

Signed-off-by: Patrice Chotard 
Acked-by: Peter Griffin 
Signed-off-by: Linus Walleij

pinctrl: st: Remove STiH415/6 SoC pinctrl driver support.

2016-09-15T13:55:25+00:00

STiH415/6 SoC support is being removed from the kernel.
This patch updates the ST pinctrl driver and removes
references to these obsolete platforms. As some structures
referenced by STiH407 based configuration were shared with
STiH416 we update these names to match the remaining
supported platform.

Signed-off-by: Peter Griffin 
Cc: 
Cc: 
Signed-off-by: Linus Walleij

pinctrl: st: Use second parameter to gpio specifier

2016-08-11T14:05:45+00:00

This patch allows to use the second parameter of gpio
specifier, which is used to specify whether the gpio is
active high or low.

Simply remove specific of_xlate callback() and of_gpio_n_cells
and use default one set by of_gpiochip_add() which allows
to use second parameter gpio specifier.

Signed-off-by: Maxime Coquelin 
Signed-off-by: Patrice Chotard 
Signed-off-by: Linus Walleij

pinctrl: fix incorrect inline keyword in multiple drivers

2016-06-15T06:37:41+00:00

When building with 'make W=1', we get harmless warnings about
five drivers in drivers/pinctrl, which all contain a copy of
the same line:

drivers/pinctrl/freescale/pinctrl-imx1-core.c:160:1: error: 'inline' is not at beginning of declaration [-Werror=old-style-declaration]

This replaces the somewhat nonstandard 'static const inline'
with 'static inline const', which has the same meaning but
does not cause this warning.

Signed-off-by: Arnd Bergmann 
Signed-off-by: Linus Walleij

pinctrl: st: Use devm_pinctrl_register() for pinctrl registration

2016-04-21T07:23:25+00:00

Use devm_pinctrl_register() for pin control registration.

Signed-off-by: Laxman Dewangan 
Cc: Srinivas Kandagatla 
Cc: Maxime Coquelin 
Cc: Patrice Chotard 
Cc: kernel@stlinux.com
Signed-off-by: Linus Walleij

pinctrl: st: activate strict mux mode

2016-01-27T10:02:53+00:00

This activates strict mode muxing for the ST pin controllers,
as these do not allow GPIO and functions to use the same pin
simultaneously.

Signed-off-by: Patrice Chotard 
Signed-off-by: Linus Walleij

pinctrl: st: use gpiochip data pointer

2016-01-05T13:14:57+00:00

This makes the driver use the data pointer added to the gpio_chip
to store a pointer to the state container instead of relying on
container_of().

Cc: Srinivas Kandagatla 
Cc: Patrice Chotard 
Acked-by: Maxime Coquelin 
Signed-off-by: Linus Walleij

gpio: change member .dev to .parent

2015-11-19T08:24:35+00:00

The name .dev in a struct is normally reserved for a struct device
that is let us say a superclass to the thing described by the struct.
struct gpio_chip stands out by confusingly using a struct device *dev
to point to the parent device (such as a platform_device) that
represents the hardware. As we want to give gpio_chip:s real devices,
this is not working. We need to rename this member to parent.

This was done by two coccinelle scripts, I guess it is possible to
combine them into one, but I don't know such stuff. They look like
this:

@@
struct gpio_chip *var;
@@
-var->dev
+var->parent

and:

@@
struct gpio_chip var;
@@
-var.dev
+var.parent

and:

@@
struct bgpio_chip *var;
@@
-var->gc.dev
+var->gc.parent

Plus a few instances of bgpio that I couldn't figure out how
to teach Coccinelle to rewrite.

This patch hits all over the place, but I *strongly* prefer this
solution to any piecemal approaches that just exercise patch
mechanics all over the place. It mainly hits drivers/gpio and
drivers/pinctrl which is my own backyard anyway.

Cc: Haavard Skinnemoen 
Cc: Rafał Miłecki 
Cc: Richard Purdie 
Cc: Mauro Carvalho Chehab 
Cc: Alek Du 
Cc: Jaroslav Kysela 
Cc: Takashi Iwai 
Acked-by: Dmitry Torokhov 
Acked-by: Greg Kroah-Hartman 
Acked-by: Lee Jones 
Acked-by: Jiri Kosina 
Acked-by: Hans-Christian Egtvedt 
Acked-by: Jacek Anaszewski 
Signed-off-by: Linus Walleij

pinctrl: replace trivial implementations of gpio_chip request/free

2015-10-16T20:20:21+00:00

Replace all trivial request/free callbacks that do nothing but call into
pinctrl code with the generic versions.

Signed-off-by: Jonas Gorski 
Acked-by: Bjorn Andersson 
Acked-by: Heiko Stuebner 
Acked-by: Eric Anholt 
Acked-by: Mika Westerberg 
Acked-by: Andrew Bresticker 
Acked-by: Baruch Siach 
Acked-by: Matthias Brugger 
Acked-by: Lee Jones 
Acked-by: Laxman Dewangan 
Acked-by: Maxime Ripard 
Signed-off-by: Linus Walleij