From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754624AbcFIIVd (ORCPT ); Thu, 9 Jun 2016 04:21:33 -0400 Received: from devils.ext.ti.com ([198.47.26.153]:35816 "EHLO devils.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753061AbcFIIVL (ORCPT ); Thu, 9 Jun 2016 04:21:11 -0400 Subject: Re: [RFC v4 2/7] Revert "extcon: usb-gpio: switch to use pm wakeirq apis" To: Krzysztof Kozlowski , MyungJoo Ham , Chanwoo Choi , Rob Herring , Mark Rutland , Kukjin Kim , Marek Szyprowski , , , , References: <1465393686-16644-1-git-send-email-k.kozlowski@samsung.com> <1465393686-16644-3-git-send-email-k.kozlowski@samsung.com> <57592220.9030201@ti.com> <5759247E.6060901@samsung.com> CC: Peter Chen , "Ivan T. Ivanov" , Felipe Balbi , , Bartlomiej Zolnierkiewicz From: Roger Quadros Message-ID: <575926CD.80608@ti.com> Date: Thu, 9 Jun 2016 11:20:29 +0300 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.8.0 MIME-Version: 1.0 In-Reply-To: <5759247E.6060901@samsung.com> Content-Type: text/plain; charset="windows-1252" Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 09/06/16 11:10, Krzysztof Kozlowski wrote: > On 06/09/2016 10:00 AM, Roger Quadros wrote: >> -balbi@ti >> +balbi@kernel >> >> Hi, >> >> On 08/06/16 16:48, Krzysztof Kozlowski wrote: >>> This reverts commit 8106e404174253639731cc30a44f5b3ab764c5b7. >> >> The above commit id is not present in v4.7-rc1 and this patch doesn't apply properly. >> Can you please rebase this series on v4.7-rc1? > > The commit is present in extcon tree for v4.8. The extcon patches will > be applied (probably) to that tree so what is the point of rebasing on > v4.7-rc1? OK, please ignore my comment. Would be nice if you mention in the cover letter which tree the series is based on. > > Also the point of this particular patch is to show that previous change > should be just dropped. Don't replace things to complicated devm-like > functions just to simplify one path in the driver... > -- cheers, 0roger