Re: [PATCH 00/10] MA USB drivers cover letter

sostalle
2014-11-05T01:31:35Z

[with a valid Message-ID]

On Tue, Nov 04, 2014 at 10:00:32AM +0100, Bjørn Mork wrote:
> [added linux-usb@vger.kernel.org Cc]
>
> Stephanie Wallick <stephanie.s.wallick@intel.com> writes:
>
> > Media Agnostic (MA) USB enables the USB protocol to be used over a wide
> > range of physical media. MA USB is a relatively new protocol and is
> > currently unsupported in the Linux kernel. This patch set adds the
> > following drivers with the following capabilities:
> >
> > 1) MA USB Host:
> > - provides functionality of a USB Host Controller.
> > - implements MA USB protocol for a MA USB host.
> > - provides MA USB packet transport over TCP
> >
> > 2) MA USB Device:
> > - provides functionality of a USB Device Controller.
> > - implements MA USB protocol for a MA USB device.
> > - provides MA USB packet transport over TCP
>
> Nice. But don't you think this deserves the attention of the linux-usb
> mailing list? Yes, Greg's attention is of course good to have too :-)
> But I believe this is of interest to more people, who might not read
> everything on linux-kernel.

This is probably more relevant/interesting to the people on linux-usb.
Since we were trying to get this driver into staging, we thought linux-kernel
was the right place to send the patches. We can send the next set out to
linux-usb as well.

> Has there been any thought/discussion about the relationship to the
> existing usbip drivers, which just moved out of staging? Is a usbip
> userspace compatibility layer feasible sometime in the future?

That discussion hasn't happened. usbip has a different protocol than MA USB,
so much of the code cannot be reused.

I just briefly looked into the userspace code for usbip. It looks like the
userspace compatibility layer is just a utility for establishing the TCP
connection between the host and device & choosing what devices to connect,
correct? We have a similar utility (see patch 9/10) which could probably
be replaced by the usbip userspace utilities.

Thanks,
Sean