Re: [PATCH v5 0/3] usb: udc: Unify dp control
On Thu, Jan 29, 2015 at 12:27:23PM +0800, Peter Chen wrote: > Hi Felipe, > Hi Felipe, I see you tree is closed, but below three patches are not in your tree, will you queue them now or at next rc? I have some other patches based on them, so I would like to know your ideas, thanks. Peter > This is the follow-up: http://marc.info/?l=linux-usb&m=140979297227434&w=2. > Sorry for late. > > In the first patch, we introduce an internal APIs which are used to > find corresponding udc according to usb_gadget, it can simplify the code > structure. > > In the 2nd patch, it maintains flag 'vbus' as vbus status, and try to connect > or disconnect gadget according to vbus status. > In the 3rd patch, it maintains count 'deactivations' for connect or disconnect > gadget according to function/user behavior. > > If you agree with handing dp like above way, I will have two patchset, one for > udc driver for 'vbus' and another for function driver for 'deactivations'. > > Thank you. > > Changes for v5(all for patch [3/3]): > - Add mutux_lock to protect deactivation count > - Add warning if deactivation is 0, but the user still wants to activate > - Add more comments in commit log and code > > Changes for v4: > - Add Alan's Ack > > Changes for v3: > - Add mutex_lock protect for list_del(&udc->list) at usb_del_gadget_udc > [Patch 1/3] > - Drop former [Patch 2/4] which is useless > > Changes for v2: > - introduce two internal APIs to simplify the code structure. > - Teach patch 3/4 and 4/4 to use the new introduced API. > > Peter Chen (3): > usb: udc: add usb_gadget_find_udc > usb: udc: add usb_udc_vbus_handler > usb: udc: add usb_udc_activation_handler > > drivers/usb/gadget/udc/udc-core.c | 118 > +- > include/linux/usb/gadget.h| 9 +++ > 2 files changed, 101 insertions(+), 26 deletions(-) > > -- > 1.9.1 > -- Best Regards, Peter Chen -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH v5 0/3] usb: udc: Unify dp control
Peter Chen writes: > On Thu, Jan 29, 2015 at 12:27:23PM +0800, Peter Chen wrote: >> Hi Felipe, >> > > Hi Felipe, I see you tree is closed, but below three patches are > not in your tree, will you queue them now or at next rc? I have > some other patches based on them, so I would like to know your > ideas, thanks. It's up to Felipe again, but I'd say include the patches that you wrote on top of these in the same patchset, because adding an api without users makes it more difficult to validate wrt its intended use cases and it also makes potential bugs that are introduced in the api patches sit quietly in the tree until you add users, at which point it's probable that these apis will need fixing anyway. Regards, -- Alex -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
RE: [PATCH v5 0/3] usb: udc: Unify dp control
> > > On Thu, Jan 29, 2015 at 12:27:23PM +0800, Peter Chen wrote: > >> Hi Felipe, > >> > > > > Hi Felipe, I see you tree is closed, but below three patches are not > > in your tree, will you queue them now or at next rc? I have some other > > patches based on them, so I would like to know your ideas, thanks. > > It's up to Felipe again, but I'd say include the patches that you wrote on > top of > these in the same patchset, because adding an api without users makes it more > difficult to validate wrt its intended use cases and it also makes potential > bugs > that are introduced in the api patches sit quietly in the tree until you add > users, > at which point it's probable that these apis will need fixing anyway. > Ok, it seems most of you agree these two APIs, I will create two patch sets, one for vbus, and another for function driver to deactivations count, I will add use cases for both of patch sets. Peter -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html