{"action":"created","issue":{"url":"https://api.github.com/repos/MEGA65/mega65-core/issues/52","repository_url":"https://api.github.com/repos/MEGA65/mega65-core","labels_url":"https://api.github.com/repos/MEGA65/mega65-core/issues/52/labels{/name}","comments_url":"https://api.github.com/repos/MEGA65/mega65-core/issues/52/comments","events_url":"https://api.github.com/repos/MEGA65/mega65-core/issues/52/events","html_url":"https://github.com/MEGA65/mega65-core/pull/52","id":296268655,"number":52,"title":"Px100mhz","user":{"login":"kdyke","id":32885008,"avatar_url":"https://avatars0.githubusercontent.com/u/32885008?v=4","gravatar_id":"","url":"https://api.github.com/users/kdyke","html_url":"https://github.com/kdyke","followers_url":"https://api.github.com/users/kdyke/followers","following_url":"https://api.github.com/users/kdyke/following{/other_user}","gists_url":"https://api.github.com/users/kdyke/gists{/gist_id}","starred_url":"https://api.github.com/users/kdyke/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/kdyke/subscriptions","organizations_url":"https://api.github.com/users/kdyke/orgs","repos_url":"https://api.github.com/users/kdyke/repos","events_url":"https://api.github.com/users/kdyke/events{/privacy}","received_events_url":"https://api.github.com/users/kdyke/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":9,"created_at":"2018-02-12T05:15:40Z","updated_at":"2018-02-13T02:38:46Z","closed_at":null,"author_association":"NONE","pull_request":{"url":"https://api.github.com/repos/MEGA65/mega65-core/pulls/52","html_url":"https://github.com/MEGA65/mega65-core/pull/52","diff_url":"https://github.com/MEGA65/mega65-core/pull/52.diff","patch_url":"https://github.com/MEGA65/mega65-core/pull/52.patch"},"body":"This is a working (but incomplete) set of changes to be able to build a bitstream using Vivado 2017.4.  On my Core i7 7700K running Ubuntu under VMWare it's able to do it in about 12 minutes or so.\r\n\r\nRight now it drives Vivado in project mode, going through a few steps to build the project from .tcl and then runs the implementation phase to build the bitstream.   Long term it'll probably be better to switch to non project mode but that requires a lot more manual .tcl scripting.\r\n\r\nAlso, I haven't done anything other than the mega65r1 target yet.   And, to not rock the boat too much or mess with the ISE stuff, it's all done with a separate Makefile.vivado for now.    \r\n\r\nAside from the build process changes, the actual VHDL changes required were pretty minimal and are noted in the commit messages.\r\n\r\nThat being said you might find it useful as a starting point for getting faster bitstream generation.  Or, feel free to tell me to go back to the drawing board. ;)\r\n"},"comment":{"url":"https://api.github.com/repos/MEGA65/mega65-core/issues/comments/365132826","html_url":"https://github.com/MEGA65/mega65-core/pull/52#issuecomment-365132826","issue_url":"https://api.github.com/repos/MEGA65/mega65-core/issues/52","id":365132826,"user":{"login":"gardners","id":665423,"avatar_url":"https://avatars2.githubusercontent.com/u/665423?v=4","gravatar_id":"","url":"https://api.github.com/users/gardners","html_url":"https://github.com/gardners","followers_url":"https://api.github.com/users/gardners/followers","following_url":"https://api.github.com/users/gardners/following{/other_user}","gists_url":"https://api.github.com/users/gardners/gists{/gist_id}","starred_url":"https://api.github.com/users/gardners/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/gardners/subscriptions","organizations_url":"https://api.github.com/users/gardners/orgs","repos_url":"https://api.github.com/users/gardners/repos","events_url":"https://api.github.com/users/gardners/events{/privacy}","received_events_url":"https://api.github.com/users/gardners/received_events","type":"User","site_admin":false},"created_at":"2018-02-13T02:38:46Z","updated_at":"2018-02-13T02:38:46Z","author_association":"CONTRIBUTOR","body":"Speaking with the guys from Trenz, they suggested manually instantiating\nthe BRAMs in Vivado, so that the layout and timing regime is exactly as\nrequired.  My understanding is that it should be possible to get the same\ntiming semantics under Vivado using that method, but that inferring\ndual-port BRAMs with Vivado is to be considered to be in the part of the\nmap marked \"her be dragonnes\".\n\nRegarding the synchronous access, I believe the secret is that you can make\nthe inputs async and the output synchronous to effectively get zero wait\nstates on a read.  This might take some experimenting to get right, but\nshould be possible to make a little test harness that synthesises in a\nminute or two that can be used to determine the correct settings.\n\nNote that we want to avoid using any ipcore stuff if possible, because of\npotential GPL interaction problems, but for getting it working and figuring\nout the right way about it, of course it is fine.\n\nPaul.\n\nOn 13 February 2018 at 12:18, kdyke <notifications@github.com> wrote:\n\n> It's also possible that I changed the VHDL for one of the rams that wasn't\n> strictly needed to make everything fit. I probably need to go back and\n> audit things again to make sure I didn't cause a block ram to be inferred\n> that ISE did not.\n>\n> -Ken\n>\n> Sent from my iPhone\n>\n> > On Feb 12, 2018, at 5:26 PM, Ben-401 <notifications@github.com> wrote:\n> >\n> > the build was successful for nexys4ddr-target,\n> >\n> > but after programming bitfile to fpga using fpgajtag, i get:\n> > black screen, white text, text is chr#$00 (at-symbol) lower 17x-lines as\n> pictured.\n> >\n> >\n> > i then tried programming using \"monitor_load\" with existing (known\n> working) KICKUP and ROMS etc, and get atleast more text on the screen, as\n> pictured.\n> >\n> >\n> > I have discussed this with Paul, and Paul suggests that although it\n> builds a bitstream, the changes to the memory access (causing a delay)\n> impacts significantly.\n> >\n> > Paul will follow up with further discussion.\n> >\n> > ―\n> > You are receiving this because you authored the thread.\n> > Reply to this email directly, view it on GitHub, or mute the thread.\n> >\n>\n> —\n> You are receiving this because you commented.\n> Reply to this email directly, view it on GitHub\n> <https://github.com/MEGA65/mega65-core/pull/52#issuecomment-365124631>,\n> or mute the thread\n> <https://github.com/notifications/unsubscribe-auth/AAonTxkreqcM97-urj2-T1VNVyhxAJI-ks5tUOn5gaJpZM4SBtKC>\n> .\n>\n"}}
{"actor":{"display_login":"gardners"}}