185 Commits

• Replace uint8 with Go-idiomatic int, where it makes sense.
  Sebastian Zagrodzki · 2017-04-26 22:22:07 +02:00
  83276566fc
• Disable Write streams for now.
  Sebastian Zagrodzki · 2017-04-26 21:51:00 +02:00
  a55abe9076
• Handle a case where transfers channel was not closed properly.
  Sebastian Zagrodzki · 2017-04-26 21:50:10 +02:00
  cdd4223813
• update tests with the new "transfer done" logic.
  Sebastian Zagrodzki · 2017-04-26 21:40:51 +02:00
  ae8627a890
• NewStream takes int now.
  Sebastian Zagrodzki · 2017-04-26 21:36:14 +02:00
  fbd26497e6
• Re-use the same done channel for every callback of a single transfer.
  Sebastian Zagrodzki · 2017-04-26 21:35:26 +02:00
  eed6d08423
• Call back to Go function directly, it will know how to deal with an xfer

  pointer.

  Sebastian Zagrodzki · 2017-04-26 21:33:45 +02:00
  c1009ed25d
• Introduce xferMap, avoid passing Go pointer into the C code.

  Weirdness with memory barriers.

  Sebastian Zagrodzki · 2017-04-26 21:32:49 +02:00
  550fe8d1f2
• Tests for read transfers, starting on write transfers.
  Sebastian Zagrodzki · 2017-04-23 22:58:30 +02:00
  dacae32d83
• First version of streaming transfers.
  Sebastian Zagrodzki · 2017-04-23 10:45:04 +02:00
  b50bc8761f
• Add a test to catch invalud endpoint info.
  Sebastian Zagrodzki · 2017-04-10 03:25:32 +02:00
  d92ea14c33
• Fix a bug where EndpointInfo would always be picked from the last

  alternative setting.
  
  When looking up matching EndpointInfo and InterfaceSetting for
  an endpoint, a reference to range variable is stored. If range is
  allowed to continue, the value of this variable will be updated.
  It's a trick to allow use of "nil" pointer as a "found" variable, but it
  requires a careful management of the range iterations.

  Sebastian Zagrodzki · 2017-04-10 03:16:59 +02:00
  9cecdec977
• Change the iso packet size if the buffer is very small.

  Previously number of iso packets could have amounted to a transfer
  buffer larger than passed buffer, which could lead to overflow.
  
  Now the number of bytes requested from device is always smaller or equal
  to the buffer size.
  
  In theory, it could lead to inefficiencies in transfer: if max packet
  size is 1024 and user request 2046 bytes of data, transfer will return at most
  1024 bytes, while it could use a more efficient 2 packets of 1023 bytes.
  But at least it's correct now. It seems that for efficiency the user would
  always use the reported max packet size anyway.
  
  The device could still send more data than requested, but libusb takes
  care of it.

  Sebastian Zagrodzki · 2017-04-10 02:45:16 +02:00
  b0d5519c67
• iota used on the second line of const block starts from 1, not from 0...
  Sebastian Zagrodzki · 2017-04-10 01:41:36 +02:00
  5efb68e45e
• Set slice length, since we don't use append.
  Sebastian Zagrodzki · 2017-04-10 01:41:12 +02:00
  5150242633
• Add tests for failed open. Add tests for OutEndpoint.Write
  Sebastian Zagrodzki · 2017-04-10 01:28:21 +02:00
  453e3832b2
• Add a test for InEndpoint.Read. Use ID for vid/pid in OpenDevice...
  Sebastian Zagrodzki · 2017-04-10 01:25:51 +02:00
  b6794b7d45
• Add Device.OutEndpoint, simplify endpoint test.
  Sebastian Zagrodzki · 2017-04-10 01:11:38 +02:00
  b1dcaa1195
• Remove OpenEndpoint, add InEndpoint for Read transfers.
  Sebastian Zagrodzki · 2017-04-10 01:00:53 +02:00
  57b10f0dd3
• move tests around
  Sebastian Zagrodzki · 2017-04-09 20:55:37 +02:00
  dd405348e9
• Move EndpointInfo to endpoint.go
  Sebastian Zagrodzki · 2017-04-09 20:51:47 +02:00
  128c206986
• rename Setups to AltSettings
  Sebastian Zagrodzki · 2017-04-09 20:45:35 +02:00
  aaef575b06
• more comments
  Sebastian Zagrodzki · 2017-04-09 20:27:37 +02:00
  188fd8382d
• Remove BCD.Int, add BCD.Major/Minor. Remove USB spec versions

  - I don't expect them to get much use, and reuse of BCD for USB spec
  version and device revision makes it somewhat confusing.

  Sebastian Zagrodzki · 2017-04-09 20:25:16 +02:00
  56162d0105
