Re: [PATCH V5 2/2] usbtest: Add interrupt EP testcases

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Felipe,

I just checked your testing/next branch. 'git log' of that branch doesn't show the complete commit log of this patch.

On 8/22/2014 2:36 PM, Amit VIRDI wrote:
Two simple test cases for interrupt endpoints are added to the usbtest.c file.
These are simple non-queued interrupt IN and interrupt OUT transfers. Currently,
only gadget zero is capable of executing the interrupt EP test cases. However,
extending the same to other gadgets is extremely simple and can be done
on-demand.

The two new tests added are
   - Test 25: To verify Interrupt OUT transfer
   - Test 26: To verify Interrupt IN transfer

Since the default value of wMaxPacketSize is set as 1024, so interrupt
IN transfers must be specified with the size parameter = multiple of
1024. Otherwise the default value (512) in the usbtest application fails
the transfer. See [RUN 4] for sample logs

The application logs (usbtest) and corresponding kernel logs are as
following:

It only shows up to here. Rest of the commit log is missing.

---

I guess this is the culprit...

	[Run 1]
	./testusb -a -c 10 -s 2048 -t 26 -v 511
	Jul 17 10:31:13 dlhl1014 kernel: [72056.950910] usbtest 7-1:3.0: TEST
	26: read 2048 bytes 10 times

	[Run 2]
	./testusb -a -c 10 -s 1024 -t 25 -v 511
	Jul 17 10:31:29 dlhl1014 kernel: [72072.834853] usbtest 7-1:3.0: TEST
	25: write 1024 bytes 10 times

	[Run 3]
	./testusb -a -c 10 -s 1098 -t 25 -v 511
	Jul 17 10:31:39 dlhl1014 kernel: [72082.322219] usbtest 7-1:3.0: TEST
	25: write 1098 bytes 10 times

	[Run 4 - Failure case scenario]
	./testusb -a  -t 26
	unknown speed   /dev/bus/usb/007/004    0
	/dev/bus/usb/007/004 test 26 --> 75 (Value too large for defined data
	type)

	Jul 17 11:11:30 dlhl1014 kernel: [74473.347219] usbtest 7-1:3.0: TEST
	26: read 512 bytes 1000 times
	Jul 17 11:11:30 dlhl1014 kernel: [74473.348959] usb 7-1: test26 failed,
	iterations left 999, status -75 (not 0)
---

This code has been tested only with gadget zero and care has been taken so as to
not break the existing functionality. However, if anyone can test with other
gadgets then that would be great!

Signed-off-by: Amit Virdi <amit.virdi@xxxxxx>
---
  drivers/usb/misc/usbtest.c | 113 +++++++++++++++++++++++++++++++++++++++------
  1 file changed, 98 insertions(+), 15 deletions(-)


Please let me know if there's anything I can help with.

Regards
Amit Virdi
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux