Re: [PATCH v5 0/9] mfd: max14577: Add support for MAX77836

From: Chanwoo Choi <cw00.choi@samsung.com>
To: Krzysztof Kozlowski <k.kozlowski@samsung.com>
Cc: Lee Jones <lee.jones@linaro.org>,
	MyungJoo Ham <myungjoo.ham@samsung.com>,
	Samuel Ortiz <sameo@linux.intel.com>,
	Liam Girdwood <lgirdwood@gmail.com>,
	Mark Brown <broonie@kernel.org>,
	linux-kernel@vger.kernel.org,
	Kyungmin Park <kyungmin.park@samsung.com>,
	Marek Szyprowski <m.szyprowski@samsung.com>,
	Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>
Subject: Re: [PATCH v5 0/9] mfd: max14577: Add support for MAX77836
Date: Wed, 16 Apr 2014 20:36:50 +0900	[thread overview]
Message-ID: <534E6B52.9080706@samsung.com> (raw)
In-Reply-To: <1397646165.6147.16.camel@AMDC1943>

On 04/16/2014 08:02 PM, Krzysztof Kozlowski wrote:
> On śro, 2014-04-16 at 19:51 +0900, Chanwoo Choi wrote:
>> Hi Krzysztof,
>>
>> On 04/16/2014 07:44 PM, Krzysztof Kozlowski wrote:
>>> On śro, 2014-04-16 at 11:28 +0100, Lee Jones wrote:
>>>>> This is first part of fifth version of patchset adding support for
>>>>> MAX77836 device to the max14577 driver.
>>>>>
>>>>> I decided to split the patchset into smaller parts. I hope this would
>>>>> help in getting necessary review/acks from maintainers.
>>>>>
>>>>> The first part introduces changes in max14577 driver and adds support
>>>>> for MAX77836 in MFD, extcon and regulator drivers. Most of these
>>>>> patches already have required acks from maintainers.
>>>>> Full v4 patchset can be found here:
>>>>> http://thread.gmane.org/gmane.linux.kernel/1654267
>>>>
>>>> So what's the plan for this driver now? What's missing? Do you have
>>>> all the Acks you need? If so, I'm happy to apply this patch-set and
>>>> supply a pull-request to the other maintainers.
>>>
>>> Next patches (part 2):
>>> 1. charger: max14577: Add support for MAX77836 charger
>>> 2. power: max17040: Add ID for MAX77836 Fuel Gauge block
>>> 3. regulator/mfd: max14577: Export symbols for calculating charger 
>>>    current
>>> 4. charger: max14577: Configure battery-dependent settings from DTS
>>> 5. Documentation: mfd: max14577: Add device tree bindings document
>>> 6. regulator: max14577: Implement SUSPEND mode for MAX77836 LDO-s
>>>
>>> They are quite independent from each other so I think it would be the
>>> best if you could prepare an immutable branch with first part of
>>> patches.
>>
>> I have to modify all extcon device drivers for some reason,
>> If your patchset including extcon patch apply other git repository,
>> this patchset may have potential merge conflict.
>>
>> If you possible, following patchset would divide two patchset for each mfd/extcon
>> to remove potential merge conflict in 3.15 merge window.
>> - [PATCH v5 0/9] mfd: max14577: Add support for MAX77836
> 
> I think we can resolve this after preparing by Lee the immutable branch
> with this patches. You could pull it so your changes to extcon-max14577
> would be on newest version.

I'm reluctant for your suggestion.
IMO, If Lee agree, I'd like to apply this patchset on extcon git
after getting ack for all patchset from maintainer.
And you would send your patchset (part 2) after 3.15 merge window.

Thanks,
Chanwoo Choi