Re: [V2 PATCH 01/10] added media agnostic (MA) USB HCD driver

2014-11-15 Thread Alan Stern
On Fri, 14 Nov 2014, Sean O. Stalley wrote: To summarize the spec: MA USB groups a host connected devices into MA service sets (MSS). The architectural limit is 254 MA devices per MSS. If the host needs to connect more devices than that, It can start a new MSS and connect to 254 more MA

Re: [V2 PATCH 01/10] added media agnostic (MA) USB HCD driver

2014-11-14 Thread Sean O. Stalley
On Wed, Nov 12, 2014 at 05:03:18PM -0500, Alan Stern wrote: On Wed, 12 Nov 2014, Sean O. Stalley wrote: Our plan to support multiple MA devices is to have them all connected to the same virtual host controller, so only 1 would be needed. Would you prefer we have 1 host controller

Re: [V2 PATCH 01/10] added media agnostic (MA) USB HCD driver

2014-11-12 Thread Sean O. Stalley
Thanks for reviewing. My responses are inline. Greg has asked that we clean up this code internally before we send out another patchset to the mailing list. I will address the issues you pointed out, but it may be a while before you see another patchset. Thanks Again, Sean On Tue, Nov 11, 2014

Re: [V2 PATCH 01/10] added media agnostic (MA) USB HCD driver

2014-11-12 Thread Alan Stern
On Wed, 12 Nov 2014, Sean O. Stalley wrote: On Tue, Nov 11, 2014 at 10:54:30AM -0500, Alan Stern wrote: On Mon, 10 Nov 2014, Stephanie Wallick wrote: +static struct mausb_hcd mhcd; Only one statically-allocated structure? What if somebody wants to have more than one of these

[V2 PATCH 01/10] added media agnostic (MA) USB HCD driver

2014-11-10 Thread Stephanie Wallick
This is where we interface with the existing USB stack and implement the functionality of a USB host controller driver. From the host's perspective, we appear as just another USB host controller. However, instead of passing traffic along a wired USB bus, the driver hands USB packets off for

Re: [V2 PATCH 01/10] added media agnostic (MA) USB HCD driver

2014-11-10 Thread Greg KH
On Mon, Nov 10, 2014 at 06:09:32PM -0800, Stephanie Wallick wrote: +static int mausb_bus_probe(struct device *dev) +{ + return mausb_probe(dev); +} + +static int mausb_bus_remove(struct device *dev) +{ + return mausb_remove(dev); +} Wrapper functions that just call another