• more comments. Rename USBError to just Error.
  Sebastian Zagrodzki · 2017-04-09 19:41:44 +02:00
  a5c4de3a29
• Go naming for error codes. Export USBError type.
  Sebastian Zagrodzki · 2017-04-09 19:27:33 +02:00
  87c7a873e9
• more comments
  Sebastian Zagrodzki · 2017-04-09 19:18:11 +02:00
  54aed557d8
• more comments
  Sebastian Zagrodzki · 2017-04-09 19:17:13 +02:00
  b09de1cf09
• OpenEndpoint for now takes an address - a device can have two endpoints

  with the same number...

  Sebastian Zagrodzki · 2017-04-09 19:08:42 +02:00
  f668df87df
• make addr() part of EndpointInfo
  Sebastian Zagrodzki · 2017-04-09 18:55:41 +02:00
  f58798160d
• Unexport bitmasks, they are only used internally. All values present

  in exported structs are meaningful and the users should not need to use
  bitmasks.

  Sebastian Zagrodzki · 2017-04-09 18:53:49 +02:00
  13f4e95f2f
• Replace EndpointDirection with a bool
  Sebastian Zagrodzki · 2017-04-09 18:51:51 +02:00
  3d62929e05
• Rename IfClass/IfSubClass/IfProtocol to Class/SubClass/Protocol, these

  fields occur within an interface descriptor.

  Sebastian Zagrodzki · 2017-04-09 18:44:32 +02:00
  168b501d53
• Use Protocol type for USB interface protocol.
  Sebastian Zagrodzki · 2017-04-09 18:42:04 +02:00
  f91d53931a
• Use Class for classes in usbid.
  Sebastian Zagrodzki · 2017-04-09 18:29:16 +02:00
  d3428d9b35
• more comments, add Milliamperes type.
  Sebastian Zagrodzki · 2017-04-09 18:15:48 +02:00
  af00028b97
• comments for endpointinfo type fields
  Sebastian Zagrodzki · 2017-04-09 16:56:36 +02:00
  c437b39ef7
• replace class/subclass with Class type
  Sebastian Zagrodzki · 2017-04-09 16:41:38 +02:00
  3e59eeac6c
• add test for an interrupt endpoint.
  Sebastian Zagrodzki · 2017-04-09 00:22:11 +02:00
  83302ae558
• String of the endpoint should use EndpointInfo, not the default mapping

  from InterfaceInfo.

  Sebastian Zagrodzki · 2017-04-09 00:18:54 +02:00
  ab6b612d1b
• since this is the only usecase, return a *libusbDevice for a new

  pointer.

  Sebastian Zagrodzki · 2017-04-09 00:12:39 +02:00
  ca02c03241
• An invalid pointer triggers a runtime failure when passed as argument in

  Go1.7:
  runtime: bad pointer in frame
  github.com/kylelemons/gousb/usb.(*fakeLibusb).open at 0xc420045ce0: 0x1
  fatal error: invalid stack pointer
  
  Use an arbitrary C-allocated pointer instead.

  Sebastian Zagrodzki · 2017-04-09 00:10:15 +02:00
  3ec748db4c
• Modify EndpointInfo to provide meaningful data rather than raw USB

  descriptor values. E.g. for the user, the distinction between
  MaxIsoPkt and MaxPktSize is irrelevant, only the calculated max packet
  size matters.

  Sebastian Zagrodzki · 2017-04-08 23:49:10 +02:00
  e0f30623b3
• Add constants for device speed.
  Sebastian Zagrodzki · 2017-03-27 23:01:02 +02:00
  d982ef5a37
• Rename the rest of the constants.
  Sebastian Zagrodzki · 2017-03-27 22:33:29 +02:00
  f3b2f37e3a
• more constants
  Sebastian Zagrodzki · 2017-03-27 22:29:24 +02:00
  36241e869e
• start renaming the constants to Go-style.
  Sebastian Zagrodzki · 2017-03-27 22:23:47 +02:00
  b70848cf73
• replace Endpoint interface with a real Endpoint struct.
  Sebastian Zagrodzki · 2017-03-27 22:15:45 +02:00
  341fde410e
• Merge branch 'master' of https://github.com/kylelemons/gousb into libusb_wrappers_tests
  Sebastian Zagrodzki · 2017-03-26 17:46:12 +02:00
  0a7fc9fb53
• Update the comment - there's no way to create a new pointer to struct

  libusb_device, since the struct is not defined in libusb headers (only
  name is declared), as such the compiler doesn't know the size of the
  struct to allocate.

  Sebastian Zagrodzki · 2017-03-26 17:44:25 +02:00
  1f977a94aa