From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752800AbbIWHgD (ORCPT ); Wed, 23 Sep 2015 03:36:03 -0400 Received: from mail-gw1-out.broadcom.com ([216.31.210.62]:13355 "EHLO mail-gw1-out.broadcom.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752165AbbIWHf7 (ORCPT ); Wed, 23 Sep 2015 03:35:59 -0400 X-IronPort-AV: E=Sophos;i="5.17,577,1437462000"; d="scan'208";a="75870998" Subject: Re: [PATCH v3 0/1] USB DWC2 parity fix in isochronous mode To: John Youn , Greg Kroah-Hartman , , Roman Bacik References: <1441934023-6557-1-git-send-email-sbranden@broadcom.com> CC: , From: Scott Branden Message-ID: <5602565A.90304@broadcom.com> Date: Wed, 23 Sep 2015 00:35:54 -0700 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.2.0 MIME-Version: 1.0 In-Reply-To: <1441934023-6557-1-git-send-email-sbranden@broadcom.com> Content-Type: text/plain; charset="windows-1252"; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org 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(-) >