| *** tpb has joined #litex | 00:00 | |
| futarisIRCcloud | https://www.seeedstudio.com/blog/2019/04/23/free-assembly-for-5-pieces-only-at-seeed-fusion/ | 03:15 |
|---|---|---|
| tpb | Title: Free Assembly for 5 Pieces – Only at Seeed Fusion! | Seeed Studio Blog (at www.seeedstudio.com) | 03:15 |
| keesj | tempting | 06:33 |
| keesj | I need to spend a few hours creating the correct bom file but it would be a great experience.. if they have ice40 | 06:34 |
| futarisIRCcloud | https://www.seeedstudio.com/opl.html - I couldn't see any Ice40 at their OPL. | 07:02 |
| tpb | Title: Fusion Open Part Library(OPL) Service - Seeed Studio (at www.seeedstudio.com) | 07:02 |
| futarisIRCcloud | https://www.seeedstudio.com/fusion_pcb.html says that they can use digikey though... | 07:03 |
| tpb | Title: Fusion PCB Manufacturing & Prototype PCB Assembly - Seeed Studio (at www.seeedstudio.com) | 07:03 |
| keesj | I tried multiple ways to use their open parts library with kicad but failed all times. | 07:04 |
| keesj | (incompatible with kicad 5 and other similar problems) (this was last year) perhaps I need to try again https://www.seeedstudio.com/blog/2019/01/30/kicad-and-eagle-libraries-you-can-use-with-confidence/ (artible from Jan this year) | 07:06 |
| futarisIRCcloud | Could probably do a S7 Mini or Ice40. | 07:07 |
| futarisIRCcloud | tinyfpga_ : Did Greg Davill get https://github.com/tinyfpga/TinyFPGA-EX/tree/master/board to work? | 07:08 |
| tpb | Title: TinyFPGA-EX/board at master · tinyfpga/TinyFPGA-EX · GitHub (at github.com) | 07:08 |
| *** felix_ has quit IRC | 07:50 | |
| *** felix_ has joined #litex | 07:51 | |
| keesj | the new litex_server does not work as reliable as the previous one | 08:15 |
| keesj | specially it does not matter if the server is up or not I will get the same strange error | 08:16 |
| keesj | ValueError: not enough values to unpack (expected 5, got 1) | 08:17 |
| keesj | hmm it looks like the parsing of the csr is to blame the newly generated comment on top of the csr causes problems (e.g. the ##### Generated by litex) | 08:22 |
| keesj | https://stackoverflow.com/questions/14158868/python-skip-comment-lines-marked-with-in-csv-dictreader is needed | 08:29 |
| tpb | Title: Python: skip comment lines marked with # in csv.DictReader - Stack Overflow (at stackoverflow.com) | 08:29 |
| keesj | http://paste.ubuntu.com/p/7ghdGZg6Kh/ (the new header format to reproduce the problem) | 09:04 |
| tpb | Title: Ubuntu Pastebin (at paste.ubuntu.com) | 09:04 |
| _florent_ | keesj: sorry, i finish something and i do fix that right after | 09:06 |
| keesj | I have "fixed" it locally if you want I create a issue on github | 09:12 |
| keesj | the class would need a little refactoring to not have to much duplication | 09:13 |
| _florent_ | keesj: i'm looking at that, you can create a pull request yes if you want | 09:46 |
| _florent_ | keesj: https://github.com/enjoy-digital/litex/commit/27fbb814abd727b35bc8ed989808282b0eebbee7 | 10:35 |
| tpb | Title: tools/remote/csr_builder: allow comments in csv file and cleanup · enjoy-digital/litex@27fbb81 · GitHub (at github.com) | 10:35 |
| keesj | you beat me to it | 10:53 |
| keesj | https://github.com/keesj/litex/commit/0a34b3d05b1b4f65359b4b1d08a3b4ddc6234192 | 10:53 |
| tpb | Title: Fix parsing of CSR files containing a header. · keesj/litex@0a34b3d · GitHub (at github.com) | 10:53 |
| keesj | the @staticmethod is nicer | 11:08 |
| somlo | _florent_: about get_*_git_revision(), doing something like "litex/litex/tools/litex_sim.py --cpu-type=vexriscv" used to work before, now it crashes with "fatal: not a git repository (or any of the parent directories): .git" | 13:18 |
| somlo | I assume it now only works if you run it from within the right folder (e.g., the litex project directory)? | 13:18 |
| somlo | and, what if one has a distro-packaged migen, and no git repo in which to run `git foo` commands (re. get_migen_git_revision(), specifically)? | 13:21 |
| somlo | ok, it's only get_migen_git_revision() that fails for me right now (distro-packaged migen in Fedora), but can we talk about codifying the assumption that litex (and migen) are being deployed as git repos? | 13:27 |
| *** futarisIRCcloud has quit IRC | 14:22 | |
| somlo | _florent_: how about https://github.com/enjoy-digital/litex/pull/173 ? | 17:01 |
| tpb | Title: build: handle exceptional case when litex/migen not deployed as git repo by gsomlo · Pull Request #173 · enjoy-digital/litex · GitHub (at github.com) | 17:01 |
| keesj | it the checking of git version not also only relevant when in "developer" mode? can this not reliably be detected? | 20:17 |
| somlo | just dug around through litex to see how generated_banner() is actually *used* | 20:26 |
| somlo | btw, _florent_, this line should be "+=", missed that in my PR: https://github.com/enjoy-digital/litex/blob/master/litex/soc/integration/cpu_interface.py#L75 | 20:27 |
| tpb | Title: litex/cpu_interface.py at master · enjoy-digital/litex · GitHub (at github.com) | 20:27 |
| somlo | it appears it's just there to add comments re. which migen and litex git commit was current when various generated .h files (and top.v) were produced | 20:29 |
| keesj | won't break anything indeed | 20:29 |
| keesj | (from the looks of it) | 20:29 |
| somlo | so "------" as a fallback is fine with me for now -- the interesting question IMHO (and to which I don't have a reasonable answer yet) is how I'd get the migen/litex version if/when they're not in a git repo (developer-mode or not) | 20:30 |
| somlo | I'd really hate to go down the "rpm? deb? other?" rabbit hole, just for a bunch of comments :) | 20:30 |
| somlo | cheesy cop-out, the packager (me, in Fedora's case) could sed -i 's/------/<version>/' as part of the package build process :) | 20:32 |
| somlo | i.e. just monkeypatch the version into the sources. But I'm not sure I want to go down in history as the guy who suggested that :) | 20:33 |
| keesj | this problem has been solved many times already and also keeps comming back. I try to keep away from it | 20:36 |
| keesj | testing for the existance of the .git folder is probably way nicers of a solution | 20:38 |
| somlo | keesj: right, but the question is what to do if .git is *not* there (the "except:" branch) | 20:39 |
| _florent_ | somlo: i think your solution is fine for now. If migen/litex are installed from .rpm/.deb we'll fall back to the "--------" case, we could fix that later if really needed | 20:50 |
| somlo | _florent_: thanks, that works for me! | 20:53 |
| _florent_ | somlo: have you been able to do progress on the rocket integration? | 20:56 |
| _florent_ | somlo: are you still stuck with the missing burst support? | 20:56 |
| somlo | _florent_: I found a third-party axi-wb bridge here: https://bitbucket.org/danstrother/dls_cores/get/beb677ea1919.tar.gz | 20:58 |
| somlo | which I'm going to try tomorrow | 20:58 |
| somlo | if that won't work, I'll try to start hacking on the LiteX internal axi implementation | 20:59 |
| somlo | I may be working harder than needed just to avoid learning something :) but I really just want to see something (anything at all) work (even if horribly inefficient) before deciding where to spend serious effort optimizing things (and that's when I'll actually *have* to learn things :) | 21:00 |
| _florent_ | ok, if you are stuck i can try to help creating the axi to wishbone bridge | 21:00 |
| _florent_ | yes sure, i understand | 21:00 |
| somlo | I read somewhere that if laziness were a martial art, they'd call it "the Path of Least Resistance" :) | 21:01 |
| _florent_ | :) | 21:02 |
| somlo | it's almost quittin' time here (EDT) but I'm planning to lock myself in my office tomorrow and do nothing but this, so chances are I'll be here crying about it... | 21:07 |
Generated by irclog2html.py 2.13.1 by Marius Gedminas - find it at mg.pov.lt!