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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20170228094740.kgsanxtum6xj2wdt@sirena.org.uk>
Date:   Tue, 28 Feb 2017 09:47:40 +0000
From:   Mark Brown <broonie@...nel.org>
To:     Bartlomiej Zolnierkiewicz <b.zolnierkie@...sung.com>
Cc:     Krzysztof Kozlowski <krzk@...nel.org>,
        Lee Jones <lee.jones@...aro.org>,
        Chanwoo Choi <cw00.choi@...sung.com>,
        Liam Girdwood <lgirdwood@...il.com>,
        linux-kernel@...r.kernel.org
Subject: Re: [RFT 0/4] mfd: regulator: max14577: Remove support for platform
 data

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.

Download attachment "signature.asc" of type "application/pgp-signature" (489 bytes)