
Hi Remy,
I would like to add support for Ethernet over USB (CDC) for the musb (musb_udc) controller, basing on your cdc branch in u-boot-usb.
I have seen there are some conflicts and some duplicated structure between the header imported from linux (linux/usb/ch9.h) and the headers in u-boot (usbdescriptors.h, for example), and I have started to solve these issues.
However, before doing some unnecessary work, I prefer to ask you which are your plans about this branch and if you think it could be merged soon to the u-boot-usb master and, at the end, brought to the mainline.
Regards, Stefano

FYI I'm doing some work on this branch too. Actually I have ported it to 2010.03, added RNDIS support and made integration into 'usb' command. However these changes aren't tested yet so the work is still under progress.
Probably someone knows about any possible issues with CDC/RNDIS support? And also I would like to know the answers to Stefano's questions.
On 07/13/2010 11:26 AM, Stefano Babic wrote:
Hi Remy,
I would like to add support for Ethernet over USB (CDC) for the musb (musb_udc) controller, basing on your cdc branch in u-boot-usb.
I have seen there are some conflicts and some duplicated structure between the header imported from linux (linux/usb/ch9.h) and the headers in u-boot (usbdescriptors.h, for example), and I have started to solve these issues.
However, before doing some unnecessary work, I prefer to ask you which are your plans about this branch and if you think it could be merged soon to the u-boot-usb master and, at the end, brought to the mainline.
Regards, Stefano

Hi,
2010/7/13 Vitaly Kuzmichev vkuzmichev@mvista.com:
FYI I'm doing some work on this branch too.
Great!
Actually I have ported it to 2010.03
What do you mean with ported to 2010.03? It was already working on latest git, do you mean you backported it to an older version? (I would not expect this would be a big difference though)
added RNDIS support
Is RNDIS working stable? We had quite some troubles with it, especially with BSODs on those Windows boxes... The device unplug/plug behaviour was a pain for RNDIS, hence the reason why we removed that from the code. But, I do not want to scare you off from this rndis work, please continue. Maybe you can get it to work properly :-)
We use it to communicate with a CDC driver on a Windows XP-embedded host (http://www.thesycon.de/eng/usb_cdcecm.shtml). CDC works much more stable, and much more faster compared to rndis.
and made integration into 'usb' command.
Great!
However these changes aren't tested yet so the work is still under progress.
Probably someone knows about any possible issues with CDC/RNDIS support?
See above. and CDC is working stable, no known issues with it.
At least one hint: First get CDC working on your hardware before you start to connect rndis, that would allow you to get the udc driver and all other levels of SW working properly and that you would only need to focus on rndis protocol rarities after that. And to prevent that you are debugging all kinds of undocumented rndis calls from the Windows host you could use a Linux host instead with the Linux-rndis-usb-host driver ;-)
And also I would like to know the answers to Stefano's questions.
See my other reply.
I would suggest to merge all work in progress in the u-boot-usb/CDC branch from which we push it to mainline.
Kind regards,
Remy

On Wed, Jul 14, 2010 at 4:48 AM, Remy Bohmer linux@bohmer.net wrote:
Hi,
2010/7/13 Vitaly Kuzmichev vkuzmichev@mvista.com:
FYI I'm doing some work on this branch too.
Great!
Actually I have ported it to 2010.03
What do you mean with ported to 2010.03? It was already working on latest git, do you mean you backported it to an older version? (I would not expect this would be a big difference though)
added RNDIS support
Is RNDIS working stable? We had quite some troubles with it, especially with BSODs on those Windows boxes... The device unplug/plug behaviour was a pain for RNDIS, hence the reason why we removed that from the code. But, I do not want to scare you off from this rndis work, please continue. Maybe you can get it to work properly :-)
We use it to communicate with a CDC driver on a Windows XP-embedded host (http://www.thesycon.de/eng/usb_cdcecm.shtml). CDC works much more stable, and much more faster compared to rndis.
We also implemented the CDC and RNDIS on our board and working well. If you merge your usb gadget tree then we will send the patches also.
To Minkyu, Can you prepare the patches and send to u-boot?
Thank you, Kyungmin Park
and made integration into 'usb' command.
Great!
However these changes aren't tested yet so the work is still under progress.
Probably someone knows about any possible issues with CDC/RNDIS support?
See above. and CDC is working stable, no known issues with it.
At least one hint: First get CDC working on your hardware before you start to connect rndis, that would allow you to get the udc driver and all other levels of SW working properly and that you would only need to focus on rndis protocol rarities after that. And to prevent that you are debugging all kinds of undocumented rndis calls from the Windows host you could use a Linux host instead with the Linux-rndis-usb-host driver ;-)
And also I would like to know the answers to Stefano's questions.
See my other reply.
I would suggest to merge all work in progress in the u-boot-usb/CDC branch from which we push it to mainline.
Kind regards,
Remy _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot

