Hi Pavel,

On 5/7/20 7:45 PM, Pavel Machek wrote:
Hi!

This patchset add regulator vibrator driver for MTK Soc. The driver
controls vibrator through regulator's enable and disable.

We'd prefer not to have vibrators in led subsystem.


Xing Zhang (3):
   dt-bindings: add regulator vibrator documentation
   arm64: mediatek: Add regulator vibrator support
   Vibrator: Add regulator vibrator driver

  .../bindings/leds/regulator-vibrator.txt      |  39 ++
  arch/arm64/configs/defconfig                  |   1 +
  drivers/leds/Kconfig                          |  10 +
  drivers/leds/Makefile                         |   1 +
  drivers/leds/regulator-vibrator.c             | 450 ++++++++++++++++++

OTOH, connecting LED to regulator might make some sense. I can take the driver 
with
vibrator functionality stripped, provided it is named the usual way...

We already had an attempt of solving this in more generic way [0],
but you opposed then [1]. Just for the record.

[0] https://lore.kernel.org/linux-leds/20170913175400.42744-1-dtw...@google.com/
[1] https://lore.kernel.org/linux-leds/20170914205804.GA24339@amd/

--
Best regards,
Jacek Anaszewski

Reply via email to