Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1270402 > unrolled thread

Re: [PATCH 2/2] usb: phy: msm: fix connect/disconnect bug for dragonboard OTG port

Started byFelipe Balbi <balbi@ti.com>
First post2015-11-16 18:30 +0100
Last post2015-11-21 02:00 +0100
Articles 3 — 2 participants

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.


Contents

  Re: [PATCH 2/2] usb: phy: msm: fix connect/disconnect bug for dragonboard OTG port Felipe Balbi <balbi@ti.com> - 2015-11-16 18:30 +0100
    Re: [PATCH 2/2] usb: phy: msm: fix connect/disconnect bug for  dragonboard OTG port Tim Bird <tim.bird@sonymobile.com> - 2015-11-20 23:40 +0100
      Re: [PATCH 2/2] usb: phy: msm: fix connect/disconnect bug for dragonboard OTG port Felipe Balbi <balbi@ti.com> - 2015-11-21 02:00 +0100

#1270402 — Re: [PATCH 2/2] usb: phy: msm: fix connect/disconnect bug for dragonboard OTG port

FromFelipe Balbi <balbi@ti.com>
Date2015-11-16 18:30 +0100
SubjectRe: [PATCH 2/2] usb: phy: msm: fix connect/disconnect bug for dragonboard OTG port
Message-ID<qvvLj-1Mp-7@gated-at.bofh.it>

[Multipart message — attachments visible in raw view] — view raw

Hi,

Peter Chen <peter.chen@freescale.com> writes:
> On Wed, Nov 11, 2015 at 09:48:00AM -0800, Tim Bird wrote:
>> 
>> 
>> On 11/10/2015 07:14 PM, Peter Chen wrote:
>> > On Tue, Nov 10, 2015 at 04:46:51PM -0800, Tim Bird wrote:
>> >> This fixes a bug where if you disconnect and re-connect the USB cable,
>> >> the gadget driver stops working.
>> >>
>> >> Add support for async_irq to wake up driver from low power mode.
>> >> Without this, the power management code never calls resume.
>> >> Also, have the phy driver kick the gadget driver (chipidea otg)
>> >> by having the chipidea driver register with it, for vbus connect
>> >> notifications.
>> >>
>> >> Signed-off-by: Tim Bird <tim.bird@sonymobile.com>
>> >> ---
>> >>  drivers/usb/chipidea/udc.c    |  6 ++++++
>> >>  drivers/usb/phy/phy-msm-usb.c | 16 ++++++++++++++++
>> >>  include/linux/usb/msm_hsusb.h |  1 +
>> >>  3 files changed, 23 insertions(+)

I just wanna know how you guys want this to be handled ? Through my tree
or chipidea's ? Or do we break the dependencies between the changes ?

-- 
balbi

[toc] | [next] | [standalone]


#1274465 — Re: [PATCH 2/2] usb: phy: msm: fix connect/disconnect bug for dragonboard OTG port

FromTim Bird <tim.bird@sonymobile.com>
Date2015-11-20 23:40 +0100
SubjectRe: [PATCH 2/2] usb: phy: msm: fix connect/disconnect bug for dragonboard OTG port
Message-ID<qx2vw-50g-13@gated-at.bofh.it>
In reply to#1270402

On 11/16/2015 09:21 AM, Felipe Balbi wrote:
> 
> Hi,
> 
> Peter Chen <peter.chen@freescale.com> writes:
>> On Wed, Nov 11, 2015 at 09:48:00AM -0800, Tim Bird wrote:
>>>
>>>
>>> On 11/10/2015 07:14 PM, Peter Chen wrote:
>>>> On Tue, Nov 10, 2015 at 04:46:51PM -0800, Tim Bird wrote:
>>>>> This fixes a bug where if you disconnect and re-connect the USB cable,
>>>>> the gadget driver stops working.
>>>>>
>>>>> Add support for async_irq to wake up driver from low power mode.
>>>>> Without this, the power management code never calls resume.
>>>>> Also, have the phy driver kick the gadget driver (chipidea otg)
>>>>> by having the chipidea driver register with it, for vbus connect
>>>>> notifications.
>>>>>
>>>>> Signed-off-by: Tim Bird <tim.bird@sonymobile.com>
>>>>> ---
>>>>>  drivers/usb/chipidea/udc.c    |  6 ++++++
>>>>>  drivers/usb/phy/phy-msm-usb.c | 16 ++++++++++++++++
>>>>>  include/linux/usb/msm_hsusb.h |  1 +
>>>>>  3 files changed, 23 insertions(+)
> 
> I just wanna know how you guys want this to be handled ? Through my tree
> or chipidea's ? Or do we break the dependencies between the changes ?

I'm fine with splitting it up.  I'm sending a new series with 3 patches
right after this message.  Do both trees go to linux-next?
 -- Tim

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [next] | [standalone]


#1274581

FromFelipe Balbi <balbi@ti.com>
Date2015-11-21 02:00 +0100
Message-ID<qx4GZ-6lN-7@gated-at.bofh.it>
In reply to#1274465

[Multipart message — attachments visible in raw view] — view raw

Hi,

Tim Bird <tim.bird@sonymobile.com> writes:
> On 11/16/2015 09:21 AM, Felipe Balbi wrote:
>> 
>> Hi,
>> 
>> Peter Chen <peter.chen@freescale.com> writes:
>>> On Wed, Nov 11, 2015 at 09:48:00AM -0800, Tim Bird wrote:
>>>>
>>>>
>>>> On 11/10/2015 07:14 PM, Peter Chen wrote:
>>>>> On Tue, Nov 10, 2015 at 04:46:51PM -0800, Tim Bird wrote:
>>>>>> This fixes a bug where if you disconnect and re-connect the USB cable,
>>>>>> the gadget driver stops working.
>>>>>>
>>>>>> Add support for async_irq to wake up driver from low power mode.
>>>>>> Without this, the power management code never calls resume.
>>>>>> Also, have the phy driver kick the gadget driver (chipidea otg)
>>>>>> by having the chipidea driver register with it, for vbus connect
>>>>>> notifications.
>>>>>>
>>>>>> Signed-off-by: Tim Bird <tim.bird@sonymobile.com>
>>>>>> ---
>>>>>>  drivers/usb/chipidea/udc.c    |  6 ++++++
>>>>>>  drivers/usb/phy/phy-msm-usb.c | 16 ++++++++++++++++
>>>>>>  include/linux/usb/msm_hsusb.h |  1 +
>>>>>>  3 files changed, 23 insertions(+)
>> 
>> I just wanna know how you guys want this to be handled ? Through my tree
>> or chipidea's ? Or do we break the dependencies between the changes ?
>
> I'm fine with splitting it up.  I'm sending a new series with 3 patches
> right after this message.  Do both trees go to linux-next?

I have my fixes and next branches both on next. Not sure about chipidea.

-- 
balbi

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web