Hi,
We also implemented the CDC and RNDIS on our board and working well. If you merge your usb gadget tree then we will send the patches also.
Have you shared this anywhere (public git?)? I would like to look at your patches to sync them with mine.
___ WBR, Vitaly.
On 07/14/2010 11:10 AM, Kyungmin Park wrote:
On Wed, Jul 14, 2010 at 4:48 AM, Remy Bohmer linux@bohmer.net wrote:
Hi,
2010/7/13 Vitaly Kuzmichev vkuzmichev@mvista.com:
FYI I'm doing some work on this branch too.
Great!
Actually I have ported it to 2010.03
What do you mean with ported to 2010.03? It was already working on latest git, do you mean you backported it to an older version? (I would not expect this would be a big difference though)
added RNDIS support
Is RNDIS working stable? We had quite some troubles with it, especially with BSODs on those Windows boxes... The device unplug/plug behaviour was a pain for RNDIS, hence the reason why we removed that from the code. But, I do not want to scare you off from this rndis work, please continue. Maybe you can get it to work properly :-)
We use it to communicate with a CDC driver on a Windows XP-embedded host (http://www.thesycon.de/eng/usb_cdcecm.shtml). CDC works much more stable, and much more faster compared to rndis.
We also implemented the CDC and RNDIS on our board and working well. If you merge your usb gadget tree then we will send the patches also.
To Minkyu, Can you prepare the patches and send to u-boot?
Thank you, Kyungmin Park
and made integration into 'usb' command.
Great!
However these changes aren't tested yet so the work is still under progress.
Probably someone knows about any possible issues with CDC/RNDIS support?
See above. and CDC is working stable, no known issues with it.
At least one hint: First get CDC working on your hardware before you start to connect rndis, that would allow you to get the udc driver and all other levels of SW working properly and that you would only need to focus on rndis protocol rarities after that. And to prevent that you are debugging all kinds of undocumented rndis calls from the Windows host you could use a Linux host instead with the Linux-rndis-usb-host driver ;-)
And also I would like to know the answers to Stefano's questions.
See my other reply.
I would suggest to merge all work in progress in the u-boot-usb/CDC branch from which we push it to mainline.
Kind regards,
Remy _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot

Hi Remi,
What do you mean with ported to 2010.03? It was already working on latest git, do you mean you backported it to an older version?
I mean that I have to work over our fork of main u-boot 2010.03. Of course before submitting the patches I will test them on both 'master' branches of main u-boot and u-boot-usb.
Is RNDIS working stable?
I didn't make tests yet.
We had quite some troubles with it, especially with BSODs on those Windows boxes...
Looks like I have to fix this :)
The device unplug/plug behaviour was a pain for RNDIS,
Do you mean that Windows and u-boot do not recognize cable unplug?
But, I do not want to scare you off from this rndis work
I have no choice :)
We use it to communicate with a CDC driver on a Windows XP-embedded host (http://www.thesycon.de/eng/usb_cdcecm.shtml).
Fine.
And thanks for hint.
___ WBR, Vitaly.
On 07/13/2010 11:48 PM, Remy Bohmer wrote:
Hi,
2010/7/13 Vitaly Kuzmichev vkuzmichev@mvista.com:
FYI I'm doing some work on this branch too.
Great!
Actually I have ported it to 2010.03
What do you mean with ported to 2010.03? It was already working on latest git, do you mean you backported it to an older version? (I would not expect this would be a big difference though)
added RNDIS support
Is RNDIS working stable? We had quite some troubles with it, especially with BSODs on those Windows boxes... The device unplug/plug behaviour was a pain for RNDIS, hence the reason why we removed that from the code. But, I do not want to scare you off from this rndis work, please continue. Maybe you can get it to work properly :-)
We use it to communicate with a CDC driver on a Windows XP-embedded host (http://www.thesycon.de/eng/usb_cdcecm.shtml). CDC works much more stable, and much more faster compared to rndis.
and made integration into 'usb' command.
Great!
However these changes aren't tested yet so the work is still under progress.
Probably someone knows about any possible issues with CDC/RNDIS support?
See above. and CDC is working stable, no known issues with it.
At least one hint: First get CDC working on your hardware before you start to connect rndis, that would allow you to get the udc driver and all other levels of SW working properly and that you would only need to focus on rndis protocol rarities after that. And to prevent that you are debugging all kinds of undocumented rndis calls from the Windows host you could use a Linux host instead with the Linux-rndis-usb-host driver ;-)
And also I would like to know the answers to Stefano's questions.
See my other reply.
I would suggest to merge all work in progress in the u-boot-usb/CDC branch from which we push it to mainline.
Kind regards,
Remy

Hi,
The device unplug/plug behaviour was a pain for RNDIS,
Do you mean that Windows and u-boot do not recognize cable unplug?
Device unplug/plug behaviour on Windows is so slooooow... What I mean with unplug/plug is removing and adding the pullup resistor that makes the devices visible to the host. We had huge timeouts between unplug/plug up to more then 10 seconds, and the host could sometimes not even keep up. It frequently ran into conditions that the device was not seen at all anymore, up to even a very slow mouse responsiveness, and to malfunction of the context menus and so on, either eventually resulting in BSOD or manual reboot. All these problems vanished with the use of a CDC device driver on Windows...
Kind regards,
Remy

Hi Stefano,
2010/7/13 Stefano Babic sbabic@denx.de:
Hi Remy,
I would like to add support for Ethernet over USB (CDC) for the musb (musb_udc) controller, basing on your cdc branch in u-boot-usb.
Great!
I have seen there are some conflicts and some duplicated structure between the header imported from linux (linux/usb/ch9.h) and the headers in u-boot (usbdescriptors.h, for example), and I have started to solve these issues.
However, before doing some unnecessary work, I prefer to ask you which are your plans about this branch
Well, the branch exist already for quite some time, and I did not discover much interest in it for at least the last year, so that did not convince me to invest much time in it to get it mainline ready. However, it seems time has changed by now, so lets get this beast to mainline. The current status is that it is still up to date and we use it ourselves, even in combination with the U-boot 2010.06 release.
and if you think it could be merged soon to the u-boot-usb master and, at the end, brought to the mainline.
Of course this could be merged to mainline! We first have to get it reviewed on the ML because that has never been done before...
Kind regards,
Remy
participants (4)
-
Kyungmin Park
-
Remy Bohmer
-
Stefano Babic
-
Vitaly Kuzmichev