Re: [PATCH v8 00/23] usb gadget update for OTG 2.0

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Aug 04, 2015 at 08:00:31AM +0800, Li Jun wrote:
> On Thu, Jul 30, 2015 at 09:10:06AM -0500, Felipe Balbi wrote:
> > On Thu, Jul 30, 2015 at 09:46:58AM +0800, Li Jun wrote:
> > > On Wed, Jul 29, 2015 at 09:11:41PM -0500, Felipe Balbi wrote:
> > > > On Thu, Jul 30, 2015 at 07:24:03AM +0800, Li Jun wrote:
> > > > > On Wed, Jul 29, 2015 at 10:04:27AM -0500, Felipe Balbi wrote:
> > > > > > On Mon, Jul 27, 2015 at 03:21:59PM +0800, Peter Chen wrote:
> > > > > > > On Thu, Jul 23, 2015 at 11:37:24AM +0800, Li Jun wrote:
> > > > > > > > Change for v8:
> > > > > > > > - Add Peter's ACk for chipidea driver; and Roger's Reviewed-by for patch
> > > > > > > >   07, 21~23.
> > > > > > > > - Add ci->is_otg condition before enable ci->gadget.is_otg for chipidea
> > > > > > > >   driver in patch 8.
> > > > > > > > 
> > > > > > > 
> > > > > > > Hi Felipe,
> > > > > > > 
> > > > > > > For chipidea patches in this series, help to queue them in your tree 
> > > > > > > please, thanks.
> > > > > > 
> > > > > > all there, please make sure they're correct. There were so many versions
> > > > > > of this series that I might have picked up the wrong one :-p
> > > > > > 
> > > > > 
> > > > > Patch [8/23]: "usb: chipidea: set usb otg capabilities", you picked up the V7,
> > > > > I have a small change for V8, others are correct.
> > > > 
> > > > Can you send an incremental diff, please ?
> > > > 
> > > 
> > > diff --git a/drivers/usb/chipidea/udc.c b/drivers/usb/chipidea/udc.c
> > > index 61fde89..c6d1595 100644
> > > --- a/drivers/usb/chipidea/udc.c
> > > +++ b/drivers/usb/chipidea/udc.c
> > > @@ -1838,8 +1838,8 @@ static int udc_start(struct ci_hdrc *ci)
> > >  	ci->gadget.name         = ci->platdata->name;
> > >  	ci->gadget.otg_caps	= otg_caps;
> > >  
> > > -	if (otg_caps->hnp_support || otg_caps->srp_support ||
> > > -					otg_caps->adp_support)
> > > +	if (ci->is_otg && (otg_caps->hnp_support || otg_caps->srp_support ||
> > > +						otg_caps->adp_support))
> > >  		ci->gadget.is_otg = 1;
> > >  
> > >  	INIT_LIST_HEAD(&ci->gadget.ep_list);
> > 
> > I need it as a real patch with Signed-off-by and all, so I can apply :-)
> > 
> Hi Felipe,
> 
> I have sent it out as a formal patch for this incremental diff, but still not
> in your next tree.

it's in testing/next :-) I should be able to merge that back into next
today.

-- 
balbi

Attachment: signature.asc
Description: Digital signature


[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux