*** tpb has joined #timvideos | 00:00 | |
*** CarlFK has joined #timvideos | 00:36 | |
*** ChanServ sets mode: +v CarlFK | 00:36 | |
*** Kripton has quit IRC | 02:56 | |
*** Kripton has joined #timvideos | 02:57 | |
*** rohitksingh_work has joined #timvideos | 03:44 | |
*** nancychauhan has joined #timvideos | 03:54 | |
nancychauhan | HELP | 03:55 |
---|---|---|
*** nancychauhan has left #timvideos | 04:17 | |
*** sb0 has joined #timvideos | 05:10 | |
*** sb0 has quit IRC | 05:14 | |
*** CarlFK has quit IRC | 05:28 | |
*** sb0 has joined #timvideos | 06:58 | |
*** sb0 has quit IRC | 07:07 | |
futarisIRCcloud | https://talosintelligence.com/pyrebox | 09:03 |
*** rohitksingh_work has quit IRC | 11:03 | |
*** rohitksingh_work has joined #timvideos | 11:04 | |
*** futarisIRCcloud has quit IRC | 11:16 | |
*** harshgugale has joined #timvideos | 11:21 | |
*** harshgugale has quit IRC | 11:23 | |
*** rohitksingh_work has quit IRC | 12:39 | |
mithro | cr1901_modern: gah - OSError: Unable to locate Vivado directory or settings. | 13:19 |
cr1901_modern | Why is "works on my machine" rearing it's ugly head? Why now T_T? | 13:30 |
mithro | cr1901_modern: That is the setting Xilinx dir issue | 13:31 |
cr1901_modern | But I fixed that in LiteX: https://github.com/enjoy-digital/litex/commit/2b00b7eba4204f22eccaedf59b257609ab99f6b3 | 13:36 |
tpb | Title: xilinx/vivado: Provide a fallback mechanism for using the same root f… · enjoy-digital/litex@2b00b7e · GitHub (at github.com) | 13:36 |
cr1901_modern | First off, is it the PR that's failing or what? Because I made sure to incorporate that linked commit before sending my PR | 13:37 |
cr1901_modern | mithro: ^^ | 13:37 |
mithro | cr1901_modern: It appears not | 13:38 |
cr1901_modern | mithro: Please link me to exactly where its failing. Now there's _three_ issues that need to be fixed | 13:39 |
mithro | Of course _florent_ removed all my debugging information about were it looked for the settings file | 13:39 |
cr1901_modern | * the "TCL file can't be found" issue | 13:39 |
mithro | https://travis-ci.org/mithro/HDMI2USB-litex-firmware/jobs/352554810#L3045 | 13:39 |
* cr1901_modern weeps internally... | 13:41 | |
cr1901_modern | my fix broke T_T | 13:42 |
cr1901_modern | So issue 2 is this: ^^ | 13:42 |
cr1901_modern | And issue 3 is... | 13:42 |
* cr1901_modern searches for the link | 13:42 | |
_florent_ | mithro: if i broke something that was probably not wanted, what have i removed? | 13:43 |
mithro | https://github.com/enjoy-digital/litex/commit/e07bd71b16932a504b7435a96901a3b0eb5a99cd | 13:43 |
tpb | Title: build/xilinx: Fixing settings finding. · enjoy-digital/litex@e07bd71 · GitHub (at github.com) | 13:43 |
cr1901_modern | mithro: Issue 3 is https://travis-ci.org/timvideos/HDMI2USB-litex-firmware/jobs/349061105#L3799 | 13:43 |
mithro | cr1901_modern: I've fixed Issue 3 | 13:43 |
cr1901_modern | mithro: What was wrong with it then? | 13:44 |
mithro | cr1901_modern: Your looking at old build there | 13:45 |
mithro | cr1901_modern: https://github.com/timvideos/HDMI2USB-litex-firmware/pull/415 | 13:45 |
tpb | Title: WIP: Update to new LiteX by mithro · Pull Request #415 · timvideos/HDMI2USB-litex-firmware · GitHub (at github.com) | 13:45 |
cr1901_modern | I know, but it seemed like it was my fault, and I still don't know what was wrong with it | 13:45 |
cr1901_modern | i.e. _You_ fixed something that _I_ broke. :P | 13:46 |
mithro | cr1901_modern: you just updated litevideo which was broken | 13:46 |
mithro | cr1901_modern: https://github.com/enjoy-digital/litevideo/commit/9b4169d5d1e2c400a86ea0cbdb800730d84dc40b | 13:46 |
tpb | Title: input/clocking/S7Clocking: fix case when split_clocking=False · enjoy-digital/litevideo@9b4169d · GitHub (at github.com) | 13:46 |
cr1901_modern | Oh yes, that would explain why I didn't explain it. | 13:47 |
cr1901_modern | mithro: So the reason https://github.com/enjoy-digital/litex/commit/e07bd71b16932a504b7435a96901a3b0eb5a99cd was reverted was compatibility with migen.builf | 13:48 |
tpb | Title: build/xilinx: Fixing settings finding. · enjoy-digital/litex@e07bd71 · GitHub (at github.com) | 13:48 |
cr1901_modern | I tried to fix it by providing a fallback to search for extra directories if the migen-compatible behavior failed | 13:48 |
mithro | cr1901_modern: Yeah -- that change should probably go into migen | 13:48 |
cr1901_modern | It was on my todolist (after my PR got merged) | 13:49 |
mithro | it should really have tests :-/ | 13:49 |
cr1901_modern | I'd rather see less code duplication | 13:50 |
cr1901_modern | mithro: In any case, the migen-compatible behavior uses a different type signature from your solution in e07db71... looks like I didn't make the fallback behavior all-encompassing, so it broke on a platform I didn' | 13:53 |
cr1901_modern | t test* | 13:53 |
mithro | cr1901_modern: Yes - it is super tricky to get right | 13:53 |
mithro | cr1901_modern: I'm reasonably confident your version is probably broken if you have multiple ISE and Vivado versions installed as well | 13:53 |
cr1901_modern | Can you explain? I really don't want to install a second copy of ISE/Vivado to test except as a last resort | 13:55 |
cr1901_modern | maybe this behavior will just have to diverge in migen/litex for now b/c too much existing code depends on the litex behavior | 13:56 |
mithro | cr1901_modern: The migen behavior is kind of broken - I think we should just fix in migen? | 13:56 |
cr1901_modern | mithro: Sure I agree, but... | 13:57 |
cr1901_modern | end privmsg :) | 13:58 |
*** CarlFK has joined #timvideos | 14:00 | |
*** ChanServ sets mode: +v CarlFK | 14:00 | |
cr1901_modern | mithro: What is broken in the migen behvaior? | 14:07 |
mithro | cr1901_modern: I should be able to provide the directory I have vivado/ise installed to without needing to know if Vivado or ISE is going to be used | 14:08 |
cr1901_modern | Yes, this is what my patch attempts to fix (by adding an alternate fallback behavior). What makes you think it breaks in the presence of multiple ISE/Vivado versions? | 14:09 |
cr1901_modern | And I'll examine why NeTV fails | 14:10 |
cr1901_modern | mithro: Check #m-labs channel when you get the chance | 15:30 |
*** MK_18 has joined #timvideos | 16:33 | |
*** shivm_ has joined #timvideos | 16:38 | |
*** MK_18 has quit IRC | 16:41 | |
*** felix_ has quit IRC | 17:27 | |
*** felix_ has joined #timvideos | 17:28 | |
CarlFK[m] | mithro (IRC): have you seen this group? Black Mesa Labs https://blackmesalabs.wordpress.com/2018/02/28/sump2-py-as-a-vcd-waveform-viewer-for-xilinx-vivado-rtl-simulator/ | 18:20 |
tpb | Title: SUMP2.py as a VCD waveform viewer for Xilinx Vivado RTL Simulator Black Mesa Labs (at blackmesalabs.wordpress.com) | 18:21 |
*** CarlFK has quit IRC | 18:25 | |
*** shivm_ has joined #timvideos | 18:31 | |
*** CarlFK has joined #timvideos | 18:36 | |
*** ChanServ sets mode: +v CarlFK | 18:36 | |
mithro | CarlFK[m]: yes | 18:42 |
CarlFK[m] | mithro (IRC): Peter (ps1 member) was asking what the difference was between the pmod boards you sent me and https://blackmesalabs.wordpress.com/2017/12/15/bml-hdmi-video-for-fpgas-over-pmod/ | 18:44 |
tpb | Title: BML DVI digital video for FPGAs over PMOD Black Mesa Labs (at blackmesalabs.wordpress.com) | 18:44 |
mithro | CarlFK[m]: mine drives things directly from the pins, his has an conversion IC in between | 18:45 |
CarlFK[m] | mithro (IRC): - k. I guess thats better? (I'll just pass it on..) | 18:46 |
mithro | each has their advantages / disadvantages | 18:47 |
CarlFK[m] | mithro (IRC): he also said "please don't use such tiny components next time. they are hard to work with." | 18:47 |
CarlFK[m] | mithro (IRC): is the edid fix still blocked? https://github.com/enjoy-digital/litevideo/commit/18b88dfee6bf6f4ab55d196747ca00c6c84c2ef2 | 19:45 |
tpb | Title: input/edid: fix scl polarity · enjoy-digital/litevideo@18b88df · GitHub (at github.com) | 19:45 |
mithro | CarlFK[m]: correct - see the cr1901_modern thing above | 19:45 |
cr1901_modern | mithro: See #m-labs chat | 19:50 |
cr1901_modern | mithro: If I commit some debug info for paths, could you rerun travis for the NeTV target? | 19:54 |
mithro | cr1901_modern: Sure | 19:54 |
cr1901_modern | so I can see what paths litex attempted to search | 19:54 |
cr1901_modern | mithro: Alright, I'll get on that in 15 mins | 19:55 |
*** Kripton has quit IRC | 21:51 | |
*** Kripton has joined #timvideos | 21:54 | |
cr1901_modern | mithro: I'll be using this branch to stage changes/test stuff. Eventually it'll be squashed onto new-litex, where I'll make the PR https://github.com/cr1901/HDMI2USB-litex-firmware/tree/new-litex-tmp | 23:19 |
tpb | Title: GitHub - cr1901/HDMI2USB-litex-firmware at new-litex-tmp (at github.com) | 23:19 |
cr1901_modern | err, mithro: https://travis-ci.org/cr1901/HDMI2USB-litex-firmware/jobs/353104732#L3626 | 23:56 |
cr1901_modern | well, that's the wrong commit, nevermind | 23:58 |
Generated by irclog2html.py 2.13.1 by Marius Gedminas - find it at mg.pov.lt!