| *** tpb has joined #timvideos | 00:00 | |
| *** TheAssassin has quit IRC | 00:16 | |
| *** TheAssassin has joined #timvideos | 00:21 | |
| *** futarisIRCcloud has joined #timvideos | 01:42 | |
| *** CarlFK has quit IRC | 01:48 | |
| xobs | mithro: I remember an ancient article about being able to write FORTRAN in any language. I'm sure it applies somehow. | 02:21 |
|---|---|---|
| *** Kripton has quit IRC | 03:11 | |
| *** Kripton has joined #timvideos | 03:19 | |
| *** froztbyte has quit IRC | 03:37 | |
| *** froztbyte has joined #timvideos | 03:45 | |
| *** olasd has quit IRC | 03:58 | |
| *** CarlFK has joined #timvideos | 04:02 | |
| *** ChanServ sets mode: +v CarlFK | 04:02 | |
| *** olasd has joined #timvideos | 04:05 | |
| mithro | tannewt: ping? | 05:19 |
| mithro | tannewt: Is "set_config #:1, process_set_config: 463: failed" expected? | 05:19 |
| tannewt | No it’s not. I’m at choir now though. Probably an endian issue | 05:21 |
| tannewt | I think that was due to the stall not being cleared by the setup token | 05:22 |
| tannewt | The setup packet should always be acked | 05:22 |
| mithro | https://www.irccloud.com/pastebin/jYQ9K5G2/ | 05:22 |
| tpb | Title: Snippet | IRCCloud (at www.irccloud.com) | 05:22 |
| mithro | tannewt: that is happening now | 05:22 |
| tannewt | I’m not sure what that output means. I may have time tomorrow to look. I’m on my phone now | 05:23 |
| mithro | tannewt: Okay | 05:24 |
| xobs | mithro: your string descriptors look like you're feeding them utf8/ascii data when they should be utf16 (or is it ucs2?) | 05:36 |
| xobs | It also sounds like you might be missing the SET_CONFIGURATION command? For that you just need to respond with a zero-byte DATA packet. | 05:44 |
| *** CarlFK has quit IRC | 05:44 | |
| xobs | Here's what it looks like in Grainuum: https://github.com/xobs/grainuum/blob/master/grainuum-state.c#L264 (note that response_len remains 0 for both SET_ADDRESS and SET_CONFIGURATION) | 05:44 |
| tpb | Title: grainuum/grainuum-state.c at master · xobs/grainuum · GitHub (at github.com) | 05:44 |
| mithro | tannewt: I'm not seeing any of the endpoint opens.... | 05:47 |
| mithro | [278609.785183] cdc_acm 2-13.1.3:1.0: acm_port_activate - usb_submit_urb(ctrl irq) failed | 05:47 |
| mithro | also this suspect line... | 05:48 |
| mithro | [278494.791232] xhci_hcd 0000:00:14.0: xHCI xhci_add_endpoint called with enabled ep 00000000150719c9 | 05:48 |
| mithro | Oh | 05:52 |
| mithro | the 463 == line 463 | 05:52 |
| mithro | The issue seems to be | 05:56 |
| mithro | TU_ASSERT( 0xff == _usbd_dev.itf2drv[desc_itf->bInterfaceNumber] ); | 05:56 |
| xobs | Is there an off-by-one error with bInterfaceNumber? I can't remember if it's 0-indexed or 1-indexed. | 05:57 |
| *** rohitksingh_work has joined #timvideos | 06:14 | |
| mithro | I removed that assert and now I get endpt_open | 06:30 |
| mithro | It seems to have an issue with | 06:52 |
| mithro | [282354.299357] cdc_acm 2-13.1.3:1.0: acm_ctrl_msg - rq 0x20, val 0x0, len 0x7, result -110 | 06:52 |
| mithro | xobs / tannewt: Pushed the latest version with a lot of updates, reworked / rewrote a lot of the endpoint interface stuff | 06:59 |
| xobs | mithro: sounds like a lot of really good progress! | 06:59 |
| mithro | It'll now correctly always ACK a setup packet | 07:00 |
| mithro | and you can stall endpoints | 07:00 |
| mithro | as far as I can tell the hardware seems to be working pretty well | 07:01 |
| mithro | Everything should be pushed now | 07:06 |
| tannewt | Nice job mithro! I may have time to test tonight | 07:15 |
| mithro | tannewt: I also put everything into a nice structure -> https://github.com/mithro/tinyusb/blob/develop/src/portable/valentyusb/dcd_valentyusb.c#L93-L106 | 07:34 |
| tpb | Title: tinyusb/dcd_valentyusb.c at develop · mithro/tinyusb · GitHub (at github.com) | 07:34 |
| *** thaytan has quit IRC | 07:39 | |
| *** thaytan has joined #timvideos | 07:40 | |
| *** ChanServ sets mode: +v thaytan | 07:40 | |
| tannewt | mithro, updating everything now! | 07:42 |
| *** swalladge_ has joined #timvideos | 07:47 | |
| mithro | tannewt: I think I'm going to head home in a second | 07:47 |
| *** swalladge has quit IRC | 07:47 | |
| tannewt | k np. I'm surprised you stay at work to work on it | 07:47 |
| mithro | https://www.irccloud.com/pastebin/w21iJyo4/ | 07:47 |
| tpb | Title: Snippet | IRCCloud (at www.irccloud.com) | 07:47 |
| mithro | tannewt: My work setup is much nicer than my home setup, 3 * 4k monitors | 07:48 |
| tannewt | nice! I have one ultrawide and its my work and home setup :-) | 07:48 |
| tannewt | that looks like its working | 07:48 |
| tannewt | what are you working on now? | 07:49 |
| mithro | tannewt: https://github.com/mithro/valentyusb/blob/master/valentyusb/test_usbcore.py#L2223-L2258 | 07:49 |
| tpb | Title: valentyusb/test_usbcore.py at master · mithro/valentyusb · GitHub (at github.com) | 07:49 |
| tannewt | awesome | 07:49 |
| mithro | tannewt: I'm pretty happy with how the tests are starting to turn out - but they are very slow.... | 07:50 |
| mithro | hrm, I should finish my dinner before going home... | 07:50 |
| tannewt | ya, definitely | 07:51 |
| tannewt | are you updating litex-buildenv or just valentyusb? | 07:52 |
| mithro | tannewt: you'll want to update everything | 07:52 |
| mithro | tannewt: there are some changes in litex, valentyusb and tinyusb packages | 07:52 |
| tannewt | kk | 07:53 |
| tannewt | mithro, trace looks wayyyyyy cleaner! | 07:58 |
| tannewt | https://usercontent.irccloud-cdn.com/file/L0TLWvbe/Screen%20Shot%202018-12-11%20at%2011.52.41%20PM.png | 07:59 |
| tannewt | very nice job! | 07:59 |
| mithro | tannewt: so, need to figure out what is going that means cdc-acm isn't working | 08:02 |
| tannewt | ya, looking at that now | 08:03 |
| tannewt | its trying to get or set the linecoding I think | 08:03 |
| tannewt | the strings are the incorrect endian as well | 08:03 |
| tannewt | my bx isn't staying in the bootloader after reset | 08:12 |
| mithro | tannewt: I had that - needed to reset the usb bus | 08:12 |
| tannewt | how do you do that? | 08:13 |
| tannewt | https://www.irccloud.com/pastebin/XIiLZiNL/ | 08:13 |
| tpb | Title: Snippet | IRCCloud (at www.irccloud.com) | 08:13 |
| mithro | tannewt: sudo rmmod xhci; sudo modprobe xhci | 08:13 |
| mithro | IIRC | 08:13 |
| mithro | tannewt: Actually seems to be "sudo rmmod xhci_pci; sudo modprobe xhci_pci" | 08:14 |
| tannewt | kk thanks | 08:14 |
| tannewt | still not happy | 08:16 |
| tannewt | may need to restart | 08:16 |
| tannewt | ya, not enumerating | 08:17 |
| mithro | well, I'm *actually* heading home now | 08:17 |
| tannewt | k thanks! I'll see if I can figure this out in the next 20 minutes or so | 08:17 |
| tannewt | then bedtime :-) | 08:17 |
| *** CarlFK has joined #timvideos | 08:25 | |
| *** ChanServ sets mode: +v CarlFK | 08:25 | |
| *** swalladge_ is now known as swalladge | 09:02 | |
| *** cr1901_modern has quit IRC | 09:06 | |
| tannewt | mithro: I got the strings looking ok and the endpoint size correct but its still crashing the xhci every time. (changes here: https://github.com/mithro/tinyusb/compare/develop...tannewt:endian_cleanup?expand=1) The beagle doesn't show any token coming from the host after the zero length IN after the set line coding. Is it possible its something lower level? Anyway I'm off to bed. Great work on this! Sooo close. :-) | 09:06 |
| tpb | Title: Comparing mithro:develop...tannewt:endian_cleanup · mithro/tinyusb · GitHub (at github.com) | 09:06 |
| *** futarisIRCcloud has quit IRC | 09:12 | |
| thaytan | anyone else get email from Ark Jadoon <[email protected]> about FPGA video compression? | 11:01 |
| *** tumbleweed_ has quit IRC | 11:05 | |
| *** tumbleweed_ has joined #timvideos | 11:05 | |
| *** tumbleweed_ is now known as tumbleweed | 11:05 | |
| tumbleweed | yeah, assuming because of timvideos | 11:06 |
| tumbleweed | sounds like someone who wants people to do his schoolwork for him | 11:07 |
| *** puck_ has quit IRC | 11:23 | |
| *** puck_ has joined #timvideos | 11:25 | |
| *** cr1901_modern has joined #timvideos | 12:02 | |
| *** rohitksingh_work has quit IRC | 13:33 | |
| *** rohitksingh has joined #timvideos | 14:30 | |
| *** rohitksingh has quit IRC | 15:36 | |
| *** thaytan has quit IRC | 16:18 | |
| *** thaytan has joined #timvideos | 16:22 | |
| *** ChanServ sets mode: +v thaytan | 16:22 | |
| mithro | tmmmcci: morning? | 16:45 |
| *** rohitksingh has joined #timvideos | 18:17 | |
| *** rohitksingh has quit IRC | 21:02 | |
| *** rpifan has joined #timvideos | 21:02 | |
| *** rohitksingh has joined #timvideos | 21:05 | |
| *** rpifan has quit IRC | 21:51 | |
| mithro | tannewt: morning? | 21:58 |
| tannewt | hi hi mithro | 21:58 |
| mithro | So I'm getting to | 21:59 |
| mithro | https://www.irccloud.com/pastebin/HCPa3QWt/ | 21:59 |
| tpb | Title: Snippet | IRCCloud (at www.irccloud.com) | 21:59 |
| mithro | tannewt: It looks like it was expecting a status packet and got a setup packet? | 22:00 |
| mithro | but it was a tok == out.... | 22:00 |
| tannewt | ya, that looks like a set config setup packet | 22:01 |
| mithro | tannewt: I'm going to change to tok being per endpoint | 22:02 |
| tannewt | good call | 22:02 |
| tannewt | only endpoint 0 should get setup | 22:02 |
| tannewt | I think its the only endpoint that will get a mix of tokens | 22:03 |
| mithro | tannewt: Yeah - it's confusing like that | 22:03 |
| tannewt | ya | 22:03 |
| mithro | I was pondering making the endpoint "address" == tok_type + epnum | 22:04 |
| tannewt | address is usually num + direction | 22:04 |
| mithro | tannewt: Yeah - but setup is kind of special.... | 22:05 |
| tannewt | ya, it is | 22:06 |
| mithro | The only thing which stopped me is the stall state being kind of shared... | 22:07 |
| mithro | Hrm, it didn't seem to change anything... | 22:08 |
| mithro | [337536.763623] usb 2-13.1.3: can't set config #1, error -110 | 22:10 |
| tannewt | I wasn't getting that last night | 22:11 |
| tannewt | did you take a peek at the changes I posted last night? | 22:11 |
| mithro | Yes | 22:12 |
| mithro | Hrm, I was getting /dev/ttyACM0 last night.... | 22:13 |
| mithro | So, the "ep0: tok out w:0 0 9 1 0 0 0 0 0 g:8 -- ERR long!!" is the "SET CONFIGURATION" request... | 22:17 |
| mithro | tannewt: It kind of looks like the status token and setup token are getting mundged together or something? | 22:18 |
| tannewt | maybe. I don't know. I can't look now. adding midi to tinyusb | 22:19 |
| *** rohitksingh has quit IRC | 22:22 | |
| *** Toba__ has joined #timvideos | 22:32 | |
| *** nancy[m]1 has joined #timvideos | 22:35 | |
| *** pzieba[m]1 has joined #timvideos | 22:35 | |
| *** Toba has quit IRC | 22:35 | |
| *** pzieba[m] has quit IRC | 22:35 | |
| *** nancy[m] has quit IRC | 22:35 | |
Generated by irclog2html.py 2.13.1 by Marius Gedminas - find it at mg.pov.lt!
