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

From: Mark Brown
Date: Tue Feb 28 2017 - 04:50:13 EST


On Mon, Feb 27, 2017 at 04:52:36PM +0100, Bartlomiej Zolnierkiewicz wrote:
> On Monday, February 27, 2017 05:32:37 PM Krzysztof Kozlowski wrote:

> > The driver supports two chipsets: max14577 and max77836. As you noted,
> > the first one is for Gear 1. The second one is present on Gear2 which
> > is supported by mainline. Simplifying the driver to support only
> > max77836 would be a task requiring testing. I think review would be in
> > such case not enough.

> I think that this shouldn't be a big problem (if no one
> else volunteers we can do the testing on Gear2 device).

OTOH is it really gaining us much to remove the functionality vs the
effort for testing? Removing the entire driver is fairly
straightforward but if it's just some extra device support that's
sitting there not bothering anyone perhaps it's not really a good use of
time.

Attachment: signature.asc
Description: PGP signature