Re: [PATCH] usb: dwc2: Fix SET/CLEAR_FEATURE and GET_STATUS flows

[Minas Harutyunyan <Minas.Harutyunyan@synopsys.com>]

Hi John,


On 2/4/2020 3:45 PM, John Keeping wrote:
> Hi Minas,
> 
> On Fri, 13 Dec 2019 11:54:11 +0000
> Minas Harutyunyan <Minas.Harutyunyan@synopsys.com> wrote:
>> On 12/10/2019 6:19 PM, Minas Harutyunyan wrote:
>>> On 12/10/2019 4:53 PM, Felipe Balbi wrote:
>>>> Minas Harutyunyan <Minas.Harutyunyan@synopsys.com> writes:
>>>>   
>>>>> SET/CLEAR_FEATURE for Remote Wakeup allowance not handled correctly.
>>>>> GET_STATUS handling provided not correct data on DATA Stage.
>>>>> Issue seen when gadget's dr_mode set to "otg" mode and connected
>>>>> to MacOS.
>>>>> Both are fixed and tested using USBCV Ch.9 tests.
>>>>>
>>>>> Signed-off-by: Minas Harutyunyan <hminas@synopsys.com>
>>>>
>>>> do you want to add a Fixes tag here?
>>>
>>> Fixes: fa389a6d7726 ("usb: dwc2: gadget: Add remote_wakeup_allowed flag")
>>>    
>> Got tested tag by issue reported.
>>
>> Tested-By: Jack Mitchell <ml@embed.me.uk>
>>
>> Should I resubmit patch as v2 with added "Fixes" and "Tested-by" tags.
> 
> Was this patch ever picked up?  I don't see it in current next.
> 
Yes, it picked up by Felipe. It under testing/fixes

https://git.kernel.org/pub/scm/linux/kernel/git/balbi/usb.git/commit/?h=testing/fixes&id=6de1e301b9cf8eb023357586ab8b02edda07320b

Thanks,
Minas

> Given Felipe's message [1], I guess it might have got lost.
> 
> Would you mind resending?
> 
> [1] https://urldefense.proofpoint.com/v2/url?u=https-3A__lore.kernel.org_linux-2Dusb_875zhd6pw0.fsf-40kernel.org_&d=DwICAg&c=DPL6_X_6JkXFx7AXWqB0tg&r=6z9Al9FrHR_ZqbbtSAsD16pvOL2S3XHxQnSzq8kusyI&m=D4RlPf0cgyzMpc-hKhI1giEWZJGcxNJFJWC5xjDVyoI&s=pw7tlVwt7OpHtYj_WpGG3K9WTodfF6FUWvyHoJ4__co&e=
> 
> 
> Thanks,
> John
>