**** BEGIN LOGGING AT Sun Sep 03 03:00:02 2017 Sep 03 09:29:11 hey maciejjo ! there ? Sep 03 09:31:53 cna you please check if this part is still correct : https://github.com/maciejjo/beaglebone-pru-dma#implementation-overview Sep 03 09:32:51 and maybe you'd like to add link to your final-demo in your readme. Sep 03 09:42:28 maciejjo: And this GPIO configuration : https://github.com/maciejjo/beaglebone-pru-dma/blob/master/dts/pru-dma-00A0.dts#L36 Sep 03 09:42:45 I think it will conflict with cape-universal Sep 03 09:43:35 and even if it wont, this should not be necessary for a DMA project. the GPIO config will depend on the specific project. I think its best to remove that . Sep 03 10:19:52 maciejjo: The code should be under `GPLv2` Sep 03 10:19:53 https://github.com/maciejjo/beaglebone-pru-dma/blob/master/kernel-patch/0001-Add-pru-dma-driver-to-Linux-tree.patch#L324 Sep 03 10:40:31 maciejjo: copyright and license info should also be mentioned in other files, like in here : https://github.com/maciejjo/beaglebone-pru-dma/tree/master/firmware/lib/pru_dma_lib Sep 03 11:30:43 maciejjo : Also, can you report the throughput achievable using PRU-DMA ? Sep 03 12:53:23 my video https://www.youtube.com/watch?v=vnOUQEhiibA Sep 03 12:53:29 sorry for being late Sep 03 12:54:03 nerdboy:,ds2: I changed circuit in documentation for single axis Sep 03 15:58:14 maciejjo: ? Sep 03 16:19:27 zeekhuge: I will be at computer in abt 2h, will address all your points then Sep 03 20:07:26 maciejjo: now ? Sep 03 22:05:51 zeekhuge: sorry, took me longer to get back. Yes you are right, implementation details section needs to be updated, I overlooked it Sep 03 22:05:57 I will add commits for all your remarks. I will push at the latest tomorrow morning. Sep 03 22:07:43 maciejjo: Okay so just to get back on it, you will fix the following : Sep 03 22:07:49 1. implementation details Sep 03 22:07:58 2. GPIO configuration in the dts Sep 03 22:08:08 3. code whould be GPLv2 Sep 03 22:08:35 4. copyright and license info in the PRU firmware files Sep 03 22:08:55 and if possible 5. the throughput achievable Sep 03 22:09:00 maciejjo: is that correct ? Sep 03 22:09:11 yes, + link to demo video in readme Sep 03 22:09:24 maciejjo: cool ! thanks ! **** ENDING LOGGING AT Mon Sep 04 03:00:00 2017