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

Groups > linux.kernel > #1231196 > unrolled thread

Re: [PATCH v3 0/1] USB DWC2 parity fix in isochronous mode

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 v3 0/1] USB DWC2 parity fix in isochronous mode Scott Branden <sbranden@broadcom.com> - 2015-09-23 09:40 +0200
    Re: [PATCH v3 0/1] USB DWC2 parity fix in isochronous mode John Youn <John.Youn@synopsys.com> - 2015-09-23 10:20 +0200
      Re: [PATCH v3 0/1] USB DWC2 parity fix in isochronous mode John Youn <John.Youn@synopsys.com> - 2015-09-23 10:20 +0200
        Re: [PATCH v3 0/1] USB DWC2 parity fix in isochronous mode Scott Branden <sbranden@broadcom.com> - 2015-09-23 17:40 +0200
          Re: [PATCH v3 0/1] USB DWC2 parity fix in isochronous mode John Youn <John.Youn@synopsys.com> - 2015-09-23 20:10 +0200

#1231196 — Re: [PATCH v3 0/1] USB DWC2 parity fix in isochronous mode

Hi John,

Could you please review the v3 Patch.  I believe we have address all of 
your comments?

On 15-09-10 06:13 PM, Scott Branden wrote:
> This patch contains a fix for a real world interop problem found
> when using the Synopsis DWC2 USB controller with isochronous audio as
> detailed in the commit message.
>
> Changes from v2:
>   - created s2c_hsotg_chage_ep_iso_parity function to call function in 3 places in code
>   - used hsotg->num_of_eps instead of MAX_EPS_CHANNELS
>
> Changes from v1:
>   - Address code review comments as per previous responses:
>   - renamed parity_set to has_correct_parity and reorder some logic
>
>
> Roman Bacik (1):
>    usb: dwc2: gadget: parity fix in isochronous mode
>
>   drivers/usb/dwc2/core.h   |  1 +
>   drivers/usb/dwc2/gadget.c | 58 ++++++++++++++++++++++++++++++++++++++++++-----
>   drivers/usb/dwc2/hw.h     |  1 +
>   3 files changed, 54 insertions(+), 6 deletions(-)
>

--
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] | [next] | [standalone]

#1231223

On 9/23/2015 12:36 AM, Scott Branden wrote:
> Hi John,
> 
> Could you please review the v3 Patch.  I believe we have address all of 
> your comments?
> 

Yes I've been meaning to test it on our platforms. I should be
able to get to it tomorrow.

Regards,
John

--
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]

#1231224

On 9/23/2015 1:11 AM, John Youn wrote:
> On 9/23/2015 12:36 AM, Scott Branden wrote:
>> Hi John,
>>
>> Could you please review the v3 Patch.  I believe we have address all of 
>> your comments?
>>
> 
> Yes I've been meaning to test it on our platforms. I should be
> able to get to it tomorrow.
> 

Also if you could rebase off of Felipe's testing/next branch it
would make it easier when it comes time to merge it in.

Regards,
John

--
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]

#1231507

On 15-09-23 01:18 AM, John Youn wrote:
> On 9/23/2015 1:11 AM, John Youn wrote:
>> On 9/23/2015 12:36 AM, Scott Branden wrote:
>>> Hi John,
>>>
>>> Could you please review the v3 Patch.  I believe we have address all of
>>> your comments?
>>>
>>
>> Yes I've been meaning to test it on our platforms. I should be
>> able to get to it tomorrow.
>>
>
> Also if you could rebase off of Felipe's testing/next branch it
> would make it easier when it comes time to merge it in.

Could you point me at Felipe's git repo as I am unfamiliar with the 
location?

Thanks,
  Scott

>
> Regards,
> John
>

--
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]

#1231620

On 9/23/2015 8:39 AM, Scott Branden wrote:
> On 15-09-23 01:18 AM, John Youn wrote:
>> On 9/23/2015 1:11 AM, John Youn wrote:
>>> On 9/23/2015 12:36 AM, Scott Branden wrote:
>>>> Hi John,
>>>>
>>>> Could you please review the v3 Patch.  I believe we have address all of
>>>> your comments?
>>>>
>>>
>>> Yes I've been meaning to test it on our platforms. I should be
>>> able to get to it tomorrow.
>>>
>>
>> Also if you could rebase off of Felipe's testing/next branch it
>> would make it easier when it comes time to merge it in.
> 
> Could you point me at Felipe's git repo as I am unfamiliar with the 
> location?
> 

git://git.kernel.org/pub/scm/linux/kernel/git/balbi/usb.git


Regards,
John

--
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] | [standalone]

Back to top | Article view | linux.kernel

csiph-web