linux-gpio.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH v4 00/20] mfd: adp5585: support keymap events and drop legacy Input driver
@ 2025-05-21 13:02 Nuno Sá via B4 Relay
  2025-05-21 13:02 ` [PATCH v4 01/20] dt-bindings: mfd: adp5585: ease on the required properties Nuno Sá via B4 Relay
                   ` (19 more replies)
  0 siblings, 20 replies; 45+ messages in thread
From: Nuno Sá via B4 Relay @ 2025-05-21 13:02 UTC (permalink / raw)
  To: linux-gpio, linux-pwm, devicetree, linux-input
  Cc: Lee Jones, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
	Uwe Kleine-König, Linus Walleij, Bartosz Golaszewski,
	Dmitry Torokhov, Laurent Pinchart, Liu Ying, Bartosz Golaszewski,
	Krzysztof Kozlowski

Hi all,

Here it goes v4. Main changes is to drop chip info based struct and
directly use an enum in the FW .data pointer, use the notifier API for
dispatching events and multiple calls to mfd_add_devices().

Regarding the last point, I think I could have used multiple calls to
devm_mfd_add_devices() and avoid those gotos in adp5585_add_devices()
but I do not feel that would have been "correct".

Thanks!
- Nuno Sá

---
Changes in v4:
 - Patch 2:
   * Directly use mfd_add_devices() to individually add devices

 - Patch 3:
   * Don't break lines at 80;
   * Add comment suggested by Laurent;
   * PWM change squashed in this patch.

 - Patch 5:
   * Pass an enum to the of_id data pointer;
   * Reference the regmap defaults directly from the variant enum;
   * Drop chip_info struct and place data directly in struct adp5585_dev.

 - Patch 6:
   * Adapt to the new way of passing per variant data;
   * Rename adp5585_fill_regmap_config() to adp5585_fill_variant_config();
   * Add struct device now to struct adp5585_dev;
   * Pass adp5585_dev to adp5585_add_devices().

 - Patch 7:
   * Add regs directly in struct adp5585_dev.

 - Patch 9:
   * Moved the per variant gpio register into the gpio driver;
   * Moved ADP558[59]_GPIO_{BANK_BIT} into the gpio driver;
   * Moved ADP5589_GPIO_MAX and dropped the max_{col|row}.

 - Patch 10:
   * Adapt to the lack of a chip_info pointer when getting ext_cfg.

 - Patch 12:
   * Refactor parsing of poll-interval;
   * Make use of the notifier API instead of "own" dispatcher mechanism;
   * Improve comments in the code.

 - Patch 13:
   * Add more comments;
   * Drop the function callback for validating events;
   * Renamed has_pin5 to has_pin6;
 
 - Patch 14:
   * Adapt to the new way of adding MFD cells;
   * Improve comments.

 - Patch 15:
   * Use the notifier API and adapt to that.

 - Patch 16:
   * Use the notifier API and adapt to that;
   * Move devm_add_action_or_reset() before the for() loop setting the
     bits in kpad->keypad;
   * Address Dmitry comment about checking for error instead of
     directly returning when calling adp5585_keys_check_special_events().

- Link to v3: https://lore.kernel.org/r/20250512-dev-adp5589-fw-v3-0-092b14b79a88@analog.com
- Link to v2: https://lore.kernel.org/r/20250415-dev-adp5589-fw-v2-0-3a799c3ed812@analog.com
- Link to v1: https://lore.kernel.org/r/20250313-dev-adp5589-fw-v1-0-20e80d4bd4ea@analog.com

---
Nuno Sá (20):
      dt-bindings: mfd: adp5585: ease on the required properties
      mfd: adp5585: only add devices given in FW
      mfd: adp5585: enable oscilator during probe
      mfd: adp5585: make use of MFD_CELL_NAME()
      dt-bindings: mfd: adp5585: document adp5589 I/O expander
      mfd: adp5585: refactor how regmap defaults are handled
      mfd: adp5585: add support for adp5589
      mfd: adp5585: add a per chip reg struture
      gpio: adp5585: add support for the adp5589 expander
      pwm: adp5585: add support for adp5589
      dt-bindings: mfd: adp5585: add properties for input events
      mfd: adp5585: add support for event handling
      mfd: adp5585: support reset and unlock events
      mfd: adp5585: add support for input devices
      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

 .../devicetree/bindings/mfd/adi,adp5585.yaml       |  240 ++++-
 .../devicetree/bindings/trivial-devices.yaml       |    2 -
 MAINTAINERS                                        |    1 +
 drivers/gpio/Kconfig                               |    1 +
 drivers/gpio/gpio-adp5585.c                        |  364 ++++++-
 drivers/input/keyboard/Kconfig                     |   21 +-
 drivers/input/keyboard/Makefile                    |    2 +-
 drivers/input/keyboard/adp5585-keys.c              |  371 +++++++
 drivers/input/keyboard/adp5589-keys.c              | 1066 --------------------
 drivers/mfd/adp5585.c                              |  751 +++++++++++++-
 drivers/pwm/pwm-adp5585.c                          |   78 +-
 include/linux/mfd/adp5585.h                        |  116 ++-
 12 files changed, 1804 insertions(+), 1209 deletions(-)
