Re: [PATCH 0/3] u8500: led and backlight driver

[Mike Rapoport]

Arun Murthy wrote:
> Arun Murthy (3):
>   mfd: ab8500-pwm: ab8500 pwm management
>   backlight: u8500: backlight driver based on ab8500 pwm
>   leds: ab8500-led: led driver based on ab8500 pwm

There are drivers/leds/leds-pwm.c and drivers/video/backlight/pwm_bl.c that 
implement led and backlight drivers based on generic pwm interface.
Have you considered using those?

>  drivers/leds/Kconfig                  |    9 ++
>  drivers/leds/Makefile                 |    1 +
>  drivers/leds/leds-ab8500.c            |  201 +++++++++++++++++++++++++++++++
>  drivers/mfd/Kconfig                   |    8 ++
>  drivers/mfd/Makefile                  |    1 +
>  drivers/mfd/ab8500-core.c             |    7 +-
>  drivers/mfd/ab8500-pwm.c              |  191 ++++++++++++++++++++++++++++++
>  drivers/video/backlight/Kconfig       |   10 ++
>  drivers/video/backlight/Makefile      |    2 +-
>  drivers/video/backlight/ab8500_bl.c   |  209 +++++++++++++++++++++++++++++++++
>  include/linux/mfd/ab8500.h            |    5 +
>  include/linux/mfd/abx500/ab8500-pwm.h |   32 +++++
>  12 files changed, 674 insertions(+), 2 deletions(-)
>  create mode 100644 drivers/leds/leds-ab8500.c
>  create mode 100644 drivers/mfd/ab8500-pwm.c
>  create mode 100644 drivers/video/backlight/ab8500_bl.c
>  create mode 100644 include/linux/mfd/abx500/ab8500-pwm.h
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at  http://www.tux.org/lkml/


--
Sincerely yours,
Mike.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/