Messages in this thread | | | Date | Mon, 10 Mar 2014 13:44:41 -0500 | From | Felipe Balbi <> | Subject | Re: [PATCH RFC] usb: gadget: Add xilinx axi usb2 device support |
| |
Hi,
On Mon, Mar 10, 2014 at 07:06:19PM +0530, sundeep subbaraya wrote: > On Fri, Feb 21, 2014 at 9:09 PM, Felipe Balbi <balbi@ti.com> wrote: > > Hi, > > > > On Fri, Feb 21, 2014 at 11:27:07AM +0000, Subbaraya Sundeep Bhatta wrote: > >> > From the looks of it, I doubt this was actually tested, you need a lot > >> > of work on this driver. > >> Tested on both ARM and Microblaze architectures with Mass storage gadget. > >> Will send a v2 after addressing all your comments. > > > > clearly you didn't try to remove and reinsert the module or you would > > see a whole bunch of errors. > > > > Yes you are correct. My console hung up as soon as i rmmod my driver. > Could you please point me where am wrong.
Many of your mistakes have already been pointed out here:
http://marc.info/?l=linux-usb&m=139292068028649&w=2
Please fix those and see if following script works:
#!/bin/sh
MODULE="xilinx_udc"
for i in $(seq 500); do modprobe $MODULE > /dev/null 2>&1; modprobe -r $MODULE > /dev/null 2>&1; if [ $? -eq 0 ]; then echo -n "." else echo -n "F" fi done
echo "Finished"
you shouldn't see any F characters and your machine shouldn't crash or hang or reboot...
Also make sure to run g_zero testcases and mass storage tescases. I wrote, long ago, a small tool to help testing UDC drivers running with the mass storage gadget, which you can find in [1]. There's a runner script which makes it easy to full test suite which you can find in [2].
[1] https://gitorious.org/usb/usb-tools/source/7eb7ef21de6cd124e0e0d0e7df9ddfff0e2f548e:msc.c [2] https://gitorious.org/usb/usb-tools/source/7eb7ef21de6cd124e0e0d0e7df9ddfff0e2f548e:msc.sh
-- balbi [unhandled content-type:application/pgp-signature] | |