Re: [RFC PATCH 0/4] usb: gadget: add is_selfpowered
On Wed, Jan 28, 2015 at 04:32:59AM +, Peter Chen wrote: > > > > > > > > > Hi Felipe, > > > > > > > > Except for Alan's comments, do you agree with this change for common > > > > struct, if you agree with it, I will send formal patch with other > > > > udc driver's cleanup. > > > > > > looks good to me. Will you provide patches to all other UDC drivers ? > > > > it's certainly not mandatory, and I can help with both musb and dwc3 > > (probably > > a few others too), but it'll help me if you do. :-) > > > > I will do it for all udcs. cool, thanks. I'll make sure to test on platforms I have around. cheers -- balbi signature.asc Description: Digital signature
RE: [RFC PATCH 0/4] usb: gadget: add is_selfpowered
> > > > > > Hi Felipe, > > > > > > Except for Alan's comments, do you agree with this change for common > > > struct, if you agree with it, I will send formal patch with other > > > udc driver's cleanup. > > > > looks good to me. Will you provide patches to all other UDC drivers ? > > it's certainly not mandatory, and I can help with both musb and dwc3 (probably > a few others too), but it'll help me if you do. :-) > I will do it for all udcs. 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
Re: [RFC PATCH 0/4] usb: gadget: add is_selfpowered
On Tue, Jan 27, 2015 at 10:07:02PM -0600, Felipe Balbi wrote: > Hi, > > On Wed, Jan 28, 2015 at 10:05:39AM +0800, Peter Chen wrote: > > On Tue, Jan 27, 2015 at 10:37:30AM +0800, Peter Chen wrote: > > > Hi Felipe, > > > > > > > Hi Felipe, > > > > Except for Alan's comments, do you agree with this change for > > common struct, if you agree with it, I will send formal patch > > with other udc driver's cleanup. > > looks good to me. Will you provide patches to all other UDC drivers ? it's certainly not mandatory, and I can help with both musb and dwc3 (probably a few others too), but it'll help me if you do. :-) cheers -- balbi signature.asc Description: Digital signature
Re: [RFC PATCH 0/4] usb: gadget: add is_selfpowered
Hi, On Wed, Jan 28, 2015 at 10:05:39AM +0800, Peter Chen wrote: > On Tue, Jan 27, 2015 at 10:37:30AM +0800, Peter Chen wrote: > > Hi Felipe, > > > > Hi Felipe, > > Except for Alan's comments, do you agree with this change for > common struct, if you agree with it, I will send formal patch > with other udc driver's cleanup. looks good to me. Will you provide patches to all other UDC drivers ? -- balbi signature.asc Description: Digital signature
Re: [RFC PATCH 0/4] usb: gadget: add is_selfpowered
On Tue, Jan 27, 2015 at 10:37:30AM +0800, Peter Chen wrote: > Hi Felipe, > Hi Felipe, Except for Alan's comments, do you agree with this change for common struct, if you agree with it, I will send formal patch with other udc driver's cleanup. Peter > We can use a common is_selfpowered flag at usb_gadget to indicate > whether the gadget is self-powered or bus-powered, it can avoid > every udc driver defining a similar flag, and the user can > read it through udc core sys entry. > > In the 1st patch, I introduce is_selfpowered for usb_gadget. > In the 2nd patch, I take chipidea driver as an example to show > how udc driver uses this flag. > In the 3rd patch, I clean up at91 udc driver for this flag. > In the 4th patch, I add this entry to sys through udc core. > > If you agree with this change, I will submit a formal patch set > after cleaning up other udc drivers. > > Peter Chen (4): > usb: gadget: introduce is_selfpowered for usb_gadget > usb: chipidea: udc: add set_selfpowered gaget ops > usb: gadget: at91_udc: use common is_selfpowered > usb: udc-core: add is_selfpowered sys entry > > drivers/usb/chipidea/udc.c| 14 ++ > drivers/usb/gadget/udc/at91_udc.c | 8 > drivers/usb/gadget/udc/at91_udc.h | 1 - > drivers/usb/gadget/udc/udc-core.c | 2 ++ > include/linux/usb/gadget.h| 2 ++ > 5 files changed, 22 insertions(+), 5 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
[RFC PATCH 0/4] usb: gadget: add is_selfpowered
Hi Felipe, We can use a common is_selfpowered flag at usb_gadget to indicate whether the gadget is self-powered or bus-powered, it can avoid every udc driver defining a similar flag, and the user can read it through udc core sys entry. In the 1st patch, I introduce is_selfpowered for usb_gadget. In the 2nd patch, I take chipidea driver as an example to show how udc driver uses this flag. In the 3rd patch, I clean up at91 udc driver for this flag. In the 4th patch, I add this entry to sys through udc core. If you agree with this change, I will submit a formal patch set after cleaning up other udc drivers. Peter Chen (4): usb: gadget: introduce is_selfpowered for usb_gadget usb: chipidea: udc: add set_selfpowered gaget ops usb: gadget: at91_udc: use common is_selfpowered usb: udc-core: add is_selfpowered sys entry drivers/usb/chipidea/udc.c| 14 ++ drivers/usb/gadget/udc/at91_udc.c | 8 drivers/usb/gadget/udc/at91_udc.h | 1 - drivers/usb/gadget/udc/udc-core.c | 2 ++ include/linux/usb/gadget.h| 2 ++ 5 files changed, 22 insertions(+), 5 deletions(-) -- 1.9.1 -- 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