| *** tpb has joined #timvideos | 00:00 | |
| mithro | _florent_: so, with your changes the Opsis firmware seems to take ~3 hours to generate? | 00:14 |
|---|---|---|
| _florent_ | which changes? | 00:14 |
| _florent_ | (ie my fork or the official repo?) | 00:15 |
| _florent_ | here it's 20 minutes, but that's still too long!! | 00:15 |
| mithro | _florent_: your jpeg ones | 00:32 |
| mithro | _florent_: when compiling my "florent" branch | 00:33 |
| _florent_ | ah ok, strange here it's fast | 00:33 |
| _florent_ | pfff the pixel issue is not easy to fix without simulation... | 00:34 |
| mithro | _florent_: how long till you pass out, if you haven't already :P | 01:03 |
| _florent_ | a last implemementation and I sleep :) | 01:05 |
| mithro | _florent_: when you get up, we need to sync everything up so I can repo your work | 01:21 |
| _florent_ | ok | 01:21 |
| _florent_ | going to sleep now | 01:21 |
| _florent_ | bye | 01:21 |
| CarlFK | mithro just drove off, will be back in hours | 01:23 |
| mithro | CarlFK: I'm right here | 01:23 |
| CarlFK | lol | 01:23 |
| CarlFK | never mind | 01:23 |
| mithro | CarlFK: can you tell micolous and deeprave that the testing document is updated with some more logical instructions | 02:37 |
| CarlFK | mithro: k - deeprave went off to get his mac fixed. | 02:37 |
| *** tija has joined #timvideos | 05:26 | |
| *** abhinav95 has joined #timvideos | 05:48 | |
| *** abhinav95 is now known as amod95 | 05:49 | |
| *** amod95 has quit IRC | 08:01 | |
| *** se6astian has joined #timvideos | 08:44 | |
| *** Bertl_oO is now known as Bertl | 08:46 | |
| mithro | _florent_: hey - I'm around now | 11:28 |
| _florent_ | ok me too | 11:29 |
| mithro | _florent_: Dinner, Fix micolous's problems, Fix xfxf's problems, now to fix your problems :P | 11:29 |
| _florent_ | Me?, I never have problems :) | 11:29 |
| _florent_ | for the encoder buffer stuff | 11:30 |
| _florent_ | can I create a branch in timvideos repo? | 11:30 |
| _florent_ | and then do a pull request? | 11:30 |
| _florent_ | I've been able to fix the bad pixel problem | 11:30 |
| mithro | _florent_: so normally you would just send a pull request directly from a branch in your own repo | 11:31 |
| mithro | but your own repo appears to be in a pretty mixed up state | 11:31 |
| _florent_ | yes but my repo has some others changes... | 11:31 |
| _florent_ | yes | 11:31 |
| _florent_ | so is it ok if I create a branch in the timvideos repo? | 11:32 |
| mithro | _florent_: yes | 11:32 |
| _florent_ | ok | 11:32 |
| mithro | _florent_: so your plan is to create a branch in the timvideos repo, manually copy the changes accross and commit it? | 11:32 |
| _florent_ | yes and wait travis result to merge it | 11:33 |
| mithro | _florent_: okay | 11:35 |
| mithro | _florent_: do you have travis enabled on your own repo? | 11:35 |
| _florent_ | no | 11:35 |
| mithro | _florent_: do you want me to show you how? | 11:36 |
| _florent_ | yes, but maybe later? | 11:37 |
| mithro | _florent_: okay sure | 11:39 |
| mithro | _florent_: so if I understand correctly, you have | 11:40 |
| mithro | * Fix for the encoder fps restriction (merged) | 11:40 |
| mithro | * Fix for the encoder in own clock domain (unmerged) | 11:40 |
| mithro | * Fix for reducing the buffering needed by the encoder (unmerged) | 11:40 |
| mithro | * Fix for the white bar pixel issue (unmerged) | 11:40 |
| mithro | _florent_: Is the last three correct? | 11:53 |
| _florent_ | sorry? | 11:54 |
| _florent_ | is it possible to stop a build on travis? | 11:55 |
| mithro | _florent_: yes | 12:01 |
| mithro | _florent_: not much reason too though | 12:02 |
| mithro | _florent_: click the little circle X | 12:02 |
| _florent_ | where?:) | 12:03 |
| _florent_ | (maybe I don't see it since not allowed to do that) | 12:04 |
| mithro | I have no idea how to give you permission to that.... | 12:05 |
| mithro | _florent_: just let it fail | 12:05 |
| _florent_ | ok | 12:06 |
| *** 64MAAUVAO has quit IRC | 12:13 | |
| mithro | _florent_: why 66MHz for the encoder? | 12:44 |
| _florent_ | timing problems it seems if we set it faster | 12:46 |
| _florent_ | I'll try to increase it | 12:46 |
| _florent_ | ok going out for lunch | 12:47 |
| _florent_ | travis should be working, and I hope to have good results when I'll be back :) | 12:47 |
| mithro | _florent_: welcome back :) | 13:55 |
| _florent_ | hehe | 13:55 |
| _florent_ | ok so I think it's good now | 13:55 |
| _florent_ | I'll create the pull request | 13:55 |
| mithro | _florent_: so, what does this branch contain? | 13:55 |
| _florent_ | it removes the large buf fifo on the jpeg encoder and fix the issues I had with this new buffer (artifacts that CarlFK saw while recording) | 13:56 |
| _florent_ | and also enable 1080p30 | 13:57 |
| mithro | _florent_: okay | 13:57 |
| mithro | _florent_: With these changes, does the encoder reach 30fps if the USB can reach that? | 13:59 |
| _florent_ | yes | 14:00 |
| mithro | _florent_: great! | 14:00 |
| mithro | _florent_: does this also fix https://github.com/timvideos/HDMI2USB-misoc-firmware/issues/180 ? | 14:00 |
| tpb | Title: Test pattern failure - black/white alternating lines · Issue #180 · timvideos/HDMI2USB-misoc-firmware · GitHub (at github.com) | 14:00 |
| _florent_ | probably yes (I did a test with a similar pattern, it was OK) | 14:01 |
| mithro | _florent_: that issue was fixed by https://github.com/enjoy-digital/HDMI2USB-misoc-firmware/commit/253e44dbbea7885490d9c76aaeed7ed70d4dc53a ? | 14:02 |
| tpb | Title: gateware: fix bad pixel for each 8x8 block · enjoy-digital/HDMI2USB-misoc-firmware@253e44d · GitHub (at github.com) | 14:02 |
| _florent_ | yes with the fifo just before the encoder | 14:06 |
| mithro | okay, great | 14:06 |
| _florent_ | mithro: can you cancel build 372 and 374? | 14:07 |
| _florent_ | thanks | 14:11 |
| _florent_ | if 375 is fine, I'll merge | 14:12 |
| _florent_ | next task will be to look at the HV30? | 14:12 |
| mithro | _florent_: yes please | 14:13 |
| mithro | _florent_: as far as I understand, they should be output 1080i at the moment | 14:14 |
| mithro | _florent_: is there anything left in your repo that hasn't been merged into ours? | 14:14 |
| _florent_ | I don't think so, but I'm going to do a cleanup of my repo | 14:15 |
| mithro | _florent_: yeah - it would be good if your repo was the same as upstream - much easier for me to follow what is happening then | 14:16 |
| _florent_ | so for the HV30, from what I understand you can to capture 1080i, compress it, send it over usb, decompress it and then deinterlace it? | 14:16 |
| mithro | _florent_: first step with the HV30s is to detect a 1080i signal coming from them | 14:17 |
| _florent_ | don't you think that the compression/decompression stages on the 1080i will mess up a little bit the image? | 14:17 |
| _florent_ | we are detecting it correctly | 14:17 |
| _florent_ | just that the image is black for now | 14:17 |
| mithro | _florent_: Should it be passing the image through? | 14:18 |
| mithro | we also don't get anything in the status output? | 14:18 |
| _florent_ | maybe not if the buffer is not updated in the software | 14:18 |
| _florent_ | maybe the software fails to check the length of the DMA and then does not update the buffer number | 14:19 |
| _florent_ | so on the output we are only reading the black buffer | 14:19 |
| _florent_ | status output is fine | 14:19 |
| _florent_ | same as the monitor I tested the HV30 with | 14:19 |
| mithro | _florent_: hrm... Last we tried, we aren't seeing any input detected here | 14:20 |
| _florent_ | here it's fine | 14:21 |
| mithro | _florent_: do you know what "pull down" is? | 14:21 |
| _florent_ | I read your issue about that | 14:21 |
| _florent_ | but that's probably a bit of work to implement it in hardware | 14:22 |
| mithro | so, according to the web - if you put the camera into "cine" mode - it should be producing a HDMI signal which is pulldown | 14:22 |
| _florent_ | ok | 14:22 |
| _florent_ | let's try to get an image first | 14:22 |
| mithro | _florent_: so, we need to verify that | 14:22 |
| mithro | _florent_: if the camera is producing a pull-down signal - then we should some HDMI frames which have identical contents | 14:24 |
| mithro | _florent_: which is one reason why I had ysionneau working on the CRC generation of HDMI frames | 14:25 |
| _florent_ | ok | 14:25 |
| mithro | _florent_: the detection algorithm for pull down is found at http://deinterlace.sourceforge.net/32Spec.htm | 14:26 |
| tpb | Title: DScaler - Mark Rejhon's 3:2 Pulldown Algorithm (at deinterlace.sourceforge.net) | 14:26 |
| mithro | the detection algorithm should only need to be written running on the lm32 - because it can just compare the CRC values | 14:27 |
| _florent_ | ok, but I'm not going to implement that now :) | 14:27 |
| _florent_ | first: get a picture | 14:28 |
| _florent_ | then try to send that to the computer and do software processing | 14:28 |
| mithro | _florent_: yeah | 14:28 |
| mithro | _florent_: that is what I'm calling "stage 1" :) | 14:28 |
| mithro | the (get a picture from the HV30 onto the computer) | 14:29 |
| mithro | stage 2 is trying to get the pull-down deinterlacing working | 14:29 |
| _florent_ | ok | 14:30 |
| *** tija has quit IRC | 14:31 | |
| mithro | _florent_: the CRC generation would also be useful for a selftest mode (send a known HDMI signal, check the CRC value captured is the same) | 14:32 |
| _florent_ | After Tim has too many projects --> Tim has too many ideas :) | 14:34 |
| mithro | I have too many projects because I have too many ideas I think :P | 14:34 |
| mithro | anyway, I should probably head to bed | 14:37 |
| mithro | _florent_: we could really use https://github.com/timvideos/HDMI2USB-misoc-firmware/issues/94 too | 14:41 |
| tpb | Title: Have frequency (and hsync/vsync) information on the inputs in the status message · Issue #94 · timvideos/HDMI2USB-misoc-firmware · GitHub (at github.com) | 14:41 |
| *** se6astian is now known as se6astian|away | 16:27 | |
| *** se6astian|away is now known as se6astian | 16:53 | |
| *** Joelw has quit IRC | 20:20 | |
| *** Joelw has joined #timvideos | 20:20 | |
| *** Bertl is now known as Bertl_zZ | 21:24 | |
| *** se6astian is now known as se6astian|away | 22:18 | |
Generated by irclog2html.py 2.13.1 by Marius Gedminas - find it at mg.pov.lt!