| From: |
| Nuno Sá via B4 Relay <devnull+nuno.sa.analog.com-AT-kernel.org> |
| To: |
| linux-gpio-AT-vger.kernel.org, linux-pwm-AT-vger.kernel.org, devicetree-AT-vger.kernel.org, linux-input-AT-vger.kernel.org |
| Subject: |
| [PATCH 00/18] mfd: adp5585: support keymap events and drop legacy Input driver |
| Date: |
| Thu, 13 Mar 2025 14:19:17 +0000 |
| Message-ID: |
| <20250313-dev-adp5589-fw-v1-0-20e80d4bd4ea@analog.com> |
| Cc: |
| Lee Jones <lee-AT-kernel.org>, Rob Herring <robh-AT-kernel.org>, Krzysztof Kozlowski <krzk+dt-AT-kernel.org>, Conor Dooley <conor+dt-AT-kernel.org>, Uwe Kleine-König <ukleinek-AT-kernel.org>, Linus Walleij <linus.walleij-AT-linaro.org>, Bartosz Golaszewski <brgl-AT-bgdev.pl>, Dmitry Torokhov <dmitry.torokhov-AT-gmail.com>, Laurent Pinchart <laurent.pinchart-AT-ideasonboard.com>, Liu Ying <victor.liu-AT-nxp.com> |
| Archive-link: |
| Article |
The adp5585 MFD driver was introduced in 6.11 adding support for gpio
and PWM. However, the gpio part of it was already supported as part of
the keyboard driver:
https://elixir.bootlin.com/linux/v6.14-rc6/source/drivers...
On top of that it also overlapped with my refactoring of the above driver [1]
to drop usage of platform data and use FW properties instead.
Now, it actually makes sense for this device to be supported under MFD
and since the "legacy" input device depends on platform data that is not
defined anywhere the plan in this series is to add support for the
keyboard and adp5589 devices as part of the MFD driver. Once the MFD
driver supports all that's supported in the Input one, we drop it...
For DT Maintainers:
The compatible for adp5589 is part of trivial devices. To me, it makes
sense to remove it in the patch where we drop the driver but doing so
would result in a warning when adding the same compatible for the MFD
bindings. Hence, I remove it in that patch. Is that ok?
Uwe:
In my eval board, I could see that reading the GPIO value (when
configured as input) does not work when OSC_EN is not set. Therefore,
commit ("pwm: adp5585: don't control OSC_EN in the pwm driver") could
very well have a Fixes tag. However I'm not 100% sure it's a real issue
or something special to my eval board.
It would be nice if Laurent or Liu could test the PWM bits or even
check that the above is also an issue for their platform.
[1]: https://lore.kernel.org/linux-input/d1395bd61ce58b3734121...
---
Nuno Sá (18):
dt-bindings: mfd: adp5585: ease on the required properties
mfd: adp5585: enable oscilator during probe
pwm: adp5585: don't control OSC_EN in the pwm driver
mfd: adp5585: make use of MFD_CELL_NAME()
dt-bindings: mfd: adp5585: document adp5589 I/O expander
mfd: adp5585: add support for adp5589
gpio: adp5585: add support for the ad5589 expander
pwm: adp5585: add support for adp5589
dt-bindings: mfd: adp5585: add properties for input events
mfd: adp5585: add support for key events
gpio: adp5585: support gpi events
Input: adp5585: Add Analog Devices ADP5585/89 support
Input: adp5589: remove the driver
mfd: adp5585: support getting vdd regulator
dt-bindings: mfd: adp5585: document reset gpio
mfd: adp5585: add support for a reset pin
pwm: adp5585: make sure to include mod_devicetable.h
gpio: adp5585: make sure to include mod_devicetable.h
.../devicetree/bindings/mfd/adi,adp5585.yaml | 243 ++++-
.../devicetree/bindings/trivial-devices.yaml | 2 -
MAINTAINERS | 1 +
drivers/gpio/Kconfig | 1 +
drivers/gpio/gpio-adp5585.c | 299 +++++-
drivers/input/keyboard/Kconfig | 21 +-
drivers/input/keyboard/Makefile | 2 +-
drivers/input/keyboard/adp5585-keys.c | 221 ++++
drivers/input/keyboard/adp5589-keys.c | 1066 --------------------
drivers/mfd/adp5585.c | 799 ++++++++++++++-
drivers/pwm/pwm-adp5585.c | 57 +-
include/linux/mfd/adp5585.h | 153 ++-
12 files changed, 1703 insertions(+), 1162 deletions(-)
---
base-commit: 4d395cb071a343196ca524d3694790f06978fe91
change-id: 20250311-dev-adp5589-fw-e04cfd945286
--
Thanks!
- Nuno Sá