Thread (31 messages) flat view 31 messages, 6 authors, 2018-03-01

[PATCH 5/6] ARM: dts: stm32: add joystick support on stm32746g-eval

From: Linus Walleij <hidden>
Date: 2018-02-22 13:52:49
Also in: linux-devicetree, linux-gpio, lkml

Hi Amelie,

thanks for your patch.

On Thu, Feb 8, 2018 at 3:27 PM, Amelie Delaunay [off-list ref] wrote:
The joystick on stm32746g-eval uses gpios on MFX gpio expander.

Signed-off-by: Amelie Delaunay <redacted>
(...)
+       joystick {
+               compatible = "gpio-keys";
+               #address-cells = <1>;
+               #size-cells = <0>;
+               button at 1 {
+                       label = "JoySel";
+                       linux,code = <KEY_ENTER>;
+                       gpios = <&mfxgpio 0 (GPIO_ACTIVE_LOW | GPIO_IN_PUSH_PULL | GPIO_PULL_UP)>;
+               };
As I think this should not all be handled by GPIO, the joystick on
gpio-keys needs a pin control handle pointing back to pin control
states on the respective pins in the pin control/GPIO driver,
where it can set up these properties.

For the individual lines the pushing and pulling flags should be
removed.

Yours,
Linus Walleij
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help