Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1588662

Re: [RFT 0/4] mfd: regulator: max14577: Remove support for platform data

Show all headers | View raw

Hi,

On Friday, February 17, 2017 10:01:56 PM Krzysztof Kozlowski wrote:
> Hi,
> 
> Minor cleanup of max14577 family of drivers. The dependency inside:
> 1. Patch #3 and #4 depends for safeness on #1 so no one would try to
>    run the driver on non-DT platform (such platform does not exist anyway...).
> 2. Patch #4 depends on patch #3 (compile time dependency).
> 
> Unfortunately I do not have the boards using these drivers, so testing
> would be appreciated.

Your patchset looks fine to me but I've noticed that
maxim,max14577 bindings are not used currently in
the mainline kernel.

AFAIK MAX14577 support is needed only for Gear1 board
which doesn't have DTS file in the upstream kernel
(moreover Exynos4212 SoC support has been removed
recently).

It seems that MAX14577 support should also go away
(unless there are some other boards using it that we
support in the mainline kernel).

Best regards,
--
Bartlomiej Zolnierkiewicz
Samsung R&D Institute Poland
Samsung Electronics

> Best regards,
> Krzysztof
> 
> Krzysztof Kozlowski (4):
>   mfd: max14577: Explicitly depend on OF
>   mfd: max14577: Export OF module alias information
>   regulator: max14577: Remove support for platform data
>   mfd: max14577: Remove platform data and instantiation from I2C
> 
>  drivers/mfd/Kconfig                    |  1 +
>  drivers/mfd/max14577.c                 | 23 +----------------------
>  drivers/regulator/max14577-regulator.c | 15 +++------------
>  include/linux/mfd/max14577.h           | 20 --------------------
>  4 files changed, 5 insertions(+), 54 deletions(-)

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[RFT 0/4] mfd: regulator: max14577: Remove support for platform data Krzysztof Kozlowski <krzk@kernel.org> - 2017-02-17 21:10 +0100
  [RFT 4/4] mfd: max14577: Remove platform data and instantiation from I2C Krzysztof Kozlowski <krzk@kernel.org> - 2017-02-17 21:10 +0100
  [RFT 3/4] regulator: max14577: Remove support for platform data Krzysztof Kozlowski <krzk@kernel.org> - 2017-02-17 21:10 +0100
    Re: [RFT 3/4] regulator: max14577: Remove support for platform data Mark Brown <broonie@kernel.org> - 2017-02-18 01:10 +0100
  [RFT 1/4] mfd: max14577: Explicitly depend on OF Krzysztof Kozlowski <krzk@kernel.org> - 2017-02-17 21:10 +0100
  Re: [RFT 0/4] mfd: regulator: max14577: Remove support for platform  data Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com> - 2017-02-27 14:10 +0100
    Re: [RFT 0/4] mfd: regulator: max14577: Remove support for platform data Krzysztof Kozlowski <krzk@kernel.org> - 2017-02-27 16:40 +0100
      Re: [RFT 0/4] mfd: regulator: max14577: Remove support for platform  data Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com> - 2017-02-27 17:00 +0100
        Re: [RFT 0/4] mfd: regulator: max14577: Remove support for platform  data Mark Brown <broonie@kernel.org> - 2017-02-28 11:00 +0100
          Re: [RFT 0/4] mfd: regulator: max14577: Remove support for platform  data Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com> - 2017-02-28 14:00 +0100

csiph-web