---
base-commit: 407f60a151df3c44397e5afc0111eb9b026c38d3
change-id: 20250311-dev-adp5589-fw-e04cfd945286
--

Thanks!
- Nuno Sá



^ permalink raw reply	[flat|nested] 45+ messages in thread

end of thread, other threads:[~2025-06-13 13:27 UTC | newest]

Thread overview: 45+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-05-21 13:02 [PATCH v4 00/20] mfd: adp5585: support keymap events and drop legacy Input driver Nuno Sá via B4 Relay
2025-05-21 13:02 ` [PATCH v4 01/20] dt-bindings: mfd: adp5585: ease on the required properties Nuno Sá via B4 Relay
2025-05-21 13:02 ` [PATCH v4 02/20] mfd: adp5585: only add devices given in FW Nuno Sá via B4 Relay
2025-05-23 14:51   ` Lee Jones
2025-05-23 15:07     ` Nuno Sá
2025-05-23 15:19       ` Lee Jones
2025-05-23 15:45         ` Nuno Sá
2025-06-12 14:17           ` Lee Jones
2025-05-21 13:02 ` [PATCH v4 03/20] mfd: adp5585: enable oscilator during probe Nuno Sá via B4 Relay
2025-06-12 14:20   ` Lee Jones
2025-06-12 14:40     ` Nuno Sá
2025-06-12 15:20       ` Lee Jones
2025-06-13  9:43         ` Nuno Sá
2025-05-21 13:02 ` [PATCH v4 04/20] mfd: adp5585: make use of MFD_CELL_NAME() Nuno Sá via B4 Relay
2025-05-23 14:53   ` Lee Jones
2025-05-21 13:02 ` [PATCH v4 05/20] dt-bindings: mfd: adp5585: document adp5589 I/O expander Nuno Sá via B4 Relay
2025-05-21 13:02 ` [PATCH v4 06/20] mfd: adp5585: refactor how regmap defaults are handled Nuno Sá via B4 Relay
2025-05-23 15:03   ` Lee Jones
2025-05-27 10:08     ` Nuno Sá
2025-05-21 13:02 ` [PATCH v4 07/20] mfd: adp5585: add support for adp5589 Nuno Sá via B4 Relay
2025-05-23 15:15   ` Lee Jones
2025-05-27 10:20     ` Nuno Sá
2025-05-21 13:02 ` [PATCH v4 08/20] mfd: adp5585: add a per chip reg struture Nuno Sá via B4 Relay
2025-06-12 14:22   ` Lee Jones
2025-05-21 13:02 ` [PATCH v4 09/20] gpio: adp5585: add support for the adp5589 expander Nuno Sá via B4 Relay
2025-05-21 13:03 ` [PATCH v4 10/20] pwm: adp5585: add support for adp5589 Nuno Sá via B4 Relay
2025-05-21 13:03 ` [PATCH v4 11/20] dt-bindings: mfd: adp5585: add properties for input events Nuno Sá via B4 Relay
2025-05-21 13:03 ` [PATCH v4 12/20] mfd: adp5585: add support for event handling Nuno Sá via B4 Relay
2025-06-12 14:29   ` Lee Jones
2025-05-21 13:03 ` [PATCH v4 13/20] mfd: adp5585: support reset and unlock events Nuno Sá via B4 Relay
2025-06-12 14:55   ` Lee Jones
2025-06-13  9:48     ` Nuno Sá
2025-06-13 13:07       ` Lee Jones
2025-06-13 13:13         ` Nuno Sá
2025-06-13 13:27           ` Lee Jones
2025-05-21 13:03 ` [PATCH v4 14/20] mfd: adp5585: add support for input devices Nuno Sá via B4 Relay
2025-06-12 15:16   ` Lee Jones
2025-05-21 13:03 ` [PATCH v4 15/20] gpio: adp5585: support gpi events Nuno Sá via B4 Relay
2025-05-21 13:03 ` [PATCH v4 16/20] Input: adp5585: Add Analog Devices ADP5585/89 support Nuno Sá via B4 Relay
2025-05-21 13:03 ` [PATCH v4 17/20] Input: adp5589: remove the driver Nuno Sá via B4 Relay
2025-05-21 13:03 ` [PATCH v4 18/20] mfd: adp5585: support getting vdd regulator Nuno Sá via B4 Relay
2025-06-12 15:17   ` Lee Jones
2025-05-21 13:03 ` [PATCH v4 19/20] dt-bindings: mfd: adp5585: document reset gpio Nuno Sá via B4 Relay
2025-05-21 13:03 ` [PATCH v4 20/20] mfd: adp5585: add support for a reset pin Nuno Sá via B4 Relay
2025-06-12 15:18   ` Lee Jones

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).