Re: [PATCH v5 1/2] power: supply: bq24190_charger: Use new extcon_register_notifier_all()

[Chanwoo Choi <cw00.choi@samsung.com>]

Hi Sebastian,

On 2017년 04월 14일 08:44, Sebastian Reichel wrote:
> Hi,
> 
> On Fri, Apr 14, 2017 at 01:35:26AM +0200, Sebastian Reichel wrote:
>> Hi,
>>
>> On Thu, Apr 13, 2017 at 02:04:11PM +0200, Hans de Goede wrote:
>>> When I submitted the extcon handling I had a patch pending for the
>>> extcon sub-system for extcon_register_notifier to take -1 as cable id
>>> for listening for all type cable events on an extcon with a single
>>> notifier.
>>>
>>> In the end it was decided to instead add a new
>>> extcon_register_notifier_all function for this, switch to using this.
>>>
>>> Signed-off-by: Hans de Goede <hdegoede@redhat.com>
>>> Acked-by: Liam Breck <kernel@networkimprov.net>
>>
>> With extcon_register_notifier_all() being new and not available
>> in my tree I can't simply queue this patch. Also it can't simply
>> go through the extcon subsystem, since it depends on changes in
>> the power-supply subsystem and results in non-trivial merge conflict.
>> I see three merge solutions:
>>
>>  1. Defer to 4.13 (since we are already quite near the merge window)
>>  2. I need a immutable branch with extcon_register_notifier_all()
>>     from the extcon subsystem
>>  3. It may count as Fix and could be queued during the 4.12-rc
>>     phase based von 4.12-rc1
> 
> Sorry for the noise, I just saw the immutable branch in the cover
> mail. I will queue this now. Unfortunately it means rebasing the
> power-supply tree to 4.11-rc4 (or ugly merge, which is annoying
> in the pull-request). Chanwoo, please try to use -rc1 as base for
> immutable branches in the future.

I'm really sorry about this. It is my fault. In the future,
I'll use the -rc1 for the immutable branch. Thanks.

-- 
Best Regards,
Chanwoo Choi
Samsung Electronics