From: "Du, Changbin" <changbin.du@intel.com>
To: Felipe Balbi <felipe.balbi@linux.intel.com>
CC: "gregkh@linuxfoundation.org" <gregkh@linuxfoundation.org>,
        "linux-usb@vger.kernel.org" <linux-usb@vger.kernel.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: RE: [PATCH] usb: dwc3: rename __dwc3_cleanup_done_trbs to
 __dwc3_cleanup_one_trb
Thread-Topic: [PATCH] usb: dwc3: rename __dwc3_cleanup_done_trbs to
 __dwc3_cleanup_one_trb
Thread-Index: AQHRim5ltLnnxHB/qU2NenJFQ795159xStuAgACHEKA=
Date: Wed, 30 Mar 2016 11:04:03 +0000
Message-ID: <0C18FE92A7765D4EB9EE5D38D86A563A05CDF3AF@SHSMSX103.ccr.corp.intel.com>
References: <1459332836-11579-1-git-send-email-changbin.du@intel.com>
 <87r3esp71y.fsf@intel.com>
In-Reply-To: <87r3esp71y.fsf@intel.com>
Accept-Language: en-US
Content-Language: en-US
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 8BIT
MIME-Version: 1.0
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 602
Lines: 24

> > @@ -1873,7 +1873,7 @@ static void dwc3_gadget_free_endpoints(struct
> dwc3 *dwc)
> >
> >  /* -------------------------------------------------------------------------- */
> >
> > -static int __dwc3_cleanup_done_trbs(struct dwc3 *dwc, struct dwc3_ep
> *dep,
> > +static int __dwc3_cleanup_one_trb(struct dwc3 *dwc, struct dwc3_ep
> *dep,
> 
> I would rather just remove that 's' from the end so the function's name
> stays as __dwc3_cleanup_done_trb() ;-)
> 
> Care to do that ?
> 
> thanks
> 
> --
> Balbi

IMO, "one_trb" is a little more clear. But both is okay for me. :) 

Thanks, 
Du, Changbin