18 Feb 2020 |
blueluna | Has there been any attempts at using the USB block in nRF52? | 11:46:27 |
jschievink | blueluna: this issue indicates that someone tried: https://github.com/mvirkkunen/usb-device/issues/4 | 12:25:14 |
jschievink | I have also briefly looked into the peripheral, it doesn't look too complicated, but I've never used the usb-device stack | 12:25:46 |
blueluna | Thanks jschievink , Will take a look. | 12:29:49 |
thalesfragoso | You may also take a look at stm32-usbd to get a feel for it | 15:59:37 |
thalesfragoso | Implementing usb-device gives you access to some usb classes crates already published too | 16:00:47 |
20 Feb 2020 |
jschievink | I've published new nRF52 PACs and opened this update PR for the nRF91: https://github.com/nrf-rs/nrf91/pull/6 | 19:31:33 |
jschievink | This should get us a step closer to a new HAL release | 19:31:47 |
Noah | nice, thx! | 20:47:00 |
thejpster | lgtm | 21:11:27 |
thejpster | Do we have bors, or do I just hit merge? | 21:11:35 |
jschievink | thejpster: thanks! You'll have to take care of publishing though since I'm not in the nrf91 team | 21:18:53 |
thejpster | Hmm #7 | 21:22:26 |
thejpster | https://github.com/nrf-rs/nrf91/issues/7 | 21:22:28 |
thejpster | I don't have push rights on that repo, so I'm going to have to take a fork to open a PR to remove those lints. | 21:26:05 |
thejpster | Ok, have opened PR #8 and #9. | 21:34:23 |
jschievink | the lints are probably an svd2rust problem, I just left them alone in the other PACs since downstream users won't see them anyways | 21:40:02 |
jschievink | ah, it's fixed in master https://github.com/rust-embedded/svd2rust/issues/415 | 21:41:37 |
jschievink | thejpster: wait, can you not merge PRs? | 21:41:56 |
thejpster | Won't anyone who compiles a HAL see them? | 21:43:03 |
thejpster | No idea if I can merge a PR, but I definitely can't push a new branch | 21:43:24 |
thejpster | "Only those with write access to this repository can merge pull requests." | 21:44:50 |
thejpster | Guess I can't. | 21:44:56 |
jschievink | HALs shouldn't make them show up, Cargo hides all warnings from dependencies | 21:53:52 |
jschievink | I guess therealprof can fix the permissions on that repo? | 21:54:25 |
jschievink | hmm, but you're also in the nrf52 team, why can I merge PRs but you can't?? | 21:55:06 |
Noah | hmm are you maybe a direct collaborator on the repo for some reason? | 21:59:09 |
jschievink | no, but jp isn't getting the member badge on github, weird! he's definitely still a member though | 22:00:31 |
jschievink | oh no lol :D | 22:01:20 |
Noah | odd, ill have a look when im in the train but i dont have more permissions than you two either :) | 22:01:28 |